Last modified: 2011-06-18 11:53:19 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 T31144, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 29144 - Move action=dublincore and action=creativecommons to extensions
Move action=dublincore and action=creativecommons to extensions
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
General/Unknown (Other open bugs)
unspecified
All All
: Low enhancement (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks: 29145
  Show dependency treegraph
 
Reported: 2011-05-25 20:20 UTC by Alexandre Emsenhuber [IAlex]
Modified: 2011-06-18 11:53 UTC (History)
5 users (show)

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


Attachments

Description Alexandre Emsenhuber [IAlex] 2011-05-25 20:20:47 UTC
Those two actions generate XML and are not intended for normal views, so it would be much more suitable to have them in api.php rather than index.php.
Comment 1 Brion Vibber 2011-05-25 21:19:12 UTC
If done, like then OpenSearch output they'll need some customization:
* compatibility with existing URLs (index.php still has to catch the actions and redirect)
* possibly force default output mode to xml (or else require putting format=xml onto the new generated URLs)
Comment 2 Sam Reed (reedy) 2011-05-25 21:22:13 UTC
Was ist das?
Comment 3 Brion Vibber 2011-05-25 22:17:01 UTC
See:

http://www.mediawiki.org/wiki/Manual:$wgEnableDublinCoreRdf
http://www.mediawiki.org/wiki/Manual:$wgEnableCreativeCommonsRdf
http://meta.wikimedia.org/wiki/RDF_metadata

Note that these are off by default and on Wikimedia sites, and are probably good candidates for bumping out to an extension.
Comment 4 Sam Reed (reedy) 2011-05-25 22:42:25 UTC
It looks like they will be rather trivial to move out to extensions.... The code isn't really intertwined

But RdfMetaData would presumably have to stay in core...?
Comment 5 Sam Reed (reedy) 2011-05-25 22:44:23 UTC
Would they still need adding to OutputPage->getHeadLinks()...

I'd guess not?
Comment 6 Sam Reed (reedy) 2011-05-26 00:15:52 UTC
r88868 and r88873 essentially just move them out to be Extensions, keeping the action=blah back compat on index.php

The only thing "lost" is the adding of them to getHeadLinks, as there isn't a hook or similar to add it in

Is that actually needed?


Also, the actual API modules need creating later
Comment 7 Sam Reed (reedy) 2011-05-26 00:59:04 UTC
(In reply to comment #6)
> The only thing "lost" is the adding of them to getHeadLinks, as there isn't a
> hook or similar to add it in
> 

r88876 fixes that up
Comment 8 Mark A. Hershberger 2011-05-27 03:12:45 UTC
if this was done, you'd want those modules included in the tarball.
Comment 9 p858snake 2011-05-27 03:14:39 UTC
(In reply to comment #8)
> if this was done, you'd want those modules included in the tarball.

Not really.... we remove stuff to reduce bloat and to make stuff simpler. or are we just going to make that pointless and stick everything back in?
Comment 10 Mark A. Hershberger 2011-05-27 16:45:53 UTC
(In reply to comment #9)
> Not really.... we remove stuff to reduce bloat and to make stuff simpler. or
> are we just going to make that pointless and stick everything back in?

My goal is not to reduce "bloat", by which I assume you mean the size of the tarball (since modules are maintained outside of phase3).  While elegance and simplicity are good things to have in the code, those goals are orthogonal to bloat.

My goal (prejudiced by my daily grind through Bugzilla) is to reduce support costs.  If you look, you'll find several (hundred?) bugs in Bugzilla of people asking why MediaWiki doesn't support <ref> tags out of the box.  Each time, the response is the same: install the Cite extension.

If we bundled the Cite extension in the tarball and the new installer provided a clear explanation of what you get if you enable it, that would reduce the number of support requests in Bugzilla, 

That is why I aiming to have modules that are split out from core included in the tarball.
Comment 11 MZMcBride 2011-05-28 03:18:44 UTC
(In reply to comment #10)
> If we bundled the Cite extension in the tarball and the new installer provided
> a clear explanation of what you get if you enable it, that would reduce the
> number of support requests in Bugzilla, 

This analogy is flawed.

There's a reasonable argument that popular extensions should come bundled with MediaWiki (particularly Cite, ParserFunctions, RenameUser, etc.). However, as Brion notes above:

> Note that these are off by default and on Wikimedia sites, and are probably
> good candidates for bumping out to an extension.

It's a bogus argument to suggest that there is going to be a noticeably higher support cost when moving these ancient (seemingly forgotten) features from core to an extension. If you have evidence to support your claims, I'd love to see it. :-)
Comment 12 Chad H. 2011-05-28 03:28:30 UTC
(In reply to comment #10)
> My goal (prejudiced by my daily grind through Bugzilla) is to reduce support
> costs.  If you look, you'll find several (hundred?) bugs in Bugzilla of people
> asking why MediaWiki doesn't support <ref> tags out of the box.  Each time, the
> response is the same: install the Cite extension.
> 

Several hundred?

I've been watching BZ for *years* and I haven't seen it crop up more than a small handful of times, if that.
Comment 13 Mark A. Hershberger 2011-05-28 03:35:50 UTC
> If you have evidence to support your claims, I'd love to see
> it. :-)

You're pretty familiar with bugzilla.  I'm sure you can find the bugs
about missing Cite functionality yourself.

Still, Chad is right.  Saying there are more than a few requests in
Bugzilla is hyperbolic.
Comment 14 Chad H. 2011-05-28 03:40:47 UTC
(In reply to comment #8)
> if this was done, you'd want those modules included in the tarball.

I disagree on this too. As has been pointed out in several comments here, this is an *almost never used feature* that has been disabled on WMF sites since time immemorial (since inception?). 

As long as it's in the REL-NOTES that it's been bumped to an extension, there's zero reason to include this in the tarball.
Comment 15 p858snake 2011-05-28 03:47:13 UTC
(Mid Air Collision)
(In reply to comment #10)
> That is why I aiming to have modules that are split out from core included in
> the tarball.

Must of the things that are pulled out of core are done because either they are unloved, unwanted, disabled by default or just too complicated for the normal user.

There has been discussion multiple times for including extensions in our tarballs or to package them in core (which in this case would be basically the same for the sake of this discussion) such as cite which you bring up as as far as I can recall have never gained consensus.


But more to point, the features that have been removed recently are Maths and these twos, These are apparently disabled by default anyway and maths as was complicated and required dependencies which is why they were moved out.

Perhaps you should start a wider discussion about including these in the tarball packages.
Comment 16 Mark A. Hershberger 2011-05-28 19:56:35 UTC
(In reply to comment #15)

> Perhaps you should start a wider discussion about including these in the
> tarball packages.

Agreed.  In fact, it was because I was intending to do that that I was so drawn to this bug.  My apologies.
Comment 17 Sam Reed (reedy) 2011-05-28 20:26:52 UTC
(In reply to comment #16)
> (In reply to comment #15)
> 
> > Perhaps you should start a wider discussion about including these in the
> > tarball packages.
> 
> Agreed.  In fact, it was because I was intending to do that that I was so drawn
> to this bug.  My apologies.

See bug 26261
Comment 18 p858snake 2011-06-18 10:37:23 UTC
marking fixed, pretty sure reedy has done this.
Comment 19 Sam Reed (reedy) 2011-06-18 11:53:19 UTC
The only really outstanding issue is moving them from index.php to the api...

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


Navigation
Links