Skip to content

[18.0][FIX] password_security: Fix expected singleton#962

Open
moylop260 wants to merge 1 commit into
OCA:18.0from
vauxoo-dev:18.0-fix-singleton-moy
Open

[18.0][FIX] password_security: Fix expected singleton#962
moylop260 wants to merge 1 commit into
OCA:18.0from
vauxoo-dev:18.0-fix-singleton-moy

Conversation

@moylop260

Copy link
Copy Markdown
Contributor

Forward-port of #961 to 18.0.

When updating the password for more than one user, ensure_one() raises an expected singleton error.

This keeps the same minimal approach used in 17.0:

  • remove ensure_one() from password_match_message()
  • remove ensure_one() from _check_password_rules()
  • add a regression test for multi-user password write

cc @moylop260

Updating the password for more than one user raises "expected singleton" error

This fix supports multi case

(cherry picked from commit OCA@b990a54 OCA#762)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mod:password_security Module password_security series:18.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants