- Issue created by @dtarc
- 🇨🇦Canada dtarc
Here's.a patch with the fix.
bat_event_get_states can take an optional 2nd parameter so it seemed to make sense to pass $conditions along to it.
- 🇬🇧United Kingdom Alina Basarabeanu
The patch from #2 fixed the error. Using Drupal Version 9.5.9 and BAT 8.x-2.1-alpha1
- 🇮🇹Italy afagioli Rome
Hi, and thanks for this contribution.
It's the other way around. Useless param "$conditions" was removed from bat_unit_state_options().
We need to update code calling this function, removing the "$conditions" argument - Status changed to Needs work
over 1 year ago 6:28am 2 August 2023 - Assigned to afagioli
- 🇨🇦Canada dtarc
$conditions is not a useless param -- it's passed into bat_event_get_states() where it is used. Please commit this patch.
- 🇮🇹Italy afagioli Rome
Can you please share with us how
$argument
insidebat_unit_state_options
is being used? - Status changed to Postponed: needs info
11 months ago 4:08pm 16 December 2023 - 🇬🇧United Kingdom Alina Basarabeanu
This issue seems to be fixed with the bat 8.x-10.1-rc3 version.
- Status changed to RTBC
8 months ago 10:32am 13 March 2024 - Issue was unassigned.
- Status changed to Fixed
8 months ago 1:42pm 22 March 2024 Automatically closed - issue fixed for 2 weeks with no activity.