Files: 
CommentFileSizeAuthor
#5 drupal-aggregator_admin_to_route-1972264-5.patch10.55 KBParisLiakos
PASSED: [[SimpleTest]]: [MySQL] 55,398 pass(es). View
#5 interdiff.txt459 bytesParisLiakos
#3 drupal-aggregator_admin_to_route-1972264-3.patch10.55 KBParisLiakos
PASSED: [[SimpleTest]]: [MySQL] 55,968 pass(es). View

Comments

ParisLiakos’s picture

Status: Active » Postponed
ParisLiakos’s picture

Status: Postponed » Active

its up for grabs now:)

ParisLiakos’s picture

Assigned: Unassigned » ParisLiakos
Status: Active » Needs review
FileSize
10.55 KB
PASSED: [[SimpleTest]]: [MySQL] 55,968 pass(es). View

Also made it to return a render array, not an HTML string

dawehner’s picture

+++ b/core/modules/aggregator/lib/Drupal/aggregator/Routing/AggregatorController.phpundefined
@@ -56,4 +70,86 @@ public function feedAdd() {
+  }

Ha, an empty missing line. I knew that there has to be something wrong on that patch :P

ParisLiakos’s picture

FileSize
459 bytes
10.55 KB
PASSED: [[SimpleTest]]: [MySQL] 55,398 pass(es). View

Ha! you got me:P
i noticed it after posting the patch but i was lazy:P

dawehner’s picture

Status: Needs review » Reviewed & tested by the community

Awesome!

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed 9b1c15f and pushed to 8.x. Thanks!

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