#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 patch below;
Comment | File | Size | Author |
---|---|---|---|
#7 | uuid_features-undefined_index_when_processing_orphanned_bundles-2153157-7.patch | 642 bytes | hswong3i |
Comments
Comment #1
joseph.olstadPATCHED against commit 3b0dca6
Please review attached patch
Comment #2
joseph.olstadpatched, see comment #1
Comment #3
jayson CreditAttribution: jayson commentedI 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!
Comment #4
joseph.olstadComment #5
joseph.olstadThis patch will avoid undefined index warnings for orphaned bundles/nodes.
Applying this patch should not have any negative effects.
Please review.
Comment #6
sylus CreditAttribution: sylus commentedMakes sense to have a check before assigning setting to RTBC.
Comment #7
hswong3i CreditAttribution: hswong3i commentedRe-roll #2 via latest 7.x-1.x-dev
Comment #8
hswong3i CreditAttribution: hswong3i commentedConflicted with #2060587: Split variables as entity type specific
Comment #9
das-peter CreditAttribution: das-peter commentedOh thanks for the re-roll. That looks very straight forward - committed & pushed.
Comment #11
hswong3i CreditAttribution: hswong3i commented#2060587: Split variables as entity type specific re-rolled due to recent commit ;-)