Problem/Motivation

Deprecated: Drupal\feeds_http_auth_fetcher\Feeds\Fetcher\Form\HttpAuthFetcherFeedForm::buildConfigurationForm(): Implicitly marking parameter $feed as nullable is deprecated, the explicit nullable type must be used instead in /var/www/html/backend/web/modules/contrib/feeds_http_auth_fetcher/src/Feeds/Fetcher/Form/HttpAuthFetcherFeedForm.php on line 26

Deprecated: Drupal\feeds_http_auth_fetcher\Feeds\Fetcher\Form\HttpAuthFetcherFeedForm::submitConfigurationForm(): Implicitly marking parameter $feed as nullable is deprecated, the explicit nullable type must be used instead in /var/www/html/backend/web/modules/contrib/feeds_http_auth_fetcher/src/Feeds/Fetcher/Form/HttpAuthFetcherFeedForm.php on line 59

Steps to reproduce

visit feeds edit form

Proposed resolution

fix types with "?" operator

Remaining tasks

Review merge request.

User interface changes

none

API changes

none

Data model changes

none

CommentFileSizeAuthor
#2 feeds_http_auth_fetcher-3548667.patch1.16 KBklausi

Issue fork feeds_http_auth_fetcher-3548667

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

klausi created an issue. See original summary.

klausi’s picture

klausi
he/him||they/them
Primary language German
Location 🇦🇹 Vienna
commented
Status: Active » Needs review
StatusFileSize
new feeds_http_auth_fetcher-3548667.patch1.16 KB

Merge request: https://git.drupalcode.org/project/feeds_http_auth_fetcher/-/merge_reque... (seems broken, not sure why)

Attaching stable patch for composer patches.

klausi opened merge request !16

alesbencina’s picture

alesbencina commented
Status: Needs review » Reviewed & tested by the community

thanks Klausi!

bwoods’s picture

bwoods commented

Applied the patch, and it works as expected, thanks!

yogeshmpawar’s picture

yogeshmpawar
He/Him/His
Primary language English
Location Pune(MH), INDIA 🇮🇳
commented
Assigned: Unassigned » yogeshmpawar
yogeshmpawar’s picture

yogeshmpawar
He/Him/His
Primary language English
Location Pune(MH), INDIA 🇮🇳
commented
Assigned: yogeshmpawar » Unassigned
Status: Reviewed & tested by the community » Fixed

Merged MR.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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