Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Notice: Undefined index: field_page_attachments in /var/www/glowbox/company/sites/all/modules/contrib/apachesolr/apachesolr.module on line 1551
If a module is implementing hook_apachesolr_cck_fields_alter line 1551 throws errors because any added fields in the alter don't exist in $fields[$row->field_name]. Patch attached fixes this issue.
Note, this might be related also (I think) with #664866: Multiple indexes for one CCK field.
Comment | File | Size | Author |
---|---|---|---|
#4 | apachesolr_notices.patch | 2.56 KB | robertDouglass |
#3 | apachesolr_filters_notices.patch | 572 bytes | swentel |
#3 | apachesolr_operator_settings.patch | 550 bytes | swentel |
apachesolr_cck_fields_alter_notices.patch | 840 bytes | swentel | |
Comments
Comment #1
robertDouglass CreditAttribution: robertDouglass commentedCan you paste some example implementation that causes the error for my testing? Thanks.
Comment #2
swentel CreditAttribution: swentel commentedTaken from nd_search.module (from nd_contrib branch 6--2)
Comment #3
swentel CreditAttribution: swentel commentedI've attached two other patches fixing other notices. I did them seperately so you can review them easier, changed title to reflect that we're trying to fix several notices at once :)
I found two others which I couldn't fix immediately or didn't really understand what exactly was going on in there:
For the second notice, a backtrace leads to ../Solr_Base_Query.php:388 where the $themed variable has become an array instead of text. Probably the theme($breadcrum_name, $field) function errors here. The backtrace for the first one seems related too
Comment #4
robertDouglass CreditAttribution: robertDouglass commentedUnified patch.
Comment #5
robertDouglass CreditAttribution: robertDouglass commented#664884 by swentel: Fixed Several php notices.
Committing to 6.2. Thanks!
Comment #6
robertDouglass CreditAttribution: robertDouglass commentedDon't think these need porting.
Comment #7
pwolanin CreditAttribution: pwolanin commentedLooks like the code is rather different in 6.x-1.x