- Issue created by @joevagyok
- π·π΄Romania claudiu.cristea Arad π·π΄
claudiu.cristea β made their first commit to this issueβs fork.
- Merge request !35Keep the status if it has been set by an upstream subscriber β (Merged) created by claudiu.cristea
- Status changed to Needs work
about 1 month ago 2:46pm 27 May 2024 - π·π΄Romania claudiu.cristea Arad π·π΄
Thank you for reporting. Indeed,
CasHelper::EVENT_PRE_REGISTER
events are pretending that you can alter the user properties but the recent issue just overrides any subscriber changes* Subscribers to this event can: * - ... * - Change the username that will be assigned to the Drupal account. By * default it is the same as the CAS username. * - Set properties on the user account that will be created, like user roles * or a custom first name field (for example by populating it with data from * the CAS attributes available in $casPropertyBag).
I've proposed something in the MR but we still need to weight on that and we need tests.
- π·π΄Romania claudiu.cristea Arad π·π΄
Linking the initial issue
- Status changed to Needs review
about 1 month ago 3:56pm 27 May 2024 - π§πͺBelgium joevagyok
I pushed a basic test involving the
CasHelper::EVENT_PRE_REGISTER
event. - π§πͺBelgium joevagyok
Uploading a patch file for composer patching.
- Status changed to RTBC
29 days ago 12:29am 30 May 2024 - πΊπΈUnited States bkosborne New Jersey, USA
This looks good to me. Feel free to merge and release 2.3.2 claudiu.cristea
-
claudiu.cristea β
committed 4cc7a08a on 2.x
Issue #3450140 by joevagyok, claudiu.cristea, bkosborne: Status property...
-
claudiu.cristea β
committed 4cc7a08a on 2.x
- Status changed to Fixed
29 days ago 7:36am 30 May 2024 Automatically closed - issue fixed for 2 weeks with no activity.