Hi,

I find this error in the new branch 6.3-x

@ function apachesolr_field_name_map().

cause an foreach(apachesolr_entity_fields('nore') as $field_nm => $nodefields) { ...

I just did git clone --recursive --branch 6.x-3.x http://git.drupal.org/project/apachesolr.git

PS: I look for this error after create this issue. Sorry if I recreate one did not find it.

CommentFileSizeAuthor
#2 offset-as-an-array-1630222-2.patch523 bytesbasvredeling
#1 offset-as-an-array-1630222-1.patch2.04 KBkillua99

Comments

killua99’s picture

killua99 commented
StatusFileSize
new offset-as-an-array-1630222-1.patch2.04 KB

Some can see this "patch"

basvredeling’s picture

basvredeling
Primary language Dutch
Location Amsterdam
commented
Status: Active » Needs review
StatusFileSize
new offset-as-an-array-1630222-2.patch523 bytes

This is a result of the patch in #1494576: Taxonomy facets
It's an issue of proper array indexing. Attached patch should solve this. The patch from #1 just catches the error but doesn't solve the issue that taxonomy vocabularies aren't handled by the latest apachesolr 6.x-3.x-dev

killua99’s picture

killua99 commented

Yeah, thats the problem. Thanks for take a look at this issue!

basvredeling’s picture

basvredeling
Primary language Dutch
Location Amsterdam
commented
Status: Needs review » Reviewed & tested by the community

It's a small fix and it's been confirmed by another community member. I'd say RTBC.

nick_vh’s picture

nick_vh
he/him
Location Ghent
commented

So, which of the patches should I commit, both?

killua99’s picture

killua99 commented

#1630222-2: Fatal error: Cannot use string offset as an array in /sites/all/modules/contrib/apachesolr/apachesolr.module on line 1698

nick_vh’s picture

nick_vh
he/him
Location Ghent
commented
Status: Reviewed & tested by the community » Closed (duplicate)

Closing this one, handling it in #1588256: Warning: Invalid argument supplied for foreach() in apachesolr_entity_update() (line 1766 of /apachesolr/apachesolr.module)