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

WordPress Trac noreply at wordpress.org
Thu Nov 7 17:22:43 UTC 2013


#25864: jshint shouldn't throw errors - wp-admin/revisions.js
-----------------------------+------------------------------
 Reporter:  adamsilverstein  |       Owner:
     Type:  enhancement      |      Status:  new
 Priority:  normal           |   Milestone:  Awaiting Review
Component:  Unit Tests       |     Version:  trunk
 Severity:  normal           |  Resolution:
 Keywords:  has-patch        |
-----------------------------+------------------------------

Comment (by adamsilverstein):

 As a check (per azaozz's suggestion), I compared the minified versions of
 the original and patched files (kaleidoscope helpful since its all one
 line). differences were minimal: better variable name optimization where
 variables now declared properly. the one other change was that I switched
 console reference to window.console to clear errors - the other approach
 would be to add console to the file's global declaration.

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


More information about the wp-trac mailing list