AggregatorController is a route controller, but lives under Drupal\aggregator\Routing. All other controllers live under Drupal\{module}\Controller.

This moves AggregatorController to Drupal\aggregator\Controller.

Files: 
CommentFileSizeAuthor
#5 2015377-move-aggregator-controller-5.patch7.12 KBkim.pepper
PASSED: [[SimpleTest]]: [MySQL] 55,065 pass(es).
[ View ]
#5 interdiff.txt4.07 KBkim.pepper
#4 2015377-move-aggregator-controller-3.patch64.87 KBkim.pepper
PASSED: [[SimpleTest]]: [MySQL] 56,327 pass(es).
[ View ]
#4 interdiff.txt4.07 KBkim.pepper
#1 2015377-move-aggregator-controller-1.patch3.05 KBkim.pepper
PASSED: [[SimpleTest]]: [MySQL] 55,784 pass(es).
[ View ]

Comments

kim.pepper’s picture

Status:Active» Needs review
StatusFileSize
new3.05 KB
PASSED: [[SimpleTest]]: [MySQL] 55,784 pass(es).
[ View ]

Initial patch.

ParisLiakos’s picture

if we are doing this, we should move views_ui controller too..its also in wrong folder

dawehner’s picture

Status:Needs review» Reviewed & tested by the community

Whether we do it in this issue or in another issue ... I don't care

kim.pepper’s picture

Title:Move AggregatorController to Controller package» Move Controllers in 'Routing' to 'Controller' namespace
Status:Reviewed & tested by the community» Needs review
StatusFileSize
new4.07 KB
new64.87 KB
PASSED: [[SimpleTest]]: [MySQL] 56,327 pass(es).
[ View ]

Moved views_ui controller as well. Updated issue title.

kim.pepper’s picture

StatusFileSize
new4.07 KB
new7.12 KB
PASSED: [[SimpleTest]]: [MySQL] 55,065 pass(es).
[ View ]

Messy rebase. Let's try that again.

ParisLiakos’s picture

Status:Needs review» Reviewed & tested by the community

thanks!

ParisLiakos’s picture

Status:Reviewed & tested by the community» Fixed

this was committed 466c8c4
thanks!

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