Problem/Motivation

drupal-check results on commit hash: 735d4b095aff18e63a90f559c3e1a41691ebad0a


------ ---------------------------------------------------
Line src/Form/EuCookieComplianceConfigForm.php
------ ---------------------------------------------------
835 Call to deprecated function drupal_set_message().
------ ---------------------------------------------------

[ERROR] Found 1 error

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

CommentFileSizeAuthor
#3 eu_cookie_compliance-replaced_drupal_set_message-3042637-3.patch1.04 KBi-trokhanenko

Comments

mcdwayne created an issue. See original summary.

svenryen’s picture

svenryen commented

Thanks for the report. Seems like it should be easy to resolve.

i-trokhanenko’s picture

i-trokhanenko
Location Lutsk 🇺🇦
commented
Status: Active » Needs review
StatusFileSize
new eu_cookie_compliance-replaced_drupal_set_message-3042637-3.patch1.04 KB

Please review!

Status: Needs review » Needs work

The last submitted patch, 3: eu_cookie_compliance-replaced_drupal_set_message-3042637-3.patch, failed testing. View results

i-trokhanenko’s picture

i-trokhanenko
Location Lutsk 🇺🇦
commented
Status: Needs work » Needs review

It seems that test bot marks patches for modules with no tests as Build Successful which turns into Needs Work (failed testing), but this patch does not break stuff. Putting this back to Needs Review.

Roman Dyn’s picture

Roman Dyn commented
Status: Needs review » Reviewed & tested by the community

Checked patch #3. All Changes works good for me. Please commit.

svenryen’s picture

svenryen commented

Thanks for the patch. I'll commit it later.

svenryen’s picture

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

Status: Fixed » Closed (fixed)

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