Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Don´t validate Customer password field if "Allow login" isn´t checked #158

Closed
merylen opened this issue Dec 21, 2017 · 2 comments
Closed
Labels
Milestone

Comments

@merylen
Copy link

merylen commented Dec 21, 2017

Hi,

i´m on a fresh installation (Version 1.1.5).

I discovered an strange behavior when i tried to add an customer.
Adding all required information and hitting "Save" didn´t Save the new customer. In fact i didn´t get any response.

APP_Debug = true shows:

The given data failed to pass validation.
/var/www/akaunting/vendor/laravel/framework/src/Illuminate/Foundation/Http/FormRequest.php#125

All must-have-field have been filled, but i wasn´t able to save.
Adding an customer when writing an Invoice is possible, but then editing wasn´t possible.

After a longer research i hit the bug.

I´m using a password-Manager which autofills password-fields.
In this case, the "hidden" field under "Allow login" [Which was not marked by me!] was partial (only the first), auto-filled and this hits the "validation".

In my opinion, the passwort-field shouldn´t be validated when the "Allow login"-field isn´t checked.

Greetings

@denisdulici
Copy link
Member

I can confirm this Markus, thanks for reporting.

@denisdulici denisdulici added the Bug label Jan 8, 2018
@sweisgerber-dev
Copy link

To hop on this train, it would be cool to auto-generate the passwords for users if fields are left blank or a checkbox is set.

The password is then only stored in the DB and not sent to anyone.

Users could then reset their password via the reset form.

@cuneytsenturk cuneytsenturk added this to the 1.1.11 milestone Feb 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants