Problem/Motivation

 204 | WARNING | \Drupal calls should be avoided in classes, use
     |         | dependency injection instead

Proposed resolution

inject the object use Drupal\Core\State\State;

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#2 xmlsitemap-drupal_injection_on_XmlSitemapEntitiesSettingsForm-3015350.patch2.05 KBthalles

Comments

thalles created an issue. See original summary.

thalles’s picture

thalles
Primary language Portuguese, Brazil
Location Teófilo Otoni - MG
commented
Status: Needs work » Needs review
StatusFileSize
new xmlsitemap-drupal_injection_on_XmlSitemapEntitiesSettingsForm-3015350.patch2.05 KB
alex_optim’s picture

alex_optim
Location Tallinn
commented

Good for me.

voleger’s picture

voleger
he/his
Primary language Ukrainian
Location Ukraine, Rivne
commented
Status: Needs review » Reviewed & tested by the community

Looks good, injection of state service is properly done.

pifagor’s picture

pifagor
Location 🇺🇦 Rivne
commented

  • pifagor committed 967b598 on 8.x-1.x authored by thalles 
    Issue #3015350 by thalles, alex_optim, voleger, pifagor: Drupal...
pifagor’s picture

pifagor
Location 🇺🇦 Rivne
commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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