- First commit to issue fork.
- @royalpinto007 opened merge request.
- Assigned to royalpinto007
- Status changed to Needs review
over 1 year ago 3:39am 8 February 2023 - Status changed to Needs work
over 1 year 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
over 1 year 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
over 1 year ago 3:29pm 18 February 2023 - First commit to issue fork.
- Status changed to Needs review
over 1 year ago 8:14am 20 February 2023 - ๐ฎ๐ณIndia sourabhjain
I have removed the changes suggested in #7. Please review.
- Status changed to Needs work
over 1 year ago 4:16pm 20 February 2023 - Status changed to Needs review
over 1 year 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 - ๐ฎ๐ณIndia Rashmisoni Bangalore
Rassoni โ made their first commit to this issueโs fork.
- First commit to issue fork.
- last update
about 1 year ago 28,523 pass - last update
about 1 year ago 28,520 pass, 1 fail - Status changed to Needs review
about 1 year ago 8:28pm 7 June 2023 - ๐จ๐ฆCanada bbombachini London, ON
Rebased and added a pseudo example. Please review.
- last update
about 1 year ago 28,523 pass - ๐บ๐ธUnited States smustgrave
99% positive it was a random failure but requeued the tests.
- Status changed to RTBC
about 1 year ago 1:28pm 8 June 2023 - ๐บ๐ธUnited States smustgrave
All green and pseudo code looks like a decent example.
- last update
about 1 year ago 28,523 pass - last update
about 1 year ago 28,523 pass - last update
about 1 year ago 28,523 pass - Status changed to Fixed
about 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
12 months 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.)