When I was doing some work on #2002606: Allow themes to provide services.yml, I noticed a few minor coder issues that should probably be fixed.

CommentFileSizeAuthor
#1 drupal-drupalkernal.php-2031589-1.patch1.99 KBmarkhalliwell

Comments

markhalliwell’s picture

markhalliwell
Primary language English
commented
Status: Active » Needs review
StatusFileSize
new drupal-drupalkernal.php-2031589-1.patch1.99 KB

Patch

markhalliwell’s picture

markhalliwell
Primary language English
commented
Issue tags: +Novice, +Coding standards, +coder
markhalliwell’s picture

markhalliwell
Primary language English
commented
Component: other » base system
Issue tags: +kernel-followup
markhalliwell’s picture

markhalliwell
Primary language English
commented
Title: DrupalKernal.php coder review » DrupalKernel.php coder review
Issue tags: -Novice, -kernel-followup
tr’s picture

tr commented

#1: drupal-drupalkernal.php-2031589-1.patch queued for re-testing.

Coding standards issues are still there, patch still applies (with minor offsets). Let's see what the testbot says.

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Status: Needs review » Reviewed & tested by the community

Passes tests, looks good to me.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Issue tags: -Coding standards, -coder

#1: drupal-drupalkernal.php-2031589-1.patch queued for re-testing.

xano’s picture

xano
he/they
Primary language English
Location Southampton
commented

1: drupal-drupalkernal.php-2031589-1.patch queued for re-testing.

xano’s picture

xano
he/they
Primary language English
Location Southampton
commented

1: drupal-drupalkernal.php-2031589-1.patch queued for re-testing.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Issue summary: View changes
Status: Reviewed & tested by the community » Fixed

Committed 6856f73 and pushed to 8.x. Thanks!

Status: Fixed » Closed (fixed)

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