Feature Proposal: Making clear attachments are versioned

Motivation

Quite a few users, even long time users, are not aware of the fact that attachments are versioned.

Description and Documentation

This proposal introduces a new row to the attachment table displayed at the bottom of a topic. The new column is titled "History" and has hyperlinks to retrieve past versions of an attachment. This not only makes clear that attachments are versioned, but also is handy retrieve past versions.

On each row, the History column shows the latest version (not a hyperlink) and at most $TWiki::cfg{NumberOfRevisions} of the past versions (hyperlinks to retrieve corresponding versions). This is similar to how a topic's history is shown at the action buttons area.

There are cases where an attachment has no versions. The attachments of the TWiki06x01 web topics are as such. For those attachments, nothing is shown on the History column.

Examples

Impact

Implementation

-- Contributors: HideyoImazu - 2011-12-02

templates/attachtables.tmpl

• ATTACH:files:header(toprow)
• ATTACH:files:row(basic)
  • It will have %A_HISTORY% to show the version history of the attachment.

lib/TWiki/Attach.pm

• _expandAttrs()

Discussion

Simple proposal I support. Small disadvantage though is that it adds a bit more clutter to the attachment table, but I think that is OK.

Question is what to do if there are more then a few revisions. This could be solved by limiting to the 4 latest revisions and a ... link pointing to the attachment screen of that attachment if there are more than 4 revisions.

The clock for 7 day review period for acceptance starts once the CommittedDeveloper and DateOfCommitment is filled out.

-- PeterThoeny - 2011-12-02

I agree with the way to handle many revisions. It would be like r10 < r9 < r8 ... on the bottom toolbar.

-- HideyoImazu - 2011-12-16

Accepted now based on 7 day feedback period rule.

-- PeterThoeny - 2012-01-02

Feature looks good in SVN trunk!

-- PeterThoeny - 2012-04-16

Once finished flip the state here and in TWikibug:Item6826.

-- PeterThoeny - 2012-04-16