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

Status:Active» Postponed

Status:Postponed» Active

its up for grabs now:)

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

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

+++ 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

StatusFileSize
new459 bytes
new10.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

Status:Needs review» Reviewed & tested by the community

Awesome!

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.