- 🇺🇸United States jive01
I get the following error when testing patch #6. :
Fatal error: Class Drupal\password_policy\PasswordPolicyValidator contains 1 abstract method and must therefore be declared abstract or implement the remaining methods (Drupal\password_policy\PasswordPolicyValidatorInterface::buildPasswordPolicyConstraintsTableRows) in /app/web/modules/contrib/password_policy/src/PasswordPolicyValidator.php on line 16
- Status changed to Needs work
over 1 year ago 12:21pm 18 July 2023 - Status changed to Needs review
over 1 year ago 8:06am 26 July 2023 - Open on Drupal.org →Core: 10.0.7 + Environment: PHP 7.4 & MySQL 5.7last update
over 1 year ago Waiting for branch to pass - Status changed to Needs work
over 1 year ago 8:11am 9 August 2023 - 🇬🇧United Kingdom Alina Basarabeanu
Using Drupal Version 9.5.10 and password_policy 4.0.0 the patch applies cleaning but the markup generated has a duplicated div element with id password-policy-status
See the screenshot attached. - Assigned to samit.310@gmail.com
- Issue was unassigned.
- Status changed to Needs review
over 1 year ago 11:30am 16 August 2023 - Open on Drupal.org →Core: 10.0.7 + Environment: PHP 7.4 & MySQL 5.7last update
over 1 year ago Waiting for branch to pass - 🇮🇳India samit.310@gmail.com
Hi @Alina,
The duplicated div element with id password-policy-status issue has been fixed, Please review again.
Thanks
Samit K. - Status changed to Postponed: needs info
9 months ago 9:15pm 7 April 2024 - 🇺🇸United States Kristen Pol Santa Cruz, CA, USA
Thanks to everyone for the work on this issue.
I'm going through all the 8.x issues.
As the 8.x is no longer supported, I'm postponing this issue for now and need feedback as to whether or not this issue is relevant to 4.0.x.
If it is, please reopen and change the version, make sure the issue summary is clear and complete, including steps to reproduce, and reroll the patch. If it's not, please close.
If there is no response to this in a month addressing the above, it can be closed.