Last modified: 2012-05-25 02:43:24 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 T25137, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 23137 - no diff when click "show changes" by section=new on non existing page
no diff when click "show changes" by section=new on non existing page
Status: NEW
Product: MediaWiki
Classification: Unclassified
Page editing (Other open bugs)
unspecified
All All
: Low enhancement (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2010-04-10 10:30 UTC by Umherirrender
Modified: 2012-05-25 02:43 UTC (History)
1 user (show)

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


Attachments
remove "bogus page" check (757 bytes, patch)
2011-09-02 20:53 UTC, Umherirrender
Details

Description Umherirrender 2010-04-10 10:30:47 UTC
When create a new section an non existing page with section=new the "show changes" button gives no diff. Works fine on existing page and without section=new.

Steps to reproduce:
* Go to non existing talk page
* click '+' (new section)
* add a subject and text
* click "show changes"
Comment 1 Church of emacs 2010-04-15 21:36:03 UTC
I suppose that one could be fixed quite easily, but the question is if "show changes" really makes sense.
The diff will be just the new text, which is also in the text field. We really should think about just omitting the "show changes" button/tab when section=new. Although we might keep it for UI consistency.
Comment 2 Umherirrender 2010-04-16 21:03:44 UTC
The inconsistency is that the show changes button give a diff, when not create a new section on an non-existing page.

The button is helpful to detect/prevent edit conflicts, because the function diffs always against the new version of a page and so changes is shown in the diff.
Comment 3 Umherirrender 2011-09-02 20:53:36 UTC
Created attachment 9007 [details]
remove "bogus page" check

The patch removed a "bogus page" check in WikiPage::replaceSection, because this fails on non-existing pages.

Feel free to modify the patch.

Thanks.
Comment 4 Umherirrender 2012-04-05 19:20:59 UTC
Comment on attachment 9007 [details]
remove "bogus page" check

Patch is obsolete because with r107319 some things changed, which makes that patch useless.

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


Navigation
Links