Message TypeError: array_key_exists(): Argument #2 ($array) must be of type array, null given in array_key_exists()

Created on 11 January 2024, 8 months ago
Updated 25 July 2024, about 1 month ago

Problem/Motivation

In the Security Review module, when updating to 3.0.0-alpha1, I receive the message "Message TypeError: array_key_exists(): Argument #2 ($array) must be of type array, null given in array_key_exists()" .

Steps to reproduce

1. Access the Drupal administration panel.
2. Navigate to "Reports" and select "Security review".
3. After that, the issue appears as described above.

๐Ÿ› Bug report
Status

Closed: outdated

Version

3.0

Component
Systemย  โ†’

Last updated 1 day ago

No maintainer
Created by

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

  • Issue created by @Darsana Prasad
  • ๐Ÿ‡บ๐Ÿ‡ธUnited States greggles Denver, Colorado, USA

    Thanks for this bug report, @Darsana Prasad. The 3.0.x-dev version is being stabilized, so it is very helpful to have testers reporting any issue they find.

  • Status changed to Postponed: needs info 8 months ago
  • ๐Ÿ‡บ๐Ÿ‡ธUnited States smustgrave

    did an rc1 release can you confirm if you're still seeing the issue.

  • ๐Ÿ‡ฎ๐Ÿ‡ณIndia Athul Dinesh

    After updating to 3.0.0-rc1, the issue remains.

  • Status changed to Active 8 months ago
  • Status changed to Needs review 8 months ago
  • ๐Ÿ‡ฎ๐Ÿ‡ณIndia sarwan

    Hi,
    I have fixed this issue "Message TypeError: array_key_exists(): Argument #2 ($array) must be of type array, null given in array_key_exists()"
    and also attached patch ,
    please review and verify .

  • Status changed to Needs work 8 months ago
  • ๐Ÿ‡บ๐Ÿ‡ธUnited States greggles Denver, Colorado, USA

    The patch has some unrelated changes from IGNOREME.txt and introduces some formatting changes that seem unrelated, so it needs work.

    Conceptually this probably makes sense.

    I guess another solution would be to save the settings to get the data more complete? I wonder which check isn't being found after upgrading to 3.0.x

  • ๐Ÿ‡บ๐Ÿ‡ธUnited States smustgrave

    Yea Iโ€™d like to track down where this is getting null vs just putting if not null

  • Status changed to Postponed: needs info 8 months ago
  • ๐Ÿ‡บ๐Ÿ‡ธUnited States smustgrave

    Just tested on a fresh install of 10.2.x and rc1 and with the steps in the issue summary not able to replicate.

  • ๐Ÿ‡บ๐Ÿ‡ธUnited States greggles Denver, Colorado, USA

    What about an upgrade of 2.0.2 to 3.0.x?

  • ๐Ÿ‡บ๐Ÿ‡ธUnited States smustgrave

    Maybe, so much changed in 3.0.x that upgrade path from 2.0.x was low priority for me.

  • ๐Ÿ‡บ๐Ÿ‡ธUnited States smustgrave

    I did get an error in rc1 and just released rc2 though as I had a typo. But wasn't this warning at all though.

  • Status changed to Closed: outdated 7 months ago
  • ๐Ÿ‡บ๐Ÿ‡ธUnited States smustgrave

    Because of the refactoring of the configuration not sure there is an upgrade path between 2.0.x and 3.0.x to be had. And security review doesn't have a huge number of configuration to begin with I think uninstalling 2.0.x, upgrading, installing 3.0.x should work. I added that to the homepage.

  • ๐Ÿ‡ฏ๐Ÿ‡ดJordan Rajab Natshah Jordan

    Rajab Natshah โ†’ made their first commit to this issueโ€™s fork.

Production build 0.71.5 2024