Moodle

limit size of tags

Details

  • Type: Sub-task Sub-task
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 1.9
  • Fix Version/s: None
  • Component/s: Other
  • Labels:
    None
  • Affected Branches:
    MOODLE_19_STABLE

Description

Maybe it´s a good idea to lower the limit for the size of tags ?? 50 characters max??

Take a look at this:
http://moodle.org/tag/index.php?id=71

Activity

Hide
Martin Dougiamas added a comment -

Yes, good idea. Can you make it truncate to 50 (in code only, don't touch db)

Show
Martin Dougiamas added a comment - Yes, good idea. Can you make it truncate to 50 (in code only, don't touch db)
Hide
Martin Dougiamas added a comment -

Best would be just to ignore and not add any new tags that are over the constant value (ie 50)

Show
Martin Dougiamas added a comment - Best would be just to ignore and not add any new tags that are over the constant value (ie 50)
Hide
Mathieu Petit-Clair added a comment -

del.icio.us has a length limit set to 128 chars. As tags are part of the url (which has a 255 chars limit), I guess this makes sense.

If the length is too long to get a good presentation, maybe these tags should be excluded from the tag cloud, or be abbreviated (in the way internationalization gets abbbreviated to i18n, or by just putting "..." at the end, etc.).

I don't think there should be a hard-coded limit on content for a question of presentation.

Show
Mathieu Petit-Clair added a comment - del.icio.us has a length limit set to 128 chars. As tags are part of the url (which has a 255 chars limit), I guess this makes sense. If the length is too long to get a good presentation, maybe these tags should be excluded from the tag cloud, or be abbreviated (in the way internationalization gets abbbreviated to i18n, or by just putting "..." at the end, etc.). I don't think there should be a hard-coded limit on content for a question of presentation.
Hide
Mathieu Petit-Clair added a comment -

We keep the current limit (50). I put a comment in the code not to exceed 85 chars, which could make the tag impossible to store in a varchar(255).

Show
Mathieu Petit-Clair added a comment - We keep the current limit (50). I put a comment in the code not to exceed 85 chars, which could make the tag impossible to store in a varchar(255).

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: