Last modified: 2013-08-25 13:46:30 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 T31984, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 29984 - SocialProfile needs to be ported to ResourceLoader
SocialProfile needs to be ported to ResourceLoader
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
SocialProfile (Other open bugs)
unspecified
All All
: High enhancement with 3 votes (vote)
: ---
Assigned To: Nobody - You can work on this!
http://www.mediawiki.org/wiki/Extensi...
:
Depends on:
Blocks: 29713
  Show dependency treegraph
 
Reported: 2011-07-20 16:06 UTC by Jack Phoenix
Modified: 2013-08-25 13:46 UTC (History)
4 users (show)

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


Attachments

Description Jack Phoenix 2011-07-20 16:06:42 UTC
Current trunk version of SocialProfile has been tested and developed against MediaWiki 1.16 and it seems that it will work only on 1.16 at the moment; it appears quite broken on current trunk version of MediaWiki (1.19alpha).

I've tried porting SocialProfile to use ResourceLoader a few times now and the JS is always giving me a headache. I've rewritten SocialProfile/UserGifts/UserGifts.js to be more object-oriented, but the way its functions are (currently) used seems to be problematic when combined with the ResourceLoader; see SocialProfile/UserGifts/SpecialGiveGift.php, lines 244, 313 and 372.

I'd like to retain backwards compatibility with MediaWiki 1.16 for the time being, as I need to deploy SocialProfile on some ShoutWiki sites, which still run 1.16.

SystemGifts, UserActivity, UserStats and UserWelcome have only CSS files; UserSystemMessages has no CSS nor JS and the remaining "modules" (UserBoard, UserGifts, UserProfile, UserRelationship and UserStatus) have both CSS and JS files.

Assigning to Krinkle as per Reedy's suggestion on #mediawiki:

08:02 < ashley> to whom might I assign a ResourceLoader-related bug? specifically, my SocialProfile extension appears to be rather...broken in current trunk (and probably for 1.17+ in general) and it needs to be ported to use RL (retaining backwards compat w/ 1.16) but jQuery and RL are driving me crazy :-/
08:02 < Reedy> ashley, Roan, Trevor or maybe Krinkle
Comment 1 Robin Pepermans (SPQRobin) 2011-07-23 16:35:57 UTC
This would improve RTL support for SocialProfile.
If JS is a problem, maybe it can be done only for CSS, temporarily?
Comment 2 Jack Phoenix 2011-07-23 16:47:54 UTC
From what I've understood, porting SocialProfile to ResourceLoader before ResourceLoader 2/MediaWiki 1.19 will be quite difficult, if not impossible, because of the way how SocialProfile uses inline JS in certain places.
According to http://www.mediawiki.org/wiki/ResourceLoader/Migration_guide_for_extension_developers#Inline_JavaScript, MW 1.19 will ship with RL2, which has the top-load queue; by loading JS in the top (just like how we've done it until RL/MW 1.17), we can avoid rewriting lots and lots of code.

As for porting CSS only for the time being...I have to say that I'm not a big fan of the idea. I'd much rather have this issue fixed properly for once and for all; sure, we /can/ port CSS to use RL right away, but the fact is that JS will still be broken.
Comment 3 Daniel Fernandes 2011-08-19 00:54:50 UTC
i'm user of socialprofile, and the js is debugging, i think if socialprofile supported the newest releases of mediawiki will make my life easier, in version 1.17.0 i cant run socialprofile because have js errors and loss of functionality (i can't send board messages), :((
Comment 4 Jack Phoenix 2011-08-19 19:32:18 UTC
Marking this as high as per my discussion with Mark on #mediawiki today; there's major loss of functionality (pretty much everything that uses JS doesn't work, i.e. sending board messages, as mentioned by Daniel above, picking a friend from the list on Special:GiveGift, etc. etc.) and users have every right to assume that latest trunk version of an extension is compatible with the latest stable release of MediaWiki.
Unfortunately for SocialProfile, that is right now not true and users wanting to use SocialProfile are forced to use the old 1.16 branch of MediaWiki.
Comment 5 Krinkle 2011-08-24 18:23:55 UTC
Resetting assignee. I don't know the extension and currently have a agenda packed with other projects. I'm happy to give some general advice on IRC but no time to go the actual porting.
Comment 6 Jack Phoenix 2013-08-25 13:46:30 UTC
I've (mostly) done this in Gerrit change #79830 so I'm closing this ticket.

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


Navigation
Links