I'd like to limit the terms extractor to a single content type. The attached patch adds an option to admin/settings/yahoo_terms so that the administrator can select content types to act on. If no content types are selected, then it continues to work as-is, on all content types.

CommentFileSizeAuthor
yttypes.patch2.69 KBdouggreen

Comments

electblake’s picture

electblake commented
Version: 5.x-1.x-dev » 6.x-6.x-dev

I checked out latest from cvs and applied this patch and YES, it worked like gang busters. ping me to get patched version of yahoo_terms.

thanks doug :)

Only thing I can think of is that at line 228.

'#options' => node_get_types('names'),

maybe we don't use just full array details of a type and merely the name, a suggestion. Overall this works and if someone else wants to review this should be comited for others wicked soon.

Shiny’s picture

Shiny commented
Assigned: Unassigned » Shiny
Status: Needs review » Patch (to be ported)

will be ported to D6

Shiny’s picture

Shiny commented
Status: Patch (to be ported) » Fixed

D6 version in CVS - all unit tests passing

Shiny’s picture

Shiny commented
Version: 6.x-6.x-dev » 7.x-2.x-dev
Status: Fixed » Patch (to be ported)

Needs porting to D7 now