Last modified: 2010-09-01 14:17:14 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 T19913, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 17913 - Show no "← Older edit" is no older version available
Show no "← Older edit" is no older version available
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
History/Diffs (Other open bugs)
unspecified
All All
: Normal enhancement (vote)
: ---
Assigned To: Nobody - You can work on this!
http://en.wikipedia.org/w/index.php?t...
: patch
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-03-10 20:34 UTC by Umherirrender
Modified: 2010-09-01 14:17 UTC (History)
1 user (show)

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


Attachments
untested patch (877 bytes, patch)
2010-03-26 20:12 UTC, Umherirrender
Details

Description Umherirrender 2009-03-10 20:34:51 UTC
On the last diff of a page "Newer edit →" is not shown, but when look at the first diff of a page "← Older edit" is shown and suggest there is any diff, but when click that link the first revision is load and no diff. I think that is not intuition. The link should not shown, when a older version is not available.
Comment 1 Umherirrender 2009-03-28 18:50:45 UTC
function 'showDiffPage' in 'DifferenceEngine.php' should test if the oldid the first one, and then set "$prevlink = ' ';" in the same way like $nextlink. For example:

		if ( $this->mOldRev->getPrevious() != null ) {
			$prevlink = $sk->makeKnownLinkObj( $this->mTitle, wfMsgHtml( 'previousdiff' ),
				'diff=prev&oldid='.$this->mOldid.$htmldiffarg, '', '', 'id="differences-prevlink"' );
		} else {
			$prevlink = ' ';
		}

I am not so familiar with php and mediawiki, so I cannot give you a patch, only this idea.
Comment 2 Umherirrender 2010-03-26 20:12:35 UTC
Created attachment 7248 [details]
untested patch

(In reply to comment #1)
> function 'showDiffPage' in 'DifferenceEngine.php' should test if the oldid the
> first one, and then set "$prevlink = ' ';" in the same way like $nextlink.
> For example:
>         if ( $this->mOldRev->getPrevious() != null ) {
>             $prevlink = $sk->makeKnownLinkObj( $this->mTitle, wfMsgHtml(
> 'previousdiff' ),
>                 'diff=prev&oldid='.$this->mOldid.$htmldiffarg, '', '',
> 'id="differences-prevlink"' );
>         } else {
>             $prevlink = ' ';
>         }
> I am not so familiar with php and mediawiki, so I cannot give you a patch, only
> this idea.

patch attached
Comment 3 Chad H. 2010-09-01 14:17:14 UTC
Done in r72107.

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


Navigation
Links