Sorry for being new here. Patch #15 works for us with the minor caveat that for Drupal 10.2.4 with PHP 8.1.27 I get the following two variables need to be typed, likely due to when this patch was originally made.
public string $value_value; public ?array $valueOptions;
After applying the patch I can use actions and bulk assign domains via /admin/content
- Status changed to Postponed: needs info
about 1 month ago 7:17pm 28 May 2024 - π©πͺGermany diqidoq Berlin | Hamburg | New York | London | Paris
Is this issue still relevant in 2.0 branch? It is recommended to use latest beta 2.0 release and new patches against latest 2.0-dev since branch 8.x is not supported no more.
- πΊπΈUnited States eswiderski
@dqd yes, it still applies to 2.0-dev
- Status changed to Needs work
about 1 month ago 9:13pm 28 May 2024 - π©πͺGermany diqidoq Berlin | Hamburg | New York | London | Paris
Thanks @eswiderski good to now.
#22 @cpocket: Thanks for the review.
I get the following two variables need to be typed, likely due to when this patch was originally made.
public string $value_value;
public ?array $valueOptions;I think this is related to: π DomainAccessCurrentAllFilter missing variable type inheritance Needs work and would be fixed there. I would encourage to commit the other first and then try to reroll this one here to be committed afterwards.