Advertisement:

Author Topic: Reason For Editing Mod  (Read 227215 times)

Offline winrules

  • SMF Friend
  • SMF Hero
  • *
  • Posts: 5,085
  • Gender: Male
Re: Reason For Editing Mod
« Reply #40 on: June 10, 2006, 03:14:25 PM »
Quote
$posterOptions = array();

      if (time() - $row['posterTime'] > $modSettings['edit_wait_time'] || $ID_MEMBER != $row['ID_MEMBER'])
      {
         $msgOptions['modify_time'] = time();
         $msgOptions['modify_name'] = addslashes($user_info['name']);
      }
      $msgOptions['edit_reason'] = addslashes(strtr(htmlspecialchars($_POST['edit_reason']), array("\r" => '', "\n" => '', "\t" => '')));

      modifyPost($msgOptions, $topicOptions, $posterOptions);

      // Changing the first subject updates other subjects to 'Re: new_subject'.

The red line is 2263
OK, I will fix that in th next version.
winrules
SMF Developer
               My Mods
Please do not PM me for support.


Offline littleone

  • Full Member
  • ***
  • Posts: 482
  • Gender: Male
Re: Reason For Editing Mod
« Reply #41 on: June 10, 2006, 03:44:40 PM »
ok when you do, can you post the code fix here, so I can just go in and edit that one piece?  Thanks ;)

Offline Vinspire

  • Sr. Member
  • ****
  • Posts: 852
  • Gender: Male
    • The Box Of Heart Forum
Re: Reason For Editing Mod
« Reply #42 on: June 11, 2006, 08:39:59 AM »
Winrules, manage to fix it already :)

Offline winrules

  • SMF Friend
  • SMF Hero
  • *
  • Posts: 5,085
  • Gender: Male
Re: Reason For Editing Mod
« Reply #43 on: June 11, 2006, 10:31:51 AM »
It's comming...
winrules
SMF Developer
               My Mods
Please do not PM me for support.


Offline winrules

  • SMF Friend
  • SMF Hero
  • *
  • Posts: 5,085
  • Gender: Male
Re: Reason For Editing Mod
« Reply #44 on: June 11, 2006, 11:26:36 AM »
Version 1.12 has been released! This version fixes a bug with 8: Undefined index: edit_reason errors in the error log.
winrules
SMF Developer
               My Mods
Please do not PM me for support.


Offline Vinspire

  • Sr. Member
  • ****
  • Posts: 852
  • Gender: Male
    • The Box Of Heart Forum
Re: Reason For Editing Mod
« Reply #45 on: June 11, 2006, 01:57:26 PM »
winrules, I've updated my version to 1.12. Its working perfectly. Thank you  :D :D :D

Offline geezmo

  • Full Member
  • ***
  • Posts: 441
Re: Reason For Editing Mod
« Reply #46 on: June 12, 2006, 11:49:02 AM »
I installed and uninstalled this mod before and now that i want to re-install it, I'm getting an error that the tables are already in the db. Can you tell me what to do with the existing tables so I can properly install this mod? I think these tables are "edit_reason" and "er_who"

Offline DemonicInfluence

  • Full Member
  • ***
  • Posts: 564
  • Gender: Male
    • The House Of Politics
Re: Reason For Editing Mod
« Reply #47 on: June 12, 2006, 11:57:48 AM »
Thanks for the great mod :)

Offline winrules

  • SMF Friend
  • SMF Hero
  • *
  • Posts: 5,085
  • Gender: Male
Re: Reason For Editing Mod
« Reply #48 on: June 12, 2006, 03:41:22 PM »
I installed and uninstalled this mod before and now that i want to re-install it, I'm getting an error that the tables are already in the db. Can you tell me what to do with the existing tables so I can properly install this mod? I think these tables are "edit_reason" and "er_who"
It's just the edit_reason. Go to phpmyadmin and delete it.
winrules
SMF Developer
               My Mods
Please do not PM me for support.


Offline littleone

  • Full Member
  • ***
  • Posts: 482
  • Gender: Male
Re: Reason For Editing Mod
« Reply #49 on: June 12, 2006, 07:09:15 PM »
Version 1.12 has been released! This version fixes a bug with 8: Undefined index: edit_reason errors in the error log.

