Problem/Motivation

Deprecated function: mb_strtolower(): Passing null to parameter

Steps to reproduce

Create a new flagging collection

Proposed resolution

Check if the string is null.
The following is used in other modules:

$id = $id ? str_replace([' ', '_', '[', ']'], ['-', '-', '-', ''], mb_strtolower($id)) : '';

Remaining tasks

-

User interface changes

-

API changes

-

Data model changes

-

CommentFileSizeAuthor
#7 issue-3407890-deprecated-function-mbstrtolower.patch831 bytesviren18febs

Issue fork flag_lists-3407890

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

sl27257 created an issue. See original summary.

chetan 11 made their first commit to this issue’s fork.

chetan 11 opened merge request !2

chetan 11’s picture

chetan 11 commented
Assigned: sl27257 » Unassigned
Status: Active » Needs review

Hi,
I have fixed the above error on "flag list" module.
Please check the raised MR.
Thanks.

sl27257’s picture

sl27257
Location Stockholm
commented
Status: Needs review » Active

Hi,

I am sorry but that is not the problem here :(

/Thomas

sl27257’s picture

sl27257
Location Stockholm
commented

The problem here is exactly this:

#3302838: Querying with NULL values results in warning mb_strtolower(): Passing null to parameter is deprecated

But as stated in #25 this a problem in core which can be caused by a contributed module. So I will not mark this as a duplicate yet. Instead this will be kept open so that it can be fixed in the flag_lists if possible.

viren18febs’s picture

viren18febs commented
Status: Active » Needs review
StatusFileSize
new issue-3407890-deprecated-function-mbstrtolower.patch831 bytes

I have resolved the issue & added patch file, please review.

sl27257’s picture

sl27257
Location Stockholm
commented
Status: Needs review » Active

Hi,

I am sorry, but as pointed out in #5 above, this is NOT the problem here.