Trying to categorize aggregated items: when using checkbox method the checkboxes are not displayed. When using multiple-select, they are displayed, but clicking "Save Categories" does not add selected items to the category.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | aggregator-categorization.patch | 1.43 KB | joshk |
Comments
Comment #1
mgiffordVerified this with a cvs version (original bug report was against the last beta release).
Went here:
aggregator/sources/2/categorize
Categorized a couple of items, hit Save Category and then there is an error message that says:
warning: Invalid argument supplied for foreach() in /var/www/wlp_play/modules/aggregator.module on line 1007.
The categories have been saved.
I created a patch to check to see that $form_values['categories'] was an array in:
foreach ($form_values['categories'] as $iid => $selection) {
But that didn't actually save the changes, so it's not worth contributing as a patch.
I think there's a problem with $form_values in 4.7, but don't know it enough to track it down any further.
This is set up with the Category module, but don't think that this is throwing things off here.
Mike
Comment #2
joshk commentedHere's a patch. It does two things:
1) Changes the settings so that the variable stored if checkboxes are selected is 'checkboxes' rather than 'check.'
2) Changes the form_submit handler for categorization to correctly implement the new Form API
With these changes, categorization-per-item is back in action.
Comment #3
dries commentedCommitted to HEAD. Thanks.
Comment #4
(not verified) commented