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

CommentFileSizeAuthor
#13 2112705-interdiff.txt1.03 KBmikey_p
#13 2112705-entity-type-provider-13.patch33.28 KBmikey_p
#11 2112705-11.patch32.18 KBswentel
#7 2112705-7.patch32.91 KBswentel
#7 interdiff.txt1.31 KBswentel
#5 2112705-5.patch31.91 KBswentel
#5 interdiff.txt2.01 KBswentel
#1 2112705-1.patch31.19 KBswentel
#1 2112705-1-no-fallback.patch29.9 KBswentel

Comments

swentel’s picture

swentel commented
Status: Active » Needs review
StatusFileSize
new 2112705-1-no-fallback.patch29.9 KB
new 2112705-1.patch31.19 KB

two patches - one with a fallback - one without.

mikey_p’s picture

mikey_p commented

#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

berdir
Primary language German
Location Switzerland
commented

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

swentel commented
Status: Needs work » Needs review
StatusFileSize
new interdiff.txt2.01 KB
new 2112705-5.patch31.91 KB

Should fix dozens of exceptions

Status: Needs review » Needs work

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

swentel’s picture

swentel commented
Status: Needs work » Needs review
StatusFileSize
new interdiff.txt1.31 KB
new 2112705-7.patch32.91 KB

Might be green

dawehner’s picture

dawehner
Primary language German
commented
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

mikey_p commented

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

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Needs work
Issue tags: +Needs reroll

Patch no longer applies.

swentel’s picture

swentel commented
Status: Needs work » Reviewed & tested by the community
Issue tags: -Needs reroll
StatusFileSize
new 2112705-11.patch32.18 KB
alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
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

mikey_p commented
Status: Needs work » Needs review
StatusFileSize
new 2112705-entity-type-provider-13.patch33.28 KB
new 2112705-interdiff.txt1.03 KB
berdir’s picture

berdir
Primary language German
Location Switzerland
commented
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

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
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.