Problem/Motivation

Logs report
Notice: Undefined variable: variable in cookiebot_variable_info() (line 340 of /var/www/html/public/sites/all/modules/contrib/cookiebot/cookiebot.module).

Steps to reproduce

Enable the module, check the logs

Proposed resolution

cookiebot_variable_info declares $variables but returns $variable
Check: https://git.drupalcode.org/project/cookiebot/-/blob/7.x-1.x/cookiebot.mo... and phpstorm error in screenshot
cookiebot undefined variable

Remaining tasks

Review patch

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#3 undefined_variable.png115.57 KBbserem
#2 cookiebot-7.x-undefined-variable-3316540-1.patch268 bytesbserem

Comments

bserem created an issue. See original summary.

bserem’s picture

bserem commented
Assigned: bserem » Unassigned
Status: Active » Needs review
StatusFileSize
new cookiebot-7.x-undefined-variable-3316540-1.patch268 bytes

Please review attached patch

bserem’s picture

bserem commented
Issue summary: View changes
StatusFileSize
new undefined_variable.png115.57 KB
bramdriesen’s picture

bramdriesen
he/him
Primary language Dutch
Location Belgium 🇧🇪🇪🇺
commented
Status: Needs review » Reviewed & tested by the community

Looks good to me :) good catch!

  • BramDriesen committed c0d82d2 on 7.x-1.x 
    Issue #3316540 by bserem, BramDriesen: Undefined variable: variable
bramdriesen’s picture

bramdriesen
he/him
Primary language Dutch
Location Belgium 🇧🇪🇪🇺
commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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