Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0045583Retail ModulesWeb POSpublic2020-12-15 16:462021-01-20 23:58
Reporterucarrion 
Assigned Torqueralta 
PrioritynormalSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in VersionRR21Q2 
Merge Request Statusapproved
Review Assigned To
OBNetwork customerGold
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0045583: Impossible to complete the Terminal Authentication process with a expired password user
DescriptionLet's imagine that a new user has just been created with a standard password in order to let the user set a new one in the first login.

In case this user needs to complete the Terminal Authentication process the system is showing a message informing about that expired password
Steps To ReproduceLogin backend
Go to user window
Create a new user, set a password and check the 'Expired password' flag
Enable the Terminal Authentication process (preference)
Open a new unlinked till and try to login with the just created new user
Check the message displayed by the system
Proposed SolutionFrom a functional point of view I would say that the expired password should be only take into account in the login process and not in the terminal authentication process. Anyway, whatever the decission is, the user must be able to arrive to the login screen where the new password will be requested by the system
Additional Information
TagsNo tags attached.
Relationships
Attached Filespatch Fixed-BUG-45583-Core-19Q3.patch (1,427) 2021-01-20 23:58
https://issues.openbravo.com/file_download.php?file_id=15270&type=bug
patch Fixed-BUG-45583-Posterminal-19Q3.patch (6,582) 2021-01-20 23:58
https://issues.openbravo.com/file_download.php?file_id=15271&type=bug
Issue History
Date ModifiedUsernameFieldChange
2020-12-15 16:46ucarrionNew Issue
2020-12-15 16:46ucarrionAssigned To => Retail
2020-12-15 16:46ucarrionOBNetwork customer => Gold
2020-12-15 16:46ucarrionResolution time => 1609455600
2020-12-15 16:46ucarrionTriggers an Emergency Pack => No
2020-12-16 08:23guillermogilSeverityminor => major
2020-12-23 17:57rqueraltaAssigned ToRetail => rqueralta
2020-12-23 17:58rqueraltaStatusnew => scheduled
2021-01-05 16:30rqueraltaStatusscheduled => resolved
2021-01-05 16:30rqueraltaResolutionopen => fixed
2021-01-05 16:31rqueraltaStatusresolved => closed
2021-01-05 16:31rqueraltaStatusclosed => new
2021-01-05 16:31rqueraltaResolutionfixed => open
2021-01-05 16:31rqueraltaStatusnew => scheduled
2021-01-07 16:37hgbotMerge Request Status => open
2021-01-07 16:37hgbotNote Added: 0125295
2021-01-07 16:37hgbotNote Added: 0125296
2021-01-07 19:48hgbotMerge Request Statusopen => approved
2021-01-07 19:49hgbotResolutionopen => fixed
2021-01-07 19:49hgbotStatusscheduled => closed
2021-01-07 19:49hgbotNote Added: 0125301
2021-01-07 19:49hgbotFixed in Version => RR21Q2
2021-01-07 19:49hgbotNote Added: 0125302
2021-01-07 19:49hgbotNote Added: 0125303
2021-01-07 19:49hgbotNote Added: 0125304
2021-01-07 19:49hgbotNote Added: 0125305
2021-01-08 10:00hgbotNote Added: 0125318
2021-01-08 12:07hgbotNote Added: 0125320
2021-01-08 12:08guilleaerNote Added: 0125321
2021-01-08 12:08guilleaerStatusclosed => new
2021-01-08 12:08guilleaerResolutionfixed => open
2021-01-08 12:08guilleaerFixed in VersionRR21Q2 =>
2021-01-15 04:37hgbotNote Added: 0125473
2021-01-15 04:38hgbotNote Added: 0125474
2021-01-20 12:59hgbotResolutionopen => fixed
2021-01-20 12:59hgbotStatusnew => closed
2021-01-20 12:59hgbotFixed in Version => RR21Q2
2021-01-20 12:59hgbotNote Added: 0125691
2021-01-20 12:59hgbotNote Added: 0125692
2021-01-20 12:59hgbotNote Added: 0125693
2021-01-20 12:59hgbotNote Added: 0125694
2021-01-20 23:58rqueraltaFile Added: Fixed-BUG-45583-Core-19Q3.patch
2021-01-20 23:58rqueraltaFile Added: Fixed-BUG-45583-Posterminal-19Q3.patch

Notes
(0125295)
hgbot   
2021-01-07 16:37   
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/merge_requests/245 [^]
(0125296)
hgbot   
2021-01-07 16:37   
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/merge_requests/123 [^]
(0125301)
hgbot   
2021-01-07 19:49   
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/merge_requests/245 [^]
(0125302)
hgbot   
2021-01-07 19:49   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal [^]
Changeset: 4b2d28744e7b76a7d878cc95a46ac89ca5e89159
Author: Rafael Queralta <rafaelcuba81@gmail.com>
Date: 2021-01-07T15:38:12+00:00
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/commit/4b2d28744e7b76a7d878cc95a46ac89ca5e89159 [^]

