When the teaser length is set to "Unlimited" the option list value is set to 0 (zero). However this makes the teaser the same content and length as the body. This doubles the size of the node_revisions table with no obvious value to the site. I think "Unlimited" should set the length to 0 (zero) and always truncate the teaser.

Thanks
Emery Gordon

Comments

Jamie Holly’s picture

I don't agree on the teaser should always be truncated. There are sites/people out there that want to manually set their teaser lengths via the break slug, and no automatic teaser values.

As far as doubling node_revisions, I can see that being a problem - especially on sites using shared hosting with more limitations on data size. Perhaps a good alternative would be to not set teaser if it is the same as body. Then on node_load, if no teaser is found it automatically assigns node->body to node->teaser. This is something some of the more seasoned Drupal people should weigh in on though.

dragonwize’s picture

Title: Teaser length » Unlimited teaser length space saver
Category: bug » feature
Priority: Critical » Normal

This is defiantly not a normal way to handle the teaser. Unlimited is a valid option. If the space is not appropriate for you than that is an issue with your server not everyone's. It might make sense to just use the body out of the node table instead of storing it again, but that would probably cause some major reworking of the system to pull info correctly in all cases.

profjk’s picture

Project: Drupal core » Search Engine Referer API
Version: 6.6 » 6.x-1.x-dev
Component: node system » User interface

Does it affect SEO in any way?
This question is only for the Big daddies of Drupal!
I am also surprised that, under Project, you don't have Search Engine Optimization Section!!!!
Which forced me to post it here :(

dragonwize’s picture

Project: Search Engine Referer API » Drupal core
Version: 6.x-1.x-dev » 6.6
Component: User interface » node system

This has nothing to do with SEO. It is a question of database usage only. The system is working as designed. Please do not put issues in random project queues. That project has nothing to do with SEO other than it shares the words search engine in common.

mdupont’s picture

Status: Active » Closed (works as designed)

Works as designed in D6, and I don't think it would be changed at this stage. However, it seems that it's better in D7 (teaser only stored separately if it's different from full content).