Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
base system
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
20 May 2013 at 22:03 UTC
Updated:
29 Jul 2014 at 22:21 UTC
Jump to comment: Most recent file
Comments
Comment #1
aaronott commentedComment #2
aaronott commentedComment #4
arknoll commented#1: 1999344_1-replace-raw-variables-edit.patch queued for re-testing.
Comment #6
aaronott commentedThis time with PSR-0 Format
Comment #7
aaronott commentedComment #8
kim.pepperAs this is a method being called in a controller route, we can just pass in Request as a magic parameter, instead of using \Drupal::request()
Comment #9
aaronott commented@kim.pepper
I'm not sure how to do this or what you mean by passing in Request as a magic parameter. Do you have an example or link that I can read up on?
Comment #10
kim.pepperHave a look at \Drupal\aggregator\Routing\AggregatorController::feedRefresh() http://drupalcode.org/project/drupal.git/blob/HEAD:/core/modules/aggrega...
The Request parameter can be passed into the method (in any order) and you don't need to specify it in your route configuration.
Comment #11
aaronott commentedOkay so I think I've got this.
Now allowing for the Request parameter to be passed in.
Comment #12
kim.pepperLooking good!
One minor nitpick:
Should have a new line after last param comment
Comment #13
kim.pepperdouble post
Comment #14
aaronott commentedI thought so but missed it. here it is again. Thanks!
Comment #15
dawehnerLooks great!
Comment #16
alexpottCommitted e0db7eb and pushed to 8.x. Thanks!