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.
There's a @todo in Drupal\comment\Plugin\Core\Entity\Comment
Needs to fix a lot of places + hook_update_N()
Comment | File | Size | Author |
---|---|---|---|
#6 | comment_id-2031935-6.patch | 65.8 KB | plopesc |
#6 | interdiff_4-6.txt | 3.28 KB | plopesc |
Comments
Comment #1
plopescWorking on this one...
Comment #2
plopescFirst Round
Regards
Comment #4
plopescRe-rolling patch.
Regards.
Comment #6
plopescFixed unwanted file move in block module and changed last cid references in system and tracker modules.
It should be green now...
Comment #7
cweagansWait wait wait. If we're going to do this, then we also need to convert other things for consistency:
$node->nid => $node->id
$file->fid => $file->id
$user->uid => $user->id
This sounds like 9.x material to me.
Comment #8
BerdirYes, I think it's too late for this, too many changes.
We have the id() method now, so nobody should access cid exept for queries and other special cases anyway.
Comment #9
dixon_This doesn't make sense this late. Pushing to 9.x.
Comment #10
catchThis is theoretically possible in a minor release.
Whether it's possible in practice is a different issue, but moving back for now.
Comment #11
andypostMaybe sort of *field alias* exists? but that will affect db schema as well....
Otoh sort of *proxy* or lazy field doable
Comment #12
andypostbtw Better to provide id property from base class/method
Comment #25
smustgrave CreditAttribution: smustgrave at Mobomo commentedPart of the new review initiative this one came up.
Wonder if this is a valid task still?
Won't this cause breaks for contrib and custom modules.
Comment #26
andypostAccording to #10 it still doable in BC way, so I think if there's a usage it can be proxied to id
Comment #27
smustgrave CreditAttribution: smustgrave at Mobomo commentedGotcha moving to nw