Now that the "Reason" truncation setting has been committed to the main Userpoints module, we need to remove the equivalent truncation setting from User points Nodes and Comments. It isn't needed anymore!

--Ben

CommentFileSizeAuthor
#1 remove_truncation.patch1.58 KBBerdir
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Berdir’s picture

Status: Active » Needs review
FileSize
1.58 KB

Patch is simple, doesn't need much testing I guess.

BenK’s picture

Status: Needs review » Needs work

I tested this and the truncation setting went away without any ill effects. And all of my "Added [Title]" reasons had truncation successfully removed.

Only thing thing I noticed: If the points transaction was provided by the Revisions sub-module, then the truncation didn't go away. It appears to still be truncated according to the old setting. (I'm talking about reasons in the format "Revised [Title]".

Is there some code in the Revisions sub-module that needs to be deleted too (now that this module is part of up-nc)?

--Ben

Berdir’s picture

Status: Needs work » Fixed

Yes, there is code. Removed that and commited the patch.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.