In-Portal Issue Tracker - In-Portal CMS
Viewing Issue Advanced Details
227 [In-Portal CMS] Front End feature request N/A 2009-08-14 01:06 2012-03-19 03:55
alex  
alex  
normal  
closed 5.0.0  
fixed  
 
none 5.0.1  
Fixes "*" not displayed after password fields during registration
1
0000227: Password required mark logic change
Currently Password and VerifyPassword fields are not marked as required. This results in missing asterisk ("*" - required mark) on new user add form (in administrative console and on front-end). Then when value is set to the field, kPasswordFormatter class makes them required for new users. This gives us another problem - inability to dynamically make password non-required for new users.

I propose to remove "making required" code from kPasswordFormatter class and do the following:

1. in u:OnPreCreate (for administrative console) and in u:OnNew (for front-end) events do $object->setRequired for both password fields - helps to initially show "*" on user registration form;
2. in u:OnBeforeItemCreate do the same - to make passwords required when user is saved to database.

This way it will be highly customizable and more transparent logic to the naked eye.

patch password_logic_change.patch (8,582) 2012-03-19 03:54
http://tracker.in-portal.org/file_download.php?file_id=1557&type=bug
Issue History
2012-03-19 03:55 alex Change Log Message => Fixes "*" not displayed after password fields during registration
2012-03-19 03:54 alex Estimate Points => 1
2012-03-19 03:54 alex File Added: password_logic_change.patch
2009-10-03 07:56 administrator Note Added: 0000730
2009-10-03 07:56 administrator Status resolved => closed
2009-09-18 07:18 alex Type bug report => feature request
2009-09-18 07:18 alex Reproducibility always => N/A
2009-08-15 10:54 alex Fixed in Version => 5.0.1
2009-08-15 10:54 alex Changeset attached 5.0.x r12296
2009-08-15 10:54 alex Note Added: 0000320
2009-08-15 10:54 alex Status reviewed and tested => resolved
2009-08-15 10:54 alex Resolution open => fixed
2009-08-14 10:58 Dmitry Assigned To => alex
2009-08-14 10:58 Dmitry Status active => reviewed and tested
2009-08-14 10:58 Dmitry Category Admin Interfaces => Front End
2009-08-14 10:58 Dmitry version 5.0.1 => 5.0.0
2009-08-14 10:58 Dmitry Target Version => 5.0.1
2009-08-14 01:06 alex New Issue

Notes
(0000320)
alex   
2009-08-15 10:54   
Fix committed to 5.0.x branch. Commit Message:

Fixes 0000227: Password required mark logic change
(0000730)
administrator   
2009-10-03 07:56   
Closing issues from 5.0.1 version, because version was already released.