patch - associate term description with node teaser

domas - October 21, 2007 - 10:34
Project:Node Auto Term [NAT]
Version:HEAD
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:won't fix
Description

Here is a patch that allows the user more options regarding population of the term's description. Had to change a few things in the code to do this (nothing major though).

AttachmentSize
term_teaser.patch5.53 KB

#1

domas - October 21, 2007 - 19:21
Status:active» needs review

Sorry, forgot to set status to patch

#2

Zen - October 24, 2007 - 08:25
Status:needs review» needs work

Thanks for the patch Domas. I've only looked at the patch visually and wanted to bring a couple of issues to light:

  1. If the variable name 'body' is changed to 'desc', an update will be required. I recommend just leaving it as 'body'. It's not all that much of a misnomer.
  2. There are a few style issues. Please run the patched module through the coder module.
  3. Ideally, comments would be capitalised and terminated with a period.
  4. Please also fix the doxygen comments for the new function.

Cheers,
-K

#3

Summit - January 20, 2009 - 08:47

Subscribing,
very interested in this feature, is there still light in this tunnel?

Greetings,
Martijn

#4

Zen - February 14, 2009 - 21:50
Category:task» feature request
Status:needs work» won't fix

NAT already supports Node body -> Term description syncing. For UI reasons, adding a teaser option is highly unlikely.

#5

xn2001 - March 10, 2009 - 19:09

@Zen

Can you elaborate on the "UI reasons" why adding a teaser option is highly unlikely?

Glossary and this module make a good combination to implement my glossary system. However, it would be perfect if this module has a teaser option.

 
 

Drupal is a registered trademark of Dries Buytaert.