Following coding guideline JS8: "Follow guidelines set by JSHint," some errors should be corrected in basket.js basket.js: line 53, col 36, Use '===' to compare with '0'. basket.js: line 85, col 42, Missing semicolon. basket.js: line 210, col 5, Implied eval. Consider passing a function instead of a string. basket.js: line 219, col 44, Missing semicolon. basket.js: line 243, col 46, Missing semicolon. basket.js: line 251, col 29, Use '===' to compare with '0'. basket.js: line 270, col 13, 'valCookie' is already defined. basket.js: line 305, col 13, Use '===' to compare with 'undefined'. basket.js: line 409, col 25, The array literal notation [] is preferable. basket.js: line 416, col 25, Use '===' to compare with '0'. basket.js: line 432, col 33, Missing semicolon. basket.js: line 1, col 11, Missing semicolon.
Created attachment 48195 [details] [review] Bug 15847 - Correct JSHint errors in basket.js in the staff client This patch corrects minor errors in the staff client's basket.js according to warnings raised by JSHint. To test, apply the patch and run JSHint on basket.js. No warnings should be raised. To confirm that the behavior of the JS is not changed, clear your browser cache if necessary and: - Test adding multiple titles to the Cart from staff client search results. - Confirm that adding titles to the Cart triggers a message by the Cart link. - Test clicking the "Cart" link in the header menu when the Cart popup is open but behind the main window. - Testing emptying the Cart.
Created attachment 48197 [details] [review] [SIGNED-OFF]Bug 15847: Correct JSHint errors in basket.js in the staff client This patch corrects minor errors in the staff client's basket.js according to warnings raised by JSHint. To test, apply the patch and run JSHint on basket.js. No warnings should be raised. To confirm that the behavior of the JS is not changed, clear your browser cache if necessary and: - Test adding multiple titles to the Cart from staff client search results. - Confirm that adding titles to the Cart triggers a message by the Cart link. - Test clicking the "Cart" link in the header menu when the Cart popup is open but behind the main window. - Testing emptying the Cart. Signed-off-by: Hector Castro <hector.hecaxmmx@gmail.com> Works as described. No warnigns raised with jshint tool
Created attachment 48206 [details] [review] Bug 15847: Correct JSHint errors in basket.js in the staff client This patch corrects minor errors in the staff client's basket.js according to warnings raised by JSHint. To test, apply the patch and run JSHint on basket.js. No warnings should be raised. To confirm that the behavior of the JS is not changed, clear your browser cache if necessary and: - Test adding multiple titles to the Cart from staff client search results. - Confirm that adding titles to the Cart triggers a message by the Cart link. - Test clicking the "Cart" link in the header menu when the Cart popup is open but behind the main window. - Testing emptying the Cart. Signed-off-by: Hector Castro <hector.hecaxmmx@gmail.com> Works as described. No warnigns raised with jshint tool Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
pushed to master - should be in the May 2016 release. Thanks!
Patch pushed to 3.22.x, will be in 3.22.4