dokuwiki icon indicating copy to clipboard operation
dokuwiki copied to clipboard

Show old media when viewing old page revision #2048

Open solewniczak opened this issue 8 years ago • 8 comments

One possibility of implementing it. Allow wiki admins to change the way how the old revision pages will work.

solewniczak avatar Sep 13 '17 10:09 solewniczak

@solewniczak, thanks for your PR! By analyzing the history of the files in this pull request, we identified @akate, @Klap-in and @splitbrain to be potential reviewers.

mention-bot avatar Sep 13 '17 10:09 mention-bot

just a quick note (I have not looked at the code, yet). I don't think this should have a config option. we either decide we want this and make it the default or we don't.

splitbrain avatar Sep 13 '17 16:09 splitbrain

In my opinion, the most intuitive will be showing the old medias but keep the links pointing to the latest pages.

There is also one more thing to concern. It's important to keep in mind that what the "&at=" is actually doing is showing the old revision of page with the media files in the moment of revision creation. What most people may except is showing the old revision of page with the media files in the moment when the actual version becomes an old revision. The picture below shows the problem. dokuwiki revisions

solewniczak avatar Sep 14 '17 09:09 solewniczak

Some thoughts: First of all I don't like the only_media option, I think it's superflous, so I'm not discussing this here.

The real question is: should the list of old revisions pass the revision timestamp as rev or at parameter?

  • rev will show only the old text of the page, but images will remain current
  • at will show the old text as well as the old images (as they were when the page revision was created)

It would be possible to pass not the date of the selected revision, but the date of the next newer revision minus 1 second. This would mitigate the Problem described by @solewniczak

at is basically superior to rev in that it covers more usecases (show old text and show how wiki looked before this change). However at is also slightly more ressource hungry (it needs to look up the correct revision of the page and all media files instead of being able to just load a specific file).

A config option would move the decision to the wiki administrators. But realistically 99.99% of all users would never change whatever we set as default.

Opinions welcome.

splitbrain avatar Oct 16 '17 13:10 splitbrain

Merge conflicts are fixed in oldmediarev branch

splitbrain avatar May 16 '19 13:05 splitbrain

That is a curious little problem. Intuitively, I would go with rev, because I'm looking at the old revision of the page after all and media files are not part of the page, only embedded. Though maybe, we can make the at option more prominent somewhere?

That being said, I'm not sure how Mediawiki does it. If I remember it, I'll try it out next week.

micgro42 avatar May 29 '19 22:05 micgro42

MediaWiki always shows the latest. Quick test:

Single instance:

  • https://wiki.osmfoundation.org/wiki/File:Maps.me_logo.png
  • https://wiki.osmfoundation.org/w/index.php?title=Corporate_Members&direction=next&oldid=6089

Multi instances:

  • https://commons.wikimedia.org/wiki/File:1_kawhi_leonard_2019_(cropped).jpg
  • https://en.wikipedia.org/w/index.php?title=Kawhi_Leonard&oldid=899755455

phy25 avatar Jun 17 '19 03:06 phy25

I would really like to see this functionality merged. Many wikis use publishing mechanisms (publish or structpublish) and rely on the fact that the content of the published revision does not change.

Although technically media are references and not content, this is not necessarily how users see it. Think of online handbooks or quality management: when I view older revisions of a page, I expect to see the exact charts or screenshots that were rendered when the revision was created.

annda avatar Aug 17 '23 09:08 annda