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
StatusFileSize
new0 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

StatusFileSize
new4.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
StatusFileSize
new4.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
StatusFileSize
new4.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.