[Solved] Force Reason in Last edited

[MarkFL]

Originally Posted by Stratis

I thought that you see something else...
Now that you mentioned I see the tooltip, something that in all tests my eyes did not fall on, because I was watching editor with the post and the Required field. I will write in the phrase "Required" and the tooltips phrases.
Hope this will be ok, and not users keep trying to save it.

Thank you for your support helping me.

I made some improvements, among these is the ability to set a minimum character count for the edit reason field.

[Stratis]

At least now they wont put dots or comma,

I will write the "minimum characters allowed" in description to, this is a little issue
but makes what i want. Thanks a lot for another time

[MarkFL]

In the next update, I will include the minimum characters in the field description as well, and I am thinking to give you the option to define a list of acceptable reasons from which the user can choose from a drop-down menu (as well as use their own).

[MarkFL]

Okay Stratis,

I have attached the update containing the additional features I mentioned above.

[Stratis]

Very clever Mark, Clever things by clever man

The only is a tiny issue with google chrome and "MarkFL - Remove Post Edit History" Modification.

* When I remove as administrator the edit history, and member goes to edit his post and than select "suggested reasons" or write in reason field "Save" button does not take effect. If user press "Cancel' and again edit, than is no problem as described above, or If browser refreshed works well.

This happens only if History is cleared, if its not there is no issue.

With IE11, Edge and Firefox no problem in all situations.
This is only info for you, so you don't say that I am not a good tester.

Special thanks Mark

If finally publish it and you want, have in mind the time we have in standard vb options (after how many minutes/hours will show edited post by user)

[MarkFL]

I have attached an update, which now shows an alert on the quick edit form just like it does on the advanced edit form.

Originally Posted by Stratis

Very clever Mark, Clever things by clever man

The only is a tiny issue with google chrome and "MarkFL - Remove Post Edit History" Modification.

* When I remove as administrator the edit history, and member goes to edit his post and than select "suggested reasons" or write in reason field "Save" button does not take effect. If user press "Cancel' and again edit, than is no problem as described above, or If browser refreshed works well.

This happens only if History is cleared, if its not there is no issue.

With IE11, Edge and Firefox no problem in all situations.
This is only info for you, so you don't say that I am not a good tester.

I tested this in the updated version, and it seems to work correctly in Chrome (at least for me).

I would never say you are not a good tester...except perhaps to say you are an excellent tester.

Originally Posted by Stratis

Special thanks Mark

If finally publish it and you want, have in mind the time we have in standard vb options (after how many minutes/hours will show edited post by user)

That is an excellent suggestion to incorporate the intrinsic vBulletin permissions...so in addition to the usergroups you select to be exempt from giving an edit reason, usergroups who have "Show edited by note on edited messages?" set to "No" in the "General Permissions" section of the Usergroup Manager" will also be exempt, since requiring a reason would defeat that setting.

Also, anyone editing a post within the time limit defined in the setting "Time to Wait Before Starting to Display 'Last Edited by...'" under "Message Posting and Editing Options" will also be exempt.

So, if you don't mind Stratis, give this update a run through and let me know what you think.

[Stratis]

It is better now, the only thing is "Minimum Characters" does not work.

[MarkFL]

Originally Posted by Stratis

It is better now, the only thing is "Minimum Characters" does not work.

How so? Everything checks out for me...

[Stratis]

For me it is not ok.

Before show you some images, the previous worked and works, tested again, uninstalled the new and installed the previous, i did this 4 times to check the new xml with clean install.

Here is the limit
Attachment 155113

As you see in 4 characters i have
Attachment 155111

Saved after...
Attachment 155112

Here is with the previous xml, works fine.
Attachment 155114

[MarkFL]

In the image where you have "test" entered in the field, it says "Optional"...this could be because you are likely exempt due to editing before the time limit is up and/or because of your usergroup permissions...perhaps?