The block plugins already made this switch and it should be redundant.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

swentel’s picture

Status: Active » Needs review
FileSize
29.9 KB
31.19 KB

two patches - one with a fallback - one without.

mikey_p’s picture

#1: 2112705-1.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, 2112705-1-no-fallback.patch, failed testing.

Berdir’s picture

Yay!

I'm fine with dropping this without a fallback.

We also need to remove the documentation of this in Annotation/EntityType

swentel’s picture

Status: Needs work » Needs review
FileSize
2.01 KB
31.91 KB

Should fix dozens of exceptions

Status: Needs review » Needs work

The last submitted patch, 2112705-5.patch, failed testing.

swentel’s picture

Status: Needs work » Needs review
FileSize
1.31 KB
32.91 KB

Might be green

dawehner’s picture

Status: Needs review » Reviewed & tested by the community

I could not find any other entity beside bideo which still uses the module key on the annotation.

mikey_p’s picture

Thanks for fixing this. Little things like this add up to improve DX!

alexpott’s picture

Status: Reviewed & tested by the community » Needs work
Issue tags: +Needs reroll

Patch no longer applies.

swentel’s picture

Status: Needs work » Reviewed & tested by the community
Issue tags: -Needs reroll
FileSize
32.18 KB
alexpott’s picture

Issue summary: View changes
Status: Reviewed & tested by the community » Needs work

The module key is still used in config_get_module_config_entities() but this function is unused so I think this patch should remove it - as it is untested, unneeded and with this change, broken.

mikey_p’s picture

Status: Needs work » Needs review
FileSize
33.28 KB
1.03 KB
Berdir’s picture

Status: Needs review » Reviewed & tested by the community
Parent issue: » #2095603: [meta] Complete Entity Field API

Nice find!

Tried what I could to find any other left-over references but couldn't find anything. Back to RTBC.

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed bf3e9cf and pushed to 8.x. Thanks!

Status: Fixed » Closed (fixed)

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