I found in the status report the following error:
The User ID field needs to be updated.

So, I submitted the following command which returned without any issue but the status report keep reporting the same error:

$ drush entity-updates
The following updates are pending:

consumer entity type : 
The User ID field needs to be updated.

 Do you wish to run all pending updates? (yes/no) [yes]:
 > yes


 [success] Cache rebuild complete.
 [success] Finished performing updates.

Using:
- drupal 8.6.7
- drush 9.5.2

Comments

drikc created an issue. See original summary.

kevin.pfeifer’s picture

I can confirm that error.
I just did the Lightning Distribution Update 3.205 which included the Consumers Module to update to Version 1.8.
Problem persists even after multiple 'drush entity-updates', 'drush updb' and 'drush cr'

cobenash’s picture

I can confirm this error too. The same environment with #1 #2.

gdev2018’s picture

the same one

lamp5’s picture

After update Lightning Distribution the same issue exists on our sites.

dv-vc’s picture

Confirming as well. Updated Lightning distro to 3.2.5 and Entity/Field definitions mismatch message appears on status report: The User ID field needs to be updated.

e0ipso’s picture

Status: Active » Needs review
StatusFileSize
new1.37 KB

This should do… Can anyone test the patch?

e0ipso’s picture

Actually, this one should be more appropriate.

cobenash’s picture

@e0ipso

Thanks for the path. I've test #8 patch.

I got the following error.

Error: Using $this when not in object context in consumers_update_8106() (line 184 of web/modules/contrib/consumers/consumers.install)

e0ipso’s picture

Face-palm... Sorry I did a last minute change. I'll update the patch.

e0ipso’s picture

Let's see now…

cobenash’s picture

@e0ipso

Thanks for the path. I've test #11 patch. It work.

Core 8.6.9

e0ipso’s picture

Status: Needs review » Reviewed & tested by the community

RTBC based on #12.

  • e0ipso committed 67ab61f on 8.x-1.x
    Issue #3031068 by e0ipso, drikc, cobenash, kevin.pfeifer, gdev2018,...
e0ipso’s picture

Status: Reviewed & tested by the community » Fixed
e0ipso’s picture

I removed unnecessary use statements on-commit.

sam711’s picture

Status: Fixed » Needs review

The patch works for me too. Thank you!

However the underlying table column 'owner_id' is different depending on whether you're updating (Null => YES) or installing (Null => NO).

kevin.pfeifer’s picture

Can confirm that applying patch #11 and drush updb fixes the problem.

e0ipso’s picture

Status: Needs review » Fixed

However the underlying table column 'owner_id' is different depending on whether you're updating (Null => YES) or installing (Null => NO).

@samu711 any chance you can create a new issue for that? Thanks so much for your attention to detail on that.

Status: Fixed » Closed (fixed)

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