Config entities uses string as id but entity_usage column for storing such ids is an integer.
I think that we should not count Config entities, so I wrote a patch that excludes all fields in an entity that refer to Config entities.

CommentFileSizeAuthor
#11 entity_usage_explode-2822303-11.patch3.04 KBlussoluca
#8 entity_usage_explode-2822303-8.patch3.05 KBlussoluca
#5 entity_usage_explode-2822303-3.patch2.81 KBlussoluca
#2 entity_usage_explode-2822303-2.patch2.91 KBlussoluca

Comments

lussoluca created an issue. See original summary.

lussoluca’s picture

lussoluca
he/him
Primary language Italian
Location Italy
commented
Status: Active » Needs review
StatusFileSize
new entity_usage_explode-2822303-2.patch2.91 KB

Status: Needs review » Needs work

The last submitted patch, 2: entity_usage_explode-2822303-2.patch, failed testing.

The last submitted patch, 2: entity_usage_explode-2822303-2.patch, failed testing.

lussoluca’s picture

lussoluca
he/him
Primary language Italian
Location Italy
commented
Status: Needs work » Needs review
StatusFileSize
new entity_usage_explode-2822303-3.patch2.81 KB

Ops, wrong patch file

Status: Needs review » Needs work

The last submitted patch, 5: entity_usage_explode-2822303-3.patch, failed testing.

The last submitted patch, 5: entity_usage_explode-2822303-3.patch, failed testing.

lussoluca’s picture

lussoluca
he/him
Primary language Italian
Location Italy
commented
Status: Needs work » Needs review
StatusFileSize
new entity_usage_explode-2822303-8.patch3.05 KB

again...

Status: Needs review » Needs work

The last submitted patch, 8: entity_usage_explode-2822303-8.patch, failed testing.

The last submitted patch, 8: entity_usage_explode-2822303-8.patch, failed testing.

lussoluca’s picture

lussoluca
he/him
Primary language Italian
Location Italy
commented
Status: Needs work » Needs review
StatusFileSize
new entity_usage_explode-2822303-11.patch3.04 KB

Latest attempt

  • marcoscano committed 58b7585 on 8.x-1.x authored by lussoluca 
    Issue #2822303 by lussoluca: Fix bug when entity_reference fields point...
marcoscano’s picture

marcoscano
Primary language Portuguese, Brazil
Location Barcelona, Spain
commented
Status: Needs review » Fixed

Thanks for reporting!

While on the update manager service we are making sure we only trigger the check for content entities, it is true that if they have a field pointing to a config entity, this was not being filtered out. Thanks!

Status: Fixed » Closed (fixed)

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