Openbravo Issue Tracking System - POS2
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0058018POS2POSpublic2025-02-18 15:302025-02-19 11:44
Reporteralostale 
Assigned ToTriage Platform Base 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version 
Merge Request Status
Review Assigned To
OBNetwork customer
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0058018: Force Full Refresh Process silently fails if it lacks some privileges
DescriptionIf "Force Full Refresh Process" is executed by a role without write access to the Terminal entity, the process fails without providing feedback to the user.
Steps To Reproduce1. Create a manual role with read only access to Channel - Touchpoint window
2. Log in with that role
3. Open Channel Touchpoint window
4. Select a terminal and execute Force Full Refresh Process
  ERROR:
    * The popup remains in "Processing" forever
    * In the openbravo.log the cause of the problem is explained (not having write permissions to Terminal entity).

See https://openbravo.atlassian.net/browse/RM-20984 [^]
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2025-02-18 15:30alostaleNew Issue
2025-02-18 15:30alostaleAssigned To => Retail
2025-02-18 15:30alostaleTriggers an Emergency Pack => No
2025-02-18 15:31alostaleSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=29260#r29260
2025-02-18 15:34alostaleSummaryForce Full Refresh Process silently fail if it lacks some privileges => Force Full Refresh Process silently fails if it lacks some privileges
2025-02-18 15:38hgbotNote Added: 0175832
2025-02-19 08:47hgbotNote Added: 0175853
2025-02-19 09:13guillermogilAssigned ToRetail => Triage Platform Base
2025-02-19 11:44hgbotNote Added: 0175874
2025-02-19 11:44hgbotNote Added: 0175875
2025-02-19 11:44hgbotNote Added: 0175876
2025-02-19 11:44hgbotResolutionopen => fixed
2025-02-19 11:44hgbotStatusnew => closed
2025-02-19 11:44hgbotFixed in Version => RR25Q2
2025-02-19 11:44hgbotNote Added: 0175877

Notes
(0175832)
hgbot   
2025-02-18 15:38   
Merge Request created: https://gitlab.com/orisha-group/bu-commerce/openbravo/product/pmods/org.openbravo.retail.posterminal/-/merge_requests/1924 [^]
(0175853)
hgbot   
2025-02-19 08:47   
Merge Request created: https://gitlab.com/orisha-group/bu-commerce/openbravo/ci/mobile-api/-/merge_requests/40 [^]
(0175874)
hgbot   
2025-02-19 11:44   
Merge request merged: https://gitlab.com/orisha-group/bu-commerce/openbravo/ci/mobile-api/-/merge_requests/40 [^]
(0175875)
hgbot   
2025-02-19 11:44   
Repository: https://gitlab.com/orisha-group/bu-commerce/openbravo/ci/mobile-api [^]
Changeset: f4cd73af1b25f3f7a4bed17947392401dec8f415
Author: Asier Lostalé <a.lostale@orisha.com>
Date: 19-02-2025 08:43:50
URL: https://gitlab.com/orisha-group/bu-commerce/openbravo/ci/mobile-api/-/commit/f4cd73af1b25f3f7a4bed17947392401dec8f415 [^]

BUG-58018: API ForceFullRefreshActionHandler extends BaseProcessActionHandler

ForceFullRefreshActionHandler extends BaseProcessActionHandler instead of
BaseActionHandler, this changes execute method from non-final to final.

---
M java/reference/java.japi.gz
---
(0175876)
hgbot   
2025-02-19 11:44   
Merge request merged: https://gitlab.com/orisha-group/bu-commerce/openbravo/product/pmods/org.openbravo.retail.posterminal/-/merge_requests/1924 [^]
(0175877)
hgbot   
2025-02-19 11:44   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/orisha-group/bu-commerce/openbravo/product/pmods/org.openbravo.retail.posterminal [^]
Changeset: f9b95a1b3abcc5a3e2a4a574d603ebd0fb4844da
Author: Asier Lostalé <a.lostale@orisha.com>
Date: 19-02-2025 10:44:09
URL: https://gitlab.com/orisha-group/bu-commerce/openbravo/product/pmods/org.openbravo.retail.posterminal/-/commit/f9b95a1b3abcc5a3e2a4a574d603ebd0fb4844da [^]

fixes BUG-58018: Force Full Refresh silently fails if it lacks some privileges

Changed ForceFullRefreshActionHandler to extend BaseProcessActionHandler instead
of BaseActionHandler.

This will make:

  * The process to be execued in admin mode, so the reported case will not
    fail.
  * In case of other failures, a proper error will be displayed in the ui
    instead of hanging the application.
  * Further permissions checks are performed.

---
M src/org/openbravo/retail/posterminal/ForceFullRefreshActionHandler.java
---