- First commit to issue fork.
- @royalpinto007 opened merge request.
- Assigned to royalpinto007
- Status changed to Needs review
almost 2 years ago 3:39am 8 February 2023 - Status changed to Needs work
almost 2 years ago 4:18am 8 February 2023 - 🇦🇺Australia mstrelan
These changes aren't what the issue summary is asking for.
The
\Drupal\Core\Condition\ConditionManager
constructor makes a call to$this->alterInfo('condition_info');
This allows other modules to implementhook_condition_info_alter
, but there is no API definition for this. You should add this function to core.api.php. Use one of the existing alter functions in that file as an example. - Status changed to Needs review
almost 2 years ago 9:23am 8 February 2023 - 🇮🇳India royalpinto007
https://git.drupalcode.org/project/drupal/-/merge_requests/3408/diffs?co... Can someone check the current problem now?
It would be really helpful.
(Made an API definition for the function, which allows other modules to alter the condition definitions.
However, the function inside needs further updates to meet the requirements.) - Issue was unassigned.
- Status changed to Needs work
almost 2 years ago 3:29pm 18 February 2023 - First commit to issue fork.
- Status changed to Needs review
almost 2 years ago 8:14am 20 February 2023 - 🇮🇳India sourabhjain
I have removed the changes suggested in #7. Please review.
- Status changed to Needs work
almost 2 years ago 4:16pm 20 February 2023 - Status changed to Needs review
almost 2 years ago 1:01pm 24 February 2023 - Status changed to RTBC
over 1 year ago 4:49pm 27 February 2023 - 🇺🇸United States smustgrave
Seems like a simple enough addition. Threads were resolved.
- Status changed to Needs work
over 1 year ago 10:32am 22 March 2023 - First commit to issue fork.
- First commit to issue fork.
- last update
over 1 year ago 28,523 pass - last update
over 1 year ago 28,520 pass, 1 fail - Status changed to Needs review
over 1 year ago 8:28pm 7 June 2023 - 🇨🇦Canada bbombachini London, ON
Rebased and added a pseudo example. Please review.
- last update
over 1 year ago 28,523 pass - 🇺🇸United States smustgrave
99% positive it was a random failure but requeued the tests.
- Status changed to RTBC
over 1 year ago 1:28pm 8 June 2023 - 🇺🇸United States smustgrave
All green and pseudo code looks like a decent example.
- last update
over 1 year ago 28,523 pass - last update
over 1 year ago 28,523 pass - last update
over 1 year ago 28,523 pass - Status changed to Fixed
over 1 year ago 10:34am 14 June 2023 - 🇬🇧United Kingdom catch
Committed/pushed to 11.x and cherry-picked back through to 9.5.x, thanks!
Automatically closed - issue fixed for 2 weeks with no activity.
- Status changed to Fixed
over 1 year ago 9:04am 18 July 2023 - 🇮🇹Italy apaderno Brescia, 🇮🇹
(I am removing a tag, since tags should not duplicate what already selected for the Component field. I apologize for bumping this issue.)