- 🇧🇪Belgium tijsdeboeck Antwerp 🇧🇪 🇪🇺 🌎
I've tested the patch on Drupal 9.5.8 + commerce 2.35 - works like a charm! Marking it RTBC.
- last update
almost 2 years ago 779 pass, 2 fail - Status changed to Needs work
almost 2 years ago 9:05am 21 April 2023 - 🇮🇱Israel jsacksick
It is not RTBC as the tests need to be updated and are failing as a result of that change, the test should be updated to grant the new permission.
- 🇮🇱Israel jsacksick
Also, the page will cease to work for users with the "administer commerce_order" permission... So having the permission to administer orders wouldn't let users reassign orders which seems wrong... So looks like we're going to need to check both permissions, so definitely not RTBC.
- 🇧🇪Belgium tijsdeboeck Antwerp 🇧🇪 🇪🇺 🌎
Makes complete sense! I might have time to work on a new patch in the coming days...
- Merge request !308Issue #3164506: Order reassign without administrative privileges results in access denied → (Merged) created by rhovland
- 🇺🇸United States rhovland Oregon
Created a MR with the administrative permission check restored. Not entirely sure how to go about testing the new permission.
-
jsacksick →
committed ef3ac0e5 on 8.x-2.x authored by
rhovland →
Issue #3164506 by rhovland, ReBa: Order: reassign without administrative...
-
jsacksick →
committed ef3ac0e5 on 8.x-2.x authored by
rhovland →
- Status changed to Fixed
8 months ago 12:31pm 19 August 2024 -
jsacksick →
committed 7a360b89 on 3.0.x authored by
rhovland →
Issue #3164506 by rhovland, ReBa: Order: reassign without administrative...
-
jsacksick →
committed 7a360b89 on 3.0.x authored by
rhovland →
Automatically closed - issue fixed for 2 weeks with no activity.