Reason For Editing Mod

[ivo2296]

Hello,

Is there a way the reason text to be in red color?

The whole edited by line or just the reason?

I think whole line will be fine.

Thanks

[geezmo]

Had this for 1.1RC2 but when I upgraded to 1.1RC3 and tried to install this mod, I got this Database Error:

Duplicate column name 'edit_reason'
File: /home/xxx/public_html/forum/Packages/temp/reasonforeditinginstall.php
Line: 2

Help?

[winrules]

Had this for 1.1RC2 but when I upgraded to 1.1RC3 and tried to install this mod, I got this Database Error:

Duplicate column name 'edit_reason'
File: /home/xxx/public_html/forum/Packages/temp/reasonforeditinginstall.php
Line: 2

Help?

Delete the edit_reason column from the messages table and it should work.

[geezmo]

I dropped it but when I tried to upgrade the mod, I got another error saying I need to drop another table called "er_who". I'm a newbie at these things so please tell me if there are other tables need to dropped and what the command is in phpmyadmin. Thanks.

[winrules]

Code Select Expand


DELETE FROM `{$db_prefix}settings` WHERE `variable` = 'er_who' LIMIT 1

Substitute {$db_prefix} with your database prefix. Probobally "smf_".

[geezmo]

Needed to delete one more table "edit_reason" in settings but anyway, thanks winrules, the mod's now working...

[Vinspire]

winrules, it seems like my forum doesn't like ur mod ...

after i upgraded my forum board from RC 2 or RC 3 i kept getting this error when i wanted to install your mod

Code Select Expand

Installation Readme
Sorry, but there is no upgrade package from your version. Please uninstall the previous version, and install this version. You won't lose any data.

Install Actions
Installations actions for "Reason For Editing Mod":
The package you are trying to download or install is either corrupt or not compatible with this version of SMF.

it happened on 2 of ur mod that i had it install on the board previously .... i uninstall it and re-install it a couple of time but still didnt work.

this problem occur in Custom Action Mod too.

[NEMINI]

Is there a way to require this field when editting?

Not right now.

Is this a possibility for a future version?

[winrules]

winrules, it seems like my forum doesn't like ur mod ...

after i upgraded my forum board from RC 2 or RC 3 i kept getting this error when i wanted to install your mod

Code Select Expand

Installation Readme
Sorry, but there is no upgrade package from your version. Please uninstall the previous version, and install this version. You won't lose any data.

Install Actions
Installations actions for "Reason For Editing Mod":
The package you are trying to download or install is either corrupt or not compatible with this version of SMF.

it happened on 2 of ur mod that i had it install on the board previously .... i uninstall it and re-install it a couple of time but still didnt work.

this problem occur in Custom Action Mod too.

Remove any lines that say Reason For Editing Mod for in Packages/installed.list and it should work.

Is there a way to require this field when editting?

Not right now.

Is this a possibility for a future version?

Maybe.

[Vinspire]

Remove any lines that say Reason For Editing Mod for in Packages/installed.list and it should work.

Did what you said ... But i've received this error when I tried to apply the mod 

Code Select Expand

Database Error
Duplicate column name 'edit_reason'
File: /home/korobox/public_html/forum/Packages/temp/reasonforeditinginstall.php
Line: 2

[geezmo]

Vinspire, delete that column in the smf_settings table. I had the same problem some posts up.

[Vinspire]

Vinspire, delete that column in the smf_settings table. I had the same problem some posts up.

Is this the one ?

Code Select Expand


DELETE FROM `{$db_prefix}settings` WHERE `variable` = 'er_who' LIMIT 1


I've never played wif the database before and i am afraid i would screw it up

[geezmo]

Yes, I think you need to delete three columns in two tables, including edit_reason and er_who. These columns will appear when you try re-installing the mod, you'll get the error that the database is already existing.

[richiehale]

Problem installing upgrade.. any suggestions?? 

Apply Mod
Please remember to always make regular backups of your sources and database before installing mods, especially beta versions. 


!!Error in Package Installation

At least one error was encountered during a test installation of this package. It is strongly recommended that you do not continue with installation unless you know what you are doing, and have made a backup very recently. This error may be caused by a conflict between the package you're trying to install and another package you have already installed, an error in the package, a package which requires another package that you don't have installed yet, or a package designed for another version of SMF.
Installation Readme
This will upgrade the mod from version 1.0 to 1.12.


Install Actions
Installations actions for "Reason For Editing Mod":
Installing this package will perform the following actions:  Type Action Description
1. Execute Code reasonforeditingupgrade.php 
2. Execute Modification ./Themes/default/Post.template.php Test failed
3. Execute Modification ./Themes/default/Display.template.php Test successful
4. Execute Modification ./Sources/Subs-Post.php Test successful
5. Execute Modification ./Sources/ModSettings.php Test successful
6. Execute Modification ./Sources/Post.php Test successful

[winrules]

Have you changed anything with the mod in Display.template.php?

[richiehale]

no

[perplexed]

I have managed to install this mod finally but there is something interesting that happens with another mod

merge double posts

If you merge double posts and one of them has a 'reason for editing' comment, then it disappears

[winrules]

You'll have to modify it to get it to move the reason for editing.

[perplexed]

modify the reason for editing mod or the merge double posts mod?

(Sorry I'm catching up on things after being offline for a month or so)

[winrules]

The merge double posts mod. I'm not quite sure how it works.