Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0037470
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Retail Modules] Web POSmajoralways2017-12-13 15:522018-02-19 18:17
ReporteradrianromeroView Statuspublic 
Assigned Toranjith_qualiantech_com 
PrioritynormalResolutionfixedFixed in VersionRR18Q1
StatusclosedFix in branchFixed in SCM revision
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned Tomarvintm
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0037470: Best sellers module queries fail when changing POS language

DescriptionThe best seller product criteria method throws an error in the javascript console because the terminal ID is not in the request session.

The problem is in the following code lines:

https://code.openbravo.com/erp/pmods/org.openbravo.retail.bestsellers/file/tip/src/org/openbravo/retail/bestsellers/BrandBestSellerHQLCriteria.java#l31 [^]
https://code.openbravo.com/erp/pmods/org.openbravo.retail.bestsellers/file/tip/src/org/openbravo/retail/bestsellers/PCharacteristicBestSellerHQLCriteria.java#l34 [^]
https://code.openbravo.com/erp/pmods/org.openbravo.retail.bestsellers/file/tip/src/org/openbravo/retail/bestsellers/PCharacteristicValueBestSellerHQLCriteria.java#l34 [^]

The problem is that the session attribute "POSTerminal" does not exist in the reload process executed when updating the user profile.

In other places the POS Terminal identifier is requested from the query parameters, for example this changeset modifies how to get the POS Terminal identifier: https://code.openbravo.com/erp/pmods/org.openbravo.retail.digitalcoupons/rev/270c6f796a50 [^]

The problem is that in best sellers criteria methods there is no parameter at all.

Steps To Reproduce* Install module org.openbravo.retail.bestsellers.
* Log in WEB POS terminal VBS-1
* Select in the menu User > Profile
* In the dialog change one option. For example check "Set as Default" and apply.
* Wait for the WEB POS to reload
* Open the developer tools and verify the following errors in the console:

49a7e4275a09208a14dfa75f794f4941.js:3175 serviceSuccess error: status: -1; line: serviceSuccess (http://localhost:8080/openbravo/web/js/gen/49a7e4275a09208a14dfa75f794f4941.js:1148:12 [^]) (*,+)
49a7e4275a09208a14dfa75f794f4941.js:3175 Error in datasource: [object Object]; line: http://localhost:8080/openbravo/web/js/gen/49a7e4275a09208a14dfa75f794f4941.js:1629:14 [^] (*,+)
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]

-  Notes
(0101078)
hgbot (developer)
2017-12-16 08:18

Repository: erp/pmods/org.openbravo.retail.bestsellers
Changeset: a962807edfce9550e41f6f7b0b40efdfd31cfa76
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Sat Dec 16 12:48:45 2017 +0530
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.bestsellers/rev/a962807edfce9550e41f6f7b0b40efdfd31cfa76 [^]

Related to issue 37470 : Replaced session values with Terminal values from request in master data

---
M src/org/openbravo/retail/bestsellers/BestSellersProperties.java
---
(0102500)
hgbot (developer)
2018-02-19 18:17

Repository: erp/pmods/org.openbravo.retail.orderpreparation
Changeset: d27bf512ba1f78833177aad317696b03db05737f
Author: Prakash M <prakash <at> qualiantech.com>
Date: Wed Feb 14 17:04:01 2018 +0530
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.orderpreparation/rev/d27bf512ba1f78833177aad317696b03db05737f [^]

Replaced session values with Terminal values

* Session values will be lost after user profile update
* Similar to issue 37470

---
M src/org/openbravo/retail/orderpreparation/term/OrderPreDocnoPrefixTerminal.java
---

- Issue History
Date Modified Username Field Change
2017-12-13 15:52 adrianromero New Issue
2017-12-13 15:52 adrianromero Assigned To => Retail
2017-12-13 15:52 adrianromero Triggers an Emergency Pack => No
2017-12-13 16:10 adrianromero Summary Best sellers module queries fails when changing POS language => Best sellers module queries fail when changing POS language
2017-12-14 10:37 ranjith_qualiantech_com Assigned To Retail => ranjith_qualiantech_com
2017-12-14 10:37 ranjith_qualiantech_com Status new => scheduled
2017-12-16 08:18 hgbot Checkin
2017-12-16 08:18 hgbot Note Added: 0101078
2017-12-16 12:54 ranjith_qualiantech_com Status scheduled => resolved
2017-12-16 12:54 ranjith_qualiantech_com Resolution open => fixed
2017-12-20 19:00 marvintm Review Assigned To => marvintm
2017-12-20 19:00 marvintm Status resolved => closed
2017-12-20 19:00 marvintm Fixed in Version => RR18Q1
2018-02-19 18:17 hgbot Checkin
2018-02-19 18:17 hgbot Note Added: 0102500


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker