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

View Issue Details[ Jump to Notes ] [ Issue History ] [ Print ]
ID
0035945
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Retail Modules] Web POSmajoralways2017-05-08 17:252017-05-25 18:56
ReporteraaroncaleroView Statuspublic 
Assigned Tojorge-garcia 
PriorityhighResolutionfixedFixed in VersionRR17Q3
StatusclosedFix in branchFixed in SCM revision4af36cd4c394
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Merge Request Status
Review Assigned Tomarvintm
OBNetwork customerOBPS
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0035945: Extra Product requests done using Multipricelist functionality

DescriptionWhen the Multipricelist functionality is configured, each customer can have its own Price List in webpos.
Since different price lists can contain different prices for the same product, every time the current price list changes, a Product search is fired, applying the same filters than the current search.
However this search is fired even if the webpos is not currently showing a product search result (i.e. on the Browse or Search tabs), and in these cases an empty search (without any filter) is fired; when the Remote preference for Products is enabled, these search requests are done against the server.
Steps To ReproduceLogin in backend
Go to the preference window
Configure the 'Web POS Enable Multi Price List' preference with a value of 'Y'.
Configure the 'Enable Remote for Product' preference with a value of 'Y'.

Go to the webpos and open the Network tab on the developer console.
Login in the application.
During the end of the login process, a request to org.openbravo.retail.posterminal.master.Product will be done
Proposed SolutionBoth the Browse tab and the Search tab have an event handler for the onChangePricelist event. One possible solution is to verify the current tab (OB.MobileApp.model.get('lastPaneShown')) before executing the action inside the handler.
TagsNo tags attached.
Attached Files

- Relationships [ Relation Graph ] [ Dependency Graph ]
related to defect 0035485 closedgorka_gil Doing login in a multiprice list terminal with remote configured, do a product query without filters that is not needed 

-  Notes
(0096499)
hgbot (developer)
2017-05-10 08:39

 Repository: erp/pmods/org.openbravo.mobile.core
Changeset: 4af36cd4c3947b8b30db4f9597d93e737d37073f
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Tue May 09 12:00:00 2017 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/4af36cd4c3947b8b30db4f9597d93e737d37073f [^]

Fixed issue 35945: Extra Product requests done using Multipricelist
functionality

The solution is to avoid product search on price list change. Also, the list
of products should be reset in this case.

---
M web/org.openbravo.mobile.core/source/retail/component/ob-retail-searchproductcharacteristic.js
---
(0096656)
marvintm (viewer)
2017-05-18 09:17

 Two changes should be made:
- Instead of checking the permissions array directly, the hasPermissions API should be used.
- The code should be simplified. Instead of doing a big if, and then copying the call except for one property, a single call should be done with the property defined as the preference value:

 this.doSearchAction({
        productCat: this.$.searchProductCharacteristicHeader.$.productcategory.getValue() || '__all__',
        productName: this.$.searchProductCharacteristicHeader.$.productFilterText.getValue() || '',
        filter: this.model.get('filter'),
        skipProduct: OB.MobileApp.model.hasPermission("OBPOS_remote.product", true),
        skipProductCharacteristic: false
      });
(0096739)
hgbot (developer)
2017-05-24 08:46

 Repository: erp/pmods/org.openbravo.mobile.core
Changeset: 62af222b1afe86ac5d790be9d4fd0125982101bb
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Fri May 19 09:48:04 2017 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/62af222b1afe86ac5d790be9d4fd0125982101bb [^]

Related issue 35945: Extra Product requests done using Multipricelist
functionality

* Used API to check permissions.

* The code has been simplified.

---
M web/org.openbravo.mobile.core/source/retail/component/ob-retail-searchproductcharacteristic.js
---

- Issue History
Date Modified Username Field Change
2017-05-08 17:25 aaroncalero New Issue
2017-05-08 17:25 aaroncalero Assigned To => Retail
2017-05-08 17:25 aaroncalero OBNetwork customer => Yes
2017-05-08 17:25 aaroncalero Resolution time => 1495404000
2017-05-08 17:25 aaroncalero Triggers an Emergency Pack => No
2017-05-08 17:26 aaroncalero Relationship added related to 0035383
2017-05-09 09:54 jorge-garcia Status new => scheduled
2017-05-09 09:54 jorge-garcia Assigned To Retail => jorge-garcia
2017-05-10 08:39 hgbot Checkin
2017-05-10 08:39 hgbot Note Added: 0096499
2017-05-10 08:39 hgbot Status scheduled => resolved
2017-05-10 08:39 hgbot Resolution open => fixed
2017-05-10 08:39 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/4af36cd4c3947b8b30db4f9597d93e737d37073f [^]
2017-05-18 09:17 marvintm Note Added: 0096656
2017-05-18 09:17 marvintm Status resolved => new
2017-05-18 09:17 marvintm Resolution fixed => open
2017-05-19 09:31 jorge-garcia Status new => scheduled
2017-05-24 08:46 hgbot Checkin
2017-05-24 08:46 hgbot Note Added: 0096739
2017-05-24 08:46 jorge-garcia Status scheduled => resolved
2017-05-24 08:46 jorge-garcia Resolution open => fixed
2017-05-25 18:56 marvintm Review Assigned To => marvintm
2017-05-25 18:56 marvintm Status resolved => closed
2017-05-25 18:56 marvintm Fixed in Version => RR17Q3
2017-06-21 17:37 Sandrahuguet Relationship added related to 0035485
2017-06-21 17:37 Sandrahuguet Relationship deleted related to 0035383

Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker