Comments

isolate’s picture

isolate’s picture

Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, pathauto_persist-conflicts_with_search_api_pages-2107365-1.patch, failed testing.

isolate’s picture

Status: Needs work » Needs review
StatusFileSize
new623 bytes

New fix.

azinck’s picture

Works for me.

For the sake of others searching for this, this patch fixes the error:

Warning: Illegal string offset 'pathauto' in pathauto_persist_entity_load()
Bevan’s picture

Issue summary: View changes
Status: Needs review » Reviewed & tested by the community

Patch is clean and solves the issue.

damienmckenna’s picture

Version: 7.x-1.3 » 7.x-1.x-dev
Status: Reviewed & tested by the community » Needs review
StatusFileSize
new592 bytes

I've shortened the patch slightly.

mkhamash’s picture

I con confirm that patch number 7 does solve the problem with fieldable_panels_panes module also.

stefan.r’s picture

Status: Needs review » Reviewed & tested by the community

If the previous patch was RTBC then #7 should be as well as it does exactly the same :)

Jaesin’s picture

+1 for a commit on this (Tested).

I just wrote the exact same patch as #7 before searching on this.

hey_germano’s picture

Tested here too, thanks for the patch!

eelkeblok’s picture

Wouldn't this complain (notice) if the ->path doesn't exist at all (directly trying to set an index inside a non-existing array)? I submitted the attached patch for #2489284 which boils down to the same issue. My patch basically does the same thing, but starts out by setting an empty array when the ->path does not exist.

damienmckenna’s picture

Please don't forget to change the issue status to "needs review" when you upload a new patch ;-)

stefan.r’s picture

Status: Reviewed & tested by the community » Needs review
eelkeblok’s picture

Or when correcting someone about it ;) I wasn't sure this should be considered a new patch for this issue, but I suppose you are right.

anas_maw’s picture

Thanks 7 worked for me and fixed fieldable_panels_panes module path issue also.

leolandotan’s picture

Based from #12, I think this would work for me because I have a local environment using PHP 5.4.4 and a dev environment using PHP 5.6(just updated) and I'm getting the error message

Warning: Illegal string offset 'pathauto' in pathauto_persist_entity_load() (line 10 of /mnt/www/html/mysite/www/sites/all/modules/contrib/pathauto_persist/pathauto_persist.module).

on the dev environment. By the way I'm mainly using Core 7.39, CTools 1.9, Pathauto 1.2 and Pathauto Persistant state 1.3. I'll try this patch later.

dave reid’s picture

Status: Needs review » Fixed

Committed #12 to 7.x-1.x.

  • Dave Reid committed 64aa39a on 7.x-1.x authored by eelkeblok
    Issue #2107365 by iSoLate, DamienMcKenna, eelkeblok: Fixed conflict with...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.