Notice happens without checking disabled bundles:

Notice: Undefined index: multichoice in field_permissions_overview() (line 357 of /Drupal/sites/all/modules/field_permissions/field_permissions.admin.inc

I'm attaching the patch for this.

CommentFileSizeAuthor
#1 field_permission_Issue-2257833-Check-for-disabled-bundles-when-displa.patch1.07 KBalamp

Comments

alamp’s picture

alamp commented
Issue summary: View changes
StatusFileSize
new field_permission_Issue-2257833-Check-for-disabled-bundles-when-displa.patch1.07 KB
joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
Assigned: alamp » Unassigned
Status: Patch (to be ported) » Needs review

Thanks for the patch @alamp, is multichoice the $bundle variable?

I'm changing the status to needs review. Patch (to be ported) is rarely used.

Patch (to be ported)
The patch has been successfully committed to a branch of the project, and still needs to be committed to another, but the current patch doesn't apply to the target branch and needs to be modified in order to do so.

kupide’s picture

kupide commented
Issue tags: -field permissoin +Field Permissions

Thanks for the Patch #1, it works for me (field permissions 7.x-1.0-beta2).

  • mariacha1 committed b5df9d1 on 7.x-1.x authored by alamp 
    Issue #2257833 by alamp: Notice: Undefined index: in...
mariacha1’s picture

mariacha1 commented
Status: Needs review » Fixed
Related issues: +#1351506: Disabling or uninstalling a node-type module (e.g. Forum), or removing a node type, results in "undefined index" notices

Issue is related to a bug that also appears in the core field listings page. See https://www.drupal.org/node/1351506#comment-7194524

Patch works great and is basically the same as the one proposed there. Thanks!

mariacha1’s picture

mariacha1 commented
Version: 7.x-1.0-beta2 » 8.x-1.0-beta1
Status: Fixed » Patch (to be ported)

I also confirmed this is not a problem in D8.

mariacha1’s picture

mariacha1 commented
Version: 8.x-1.0-beta1 » 7.x-1.0-beta2
Status: Patch (to be ported) » Fixed

Status: Fixed » Closed (fixed)

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