#20
Posted by jayson on December 9, 2013 at 11:25am
Prior to this patch:

1. I'm getting errors about undefined indexes on both these pages:
> UUID Features Integrations configuration page (/admin/config/content/uuid_features)
> Create new feature page (/admin/structure/features/create)

2. The "UUID Features Integration" configuration page displays some select options that have blank values. The two content type related ones have to do with "Deploy Content Migrated" and "Deploy Content Simple". The taxonomy related one has to do with Tags. I was able to track them down to the WetKit Deployment feature (/admin/structure/features/wetkit_deployment) which is currently disabled on this website. That might explain why the options checkboxes appear with no label. However, perhaps this is a bug as the options should not be there at all.

See screenshot.

See patch below;

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

joseph.olstad’s picture

PATCHED against commit 3b0dca6

Please review attached patch

joseph.olstad’s picture

patched, see comment #1

jayson’s picture

I can confirm that this patch works. When visiting both these admin pages, there are no more undefined index errors:
> UUID Features Integrations configuration page (/admin/config/content/uuid_features)
> Create new feature page (/admin/structure/features/create)

Thanks Joseph!

joseph.olstad’s picture

Issue summary: View changes
joseph.olstad’s picture

Status: Patch (to be ported) » Needs review

This patch will avoid undefined index warnings for orphaned bundles/nodes.

Applying this patch should not have any negative effects.

Please review.

sylus’s picture

Status: Needs review » Reviewed & tested by the community

Makes sense to have a check before assigning setting to RTBC.

hswong3i’s picture

hswong3i’s picture

das-peter’s picture

Status: Reviewed & tested by the community » Fixed

Oh thanks for the re-roll. That looks very straight forward - committed & pushed.

hswong3i’s picture

#2060587: Split variables as entity type specific re-rolled due to recent commit ;-)

Status: Fixed » Closed (fixed)

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