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

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

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.