Add validation to limit discussion title length

[tobyzerner]

There should be a smaller limit on the length of a discussion title (60 characters?)

[franzliedke]

80?

[tobyzerner]

Yeah 80 sounds good.

[Chigusa0w0]

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

[franzliedke]

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

[qrokodial]

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.

[jgknight]

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.

[solhuebner]

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 ;-)

[tobyzerner]

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

[tobyzerner]

What needs to be done:

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

[franzliedke]

Fixed in #338, thanks @lbausch.

[ghost]

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: - /

[franzliedke]

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

[tobyzerner]

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?

[franzliedke]

I created an issue, thanks.