Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0034428Retail ModulesWeb POSpublic2016-11-04 11:122016-11-11 14:03
Reporterjorge-garcia 
Assigned Tomario_castello 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionRR16Q2.4Fixed in Version 
Merge Request Status
Review Assigned Tomigueldejuana
OBNetwork customerNo
Support ticket
Regression level
Regression date2016-09-19
Regression introduced in releaseRR16Q2.3
Regression introduced by commithttps://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/200182f3db29 [^]
Triggers an Emergency PackNo
Summary0034428: Login window is loaded even if the terminal does not exist in the database
DescriptionLogin window is loaded even if the terminal does not exist in the database.
Steps To ReproduceTry to login here

http://localhost:8080/openbravo/web/org.openbravo.retail.posterminal/?terminal=PEPE#login [^]

Wrong: The login window is rendered.
Solution: The login window should not be rendered and a popup should be shown indication the terminal name doesn't exist (check screenshot attached).

This is happening since 16Q2.3.
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
blocks defect 0034396 closed mario_castello Login window is loaded even if the terminal does not exist in the database 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2016-11-08 15:42mario_castelloTypedefect => backport
2016-11-08 15:42mario_castelloTarget Version => RR16Q2.4
2016-11-11 08:49hgbotCheckin
2016-11-11 08:49hgbotNote Added: 0091355
2016-11-11 08:49hgbotStatusscheduled => resolved
2016-11-11 08:49hgbotResolutionopen => fixed
2016-11-11 08:49hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/832ae441d73b3fcbc628d1a28ee321f5ae62010c [^] => http://code.openbravo.com/retail/backports/3.0RR16Q2.4/org.openbravo.retail.posterminal/rev/4afcff794f23b3606cd5c44bdec24b5b9260db7a [^]
2016-11-11 14:03migueldejuanaReview Assigned To => migueldejuana
2016-11-11 14:03migueldejuanaNote Added: 0091400
2016-11-11 14:03migueldejuanaStatusresolved => closed

Notes
(0091355)
hgbot   
2016-11-11 08:49   
Repository: retail/backports/3.0RR16Q2.4/org.openbravo.retail.posterminal
Changeset: 4afcff794f23b3606cd5c44bdec24b5b9260db7a
Author: Mario Castello <mario.castello <at> peoplewalking.com>
Date: Fri Nov 04 16:37:03 2016 -0600
URL: http://code.openbravo.com/retail/backports/3.0RR16Q2.4/org.openbravo.retail.posterminal/rev/4afcff794f23b3606cd5c44bdec24b5b9260db7a [^]

Fixed issue 34428, backport of issue 34396: Added validation for the terminal list to prevent throwed an unhandled exception.

When the terminal not exist the terminalList are empty and the process are throwing an exception when try to get an object from an empty list.

---
M src/org/openbravo/retail/posterminal/LoginUtilsServlet.java
---
(0091400)
migueldejuana   
2016-11-11 14:03   
Tested and reviewed