Files: 
CommentFileSizeAuthor
#9 drupal_remove_theme_core_lib-2008954-9.patch4.29 KBmarkie
PASSED: [[SimpleTest]]: [MySQL] 56,275 pass(es). View
#7 drupal_remove_theme_core_lib-2008954-7.patch4.08 KBmarkie
PASSED: [[SimpleTest]]: [MySQL] 56,643 pass(es). View
#4 drupal_remove_theme_core_lib-2008954-4.patch4.08 KBmarkie
FAILED: [[SimpleTest]]: [MySQL] 55,607 pass(es), 6 fail(s), and 6 exception(s). View
#2 drupal_remove_theme_core_lib-2008954-2.patch0 bytesmarkie
PASSED: [[SimpleTest]]: [MySQL] 56,026 pass(es). View

Comments

thedavidmeister’s picture

Issue tags: +Novice
markie’s picture

Status: Active » Needs review
FileSize
0 bytes
PASSED: [[SimpleTest]]: [MySQL] 56,026 pass(es). View

4 files altered.

markie’s picture

Assigned: Unassigned » markie

forgot to call dibs.

markie’s picture

FileSize
4.08 KB
FAILED: [[SimpleTest]]: [MySQL] 55,607 pass(es), 6 fail(s), and 6 exception(s). View

and.. it tends to help to save the files BEFORE doing your git diff.. Novice doesn't even scratch the surface.

Status: Needs review » Needs work

The last submitted patch, drupal_remove_theme_core_lib-2008954-4.patch, failed testing.

thedavidmeister’s picture

-      $content = theme('maintenance_page', array('content' => filter_xss_admin(t(config('system.maintenance')->get('message'), array('@site' => config('system.site')->get('name'))))));
+      $build = array(
+        '#theme' => 'maintenance_page',
+        'content' => filter_xss_admin(
+          t(config('system.maintenance')->get('message'), array('@site' => config('system.site')->get('name')))
+        ),
+      );
+      $content = drupal_render($build);

You missed the # for #content.

markie’s picture

Status: Needs work » Needs review
FileSize
4.08 KB
PASSED: [[SimpleTest]]: [MySQL] 56,643 pass(es). View

updated #content

thedavidmeister’s picture

Status: Needs review » Needs work

A minor styling thing, but we're only using $build when using #theme would conflict with another variable already in the local scope. Check the notes on the parent issue for more details :)

markie’s picture

Status: Needs work » Needs review
FileSize
4.29 KB
PASSED: [[SimpleTest]]: [MySQL] 56,275 pass(es). View

updated per request

thedavidmeister’s picture

Status: Needs review » Reviewed & tested by the community

this looks fine to me.

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed 5d83ca5 and pushed to 8.x. Thanks!

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