Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0035031Retail ModulesWeb POSpublic2017-01-23 11:202017-01-23 15:19
Reporterinaki_garcia 
Assigned ToRetail 
PrioritynormalSeverityminorReproducibilityalways
StatusnewResolutionopen 
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
Summary0035031: [UX][RR17Q1] Incomplete field validation for the Amount to Layaway popup of the Pay Open Tickets option
DescriptionThere are several missing validations for field of the Amount To Layaway popup, which appears when clicking on a Layaway on the "Pay Open Tickets" screen/option (check screenshots). Here are the lists of items which are already validated, and which should be

[1] Already validated inputs ('Invalid input' popup shows as it should, check screenshot):
- Negative amount
- Amount higher than the "Remaining to Pay" of the selected Layaway
- [1.3] Calculating percentages with non-digit non valid characters in the input (this functionality must be mentioned in http://wiki.openbravo.com/wiki/Retail:Web_POS_User_Guide#Layaway_a_order [^])

[2] Not validated inputs and their results:
- [2.1] One or more non-digit and not '%' characters and spaces (non valid characters) preceding or preceding and following a correct value resets the Amount to Layaway
- [2.2] One or more non-digit and not '%' characters and spaces (non valid characters) following a correct value updates the value correctly
- [2.3] '%' character to set the Amount to Layaway as a percentage of the Remaining to Pay can be placed preceding or following a correct value
Steps To Reproduce0. Log in the Web POS as user "vallblanca"

1. Create more than one Layaway with the following steps
 1.1. Click on the Browse button and select some products (p.e., "Avalanche transceiver" and "Headlamp ultralight")
 1.2. Click on the Total button (should be shown as "169.40")
 1.3. Open the dropdown menu and select the "Layaway this Receipt" option
 1.4. Click on the "Layaway" button to complete the receipt

2. Select "Pay Open Tickets" option from the dropdown menu
 2.1. Click on the search magnifying glass and select all the created layaways
 2.2. Click on "Done".

Testing the validations mentioned in the description on the "Pay Open Tickets" screen
3. Click on a Layaway to open the "Amount to Layaway" popup | Set the values and click on 'Apply'
 3.1. Type in '&%50', '%50&', '&50%' , '50%&', '%50%' to check [1.3]
 3.2. Type in '&40', '&40.', '&40.1' '&40.15', '&40&', '&40.&', '&40.1&' '&40.15&' to check [2.1]
 3.3. Type in '40&', '40.&', '40.1&' '40.15&' to check [2.2]
 3.4. Type in '%35', '35%' to check [2.3]
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
related to defect 0035035 closed rqueralta [UX][RR17Q1] Decimal format isn't validated in the field of the Amount to Layaway popup of the Pay Open Tickets window 
Attached Filespng payopentickets_screen.png (140,323) 2017-01-23 11:28
https://issues.openbravo.com/file_download.php?file_id=10334&type=bug
png

png amounttolayaway_popup.png (112,375) 2017-01-23 11:28
https://issues.openbravo.com/file_download.php?file_id=10335&type=bug
png

png invalidinput_popup.png (12,174) 2017-01-23 11:28
https://issues.openbravo.com/file_download.php?file_id=10336&type=bug
png
Issue History
Date ModifiedUsernameFieldChange
2017-01-23 11:20inaki_garciaNew Issue
2017-01-23 11:20inaki_garciaAssigned To => Retail
2017-01-23 11:20inaki_garciaTriggers an Emergency Pack => No
2017-01-23 11:28inaki_garciaFile Added: payopentickets_screen.png
2017-01-23 11:28inaki_garciaFile Added: amounttolayaway_popup.png
2017-01-23 11:28inaki_garciaFile Added: invalidinput_popup.png
2017-01-23 15:00inaki_garciaSummary[RR17Q1] Incomplete field validation for the Amount to Layaway popup of the Pay Open Tickets option => [UX][RR17Q1] Incomplete field validation for the Amount to Layaway popup of the Pay Open Tickets option
2017-01-23 15:05inaki_garciaDescription Updatedbug_revision_view_page.php?rev_id=14336#r14336
2017-01-23 15:17inaki_garciaRelationship addedrelated to 0035035
2017-01-23 15:19inaki_garciaSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=14338#r14338

There are no notes attached to this issue.