Fixed BUG-45583: If the user password expired, change it is required to complete the Terminal Authentication process

---
M src/org/openbravo/retail/posterminal/LoginUtilsServlet.java
M web/org.openbravo.retail.posterminal/js/login/model/login-model.js
---
(0125303)
hgbot   
2021-01-07 19:49   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core [^]
Changeset: 43dcc5f0bef78717e5731b5f1a42707dce16964b
Author: Rafael Queralta <rafaelcuba81@gmail.com>
Date: 2021-01-07T15:38:34+00:00
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/commit/43dcc5f0bef78717e5731b5f1a42707dce16964b [^]

Fixed BUG-45583: Reload location window was required to avoid screen without any component after close the popup.

---
M web/org.openbravo.mobile.core/source/component/dialog/ob-expirationpassword.js
---
(0125304)
hgbot   
2021-01-07 19:49   
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/merge_requests/123 [^]
(0125305)
hgbot   
2021-01-07 19:49   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core [^]
Changeset: a0ad9054009d1c43485593bdadf833e7dc7e0990
Author: Rafael Queralta <rafaelcuba81@gmail.com>
Date: 2021-01-07T13:44:52-05:00
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/commit/a0ad9054009d1c43485593bdadf833e7dc7e0990 [^]

Fixed BUG-45583: Reload location window was required to avoid screen without any component after close the popup.

---
M web/org.openbravo.mobile.core/source/component/dialog/ob-expirationpassword.js
---
(0125318)
hgbot   
2021-01-08 10:00   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core [^]
Changeset: a606f1474afed3546bf9a7ae6eb8047ee3c1bfb0
Author: Guillermo Alvarez de Eulate <guillermo.alvarez@openbravo.com>
Date: 2021-01-08T08:39:31+01:00
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/commit/a606f1474afed3546bf9a7ae6eb8047ee3c1bfb0 [^]

Revert "Fixed BUG-45583: Reload location window was required to avoid screen without any component after close the popup."

This reverts commit a0ad9054009d1c43485593bdadf833e7dc7e0990.

---
M web/org.openbravo.mobile.core/source/component/dialog/ob-expirationpassword.js
---
(0125320)
hgbot   
2021-01-08 12:07   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core [^]
Changeset: 79e4f7012c2a9698137993144c53b4ec90995807
Author: Guillermo Alvarez de Eulate <guillermo.alvarez@openbravo.com>
Date: 2021-01-08T12:07:20+01:00
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/commit/79e4f7012c2a9698137993144c53b4ec90995807 [^]

Revert "Fixed BUG-45583: Reload location window was required to avoid screen without any component after close the popup using cancel"

This reverts commit 43dcc5f0bef78717e5731b5f1a42707dce16964b.

---
M web/org.openbravo.mobile.core/source/component/dialog/ob-expirationpassword.js
---
(0125321)
guilleaer   
2021-01-08 12:08   
Reopened

Reverted these changes which are part of the fix
https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/commit/a606f1474afed3546bf9a7ae6eb8047ee3c1bfb0 [^]
https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/commit/79e4f7012c2a9698137993144c53b4ec90995807 [^]

The fix is not complete because if apply/cancel is pressed a gray screen is reached.
Reload is OK for terminal authentication flow but not for the rest of the places where change password popup is used.
(0125473)
hgbot   
2021-01-15 04:37   
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/merge_requests/132 [^]
(0125474)
hgbot   
2021-01-15 04:38   
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/merge_requests/259 [^]
(0125691)
hgbot   
2021-01-20 12:59   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core [^]
Changeset: ca8a5d9c1c5fa59e894bf7fe82cf934782cf095d
Author: Rafael Queralta <rafaelcuba81@gmail.com>
Date: 2021-01-20T11:57:03+00:00
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/commit/ca8a5d9c1c5fa59e894bf7fe82cf934782cf095d [^]

Fixed BUG-45583: Reload location window was required to restart the online mode

---
M web/org.openbravo.mobile.core/source/component/dialog/ob-expirationpassword.js
---
(0125692)
hgbot   
2021-01-20 12:59   
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/merge_requests/132 [^]
(0125693)
hgbot   
2021-01-20 12:59   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal [^]
Changeset: 7d3a107fe3fcd7bee97bc7ab1cecdd4910dc8025
Author: Rafael Queralta <rafaelcuba81@gmail.com>
Date: 2021-01-20T11:57:05+00:00
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/commit/7d3a107fe3fcd7bee97bc7ab1cecdd4910dc8025 [^]

Fixed BUG-45583: If the user password expired, change it is required to complete the Terminal Authentication process. Change Password popup is shown to force it

---
M web/org.openbravo.retail.posterminal/js/login/model/login-model.js
---
(0125694)
hgbot   
2021-01-20 12:59   
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/merge_requests/259 [^]