36/36 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%

------ ----------------------------------------------------------------------
Line tests/src/Functional/ParserTest.php
------ ----------------------------------------------------------------------
383 Call to deprecated function drupal_set_message():
in Drupal 8.5.0 and will be removed before Drupal 9.0.0.
Use \Drupal\Core\Messenger\MessengerInterface::addMessage() instead.
------ ----------------------------------------------------------------------

[ERROR] Found 1 error

CommentFileSizeAuthor
#2 weather-deprecated_code-3087409-2.patch742 bytesLilit_Ghazaryan
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Lilit_Ghazaryan created an issue. See original summary.

Lilit_Ghazaryan’s picture

Lilit_Ghazaryan CreditAttribution: Lilit_Ghazaryan at EPAM Systems commented
FileSize
weather-deprecated_code-3087409-2.patch742 bytes
Lilit_Ghazaryan’s picture

Lilit_Ghazaryan CreditAttribution: Lilit_Ghazaryan at EPAM Systems commented
Status: Needs work » Closed (duplicate)
Lilit_Ghazaryan’s picture

Lilit_Ghazaryan CreditAttribution: Lilit_Ghazaryan at EPAM Systems commented
Status: Closed (duplicate) » Needs review
sosevich.v’s picture

sosevich.v CreditAttribution: sosevich.v at EPAM Systems commented
Status: Needs review » Reviewed & tested by the community

After the patch applied it solved the issue.

alunyov’s picture

alunyov CreditAttribution: alunyov at EPAM Systems commented

Looks good to me. Patch was applied correctly. @Lilit_Ghazaryan thanks for your work

akyrylenko’s picture

akyrylenko CreditAttribution: akyrylenko at EPAM Systems commented

Patch reviewed and works.
Version: 8.x-1.x-dev

ysamoylenko’s picture

ysamoylenko CreditAttribution: ysamoylenko at EPAM Systems commented

Hello everyone,

Thanks for your report and review.

The patch will be committed soon.

ysamoylenko’s picture

ysamoylenko CreditAttribution: ysamoylenko at EPAM Systems commented
Status: Reviewed & tested by the community » Fixed

The patch was committed.

Thanks, everyone for your help with module development.

Status: Fixed » Closed (fixed)

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

Kristen Pol’s picture

Kristen Pol
she/her
Primary language English
Location Santa Cruz, CA, USA
CreditAttribution: Kristen Pol at Hook 42 commented
Issue tags: +Drupal 9 compatibility

Per a Slack discussion with Gábor Hojtsy regarding usage of D9 tags (Drupal 9, Drupal 9 compatibility, Drupal 9 readiness, etc.), "Drupal 9 compatibility" should be used for contributed projects that need updating and "Drupal 9" was the old tag for D8 issues before the D9 branch was ready. Doing tag cleanup here based on that discussion.