Various modifications for phpBB3 ⇒ Reason for editing post required

[martin]

Force user's to fill out the reason for editing a post.

Open /posting.php

Find This may be a partial find and not the whole line (spaces may differ)

Code: Select all

if (sizeof($message_parser->warn_msg))
    {
        $error[] = implode('<br />', $message_parser->warn_msg);
    }

Add after Add these lines on a new blank line after the preceding line(s) to find

Code: Select all

// When editing, a reason is mandatory for all users
        if( $mode== 'edit'&& !$post_data['post_edit_reason'] ) 
        {
        $error[]= 'Edit reason is required!';
        }

(around line 666):

Find This may be a partial find and not the whole line (spaces may differ)

Code: Select all

$post_data['post_edit_reason']   = (!empty($_POST['edit_reason']) && $mode == 'edit' && $auth->acl_get('m_edit', $forum_id)) ? utf8_normalize_nfc(request_var('edit_reason', '', true)) : '';

Replace with Replace the preceding lines with the following

Code: Select all

$post_data['post_edit_reason']   = (!empty($_POST['edit_reason']) && $mode == 'edit'/* && $auth->acl_get('m_edit', $forum_id)*/) ? utf8_normalize_nfc(request_var('edit_reason', '', true)) : '';

(around like 1448):

Find This may be a partial find and not the whole line (spaces may differ)

Code: Select all

'S_EDIT_REASON'            => ($mode == 'edit' && $auth->acl_get('m_edit', $forum_id)) ? true : false,

Replace with Replace the preceding lines with the following

Code: Select all

'S_EDIT_REASON'            => ($mode == 'edit'/* && $auth->acl_get('m_edit', $forum_id)*/) ? true : false,

[Raheem]

Better than forcing him write reason, you can make it like in vBulletin if member edited his post without write the reason it will be:
Last edited by USERNAME; 06-09-2013 at 08:03.

And if he wite the reason it will be like: Last edited by USERNAME; 10-12-2009 at 20:35. Reason: REASON!

[martin]

Did you look at the date of this topic ?

[Raheem]

Yes what's the problem then?

[Sniper_E]

I did not like this modification when you were using it on your site awhile back. But I did love entering in "I don't need a reason fool!"

[ Post made via Samsung Galaxy S5 Active ]

[dmzx]

I did not like this modification when you were using it on your site awhile back. But I did love entering in "I don't need a reason fool!"

[ Post made via Samsung Galaxy S5 Active ]

[martin]

I did not like this modification when you were using it on your site awhile back. But I did love entering in "I don't need a reason fool!"

[ Post made via Samsung Galaxy S5 Active ]

Might add it back but allow admins to carry on with out a reason.

[Sniper_E]

Might add it back but allow admins to carry on with out a reason.

You might give me another reason to call you a fool!

[martin]

Got it working on 3.2.1

[Sniper_E]

I was afraid that might happen.

[dmzx]

I was afraid that might happen.

[martin]

Edits are in my admin section Dont steal them Ed