![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png){kind=logo}
#26034 closed defect (bug) (fixed)
jshint shouldn't throw errors:wp-admin/js/link.js
| Reported by: | seanchayes | Owned by: | nacin |
|---|---|---|---|
| Priority: | normal | Milestone: | 3.8 |
| Component: | Build/Test Tools | Version: | |
| Severity: | normal | Keywords: | has-patch |
| Cc: | Focuses: |
Description
Resolving JSHint errors as shown:
Linting src/wp-admin/js/link.js ...ERROR
[L25:C103] W033: Missing semicolon.
newCat = $('#newcat').one( 'focus', function() { $(this).val( '' ).removeClass( 'form-input-tip' ) } );
Linting src/wp-admin/js/link.js ...ERROR
[L7:C5] W117: 'postboxes' is not defined.
postboxes.add_postbox_toggles('link');
Linting src/wp-admin/js/link.js ...ERROR
[L16:C13] W117: 'deleteUserSetting' is not defined.
deleteUserSetting('cats');
Linting src/wp-admin/js/link.js ...ERROR
[L18:C13] W117: 'setUserSetting' is not defined.
setUserSetting('cats','pop');
Linting src/wp-admin/js/link.js ...ERROR
[L21:C10] W117: 'getUserSetting' is not defined.
if ( getUserSetting('cats') )
Linting src/wp-admin/js/link.js ...ERROR
[L54:C53] W117: 'deleteUserSetting' is not defined.
$('a[href="#categories-all"]').click(function(){deleteUserSetting('cats');});
Linting src/wp-admin/js/link.js ...ERROR
[L55:C53] W117: 'setUserSetting' is not defined.
$('a[href="#categories-pop"]').click(function(){setUserSetting('cats','pop');});
Linting src/wp-admin/js/link.js ...ERROR
[L56:C19] W117: 'getUserSetting' is not defined.
if ( 'pop' == getUserSetting('cats') )
Attachments (1)
Change History (4)
Note:
See TracTickets
for help on using tickets.
In 26200: