- last update
over 1 year ago Composer require failure - Assigned to Kristen Pol
- 🇺🇸United States Kristen Pol Santa Cruz, CA, USA
Assigning to myself as I'm reviewing/merging ready RTBC fixes/updates over the next few days.
- Issue was unassigned.
- Status changed to Needs work
9 months ago 3:10am 9 February 2024 - 🇺🇸United States Kristen Pol Santa Cruz, CA, USA
This is working as expected. Note that if you already had constraints added, it doesn't check those when saving. This is only relevant for new policies.
Thanks for the work on this issue. This is a nice little UX improvement.
Reviewing the code:
1. This is misspelled:
$constaint
2. This shouldn't be inside the loop since it doesn't change:
+ $new_operation = $form_state->getValue('character_operation'); + $new_length = $form_state->getValue('character_length');
3. And, since there are other changes, I think these two variable names should change:
* Change
$other_operation
to$constraint_operation
* Change$other_length
to$constraint_length
- Status changed to Needs review
9 months ago 5:32am 9 February 2024 - Open on Drupal.org →Core: 10.2.1 + Environment: PHP 7.4 & MySQL 5.7last update
9 months ago Waiting for branch to pass - Assigned to Kristen Pol
- 🇺🇸United States Kristen Pol Santa Cruz, CA, USA
Assigning to myself to review, and changing this to 4.0.0.
-
Kristen Pol →
committed 77ac3d46 on 4.0.x authored by
chaitanyadessai →
Issue #3308607 by sophiavs, chaitanyadessai, lucassc, Kristen Pol:...
-
Kristen Pol →
committed 77ac3d46 on 4.0.x authored by
chaitanyadessai →
- Issue was unassigned.
- Status changed to Fixed
9 months ago 8:29pm 20 February 2024 - 🇺🇸United States Kristen Pol Santa Cruz, CA, USA
Thanks everyone. I improved the messages a bit. Merged and will be in the next release.
Automatically closed - issue fixed for 2 weeks with no activity.
- 🇺🇸United States Kristen Pol Santa Cruz, CA, USA
This is part of the new 4.0.1 release → .