[wp-trac] [WordPress Trac] #25947: jshint shouldn't throw errors: wp-admin/js/editor.js

WordPress Trac noreply at wordpress.org
Fri Nov 15 20:29:31 UTC 2013


#25947: jshint shouldn't throw errors: wp-admin/js/editor.js
--------------------------+------------------------------
 Reporter:  dougwollison  |       Owner:
     Type:  defect (bug)  |      Status:  new
 Priority:  normal        |   Milestone:  Awaiting Review
Component:  Build Tools   |     Version:
 Severity:  normal        |  Resolution:
 Keywords:  has-patch     |
--------------------------+------------------------------

Comment (by azaozz):

 Some more notes:
 - Don't think defining the globals for JSHint needs the `:false` part.
 This is the [http://www.jshint.com/docs/ default].
 - The globals should probably be comma separated on the same line,
 following the same standard as `var`.
 - `tinymce` is a superset of `tinyMCE` (tinyMCE exists for back-compat),
 but `tinyMCEPreInit` is another global that cannot be changed. Please run
 some tests with the patched file to confirm all is still working.

--
Ticket URL: <http://core.trac.wordpress.org/ticket/25947#comment:9>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list