Last modified: 2011-07-20 22:32:13 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 T31840, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 29840 - Add action=diff for diff pages
Add action=diff for diff pages
Status: RESOLVED WONTFIX
Product: MediaWiki
Classification: Unclassified
History/Diffs (Other open bugs)
unspecified
All All
: Unprioritized enhancement (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-07-12 12:04 UTC by DieBuche
Modified: 2011-07-20 22:32 UTC (History)
1 user (show)

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


Attachments

Description DieBuche 2011-07-12 12:04:50 UTC
Currently diff pages are internally action=view.
The JS var wgAction=="view" as well.

It would be useful to add an action=diff to seperate from action=view
Comment 1 Brion Vibber 2011-07-12 19:01:25 UTC
Diff views are a special case of page views showing an old page version, plus a diff. Like old version views without the diffs and current version views with and without diffs, they're handled by the view action, usually include full page view contents, and have all the UI of a page view.

What would adding a different action variable add? Would it have potential problems with code assuming that not having an action of 'view' means that there's not a page to view and that they shouldn't do UI hookups?
Comment 2 Brion Vibber 2011-07-20 22:29:42 UTC
Closing this one out as a WONTFIX for now; could reconsider for future if we actually do a major overhaul of the diff view feature.
Comment 3 Krinkle 2011-07-20 22:32:13 UTC
A bit related is bug 27930

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


Navigation
Links