Provide option to restrict terms extractor by content type
douggreen - August 29, 2007 - 21:29
| Project: | Yahoo Terms |
| Version: | 6.x-6.x-dev |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | needs review |
Jump to:
Description
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.
| Attachment | Size |
|---|---|
| yttypes.patch | 2.69 KB |
#1
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.