Last modified: 2012-04-19 21:11:56 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T30904, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 28904 - Update jQuery version to 1.6.1
Update jQuery version to 1.6.1
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
JavaScript (Other open bugs)
unspecified
All All
: Normal enhancement (vote)
: 1.19.0 release
Assigned To: Krinkle
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-05-09 23:16 UTC by B Baron
Modified: 2012-04-19 21:11 UTC (History)
5 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description B Baron 2011-05-09 23:16:10 UTC
With the branching of MW 1.18, we should take the chance to upgrade the jQuery bundled with MW 1.19.
http://blog.jquery.com/2011/05/03/jquery-16-released/
Comment 1 Sam Reed (reedy) 2011-05-09 23:38:06 UTC
The same should happen with various extensions also

See bug 26307, and to some extent, bug 26306
Comment 2 Mark A. Hershberger 2011-05-11 16:06:45 UTC
Reedy, I'm gonna ask you to take the lead on this.  Let me know if that isn't reasonable.
Comment 3 Sam Reed (reedy) 2011-05-11 16:24:48 UTC
Better with Roan, Trevor or Krinkle


I could replace it, but I'd be somewhat stuck as to fixing the issues

I'm the api/backend developer, they're the frontend ;)
Comment 4 B Baron 2011-05-15 16:20:18 UTC
jQuery 1.6.1
http://blog.jquery.com/2011/05/12/jquery-1-6-1-released
Comment 5 Krinkle 2011-05-15 21:31:13 UTC
MediaWiki 1.15: jQuery 1.3.2
MediaWiki 1.16: jQuery 1.4.2
MediaWiki 1.17: jQuery 1.4.4

For 1.18 we're too late in my opinion, since branching has taken place and active development is basically past us. 

I suggest switching trunk to 1.5.2 or 1.6.1.

From first look, I thnik we should go with 1.5.2 due to the new .prop() changes in 1.6.x which, together with all that was new in 1.5.x maybe too much at once (since we're coming from 1.4.4).
Comment 6 Sam Reed (reedy) 2011-05-22 21:51:03 UTC
Do one step now, then do the next later when appropriate ;)

Certainly putting it in 1.18 is asking for trouble
Comment 7 Krinkle 2011-05-22 21:55:01 UTC
(In reply to comment #5)
> MediaWiki 1.15: jQuery 1.3.2
> MediaWiki 1.16: jQuery 1.4.2
> MediaWiki 1.17: jQuery 1.4.4
> 
> For 1.18 we're too late in my opinion, since branching has taken place and
> active development is basically past us. 
> 

MediaWiki 1.19 (trunk): jQuery 1.5.2

Done in r88607.
Comment 8 Roan Kattouw 2011-05-23 15:14:27 UTC
(In reply to comment #5)
> From first look, I thnik we should go with 1.5.2 due to the new .prop() changes
> in 1.6.x which, together with all that was new in 1.5.x maybe too much at once
> (since we're coming from 1.4.4).
The backwards incompatible changes were fixed in 1.6.1 . I still see the argument for doing this slowly though.
Comment 9 Brion Vibber 2011-05-24 17:40:02 UTC
r88725 reverts the update (r88607) and references this bug. What's the reasoning for downgrading rather than fully upgrading on trunk? Should this bug be reopened?
Comment 10 Krinkle 2011-05-24 17:55:59 UTC
Yes, and renamed as well.
Comment 11 Krinkle 2011-06-11 00:27:00 UTC
Fixed in r89866.

I've been running on 1.6.1 locally for two weeks now and fixed issues that I was saving in the mean time and made sure we're still passing all unit tests.
Comment 12 B Baron 2011-06-30 21:07:53 UTC
JQuery 1.6.2 has been released: http://blog.jquery.com/2011/06/30/jquery-162-released/
Comment 13 Sam Reed (reedy) 2011-07-01 00:21:18 UTC
(In reply to comment #12)
> JQuery 1.6.2 has been released:
> http://blog.jquery.com/2011/06/30/jquery-162-released/

You could've just opened a new bug
Comment 14 Krinkle 2011-07-04 21:33:45 UTC
(In reply to comment #12)
> JQuery 1.6.2 has been released:
> http://blog.jquery.com/2011/06/30/jquery-162-released/

jQuery version has already been update for 1.19, I suggest doing the next update in 1.20, which already has an open bug 29100.

Re-marking this as fixed as it has already been upgraded to 1.6.1, to avoid confusion in release-notes and other referencs, don't re-use tickets for other requests.
Comment 15 Sam Reed (reedy) 2011-07-04 22:34:40 UTC
(In reply to comment #14)
> (In reply to comment #12)
> > JQuery 1.6.2 has been released:
> > http://blog.jquery.com/2011/06/30/jquery-162-released/
> 
> jQuery version has already been update for 1.19, I suggest doing the next
> update in 1.20, which already has an open bug 29100.
> 
> Re-marking this as fixed as it has already been upgraded to 1.6.1, to avoid
> confusion in release-notes and other referencs, don't re-use tickets for other
> requests.

Still, as it's a point release/bugfix update, shouldn't we probably upgrade to 1.6.2 in trunk as is? Then look at upgrading to whatever is the major version (1.7, 1.8?) for MW 1.20?

Granted, it should be logged as a new bug, but still.
Comment 16 B Baron 2011-07-11 01:25:05 UTC
It's only a minor point release, and it's not too late for 1.19 (is it?).
Comment 17 Sam Reed (reedy) 2011-07-11 14:35:32 UTC
(In reply to comment #16)
> It's only a minor point release, and it's not too late for 1.19 (is it?).

No, and it's already been done, I logged a new bug rather than just reopening this one. If the issue had been unresolved before 1.6.2 came out, retitling this bug etc would've been fine.

As it had been fixed, as is, bug 28904 is the more appropriate way of doing it
Comment 18 Krinkle 2011-08-29 22:42:01 UTC
(In reply to comment #17)
> As it had been fixed, as is, bug 28904 is the more appropriate way of doing it

bug 29773 :)

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links