Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
* notice: Undefined index: categories in ...\modules\aggregator\aggregator.pages.inc on line 145.
* warning: Invalid argument supplied for foreach() in ...\modules\aggregator\aggregator.pages.inc on line 145.
repro steps:
1. clean drupal cvs install
2. admin/build/modules enable Aggregator
2. admin/content/aggregator/add/feed (title: drupal, url: http://drupal.org/node/feed)
3. aggregator/sources/1/categorize click: Save categories
Comment | File | Size | Author |
---|---|---|---|
#6 | aggregator_categorize_2.patch | 1.27 KB | blackdog |
#3 | aggregator_categorize.patch | 1.21 KB | blackdog |
Comments
Comment #1
blackdog CreditAttribution: blackdog commentedThis looks like a duplicate, can you try the patch in http://drupal.org/node/198609.
Comment #2
Pasqualletested the patch in #198609 now, but it does not solve this issue.
Comment #3
blackdog CreditAttribution: blackdog commentedSorry, my bad.
I've attached a patch for this issue.
Comment #4
Pasqualletested, works
please commit after #198609 if possible, then no more rerolls needed.
Comment #5
Gábor HojtsyTechnically, we are looking for an !empty() right? Not an isset(). Seems like the selector could be a checkbox list or a select box, so we get an array in all cases, that should not be checked for, the FAPI checks for the values there anyway. Also the if() should have a space after the keyword.
Comment #6
blackdog CreditAttribution: blackdog commentedUpdated patch checks !empty instead, with spacing.
Comment #7
Pasqualleretested
Comment #8
Gábor HojtsyCommitted, thanks.
Comment #9
(not verified) CreditAttribution: commentedAutomatically closed -- issue fixed for two weeks with no activity.