Openbravo Issue Tracking System - Openbravo ERP | |||||
View Issue Details | |||||
ID | Project | Category | View Status | Date Submitted | Last Update |
0013653 | Openbravo ERP | 01. General setup | public | 2010-06-14 19:03 | 2011-10-28 20:17 |
Reporter | shuehner | ||||
Assigned To | jonalegriaesarte | ||||
Priority | normal | Severity | minor | Reproducibility | have not tried |
Status | closed | Resolution | out of date | ||
Platform | OS | 5 | OS Version | ||
Product Version | pi | ||||
Target Version | Fixed in Version | pi | |||
Merge Request Status | |||||
Review Assigned To | |||||
OBNetwork customer | |||||
Web browser | |||||
Modules | Core | ||||
Support ticket | |||||
Regression level | |||||
Regression date | |||||
Regression introduced in release | |||||
Regression introduced by commit | |||||
Triggers an Emergency Pack | No | ||||
Summary | 0013653: Multiple issues in selectors introduced by PagedDatagrid project | ||||
Description | This issue collects multiple problem most likely introduced by copy'n'paste in the Paged-Datagrid project. Some or style issues only (no functional impact), other are defects All are about naming of the session variable used to store the 'currentPage' (backendPage) variable which is used to track the current backend page (a page is 10k records by default). List created with: src/org/openbravo/erpCommon/info$ grep -Ir currentPage . | less Style issues means: 'Wrong' name if session-variable (not matching selector name) but not functional impact Defect means: Multiple uses of different names in the same selector -> must use same name (when fixing also check for style issue together with those) Style issues: ./ProductComplete.java: page = TableSQLData.calcAndGetBackendPage(vars, "ProjectData.currentPage"); ./Account.java: page = TableSQLData.calcAndGetBackendPage(vars, "DebtPaymentInfo.currentPage"); ./AccountElementValue.java: page = TableSQLData.calcAndGetBackendPage(vars, "ShipmentReceiptLine.currentPage"); ./BusinessPartner.java: page = TableSQLData.calcAndGetBackendPage(vars, "DebtPaymentInfo.currentPage"); Defects: ./ProductMultiple.java: page = TableSQLData.calcAndGetBackendPage(vars, "ProjectData.currentPage"); ./ProductMultiple.java: String strPage = vars.getSessionValue("ProductMultiple|currentPage", "0"); ./ProjectMultiple.java: page = TableSQLData.calcAndGetBackendPage(vars, "BusinessPartnerData.currentPage"); ./ProjectMultiple.java: String strPage = vars.getSessionValue("ProductMultiple|currentPage", "0"); ./BusinessPartnerMultiple.java: page = TableSQLData.calcAndGetBackendPage(vars, "BusinessPartnerData.currentPage"); ./BusinessPartnerMultiple.java: String strPage = vars.getSessionValue("ProductMultiple|currentPage", "0"); | ||||
Steps To Reproduce | |||||
Proposed Solution | |||||
Additional Information | |||||
Tags | No tags attached. | ||||
Relationships | |||||
Attached Files | |||||
Issue History | |||||
Date Modified | Username | Field | Change | ||
2010-06-14 19:03 | shuehner | New Issue | |||
2010-06-14 19:03 | shuehner | Assigned To | => adrianromero | ||
2011-06-03 10:59 | dalsasua | Assigned To | adrianromero => dalsasua | ||
2011-07-20 18:12 | dalsasua | Assigned To | dalsasua => jonalegriaesarte | ||
2011-10-28 20:17 | dmitry_mezentsev | Note Added: 0042307 | |||
2011-10-28 20:17 | dmitry_mezentsev | Status | new => closed | ||
2011-10-28 20:17 | dmitry_mezentsev | Resolution | open => out of date | ||
2011-10-28 20:17 | dmitry_mezentsev | Fixed in Version | => pi |
Notes | |||||
|
|||||
|
|