Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0031118Retail ModulesWeb POSpublic2015-10-09 10:242015-10-19 14:27
Reportermalsasua 
Assigned ToOrekaria 
PriorityhighSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS20OS VersionCommunity Appliance
Product Versionpi 
Target VersionFixed in VersionRR16Q1 
Merge Request Status
Review Assigned Tomarvintm
OBNetwork customerOBPS
Support ticket37586
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0031118: terminal authentication is not working with a role without permission to void layaway
Descriptionif you do a login with a role without permission to void a layaway, the thermintal authentification is not working.
Error is returned when receipt is paid:

serviceSuccess error: status: -1, error.message: Current user/role has no access to class org.openbravo.retail.posterminal.CheckTerminalAuth; line: serviceSuccess (http://196.201.20.234:50087/obtest/web/js/gen/1af912c5b4d8fca2b421331f55463f62_WebPOS.js:38:4 [^]) (*,+)
Steps To Reproduce. enable terminal authentication:
 preference: "Terminal Authentication enabled" to Y - Client: white valley

. disable the permission to void layaway
 preference with "Web POS action void layaway" to N - Client White Valley
. go to WEB POS -> store PS-1
. do login with posets
. do a receipt -> never is synced.
In console, error is returned:
serviceSuccess error: status: -1, error.message: Current user/role has no access to class org.openbravo.retail.posterminal.CheckTerminalAuth; line: serviceSuccess (https://livebuilds.openbravo.com/retail_pi_pgsql/web/js/gen/3c64eb4adb95408ed989f0d4f1dd6a1b_WebPOS.js:1033:10 [^]) (*,+)
Proposed Solutionfile: org.openbravo.retail.posterminal/src/org/openbravo/retail/posterminal/CheckTerminalAuth.java
protected String getProperty() {
    return "OBPOS_receipt.voidLayaway";
  }

should be replaced by:
protected String getProperty() {
   return "";
}
Additional Information
TagsNo tags attached.
Relationships
related to defect 0031120 closed fmolina autobom module is not working when there are warehouse rules with type PInstance 
Attached Filesdiff patchTermAuth.diff (401) 2015-10-09 13:51
https://issues.openbravo.com/file_download.php?file_id=8554&type=bug
Issue History
Date ModifiedUsernameFieldChange
2015-10-09 10:24malsasuaNew Issue
2015-10-09 10:24malsasuaAssigned To => Retail
2015-10-09 10:24malsasuaOBNetwork customer => Yes
2015-10-09 10:24malsasuaSupport ticket => 37586
2015-10-09 10:24malsasuaResolution time => 1445032800
2015-10-09 10:24malsasuaTriggers an Emergency Pack => No
2015-10-09 10:39OrekariaStatusnew => acknowledged
2015-10-09 10:58malsasuaRelationship addedrelated to 0031120
2015-10-09 13:51malsasuaFile Added: patchTermAuth.diff
2015-10-09 16:14hgbotCheckin
2015-10-09 16:14hgbotNote Added: 0080925
2015-10-09 16:14hgbotStatusacknowledged => resolved
2015-10-09 16:14hgbotResolutionopen => fixed
2015-10-09 16:14hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/a111297a3369db3a56f3da92366fdb730a7264e4 [^]
2015-10-13 09:45marvintmAssigned ToRetail => Orekaria
2015-10-13 12:17OrekariaReview Assigned To => marvintm
2015-10-19 11:59hgbotCheckin
2015-10-19 11:59hgbotNote Added: 0081070
2015-10-19 11:59marvintmStatusresolved => closed
2015-10-19 11:59marvintmFixed in Version => RR16Q1
2015-10-19 14:27hgbotCheckin
2015-10-19 14:27hgbotNote Added: 0081090

Notes
(0080925)
hgbot   
2015-10-09 16:14   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: a111297a3369db3a56f3da92366fdb730a7264e4
Author: Rafa Alonso <ral <at> openbravo.com>
Date: Fri Oct 09 15:52:49 2015 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/a111297a3369db3a56f3da92366fdb730a7264e4 [^]

Fixes issue 31118: There is no permission required for terminal authentication

---
M src/org/openbravo/retail/posterminal/CheckTerminalAuth.java
---
(0081070)
hgbot   
2015-10-19 11:59   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 31e085fb0546f87a211a02d4c54511e798316533
Author: Antonio Moreno <antonio.moreno <at> openbravo.com>
Date: Mon Oct 19 11:59:07 2015 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/31e085fb0546f87a211a02d4c54511e798316533 [^]

Related to issue 31118. Removed unnecessary method overrite

---
M src/org/openbravo/retail/posterminal/CheckTerminalAuth.java
---
(0081090)
hgbot   
2015-10-19 14:27   
Repository: tools/automation/pi-mobile
Changeset: 9ea0ecfc418bc9c581ea622a2677f6bcc92771bc
Author: Rafa Alonso <ral <at> openbravo.com>
Date: Tue Oct 13 15:53:32 2015 +0200
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/9ea0ecfc418bc9c581ea622a2677f6bcc92771bc [^]

Verifies issue 31118: Add the 'I31118_PermissionToAuthRequired' test

---
A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/loginout/I31118_PermissionToAuthRequired.java
---