While I was doing some random code review today, I found that there was one misspelling of EntityManager in core, in a Views test. This patch (coming soon, in comment) fixes it up. Trivial patch of the day, hopefully!

CommentFileSizeAuthor
#2 fix-typo-views-test-2860308-2.patch1.04 KBgeerlingguy

Comments

geerlingguy created an issue. See original summary.

geerlingguy’s picture

Status: Active » Needs review
StatusFileSize
new1.04 KB
krina.addweb’s picture

Status: Needs review » Reviewed & tested by the community

@geerlingguy, Thanks for the patch it works well, As I tested it manually by applying it in module file.

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed and pushed 61b0a38 to 8.4.x and d72fac1 to 8.3.x. Thanks!

This is the only instance of entityManger in core.

  • alexpott committed 61b0a38 on 8.4.x
    Issue #2860308 by geerlingguy: Fix typo in Drupal\Tests\views\Kernel\...

  • alexpott committed d72fac1 on 8.3.x
    Issue #2860308 by geerlingguy: Fix typo in Drupal\Tests\views\Kernel\...
krina.addweb’s picture

@alexpott, I think you forgot to provide me RTBC Credit, can you please do the needful.

Status: Fixed » Closed (fixed)

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