fixed it.  Works perfectly.  Thanks ;)

Offline geezmo

  • Full Member
  • ***
  • Posts: 441
Re: Reason For Editing Mod
« Reply #50 on: June 12, 2006, 07:38:19 PM »
I dropped edit_reason but the er_who table still creates a prob. Here's the error:

Duplicate entry 'er_who' for key 1
File: /home/mysite/public_html/forum/Packages/temp/reasonforeditinginstall.php
Line: 3

Offline winrules

  • SMF Friend
  • SMF Hero
  • *
  • Posts: 5,085
  • Gender: Male
Re: Reason For Editing Mod
« Reply #51 on: June 12, 2006, 08:22:03 PM »
I dropped edit_reason but the er_who table still creates a prob. Here's the error:

Duplicate entry 'er_who' for key 1
File: /home/mysite/public_html/forum/Packages/temp/reasonforeditinginstall.php
Line: 3
Oh. er_who isn't a table, it's a row. In the settings table, drop the row for variable er_who.
winrules
SMF Developer
               My Mods
Please do not PM me for support.


Offline geezmo

  • Full Member
  • ***
  • Posts: 441
Re: Reason For Editing Mod
« Reply #52 on: June 12, 2006, 08:37:10 PM »
That's the thing, I can't seem to find the er_who row in smf_settings table. (It's not really there, it's not about me being unable to find the row). What query can I run in mySQL that can hopefully address this er_who problem?

Offline winrules

  • SMF Friend
  • SMF Hero
  • *
  • Posts: 5,085
  • Gender: Male
Re: Reason For Editing Mod
« Reply #53 on: June 12, 2006, 08:44:13 PM »
That's the thing, I can't seem to find the er_who row in smf_settings table. (It's not really there, it's not about me being unable to find the row). What query can I run in mySQL that can hopefully address this er_who problem?

Code: [Select]
DELETE FROM `{$db_prefix}settings` WHERE `variable` = 'er_who' LIMIT 1
winrules
SMF Developer
               My Mods
Please do not PM me for support.


Offline geezmo

  • Full Member
  • ***
  • Posts: 441
Re: Reason For Editing Mod
« Reply #54 on: June 12, 2006, 09:10:27 PM »
That did it. Thanks very much for the help. :)

Offline PrizeLive.com

  • Full Member
  • ***
  • Posts: 683
    • PrizeLive.com
Re: Reason For Editing Mod
« Reply #55 on: June 30, 2006, 12:58:39 PM »
Installed smoothly. Thanks!
Get Paid Instantly via PayPal (or other options) at PrizeLive.com!

Offline perplexed

  • SMF Hero
  • ******
  • Posts: 2,603
  • It's quite perplexing isn't it
Re: Reason For Editing Mod
« Reply #56 on: July 14, 2006, 12:32:47 PM »
I just installed on my test site and I entered a reason for editing the post, but where is the reason displayed?  I cant see it anywhere?

Offline PrizeLive.com

  • Full Member
  • ***
  • Posts: 683
    • PrizeLive.com
Re: Reason For Editing Mod
« Reply #57 on: July 14, 2006, 12:42:20 PM »
I made a screenshot for you.. It's displayed on the bottom of the post..

Get Paid Instantly via PayPal (or other options) at PrizeLive.com!

Offline winrules

  • SMF Friend
  • SMF Hero
  • *
  • Posts: 5,085
  • Gender: Male
Re: Reason For Editing Mod
« Reply #58 on: July 14, 2006, 01:18:24 PM »
I just installed on my test site and I entered a reason for editing the post, but where is the reason displayed?  I cant see it anywhere?
Also here is a close-up: http://mods.simplemachines.org/index.php?action=download;mod=349;id=15976;image :)
winrules
SMF Developer
               My Mods
Please do not PM me for support.


Offline perplexed

  • SMF Hero
  • ******
  • Posts: 2,603
  • It's quite perplexing isn't it
Re: Reason For Editing Mod
« Reply #59 on: July 15, 2006, 11:29:03 AM »
hey thanks for that, I tried it again and the reason is there, but it didnt show up yesterday.  maybe my fault

~thanks