Don't render diffs by default
Closed, ResolvedPublic

Description

Author: mortgat

Description:
Many requests on the servers are done in order to watch changes (whether it is
vandalism or not).
When users check differences between versions, they don't necessarily want to
have the most recent version being displayed.
It would be a good enhancement if any logged user could choose: displaying the
changed version or not?


Version: unspecified
Severity: enhancement

bzimport added a subscriber: wikibugs-l.
bzimport set Reference to bz3446.
bzimport created this task.Via LegacySep 12 2005, 3:36 PM
bzimport added a comment.Via ConduitSep 12 2005, 8:28 PM

robchur wrote:

I thought that the History page allows you to choose whichever diff you wish to
see? The reason for the most recent change being shown under something like
Recent Changes is because, logically, that is what the user expects to see.
Could you explain further if I've misunderstood?

bzimport added a comment.Via ConduitSep 12 2005, 8:54 PM

mortgat wrote:

I think there is a misunderstanding!
When you compare two versions of an article, you can see what's changed between
both articles seeing their code and you see also the rendering corresponding
with the last code.
That's true when you click "diff" in the RC, and when you compare two versions
of an article via the History.

I'd like to disable the rendering in my individual preferences, since the
servers are overwhelmed with such requests only to check the nature of recent
Changes.

Nice day

Benoit Mortgat

bzimport added a comment.Via ConduitSep 12 2005, 8:56 PM

robchur wrote:

Oh, I see. Yes, I'd agree with that logic.

bzimport added a comment.Via ConduitSep 12 2005, 11:36 PM

dake.cdx wrote:

This would be very useful indeed. You don't need to see the whole page to track
down vandalism and such.

bzimport added a comment.Via ConduitFeb 3 2006, 10:30 PM

Siegel wrote:

I agree this would be useful. see bug #3365 and
<http://en.wikipedia.org/wiki/Wikipedia:Village_pump_%28technical%
29#Bare_diffs.3F>

bzimport added a comment.Via ConduitFeb 4 2006, 3:08 AM

lupin.wp wrote:

patch without user pref

This patch adds the diffonly flag which suppresses the article display if
present.

Attached: diffonly.patch

bzimport added a comment.Via ConduitFeb 5 2006, 8:13 PM

ezyang wrote:

*** Bug 3365 has been marked as a duplicate of this bug. ***

bzimport added a comment.Via ConduitFeb 5 2006, 8:16 PM

ezyang wrote:

Bug 208 may be related (the external API), but I don't think they have anything
to do with each other. Lupin, would you be able to extend user preferences to
make diffonly apply globally?

bzimport added a comment.Via ConduitFeb 6 2006, 3:58 AM

lupin.wp wrote:

add user pref

There's an option in Special:Preferences with this patch, under "Misc". You can
override that setting using diffonly=0 or diffonly=1 in the url.

attachment nodiff ignored as obsolete

bzimport added a comment.Via ConduitFeb 6 2006, 4:03 AM

lupin.wp wrote:

patch with user pref

Oops, forgot Messages.php - apologies.

Attached: nodiff

brion added a comment.Via ConduitFeb 6 2006, 6:34 AM

I'm leery of adding preference options; they complicate things.

bzimport added a comment.Via ConduitJul 20 2006, 12:00 AM

francesco.cosoleto wrote:

patch with user pref and "show diff with article" link

I added a "Show diff with article" link in diff generated page with diffonly
flag enabled. Pref or not pref, I think "diffonly" it's fine feature.

Attached: diffonlypatch.txt

bzimport added a comment.Via ConduitJan 4 2007, 2:56 PM

angus wrote:

*** Bug 8488 has been marked as a duplicate of this bug. ***

bzimport added a comment.Via ConduitJan 4 2007, 10:09 PM

ayg wrote:

*** Bug 6936 has been marked as a duplicate of this bug. ***

Ilmari_Karonen added a comment.Via ConduitJan 12 2007, 3:40 AM

Fixed in r19141 based on a combination of Francesco's patch with a similar
change I'd written myself.

ToAruShiroiNeko added a comment.Via ConduitJan 13 2007, 11:34 AM

So how does it work? How can this be implemented into our regular wiki usage?

bzimport added a comment.Via ConduitJan 13 2007, 11:36 AM

rotemliss wrote:

(In reply to comment #16)

So how does it work? How can this be implemented into our regular wiki usage?

This the the commit description:

(bug 3446) Add user preference to hide page content below diffs, can be
overridden by adding diffonly=1 or diffonly=0 to the URL of the diff page

I think it's clear enough.

bzimport added a comment.Via ConduitJan 14 2007, 10:50 AM

francesco.cosoleto wrote:

Thank you. (Not "Francesco's patch", but Lupin)

Add Comment