[drupal-devel] [bug] $teaser misnamed in themes (theme_node)

robertDouglass drupal-devel at drupal.org
Wed Jul 13 17:50:54 UTC 2005


Issue status update for 
http://drupal.org/node/26637
Post a follow up: 
http://drupal.org/project/comments/add/26637

 Project:      Drupal
 Version:      cvs
 Component:    theme system
 Category:     bug reports
 Priority:     normal
 Assigned to:  Anonymous
 Reported by:  robertDouglass
 Updated by:   robertDouglass
 Status:       patch
 Attachment:   http://drupal.org/files/issues/main_teaser.patch.txt (3.1 KB)

The documentation to theme_node clearly says the the parameter $teaser
is "Whether to display the teaser only, as on the main page." Naming
this parameter $main in the themes is confusing. Main is used elsewhere
to mean "whether to show full view or teaser view", but the meaning of
$teaser seems to be subtly different. 


An alternate to the patch I'm suggesting would be to eliminate $teaser
altogether, system-wide. The variable $main makes it clear that it is
transmitting state (whether or not on the node's main page), whereas
the variable $teaser makes on think it is transmitting the actual
teaser.




robertDouglass




More information about the drupal-devel mailing list