Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0031796Openbravo ERP01. General setuppublic2015-12-30 09:472016-02-29 16:19
Reporterjonibc 
Assigned Tojonibc 
PrioritynormalSeverityminorReproducibilityN/A
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version3.0PR16Q2 
Merge Request Status
Review Assigned Toalostale
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0031796: Extend functionality with the option to define a day limit for the password to be changed
DescriptionExtend core functionality to allow admin set a day limit for passwords to expire. The limit will be reset every time a user changes his password.

The implementation should works for the backend and WebPOS.
Documentation: http://wiki.openbravo.com/wiki/Projects:Change_password [^]
Steps To ReproduceN/A
Proposed SolutionModify AuthenticationManager to check if user limit day has been reached. Also implementation in ERP and WebPOS user interface is needed to enter and confirm the new password.

Added daysToPasswordExpiration column in AD_Client table and lastUpdatePasswordDate column in AD_User. Also fields have been added to their respective windows.
Additional Information
TagsNo tags attached.
Relationships
related to defect 0034863 closed inigosanchez "Your password has expired" message while logging in is not properly translated to the language of the application 
related to defect 0039919 closed alostale A change of Password leaves a session as Active in BackOffice 
related to defect 0049411 scheduled guillermogil Password expiration is not checked in the web service authentication flow 
blocks feature request 00322853.0PR16Q3 closed NaroaIriarte make possible to manually set a user as password expired 
Attached Filesdiff test.diff (6,641) 2016-02-24 08:57
https://issues.openbravo.com/file_download.php?file_id=9096&type=bug
Issue History
Date ModifiedUsernameFieldChange
2015-12-30 09:47jonibcNew Issue
2015-12-30 09:47jonibcAssigned To => jonibc
2015-12-30 09:47jonibcModules => Core
2015-12-30 09:47jonibcTriggers an Emergency Pack => No
2015-12-30 10:10jonibcDescription Updatedbug_revision_view_page.php?rev_id=10559#r10559
2016-01-20 08:41guillermogilReview Assigned To => alostale
2016-02-19 10:05alostaleRelationship addedblocks 0032285
2016-02-23 15:37guillermogilFile Added: test.diff
2016-02-23 15:37guillermogilNote Added: 0084435
2016-02-24 08:45guillermogilFile Deleted: test.diff
2016-02-24 08:45guillermogilFile Added: test.diff
2016-02-24 08:57guillermogilFile Deleted: test.diff
2016-02-24 08:57guillermogilFile Added: test.diff
2016-02-26 08:24guillermogilStatusnew => scheduled
2016-02-26 08:25guillermogilNote Added: 0084534
2016-02-26 08:25guillermogilStatusscheduled => resolved
2016-02-26 08:25guillermogilFixed in SCM revision => 16PRQ2
2016-02-26 08:25guillermogilResolutionopen => fixed
2016-02-26 08:25guillermogilFixed in SCM revision16PRQ2 => 3.0PR16Q2
2016-02-26 10:03alostaleNote Added: 0084554
2016-02-26 10:03alostaleStatusresolved => closed
2016-02-26 10:03alostaleFixed in Version => 3.0PR16Q2
2016-02-26 12:55hgbotCheckin
2016-02-26 12:55hgbotNote Added: 0084560
2016-02-29 16:17hgbotCheckin
2016-02-29 16:17hgbotNote Added: 0084603
2016-02-29 16:19hgbotCheckin
2016-02-29 16:19hgbotNote Added: 0084604
2017-01-09 12:59alostaleRelationship addedrelated to 0034863
2019-01-11 10:06alostaleRelationship addedrelated to 0039919
2022-05-25 13:29caristuRelationship addedrelated to 0049411

Notes
(0084435)
guillermogil   
2016-02-23 15:37   
Added diff with Test Case
(0084534)
guillermogil   
2016-02-26 08:25   
Branches already merged:
https://code.openbravo.com/erp/devel/pi/rev/f2e7ebc3ea7f [^]
https://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/4556c6a80aeb [^]
(0084554)
alostale   
2016-02-26 10:03   
Code reviewed and tested as part of the project
(0084560)
hgbot   
2016-02-26 12:55   
Repository: tools/automation/pi-mobile
Changeset: bc6ffa09bba64f3fee1f15effb264cb0c5f240d4
Author: Guillermo Gil <guillermo.gil <at> openbravo.com>
Date: Fri Feb 26 12:55:31 2016 +0100
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/bc6ffa09bba64f3fee1f15effb264cb0c5f240d4 [^]

Related to issue 31796: Add test case for ChangePassword

---
M src-test/org/openbravo/test/mobile/common/selenium/javascript/TestId.java
M src-test/org/openbravo/test/mobile/common/selenium/utils/AllowedErrorsHelper.java
A src-test/org/openbravo/test/mobile/quarantine/pack/GIG/I31796_ChangePassword.java
---
(0084603)
hgbot   
2016-02-29 16:17   
Repository: tools/automation/pi-mobile
Changeset: b2a445fdc3ec916d9622b5326583fffdc28c9814
Author: Guillermo Gil <guillermo.gil <at> openbravo.com>
Date: Mon Feb 29 16:06:59 2016 +0100
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/b2a445fdc3ec916d9622b5326583fffdc28c9814 [^]

Related to issue 31796: Add test case for ChangePassword

Applied test code review
Remove UI commonds form the finally, only execute the queries

---
M src-test/org/openbravo/test/mobile/quarantine/pack/GIG/I31796_ChangePassword.java
---
(0084604)
hgbot   
2016-02-29 16:19   
Repository: tools/automation/pi-mobile
Changeset: 6f8a6cc5e9a73e2a530fc0feff0459d52bef3ed3
Author: Guillermo Gil <guillermo.gil <at> openbravo.com>
Date: Mon Feb 29 16:18:57 2016 +0100
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/6f8a6cc5e9a73e2a530fc0feff0459d52bef3ed3 [^]

Related to issue 31796: Add test case for ChangePassword

Added missing variable

---
M src-test/org/openbravo/test/mobile/quarantine/pack/GIG/I31796_ChangePassword.java
---