Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0010364Openbravo ERPA. Platformpublic2009-08-20 18:552009-10-16 00:00
shuehner 
alostale 
urgentminorhave not tried
closedfixed 
5
pi 
pi 
Core
No
0010364: OPS changes add new DynamicJS. includes, however these are unneeded in most cases.
The new

ad_forms/InstanceManagement*.html files do include DynamicJS.js but these include is not needed anymore for most html files.

It should be removed if the error reporting in the windows works fine without it, as it adds an extra unneded http request.
OBPS-Security
Issue History
2009-08-20 18:55shuehnerNew Issue
2009-08-20 18:55shuehnerAssigned To => alostale
2009-08-20 18:55shuehnerTag Attached: OBPS-Security
2009-08-20 18:56shuehnerNote Added: 0019142
2009-08-20 21:10plujanStatusnew => scheduled
2009-10-09 10:22hgbotCheckin
2009-10-09 10:22hgbotNote Added: 0020900
2009-10-09 10:22hgbotStatusscheduled => resolved
2009-10-09 10:22hgbotResolutionopen => fixed
2009-10-09 10:22hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/f9dda963c7d7a8d7a080db755053eefce92ff45f [^]
2009-10-15 14:56shuehnerStatusresolved => closed
2009-10-15 14:56shuehnerNote Added: 0021059
2009-10-15 14:56shuehnerFixed in Version => pi
2009-10-16 00:00anonymoussf_bug_id0 => 2880173

Notes
(0019142)
shuehner   
2009-08-20 18:56   
Note some of the files include DynamicJS.js and messages.js twice, please remove the second copy when checking/fixing this defect.
(0020900)
hgbot   
2009-10-09 10:22   
Repository: erp/devel/pi
Changeset: f9dda963c7d7a8d7a080db755053eefce92ff45f
Author: Asier Lostalé <asier.lostale <at> openbravo.com>
Date: Fri Oct 09 10:22:05 2009 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/f9dda963c7d7a8d7a080db755053eefce92ff45f [^]

fixed bug 0010364: Clean up in InstanceManagement* js

---
M src/org/openbravo/erpCommon/ad_forms/InstanceManagement.html
M src/org/openbravo/erpCommon/ad_forms/InstanceManagementActivateLocal.html
M src/org/openbravo/erpCommon/ad_forms/InstanceManagementDeactivate.html
---
(0021059)
shuehner   
2009-10-15 14:56   
Verified. The includes have been removed from the files. The affected windows/popups still show sucess/error messages correctly.