Last modified: 2013-03-08 11:23: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 T47869, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 45869 - It should be possible to enable Extension:Diff for particular content models or namespaces
It should be possible to enable Extension:Diff for particular content models ...
Status: RESOLVED INVALID
Product: MediaWiki extensions
Classification: Unclassified
Diff (Other open bugs)
master
All All
: Unprioritized enhancement (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks: 45865
  Show dependency treegraph
 
Reported: 2013-03-08 00:43 UTC by Ori Livneh
Modified: 2013-03-08 11:23 UTC (History)
2 users (show)

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


Attachments

Description Ori Livneh 2013-03-08 00:43:33 UTC
As far as I can tell, Extension:Diff is all-or-nothing: you can use it to generate all diffs or none. It would be nice if this were more configurable, allowing you to enable it for a particular content model.
Comment 1 Jeroen De Dauw 2013-03-08 11:20:09 UTC
You seem to misunderstand what Diff is. It is a PHP library for computing an representing (as in PHP objects) diffs. It does not even know MediaWiki exists, much less content models or namespaces.
Comment 2 Jeroen De Dauw 2013-03-08 11:23:56 UTC
To clarify: it is the content handler linked to some content model that decides which Action class is used to handle diff view requests. Those action classes can then compute and display diffs as they like, including using the Diff library for this computation. This is what Wikibase does for its diff view handling. If you see shortcomings there, it's a Wikibase issue. If you think all content models should be able to use Diff, it is a MW core issue.

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


Navigation
Links