Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Our implementation looks broken anyway.
Comment | File | Size | Author |
---|---|---|---|
#12 | interdiff_10-12.txt | 1.16 KB | jsacksick |
#12 | profile_2844963-12.patch | 20.54 KB | jsacksick |
Comments
Comment #2
daggerhart CreditAttribution: daggerhart commented+1 this. I need to track which user modified the profile as well as accept log messages about the modification.
Comment #3
jsacksick CreditAttribution: jsacksick at Centarro for PayPal, Inc commentedI'm not sure about the deprecation format for
getRevisionAuthor()
andsetRevisionAuthorId()
.Comment #4
amateescu CreditAttribution: amateescu for Pfizer, Inc. commentedThere's no need for this, you can use
->setInitialValutFromField()
on the field storage definition before installing it :)Comment #5
jsacksick CreditAttribution: jsacksick at Centarro for PayPal, Inc commentedNot really, that's what happened when I tried that:
I could set it the initialValue to "created", but then the initial value would be "wrong".
Comment #6
amateescu CreditAttribution: amateescu for Pfizer, Inc. commentedRight, don't mind me :D
Comment #7
jsacksick CreditAttribution: jsacksick at Centarro for PayPal, Inc commentedEven if this task isn't about the UI, we need to make sure the revision_log field isn't shown by default on the user register form...
And this will be causing issues for the profile form in checkout as well.
Comment #8
jsacksick CreditAttribution: jsacksick at Centarro for PayPal, Inc commentedComment #9
steveoriolPatch works for me, no more error entities définition" on the "admin/reports/status" page.
but, after the patch apply, the module was not install? ...
and to reinstall it, I needed to remove the 4 following configuation (drush config-delete xxxx):
Fortunately, all my user profiles have remained in place ;-)
Comment #10
jsacksick CreditAttribution: jsacksick at Centarro for PayPal, Inc commentedComment #12
jsacksick CreditAttribution: jsacksick at Centarro for PayPal, Inc commentedComment #13
bojanz CreditAttribution: bojanz at Centarro for Adapt commentedThe patch is a lot more ambitious than what we initially discussed, so let's give the issue a better title.
Comment #14
bojanz CreditAttribution: bojanz at Centarro commentedComment #15
bojanz CreditAttribution: bojanz at Centarro commentedHad to add EntityOwnerInterface to profile because it only exists in Drupal 8.7.x while we still support 8.6.x.
Tested the upgrade path manually, all is well.
Thanks, jsacksick!