I think this is a regression from #2095595: Revision stores wrong uid

We kept having a few user_uuid fields that would stay overridden even when the containing feature was reverted.

It turned out that on production the content block had uid 1 and after reverting the feature it was set to 0.

CommentFileSizeAuthor
#3 uid_in_revision_lost-2625786-2.patch578 byteshelmo
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

helmo created an issue. See original summary.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented

It might be worth writing some tests to confirm the expected behavior and avoid another regression.

helmo’s picture

helmo CreditAttribution: helmo at Initfour websolutions commented
Status: Active » Needs review
FileSize
uid_in_revision_lost-2625786-2.patch578 bytes

I assume that when $user->uid == 0 we can skip this ... see patch.