- Issue created by @tijsdeboeck
- Assigned to abhishek_virasat
- Issue was unassigned.
- Status changed to Needs review
10 months ago 5:17am 20 February 2024 - 🇮🇳India abhishek_virasat
@tijsdeboeck, I have fixed the issue and created Patch. Please kindly review it.
- 🇳🇱Netherlands martin74
Got the same issue , Patch did not help , changed
bookable_calendar\Notification->__construct() (line 79
79 = $this->connection = $connection;
- Status changed to Needs work
9 months ago 7:35am 15 March 2024 - 🇮🇳India dineshkumarbollu
Getting same error patch#3 creates new error
TypeError: Drupal\bookable_calendar\Notification::__construct(): Argument #3 ($token) must be of type Drupal\Core\Utility\Token, Drupal\Core\Messenger\Messenger given, called in /var/www/html/diet/vb1020/web/core/lib/Drupal/Component/DependencyInjection/Container.php on line 259 in Drupal\bookable_calendar\Notification->__construct() (line 65 of modules/contrib/bookable_calendar/src/Notification.php). - Status changed to Needs review
9 months ago 7:45am 15 March 2024 - 🇳🇱Netherlands martin74
nope
Deprecated function: Creation of dynamic property Drupal\bookable_calendar\Notification::$messenger is deprecated in Drupal\bookable_calendar\Notification->__construct() (line 75
- 🇨🇳China marcolam
I applied the #6 patch and no more error message as in #1.
Thanks!Env: Drupal 10.1.8,PHP 8.2.18, bookable calendar 2.2.11
-
josh.fabean →
committed e1f10e0f on 2.2.x
Issue #3422474 by dineshkumarbollu, abhishek_gupta1, martin74,...
-
josh.fabean →
committed e1f10e0f on 2.2.x
- 🇺🇸United States josh.fabean
Patch is applied in latest 2.2.x and will be 2.2.12, thanks!
- Status changed to Fixed
5 months ago 1:17pm 5 August 2024 Automatically closed - issue fixed for 2 weeks with no activity.