[wp-trac] [WordPress Trac] #59329: Firefox gets "ReferenceError: Â is not defined" in unminified moxie.js
WordPress Trac
noreply at wordpress.org
Thu Dec 19 22:48:31 UTC 2024
#59329: Firefox gets "ReferenceError: Â is not defined" in unminified moxie.js
-----------------------------------------+-------------------------
Reporter: kinggmobb | Owner: (none)
Type: defect (bug) | Status: new
Priority: normal | Milestone: 6.8
Component: Upload | Version: 4.9
Severity: minor | Resolution:
Keywords: has-patch changes-requested | Focuses: javascript
-----------------------------------------+-------------------------
Comment (by azaozz):
Replying to [comment:13 desrosj]:
> `plupload.js`
[https://core.trac.wordpress.org/browser/trunk/src/js/_enqueues/vendor/plupload/plupload.js#L23
already has one], but doesn't explicitly say that the file is a de-facto
fork.
>
> I looked at some libraries in a similar situation (both PHP and JS), but
couldn't find one with a header to that effect.
Ah, I meant adding a note to the headers that the file was modified for
WP. Sorry didn't explain it well. But probably better to explain a little
more and/or mention that it is forked and will be maintained by the WP
developers.
> It may be worth standardizing this and adding to all of our adopted
libraries.
Yes, good idea! Seems some of the de-facto forked libs/files like
[https://github.com/WordPress/wordpress-
develop/blob/trunk/src/js/_enqueues/vendor/twemoji.js#L309 twemoji.js]
have some inline comments. Others like [https://github.com/WordPress
/wordpress-develop/blob/trunk/src/js/_enqueues/vendor/thickbox/thickbox.js
thickbox.js] don't have anything although I'm sure the original Thickbox
JS was modified many times to make it work with newer jQuery, etc.
(unbelievably it is from 2007, and still works?!).
--
Ticket URL: <https://core.trac.wordpress.org/ticket/59329#comment:14>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list