Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0031617Retail ModulesWeb POSpublic2015-11-27 13:392015-12-14 11:55
Reporterguilleaer 
Assigned Toguilleaer 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionRR16Q1Fixed in Version 
Merge Request Status
Review Assigned Tomigueldejuana
OBNetwork customer
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0031617: When a conflict is detected reading terminal authentication preference the user should be Informed
DescriptionIf you define to preference which create a conflict, the system use always 'N' but user is not informed about this situation
Steps To ReproduceCreate a new preference identical to the one which is in the module. If you add a breakpoint in below catch, you will see that there is an exception when preference is read.

file: LoginUtilsServlet.java
Code:

try {
      value = Preferences.getPreferenceValue("OBPOS_TerminalAuthentication", true, null, null,
          null, null, (String) null);
    } catch (PropertyException e) {
      result.put("terminalAuthentication", "Y");
      return result;
    }
Proposed Solution1. Because preference will be yes by defaut starting from 16Q1, in case of conflict the system should return Y.
2. Apart from that the user should realize that there is a confirguration problem and it should be shown.
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2015-11-27 13:39guilleaerNew Issue
2015-11-27 13:39guilleaerAssigned To => guilleaer
2015-11-27 13:39guilleaerTriggers an Emergency Pack => No
2015-11-27 13:39guilleaerStatusnew => scheduled
2015-11-30 08:48hgbotCheckin
2015-11-30 08:48hgbotNote Added: 0082456
2015-11-30 08:48hgbotStatusscheduled => resolved
2015-11-30 08:48hgbotResolutionopen => fixed
2015-11-30 08:48hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/6ffb30587007e7deaf8131bbe00ffcd7588f9d37 [^]
2015-12-09 10:13OrekariaReview Assigned To => marvintm
2015-12-14 09:04mtaalReview Assigned Tomarvintm => migueldejuana
2015-12-14 11:55migueldejuanaNote Added: 0082704
2015-12-14 11:55migueldejuanaStatusresolved => closed

Notes
(0082456)
hgbot   
2015-11-30 08:48   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 6ffb30587007e7deaf8131bbe00ffcd7588f9d37
Author: Guillermo Alvarez de Eulate <guillermo.alvarez <at> openbravo.com>
Date: Fri Nov 27 13:46:39 2015 +0100
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/6ffb30587007e7deaf8131bbe00ffcd7588f9d37 [^]

Fixed issue 31617: When conflict is detected reading terminal authentication preference the system will inform to the user

---
M src-db/database/sourcedata/AD_MESSAGE.xml
M src/org/openbravo/retail/posterminal/CheckTerminalAuth.java
M src/org/openbravo/retail/posterminal/LoginUtilsServlet.java
M web/org.openbravo.retail.posterminal/js/login/model/login-model.js
---
(0082704)
migueldejuana   
2015-12-14 11:55   
Tested and reviewed in d1d27a88215b