Problem/Motivation

Issue in 1.2.0 and 2.x:

Drupal\cookies\Entity\CookiesServiceEntity::setProcessorCookiePolicyUrl(): Argument #1 ($processorCookiePolicyUrl) must be of type string, null given, called in /usr/www/users/zmpd9/www9.zmp.de/web/modules/contrib/cookies/cookies.install on line 172

When running drush updb updating from 1.1.3

Steps to reproduce

Proposed resolution

Remaining tasks

- [] Fix the issue in 1.2.x and 2.x
- [] Set 1.2.0 supported again in https://www.drupal.org/node/3139023/edit/releases

User interface changes

API changes

Data model changes

Issue fork cookies-3326280

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

Anybody created an issue. See original summary.

anybody’s picture

anybody
Primary language German
Location Porta Westfalica
commented
Status: Active » Needs review

@Grevil please review. For me the issue was fixed that way.

Anybody opened merge request !79

Anybody opened merge request !78

tjtj’s picture

tjtj commented

Please issue a new fixed build

anybody’s picture

anybody
Primary language German
Location Porta Westfalica
commented

Thanks @tjtjt we'll do that as soon as we have an RTBC here. I think this will happen tomorrow, if no one is faster than @Grevil.

Grevil made their first commit to this issue’s fork.

grevil’s picture

grevil commented

Small additions, please review!

grevil’s picture

grevil commented

Tested updating from 1.1.x and this definitly fixes the issue!

petr illek’s picture

petr illek
Primary language Czech
commented
Status: Needs review » Reviewed & tested by the community

Hi,
I've tested the latest patch from #8 and can confirm the DB update is now working.
Setting the status to RTBC.
Thanks @Grevil and @Anybody.

grevil’s picture

grevil commented

Great, thank you, @Petr Illek!

  • 42bf424 committed on 1.2.x 
    Issue #3326280: Updating to 1.2.0 fails with Argument Argument #1 ($...
grevil’s picture

grevil commented
Version: 1.2.x-dev » 2.x-dev
grevil’s picture

grevil commented
Status: Reviewed & tested by the community » Fixed

As 2.x and 1.2.x are currently identical, I'll push this change without review.

  • a70a3da committed on 2.x 
    Issue #3326280: Updating to 1.2.0 fails with Argument Argument #1 ($...
anybody’s picture

anybody
Primary language German
Location Porta Westfalica
commented
Assigned: grevil » Unassigned

Nice, thank you all :)

Status: Fixed » Closed (fixed)

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