Openbravo Issue Tracking System - Openbravo ERP | ||||||||||||
| View Issue Details | ||||||||||||
| ID | Project | Category | View Status | Date Submitted | Last Update | |||||||
| 0015476 | Openbravo ERP | A. Platform | public | 2010-12-14 09:58 | 2011-01-18 12:17 | |||||||
| Reporter | iperdomo | |||||||||||
| Assigned To | mirurita | |||||||||||
| Priority | urgent | Severity | minor | Reproducibility | always | |||||||
| Status | closed | Resolution | fixed | |||||||||
| Platform | OS | 20 | OS Version | 2.6.35-gentoo-r12 | ||||||||
| Product Version | ||||||||||||
| Target Version | 3.0RC4 | Fixed in Version | ||||||||||
| Merge Request Status | ||||||||||||
| Review Assigned To | ||||||||||||
| OBNetwork customer | No | |||||||||||
| Web browser | ||||||||||||
| Modules | Advanced Payables and Receivables Mngmt | |||||||||||
| Support ticket | ||||||||||||
| Regression level | ||||||||||||
| Regression date | ||||||||||||
| Regression introduced in release | ||||||||||||
| Regression introduced by commit | ||||||||||||
| Triggers an Emergency Pack | No | |||||||||||
| Summary | 0015476: Fix JSLint warnings in FIN_Utilities.js | |||||||||||
| Description | All the JavaScript code included in the 3.0 modules should pass JSLint without warnings. | |||||||||||
| Steps To Reproduce | * Run jslint in FIN_Utilities.js e.g. ~/workspace/src/openbravo/pi/modules/org.openbravo.advpaymentmngt $ jslint ./web/org.openbravo.advpaymentmngt/FIN_Utilities.js jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:47:15:'i' is already defined. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:59:49:['inpRecordId0'] is better written in dot notation. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:62:7:Expected '{' and instead saw 'outstandingAmount'. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:71:36:'scheduledPaymentDetailId' is already defined. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:72:29:'outstandingAmount' is already defined. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:74:9:Expected '{' and instead saw 'outstandingAmount'. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:85:21:'j' is already defined. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:103:21:'updateTotal' was used before it was defined. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:108:4:Mixed spaces and tabs. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:113:49:['inpRecordId0'] is better written in dot notation. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:119:7:Expected '{' and instead saw 'initialize_MessageBox'. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:121:7:Expected '{' and instead saw 'total'. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:125:36:'scheduledPaymentDetailId' is already defined. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:126:25:'pendingAmount' is already defined. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:127:18:'amount' is already defined. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:131:9:Expected '{' and instead saw 'initialize_MessageBox'. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:133:9:Expected '{' and instead saw 'total'. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:141:9:Expected '{' and instead saw 'frm'. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:143:9:Expected '{' and instead saw 'frm'. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:147:4:Mixed spaces and tabs. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:147:9:Expected '{' and instead saw 'frm'. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:153:26:'updateDifference' was used before it was defined. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:156:5:Expected '{' and instead saw 'expected'. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:160:28:Use '!==' to compare with 'null'. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:161:5:Expected '{' and instead saw 'amount'. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:163:5:Expected '{' and instead saw 'amount'. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:165:5:Expected '{' and instead saw 'frm'. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:167:5:Expected '{' and instead saw 'frm'. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:169:5:Expected '{' and instead saw 'frm'. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:178:5:Expected '{' and instead saw 'selectDifferenceAction'. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:180:5:Expected '{' and instead saw 'selectDifferenceAction'. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:183:32:'selectDifferenceAction' was used before it was defined. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:192:12:Use '===' to compare with 'null'. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:192:21:Expected '{' and instead saw 'mark'. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:208:14:Expected '{' and instead saw 'frm'. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:208:27:['inpAllLines'] is better written in dot notation. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:215:13:Expected '{' and instead saw 'return'. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:219:2:Mixed spaces and tabs. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:220:2:Mixed spaces and tabs. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:228:2:Mixed spaces and tabs. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:228:28:'recordAmount' is already defined. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:229:2:Mixed spaces and tabs. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:243:21:'applyFormat' was used before it was defined. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:261:26:Use '===' to compare with 'null'. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:261:63:Expected '{' and instead saw 'result_maskNumeric'. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:262:20:Use '===' to compare with 'null'. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:262:51:Expected '{' and instead saw 'decSeparator'. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:263:22:Use '===' to compare with 'null'. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:263:55:Expected '{' and instead saw 'groupSeparator'. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:264:21:Use '===' to compare with 'null'. jslint:./web/org.openbravo.advpaymentmngt/FIN_Utilities.js:264:21:Too many errors. (53% scanned) | |||||||||||
| Proposed Solution | * Fix the warnings/errors | |||||||||||
| Additional Information | ||||||||||||
| Tags | No tags attached. | |||||||||||
| Relationships |
| |||||||||||
| Attached Files | ||||||||||||
| Issue History | ||||||||||||
| Date Modified | Username | Field | Change | |||||||||
| 2010-12-17 12:18 | mirurita | Type | defect => backport | |||||||||
| 2010-12-17 12:18 | mirurita | fix_in_branch | => 2.50 | |||||||||
| 2010-12-17 12:24 | hgbot | Checkin | ||||||||||
| 2010-12-17 12:24 | hgbot | Note Added: 0033270 | ||||||||||
| 2010-12-17 12:24 | hgbot | Status | scheduled => resolved | |||||||||
| 2010-12-17 12:24 | hgbot | Resolution | open => fixed | |||||||||
| 2010-12-17 12:24 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/pmods/org.openbravo.advpaymentmngt/rev/8126bf446c53f5ead0b4f33a77cc85c4fb82bbaa [^] | |||||||||
| 2011-01-18 12:17 | mirurita | Status | resolved => closed | |||||||||
| Notes | |||||
|
|
|||||
|
|
||||