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.This would help track who created/updated price list and price list items.
I'm wondering whether we should be setting an initial value or not?
Additionally, we'll probably need to introduce more granular permissions (e.g per store price list permissions).
i'm thinking we need more than just "administer commerce_pricelist".
| Comment | File | Size | Author |
|---|---|---|---|
| #8 | commerce_pricelist-3102979-7.patch | 6.84 KB | jsacksick |
| #8 | interdiff_6-7.txt | 1.65 KB | jsacksick |
Comments
Comment #2
jsacksick CreditAttribution: jsacksick at Centarro commentedNote that I'm not 100% sure whether we need the "uid" entity key.
Comment #3
jsacksick CreditAttribution: jsacksick at Centarro commentedWe also need to update the entity type definition (the "entity_keys"), adding that in the .install.
Note that the update hook number could conflict with #3102975: Define event classes for price list and price list items.
Comment #4
jsacksick CreditAttribution: jsacksick at Centarro commentedPatch had to be rerolled because of #3102975: Define event classes for price list and price list items.
Comment #5
jsacksick CreditAttribution: jsacksick at Centarro commentedBefore we commit that, we have to decide whether we need to set an initial default value for the uid field.
Comment #6
jsacksick CreditAttribution: jsacksick at Centarro commentedThe attached patch is setting the default price list and price list item owner to the store owner.
Comment #8
jsacksick CreditAttribution: jsacksick at Centarro commentedCommitted the attached patch!
Comment #10
zenimagine CreditAttribution: zenimagine commented@jsacksick On my production site, I have the RC1 version and I have not applied any patches.
What should I do to not break anything ? Thank you