Notice: Undefined index: in field_permissions_overview [#2257833]

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.

Comment	File	Size	Author
#1	field_permission_Issue-2257833-Check-for-disabled-bundles-when-displa.patch	1.07 KB	alamp
#1

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

alamp CreditAttribution: alamp commented 3 May 2014 at 22:49

Issue summary:

View changes

File	Size
field_permission_Issue-2257833-Check-for-disabled-bundles-when-displa.patch	1.07 KB

Comment #2

joelpittet

English

Vancouver

CreditAttribution: joelpittet commented 15 January 2015 at 01:12

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.

Comment #3

kupide CreditAttribution: kupide commented 16 January 2015 at 13:42

Issue tags:

-field permissoin

+Field Permissions

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

Comment #4

6 August 2016 at 21:45

mariacha1 committed b5df9d1 on 7.x-1.x authored by alamp

Issue #2257833 by alamp: Notice: Undefined index: in...

Comment #5

mariacha1 CreditAttribution: mariacha1 at ThinkShout commented 6 August 2016 at 21:47

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!