Advertisement:

Author Topic: [2.0] Error if a member have pm labels with high ids (>60)  (Read 2071 times)

Offline digger

  • Sr. Member
  • ****
  • Posts: 747
  • Gender: Male
    • realdigger on GitHub
    • SMF Russian Community
[2.0] Error if a member have pm labels with high ids (>60)
« on: March 04, 2018, 01:51:50 PM »
If a member have pm labels with id>60 and try to assign that label to any message it shows labels_too_many error.

There is a code in PersonalMessage.php file

Code: [Select]
// Check that this string isn't going to be too large for the database.
if ($set > 60)
$updateErrors++;
I think, here should be something like this

Code: [Select]
// Check that this string isn't going to be too large for the database.
if ($smcFunc['strlen']($set) > 60)
$updateErrors++;

Offline digger

  • Sr. Member
  • ****
  • Posts: 747
  • Gender: Male
    • realdigger on GitHub
    • SMF Russian Community
Re: [2.0] Error if a member have pm labels with high ids (>60)
« Reply #1 on: December 12, 2018, 09:39:46 AM »
Is it fixed in 2.0.16?

Offline Arantor

  • Resident Overthinker
  • SMF Friend
  • SMF Legend
  • *
  • Posts: 69,931
    • StoryBB/StoryBB on GitHub
Re: [2.0] Error if a member have pm labels with high ids (>60)
« Reply #2 on: December 12, 2018, 09:50:42 AM »
Doubtful. It was a fairly large rewrite to fix in 2.1.
Don’t try to tell me that some power can corrupt a person. You haven’t had enough to know what it’s like.

No good deed goes unpunished / No act of charity goes unresented.

Offline live627

  • Developer
  • SMF Hero
  • *
  • Posts: 5,478
  • Gender: Male
    • live627 on Facebook
    • live627 on GitHub
    • live627 on LinkedIn
    • @live627 on Twitter
    • livemods
Re: [2.0] Error if a member have pm labels with high ids (>60)
« Reply #3 on: December 12, 2018, 05:52:05 PM »
I'm testing this fix now.
Try not to become a man of success, but rather try to become a man of value.
- Albert Einstein