Add validation to limit discussion title length about framework HOT 14 CLOSED

I created an issue, thanks.

from framework.

80?

from framework.

Yeah 80 sounds good.

from framework.

Why not let site admins modify the max title length instead of write it into the source code? (but 80 is still a good length for title)
please forgive my poor english

from framework.

One reason that comes to my mind: the limit is probably enforced by database constraints (i.e. the title field in the discussions table).

from framework.

Why not let site admins modify the max title length instead of write it into the source code?

what would be the use-case for modifying the title length? adding customization for the sake of having customization adds unnecessary complexity when there is no realistic reasons for modifying it.

from framework.

what would be the use-case for modifying the title length?

The use case would be a forum admin wanting to increase or decrease the allowed length of forum titles.. While I agree with not adding complexity, searching around on Google provides a lot of results for people looking to customize the title length of different software (vBulletin, phpBB, etc.) Perhaps it's not a priority for initial release, but it certainly is not in the realm of 'unrealistic' to provide some method to change it.

from framework.

I think this is where the plugin concept can catch up on the end. Like I would like to see it as a bbPress replacement but I think this will not work out of the box. Although it would be awesome ;-)

from framework.

Also we should truncate the slug at 40 characters or so.

from framework.

What needs to be done:

• Add a validation rule to the Discussion model to limit the discussion title to 80 characters.

from framework.

Fixed in #338, thanks @lbausch.

from framework.

One reason that comes to my mind: the limit is probably enforced by database constraints (i.e. the title field in the discussions table).

The field title isn't used in foreign keys and therefore has no such restrictions.

80 characters for the title - it's VERY, very little to complete the work: - /

from framework.

@balancer I was referring to the size of the database column, but that is currently set to 200.

from framework.

It may as well be set to 255, 200 is kind of arbitrary.

The max title length should be configurable (not necessarily via default UI, but definitely via db settings). Do we have an issue for that elsewhere or should we reopen this one?

from framework.