Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0033769Openbravo ERPA. Platformpublic2016-08-23 16:332016-08-25 10:33
Reportershuehner 
Assigned Toshuehner 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version3.0PR16Q4 
Merge Request Status
Review Assigned Tocaristu
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0033769: ModuleManagement Settings page load trigger accidental double query to ad_ref_list
DescriptionThe code in ModuleManagement.printPageSettings triggers accidental double load by calling qList.list() twice

Steps To ReproduceWith debugging code from:
https://issues.openbravo.com/view.php?id=33767 [^]

With System Administrator role
Open Module Management window
Open settings tab of said window
Proposed Solutionsave result of first .list() call and reuse it
Additional Information
TagsPerformance
Relationships
related to feature request 0033767 closed platform Add code to auto-detect 'accidental double query' on same OBQuery or OBCriteria object 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2016-08-23 16:33shuehnerNew Issue
2016-08-23 16:33shuehnerAssigned To => shuehner
2016-08-23 16:33shuehnerModules => Core
2016-08-23 16:33shuehnerTriggers an Emergency Pack => No
2016-08-23 16:33shuehnerTag Attached: Performance
2016-08-23 16:37shuehnerRelationship addedrelated to 0033767
2016-08-23 22:04hgbotCheckin
2016-08-23 22:04hgbotNote Added: 0089322
2016-08-23 22:04hgbotStatusnew => resolved
2016-08-23 22:04hgbotResolutionopen => fixed
2016-08-23 22:04hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/ed70d2867dd8553688b585e88d5a49a741dcfb2d [^]
2016-08-23 22:04shuehnerReview Assigned To => caristu
2016-08-25 10:33caristuNote Added: 0089379
2016-08-25 10:33caristuStatusresolved => closed
2016-08-25 10:33caristuFixed in Version => 3.0PR16Q4

Notes
(0089322)
hgbot   
2016-08-23 22:04   
Repository: erp/devel/pi
Changeset: ed70d2867dd8553688b585e88d5a49a741dcfb2d
Author: Stefan Hühner <stefan.huehner <at> openbravo.com>
Date: Tue Aug 23 16:42:13 2016 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/ed70d2867dd8553688b585e88d5a49a741dcfb2d [^]

Fixed 33769. Avoid double query in ModuleManagement window (settings tab)

---
M src/org/openbravo/erpCommon/ad_forms/ModuleManagement.java
---
(0089379)
caristu   
2016-08-25 10:33   
Code review + testing OK.