Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue Details[ Jump to Notes ] [ Issue History ] [ Print ]
ID
0047435
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Retail Modules] Web POSmajoralways2021-07-22 09:172021-07-26 09:27
ReportermigueldejuanaView Statuspublic 
Assigned Tomigueldejuana 
PrioritynormalResolutionfixedFixed in VersionRR21Q4
StatusclosedFix in branchFixed in SCM revision
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned Toadrianromero
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0047435: Avoid asking to Login again if we are in a middle of a process

DescriptionIf we do a request and the response es 401 because the session is not valid, we show Login Again popup and we can logout. If we are in the middle of a process and we do logout, we can have bugs.

Example:
We are processing an order and before pressing Done, the server is restarted. In preOrderSave hook a module does a request to backend and we get 401 response, so the Login Again popup is shown and we can press Login Again. We will do logout and the complete ticket has not finished yet. It should not be errors but we should avoid doing this because is risky and could cause bugs if we take into account the differente implementation in clients.
Steps To Reproducen/a
Proposed SolutionLaunch login again popup just if there are no processes in execution
TagsNOR
Attached Files

- Relationships [ Relation Graph ] [ Dependency Graph ]

-  Notes
(0130685)
hgbot (developer)
2021-07-22 09:23

 Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/merge_requests/263 [^]
(0130745)
hgbot (developer)
2021-07-26 09:27

 Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core [^]
Changeset: 3714c37876029f853c481ac5749e5e8e580b79e8
Author: Miguel de Juana <miguel.dejuana@openbravo.com>
Date: 2021-07-22T15:17:27+02:00
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/commit/3714c37876029f853c481ac5749e5e8e580b79e8 [^]

Fixed ISSUE-47435: Avoid asking to Login again if we are in a middle of a process

- We remove if else when response is 401, we will have an if for 401
case and else will be execute always (unless we do return)

- Login Again popup won't be launch if there are processes in execution
to avoid breaking flows and causing bugs

---
M web/org.openbravo.mobile.core/source/data/ob-requestrouter.js
---
(0130746)
hgbot (developer)
2021-07-26 09:27

 Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/merge_requests/263 [^]

- Issue History
Date Modified Username Field Change
2021-07-22 09:17 migueldejuana New Issue
2021-07-22 09:17 migueldejuana Assigned To => migueldejuana
2021-07-22 09:17 migueldejuana Triggers an Emergency Pack => No
2021-07-22 09:17 migueldejuana Status new => scheduled
2021-07-22 09:18 migueldejuana Review Assigned To => adrianromero
2021-07-22 09:23 hgbot Note Added: 0130685
2021-07-22 23:18 rafaroda Tag Attached: NOR
2021-07-26 09:27 hgbot Resolution open => fixed
2021-07-26 09:27 hgbot Status scheduled => closed
2021-07-26 09:27 hgbot Fixed in Version => RR21Q4
2021-07-26 09:27 hgbot Note Added: 0130745
2021-07-26 09:27 hgbot Note Added: 0130746

Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker