This error arose when using mail_edit and panels_breadcrumbs. According to the documentation at http://api.drupal.org/api/drupal/modules%21system%21system.api.php/funct..., it seems that one can not assume that the item array will have a "path" element.

CommentFileSizeAuthor
#2 mail_edit-undefined_index_path-1826220-2.patch591 bytesacrollet
#1 mail_edit-undefined_index_path-1826220-1.patch590 bytesacrollet

Comments

acrollet’s picture

acrollet commented
Status: Active » Needs review
StatusFileSize
new mail_edit-undefined_index_path-1826220-1.patch590 bytes

Patch attached.

acrollet’s picture

acrollet commented
StatusFileSize
new mail_edit-undefined_index_path-1826220-2.patch591 bytes

whoops, typo in the previous patch.

salvis’s picture

salvis
he/his
commented

it seems that one can not assume that the item array will have a "path" element.

How do you come to that conclusion?

What makes you think that http://drupal.org/project/panels_breadcrumbs is correct?

salvis’s picture

salvis
he/his
commented
Status: Needs review » Postponed (maintainer needs more info)
acrollet’s picture

acrollet commented
Status: Postponed (maintainer needs more info) » Needs review

How do you come to that conclusion?

Simply the fact that 'path' is not listed under the text "Each link in the active trail must contain:" on http://api.drupal.org/api/drupal/modules%21system%21system.api.php/funct...

salvis’s picture

salvis
he/his
commented
Status: Needs review » Fixed

Thanks!

Committed to the -dev version, give it up to 12h to be repackaged.

Status: Fixed » Closed (fixed)

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

weynhamz’s picture

weynhamz
Location Ningbo
commented
Issue summary: View changes

I suggest we use 'href' instead, from the API page[1], you can be sure that 'href' key must exist.

[1]: https://api.drupal.org/api/drupal/modules%21system%21system.api.php/func...

acrollet’s picture

acrollet commented

@weynhamz - this is a 3 year old closed issue - I'd suggest you open a new issue and submit a patch if you feel strongly.