Typo in revisions count

[Jonnyauk]

If you have added a filter to wp_revisions_to_keep to alter the number of post revisions saved, an unnecessary (and possibly confusing) + character is outputted after the number of revisions in the publish metabox.

Simple patch attached that removes character.

[Jonnyauk]

Initial patch to remove + character

[ocean90]

That's not a typo. The + sign should show that there can be more revisions if they aren't limited. See also the title which should come up on mouse over:

Your site is configured to keep only the last %s revisions.

[claudiosanches]

Plus sign (+) indicates only that row has been added.
And minus sign (-) when it was deleted.
It's all right with that.

Example of how it works diff tool somewhere else: ​https://gist.github.com/helenhousandi/1573966/revisions

And see your patch file and click on "Unified".

[Jonnyauk]

Ah - I understand, thanks! However, I'd still propose that it could be potentially confusing to normal 'end' users. If the developer has limited the revisions for a reason, does the normal end user need to have an indication that there is the capability for more revisions?

@ocean90 - thanks for your theory behind this - and it does make sense from that perspective. If I have a post with 5 revisions available, then change the filter to 3 and go back to edit the post it should indicate if there are more revisions available with the + sign.

However, once I've made amendments to the post and saved again, the amount of revisions saved (and available) in the database is limited by the filter - so there are only 3 available in this example, yet it still shows +

This is why I think this is either redundant or needs a slightly different patch created to compare revisions available with the current limit - and then only in that case show the + if there are more revisions available than the current limit.

[nacin]

"Revisions: X" is not just to link to the new revisions screen, but also to provide the user a data point. For example, "this post underwent 65 individual changes while drafting it". It isn't much different than a word count metric. It is also better than the long, ridiculous meta box that otherwise serves little purpose.

If the install is restricted to 3 revisions, but a user has saved 5 times, then they have edited that post 3+ times. Technically, if they have only saved 3 times, it still shows "3+". You can read "3+" as "3 or more". The fourth save will again continue to show "3+".

So just keep in mind it isn't just about what is stored — the actual count could be useful or interesting to someone.

[Jonnyauk]

@nacin ah - I understand the decision behind this now - thanks for the explanation.

[nacin]

Replying to Jonnyauk:

@nacin ah - I understand the decision behind this now - thanks for the explanation.

No problem.