Convert system_admin_index() to a new style controller

First try at a patch, with help from the code in core/update/lib/Drupal/update/Controller/UpdateController.php

This looks really good already. Here are just some small nitpicks needed.

+++ b/core/modules/system/lib/Drupal/system/AdminController.phpundefined
@@ -0,0 +1,78 @@
+ * Definition of Drupal\system\AdminController.

Should be \Contains...

+++ b/core/modules/system/lib/Drupal/system/AdminController.phpundefined
@@ -0,0 +1,78 @@
+  /**

new empty line.

+++ b/core/modules/system/lib/Drupal/system/AdminController.phpundefined
@@ -0,0 +1,78 @@
+   * Constructs admin data.

Constructs a AdminController object.

+++ b/core/modules/system/lib/Drupal/system/AdminController.phpundefined
@@ -0,0 +1,78 @@
+   * Prints a listing of admin tasks, organized by module.

Needs @return

+++ b/core/modules/system/lib/Drupal/system/AdminController.phpundefined
@@ -0,0 +1,78 @@
+    return theme('system_admin_index', array('menu_items' => $menu_items));

Let's return a render array instead of directly render it.

Couldn't resist to do a quick new patch before bed.
Previous patch forgot to remove the page callback function from system.admin.inc

Let's move this new file into a "Controller" directory, just to make it more consistent.

Let's move this new file into a "Controller" directory, just to make it more consistent.

+++ b/core/modules/system/lib/Drupal/system/AdminController.phpundefined
@@ -0,0 +1,87 @@
+   *  a render array containing the listing

The start with an "A" and end with a dot :)

Updated + REALLY added the removal of the old page callback function from system.admin.inc

I'm sorry but the patch does not apply anymore.

Reroll of patch 3

Although it seems the testbot fluked, a new patch to correct the location of the ControllerInterface.

Looks good to me.

Needs a reroll

curl https://drupal.org/files/1987812-system_admin_index-controller-5.patch | git a
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  5533  100  5533    0     0  15209      0 --:--:-- --:--:-- --:--:-- 15674
error: patch failed: core/modules/system/system.routing.yml:87
error: core/modules/system/system.routing.yml: patch does not apply

Rerolled

Great. The page also looks fine on manual testing.

Needs a reroll...

curl https://drupal.org/files/1987812-system_admin_index-controller-6.patch | git apply
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  5523  100  5523    0     0   4273      0  0:00:01  0:00:01 --:--:--  7493
error: patch failed: core/modules/system/system.routing.yml:101
error: core/modules/system/system.routing.yml: patch does not apply

Reroll
Conflict

++<<<<<<< HEAD
 +system_status:
 +  pattern: '/admin/reports/status'
 +  defaults:
 +    _controller: 'Drupal\system\Controller\SystemInfoController::status'
 +  requirements:
 +    _permission: 'administer site configuration'
 +
 +system_php:
 +  pattern: '/admin/reports/status/php'
 +  defaults:
 +    _controller: 'Drupal\system\Controller\SystemInfoController::php'
 +  requirements:
 +    _permission: 'administer site configuration'
 +
++=======
+ system_admin_index:
+   pattern: 'admin/index'
+   defaults:
+     _content: 'Drupal\system\Controller\AdminController::index'
+   requirements:
+     _permission: 'access administration pages'
++>>>>>>> 6

Resolved

 +system_status:
 +  pattern: '/admin/reports/status'
 +  defaults:
 +    _controller: 'Drupal\system\Controller\SystemInfoController::status'
 +  requirements:
 +    _permission: 'administer site configuration'
 +
 +system_php:
 +  pattern: '/admin/reports/status/php'
 +  defaults:
 +    _controller: 'Drupal\system\Controller\SystemInfoController::php'
 +  requirements:
 +    _permission: 'administer site configuration'
 +
+ system_admin_index:
+   pattern: 'admin/index'
+   defaults:
+     _content: 'Drupal\system\Controller\AdminController::index'
+   requirements:
+     _permission: 'access administration pages'

Reroll

@jibran: thanks for the effort, but it's confusing when the issue is assigned and then someone else starts fixing it...

@oenie I do apologies for that. :) In my defiance I just want to reroll it ASAP and last patch was posted on "May 27, 2013".
Let's RTBC because no code change.

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