Support from Acquia helps fund testing for Drupal Acquia logo

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

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

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.