Skip to content
This repository has been archived by the owner on Dec 16, 2019. It is now read-only.

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 #71

Open
ispcpr opened this issue Aug 23, 2016 · 6 comments

Comments

@ispcpr
Copy link

ispcpr commented Aug 23, 2016

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

@rstuckart
Copy link

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.

@nzjs
Copy link

nzjs commented Oct 15, 2016

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

@sidgitind
Copy link

@rstuckart solution works

@Byter-Bit
Copy link

Byter-Bit commented Jan 11, 2017

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

@marco-gagliardi
Copy link

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)

@twysto
Copy link

twysto commented Jul 17, 2017

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

7 participants