In Table.php we're using a static call to get the renderer: \Drupal::service('renderer')

However, the Views PluginBase base class that we extend already has an injected renderer service that we can and should use instead: $this->getRenderer()

CommentFileSizeAuthor
#2 3119011-2-renderer.patch1.26 KBtr
renderer.patch1.27 KBtr

Comments

TR created an issue. See original summary.

tr’s picture

tr commented
StatusFileSize
new 3119011-2-renderer.patch1.26 KB

Corrected typo in the previous patch. We need to improve test coverage so that things like that typo will fail ... but that can wait until another issue.

  • TR committed e4a3ac1 on 8.x-1.x 
    Issue #3119011 by TR: Use injected renderer
tr’s picture

tr commented
Status: Needs review » Fixed

Committed.

Status: Fixed » Closed (fixed)

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