crudkit icon indicating copy to clipboard operation
crudkit copied to clipboard

Warning: Missing argument 1 for CrudKit\Pages\BasicLoginPage::preprocess(), called in C:\xampp\htdocs\kit\vendor\skyronic\crudkit\src\CrudKit\CrudKitApp.php on line 51 and defined in C:\xampp\htdocs\kit\vendor\skyronic\crudkit\src\CrudKit\Pages\BasicLoginPage.php on line 67

Open ispcpr opened this issue 9 years ago • 6 comments

Warning: Missing argument 1 for CrudKit\Pages\BasicLoginPage::preprocess(), called in C:\xampp\htdocs\kit\vendor\skyronic\crudkit\src\CrudKit\CrudKitApp.php on line 51 and defined in C:\xampp\htdocs\kit\vendor\skyronic\crudkit\src\CrudKit\Pages\BasicLoginPage.php on line 67

ispcpr avatar Aug 23 '16 08:08 ispcpr

In /vendor/skyronic/crudkit/src/CrudKit/CrudKitApp.php, public function useLogin calls $login->preprocess (); I changed it to $login->preprocess ($this); and there are no errors.

rstuckart avatar Sep 15 '16 15:09 rstuckart

Had the same issue, resolved with the fix by @rstuckart

nzjs avatar Oct 15 '16 22:10 nzjs

@rstuckart solution works

sidgitind avatar Dec 27 '16 11:12 sidgitind

Again, rstuckart's solution works; line 52 in CrudKitApp.php. Many thanks rstuckart.

Byter-Bit avatar Jan 11 '17 01:01 Byter-Bit

tried the solution, no more errors, but logging in with provided credentials apparently does nothing.. (I see the login form, and no error/wrong credentials message is shown)

marco-gagliardi avatar Jan 18 '17 12:01 marco-gagliardi

If you look at the preprocess function in the BasicLoginPage class (vendor\skyronic\crudkit\src\CrudKit\Pages\BasicLoginPage.php) you will notice that the $app parameter is never used anywhere. In my opinion a better fix to this issue would be to remove this $app parameter rather than passing an object that is useless.

For those who have difficulties to login, the Login/Pass to use are: admin/demo or user/demo

twysto avatar Jul 17 '17 09:07 twysto