Only show a single revision message

[alexvorn2]

​http://wpimpact.com/wp-content/uploads/2012/10/Untitled-3.png

• If a user edits a topic or a reply multiple times than log info is added. I think it would be nice to show ONLY the last one.

[johnjamesjacoby]

Agree it would be nice to be able to edit this meta directly; moving to future release.

[jaredatch]

jjj - what are you thinking? Possibly a filter that when set to try only shows the most recent one?

[MZAWeb]

Added patch. Introduced 'bbp_reply_revision_log_limit' filter.

[johnjamesjacoby]

Patch works, but hoping for something a bit more sexy (maybe a cumulative count of total edits?) Moving to 2.5.

[MZAWEb]

Working on a patch already.

Show 3 + total count + ajax loader to see all. Will try to post this week.

[MZAWeb]

Can't believe how lazy I was with the other patch.

1989.patch does it better, both for topics and replies. Show 1 revision (with a filter to modify) and a link "Show %d more" that will expand. Didn't implement the AJAX loading yet, so for now it reloads the page (and will work on themes without jQuery). Want to validate the approach, get it fixed (or rejected) first, then moving it to AJAX is super easy.

Because there's a way simpler approach, but it's jQuery dependent: just echo everything in a hided div and collapse/expand with jQuery.

[MZAWeb]

Same patch, without whitespace diff :)
Note that in latest trunk there are a few wrongly indented functions in those files.

[johnjamesjacoby]

Fix indentation from 1989-no-whitespace.patch

[johnjamesjacoby]

Replying to MZAWeb:

Same patch, without whitespace diff :)
Note that in latest trunk there are a few wrongly indented functions in those files.

Thanks for this. This is a pretty good iteration. Let's try to work on this early for 2.5.

[johnjamesjacoby]

Some ideas while looking at this code.

• The markup could be separated out into a template part used by both topics and replies.
• Always hate adding new parameters, since it means committing to those changes forever.
• Since this is a developer option (without a ui for tweaking the limits) we have an opportunity to really improve this significantly.
• The $total = $limit = - 1; bit is happening before $limit is set. This should probably be somewhere else?

[MZAWeb]

Replying to johnjamesjacoby:

• The markup could be separated out into a template part used by both topics and replies.

Great idea. It's was bothering me a bit that much duplicated code. Will do.

• Always hate adding new parameters, since it means committing to those changes forever.

If you mean the the params I added to bbp_get_*_revisions, the only other option is to grab always the full results and chop off the array, which seem inefficient.

If you mean the $_GET params, I'm not sure of any other approach other than going only the AJAX way.

For any of those, I'd love to hear suggestions. Maybe I'm just missing something obvious.

• Since this is a developer option (without a ui for tweaking the limits) we have an opportunity to really improve this significantly.

Sorry, don't follow. What do you want me to improve?

• The $total = $limit = - 1; bit is happening before $limit is set. This should probably be somewhere else?

The filter is inside the IF. If the user clicked to expand and we have the $_GET param, we don't want to run the filter, we want to do -1 to grab all, right? Or do you want me to filter the -1 too? Seems weird...

[johnjamesjacoby]

There are variables in your patch that are being referenced that have not been defined, triggering PHP notices. I'm uncertain where that data is suppose to come from, so I can't call this a functioning or complete patch to give accurate feedback.

Do you mind taking a look at mine, and improving on it? Moving to 2.6 so we can take our time with this one.

[johnjamesjacoby]

Bump to 2.7.

[MZAWeb]

Replying to johnjamesjacoby:

There are variables in your patch that are being referenced that have not been defined, triggering PHP notices. I'm uncertain where that data is suppose to come from, so I can't call this a functioning or complete patch to give accurate feedback.

Uhm weird. Maybe I need to refresh it to the latest trunk.

Do you mind taking a look at mine, and improving on it? Moving to 2.6 so we can take our time with this one.

Sure. Will do that asap.

[MZAWeb]

@johnjamesjacoby:

I just applied your patch to the latest trunk and the feature works as expected, I don't see any PHP notice in browser or logs and my IDE is not showing me any unreferenced variable. Can you give me a hint on how to reproduce?

Edit: Weee

Edit: Weee 2

[johnjamesjacoby]

Bumping all 2.7 to 2.8 milestone.

[Clorith]

Using the beast that is the w.org forums as my basis here (​https://wordpress.org/support/topic/forum-bugs-and-broken-things/ the edit list is... fun)

Although comment:16 is a nice approach if all you have are edits to the post content, this does not account for other scenarios such as edit logs of tag modifications, topic movement between forums and such.

I'm far more in favor of the approach from comment:8 where the full list can be expanded with a click and only seeing X ones by default.

Maybe making the modified text a link to see the diff as suggested in comment:16, but that is if the edit is a text one and not a meta one.

[netweb]

Related: ​#meta2232