Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0036664Retail ModulesWeb POSpublic2017-08-18 09:272017-09-15 13:22
Reportermarvintm 
Assigned Toguilleaer 
PriorityimmediateSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in VersionRR17Q4 
Merge Request Status
Review Assigned Tomarvintm
OBNetwork customer
Support ticket
Regression levelPackaging and release
Regression date2017-04-12
Regression introduced in releasemain
Regression introduced by commithttps://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/c0b32b2cd2e2 [^]
Triggers an Emergency PackNo
Summary0036664: Problem when using manual roles when having the sessions module installed
DescriptionThere is a problem when trying to log in the Web POS when using a manual role, if the Sessions module is installed. The login fails with the following error:

"Window access denied"

(see attached screenshot).
Steps To Reproduce- Install Retail and the sessions module
- Go to the terminal PS-1
- Try to login with user posets. Verify that the mentioned error appears.
Proposed SolutionThe cause of the problem is the following lines in the ob-terminal-model.js file:

if (destinationWindow.permission) {
        if (!OB.MobileApp.model.hasPermission(windowName)) {

The call to hasPermission should have "destinationWindow.permission" as a parameter, and not "windowName".
Additional Information
TagsNo tags attached.
Relationships
depends on backport 0036665RR17Q3 closed guilleaer Problem when using manual roles when having the sessions module installed 
Attached Filespng image.png (74,684) 2017-08-18 09:28
https://issues.openbravo.com/file_download.php?file_id=10986&type=bug
png
Issue History
Date ModifiedUsernameFieldChange
2017-08-18 09:27marvintmNew Issue
2017-08-18 09:27marvintmAssigned To => Retail
2017-08-18 09:27marvintmTriggers an Emergency Pack => No
2017-08-18 09:28marvintmFile Added: image.png
2017-08-18 09:28marvintmRegression level => Packaging and release
2017-08-18 09:28marvintmRegression date => 2017-04-12
2017-08-18 09:28marvintmRegression introduced in release => main
2017-08-18 09:28marvintmRegression introduced by commit => https://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/c0b32b2cd2e2 [^]
2017-08-18 09:28marvintmStatusnew => scheduled
2017-08-18 09:28marvintmStatusscheduled => acknowledged
2017-09-04 13:20hgbotCheckin
2017-09-04 13:20hgbotNote Added: 0098805
2017-09-04 13:23guilleaerNote Deleted: 0098805
2017-09-04 13:31guilleaerStatusacknowledged => scheduled
2017-09-05 10:09hgbotCheckin
2017-09-05 10:09hgbotNote Added: 0098823
2017-09-05 10:09hgbotStatusscheduled => resolved
2017-09-05 10:09hgbotResolutionopen => fixed
2017-09-05 10:09hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/548cb946c56637366533e3d21774e8244c612421 [^]
2017-09-06 09:31marvintmAssigned ToRetail => guilleaer
2017-09-15 13:22marvintmReview Assigned To => marvintm
2017-09-15 13:22marvintmStatusresolved => closed
2017-09-15 13:22marvintmFixed in Version => RR17Q4

Notes
(0098823)
hgbot   
2017-09-05 10:09   
Repository: erp/pmods/org.openbravo.mobile.core
Changeset: 548cb946c56637366533e3d21774e8244c612421
Author: Guillermo Alvarez de Eulate <guillermo.alvarez <at> openbravo.com>
Date: Tue Sep 05 08:38:39 2017 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/548cb946c56637366533e3d21774e8244c612421 [^]

Fixed issue 36664: Property Permission instead of prperty route will be checked to give acces to a certain window

---
M web/org.openbravo.mobile.core/source/component/ob-menu.js
M web/org.openbravo.mobile.core/source/model/ob-terminal-model.js
---