Deprecated: Drupal\search_api_saved_searches\Entity\SavedSearchType::setNotificationPlugins(): Implicitly marking parameter $notification_plugins as nullable is deprecated, the explicit nullable type must be used instead in /var/www/html/web/modules/contrib/search_api_saved_searches/src/Entity/SavedSearchType.php on line 397

Deprecated: Drupal\search_api_saved_searches\Entity\SavedSearchType::getActiveQuery(): Implicitly marking parameter $query_helper as nullable is deprecated, the explicit nullable type must be used instead in /var/www/html/web/modules/contrib/search_api_saved_searches/src/Entity/SavedSearchType.php on line 455

Deprecated: Drupal\search_api_saved_searches\SavedSearchTypeInterface::getActiveQuery(): Implicitly marking parameter $query_helper as nullable is deprecated, the explicit nullable type must be used instead in /var/www/html/web/modules/contrib/search_api_saved_searches/src/SavedSearchTypeInterface.php on line 146

Issue fork search_api_saved_searches-3495857

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

heyyo created an issue. See original summary.

heyyo’s picture

heyyo commented
Issue tags: +PHP 8.4

prem suthar made their first commit to this issue’s fork.

prem suthar opened merge request !22

prem suthar’s picture

prem suthar commented
Status: Active » Needs review

Please Review And Merge.

drunken monkey made their first commit to this issue’s fork.

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented

Thanks for reporting this, @heyyo, and thanks a lot for creating an MR, @prem suthar!
However, there were quite a few spots missing. Fixed those as well in the MR, please test/review and I can merge.

prem suthar’s picture

prem suthar commented

@drunken nomkey !22 is fix all the Deprecation code of php 8.4 So now you can merge.

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented
Status: Needs review » Fixed

Merged. Thanks again!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.