Problem/Motivation

I have a site that is using panelizer and the custom_breadcrumbs_panels and I've continually seen the error messages below because the handler does not contain a display variable. The breadcrumbs appear to work even with the Warning/Notice.

Error messages

Notice: Trying to get property of non-object in custom_breadcrumbs_panels_ctools_render_alter() (line 131 of /var/www/html/sites/all/modules/contrib/custom_breadcrumbs/custom_breadcrumbs_panels/custom_breadcrumbs_panels.module).
Warning: Invalid argument supplied for foreach() in custom_breadcrumbs_panels_ctools_render_alter() (line 131 of /var/www/html/it/sites/all/modules/contrib/custom_breadcrumbs/custom_breadcrumbs_panels/custom_breadcrumbs_panels.module).

Proposed resolution

Check that the $handler->conf['display'] exists before trying to reference it.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

zipymonkey created an issue. See original summary.

zipymonkey’s picture

FileSize
970 bytes

Created a patch to check that the $handler->conf['display'] exists before referencing it.

zipymonkey’s picture

Status: Active » Needs review
renatog’s picture

Assigned: zipymonkey » renatog
Issue summary: View changes
Status: Needs review » Reviewed & tested by the community
Issue tags: +ciandt-contrib
FileSize
12.06 KB
57.68 KB

Hi people.

I applied the patch and works good for me.

Before:

After:

Thank you very much @zipymonkey.

Good Work.

Regards.

  • RenatoG committed 10c4e84 on 7.x-2.x authored by zipymonkey
    Issue #2848472 by zipymonkey, RenatoG: Warning and Notice about non-...
renatog’s picture

Status: Reviewed & tested by the community » Fixed
FileSize
12.25 KB
58.72 KB

Fixed.

Commited in dev branch.

Thank you very much for your contribution @zipymonkey.

Good Work.

Regards.

Status: Fixed » Closed (fixed)

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