Title and description MINIMUM LENGTH, how to avoid that users use SPACE to fulfil the obligation

Hi,

There is no current way. My god, that's nasty of your users...

Fixing it is probably semi trivial. Line 884 in node.module:

// Check the length of the title and the body
//
if(variable_get('drigg_body_minimum_length',50 ) != 0 && strlen(strip_tags($node->body)) < variable_get('drigg_body_minimum_length',50)){
form_set_error('body', t('The description is too short!'));
}
...
if(variable_get('drigg_title_minimum_length',10) != 0 && strlen(strip_tags($node->title)) < variable_get('drigg_title_minimum_length',10)){
form_set_error('title', t('The title is too short!'));
}

Can you send me a patch that does the same checks but trims spaces before and after the body and the title?

I can't believe they are doing it.

Merc.

I'm not sending a patch because I don't know how to do one, but just to tell the author what is needed, it is simply adding the trim() function to those lines : strlen(strip_tags($node->body)

so the code would look like :

// Check the length of the title and the body
//
if(variable_get('drigg_body_minimum_length',50 ) != 0 && strlen(strip_tags(trim($node->body))) < variable_get('drigg_body_minimum_length',50)){
form_set_error('body', t('The description is too short!'));
}
...
if(variable_get('drigg_title_minimum_length',10) != 0 && strlen(strip_tags(trim($node->title))) < variable_get('drigg_title_minimum_length',10)){
form_set_error('title', t('The title is too short!'));
}

but if your users then use dots or other characters to fill in the body and title, find out where they live and kill them!

Hi,

Sorry, I got the name wrong.
I am afraid it's impossible to fix it if users use dots etc. There will always be a way to fool even the most complex AI.

Hey, feel free to send over your modifications, so that I can add them to Drigg!

Merc.

Hi,

Silly me, I completely missed Alliax's patch!
Sorry about that. And yes, applied to CVS.

Merc.