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

geerlingguy commented
Status: Active » Needs review
StatusFileSize
new fix-typo-views-test-2860308-2.patch1.04 KB
krina.addweb’s picture

krina.addweb commented
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

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
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

krina.addweb commented

@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.