Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0011552Openbravo ERPA. Platformpublic2009-11-30 12:302009-12-04 09:14
Reporteriperdomo 
Assigned Toiperdomo 
PriorityurgentSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Versionpi 
Target VersionFixed in Version 
Merge Request Status
Review Assigned To
OBNetwork customerNo
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0011552: ajax.js changes based on JSLint recomendations
DescriptionJSLint [1] tool provides best-practices about JavaScript code. All our .js files should pass this tool. There is a command-line wrapper to jslint [2].



[1] http://jslint.com/ [^]
[2] http://code.google.com/p/jslint4java/ [^]
Steps To Reproduce* Download jslint4java and check web/js/ajax.js
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2009-11-30 12:30iperdomoNew Issue
2009-11-30 12:30iperdomoAssigned To => iperdomo
2009-11-30 12:30iperdomoOBNetwork customer => No
2009-11-30 12:31iperdomoStatusnew => scheduled
2009-11-30 12:31iperdomofix_in_branch => pi
2009-11-30 12:35hgbotCheckin
2009-11-30 12:35hgbotNote Added: 0022255
2009-11-30 12:35hgbotStatusscheduled => resolved
2009-11-30 12:35hgbotResolutionopen => fixed
2009-11-30 12:35hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/4afcf80816b3a6ae5263480fd6be4bf6d4cd4152 [^]
2009-12-02 16:20shuehnerNote Added: 0022367
2009-12-02 16:20shuehnerStatusresolved => closed
2009-12-03 00:00anonymoussf_bug_id0 => 2907818
2009-12-04 09:14hudsonbotCheckin
2009-12-04 09:14hudsonbotNote Added: 0022424

Notes
(0022255)
hgbot   
2009-11-30 12:35   
Repository: erp/devel/pi
Changeset: 4afcf80816b3a6ae5263480fd6be4bf6d4cd4152
Author: Iván Perdomo <ivan.perdomo <at> openbravo.com>
Date: Mon Nov 30 12:34:46 2009 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/4afcf80816b3a6ae5263480fd6be4bf6d4cd4152 [^]

Fixes issue 11552: Changes based on JSLint best practices

---
M web/js/ajax.js
---
(0022367)
shuehner   
2009-12-02 16:20   
Tested.
JSLint does not report any more issues with changed file
Several ajax users in erp still working fine; tested: mmc,alert
(0022424)
hudsonbot   
2009-12-04 09:14   
---
A changeset related to this issue has passed a series of tests and has been promoted to main:

Changeset: http://code.openbravo.com/erp/devel/main/rev/f6bc559b1872 [^]
Tests: http://builds.openbravo.com/view/devel-int/ [^]
---