From 35cc65820c2a02f97936d16622374bef1eb9b68e Mon Sep 17 00:00:00 2001 From: Hamish Friedlander Date: Thu, 19 Jul 2012 13:57:48 +1200 Subject: [PATCH] BUG Make RequestProcessor->filters settable as a property too filters was a DI property that could only be set via constructor. This meant that modules couldnt add a filter without interfering with other modules. With this change you can now add a config block like: Injector: RequestProcessor: properties: filters: - "%$MyFilter" Which will add a filter to RequestProcessors list of filters --- control/RequestProcessor.php | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/control/RequestProcessor.php b/control/RequestProcessor.php index f82476fdc..f978a60f7 100644 --- a/control/RequestProcessor.php +++ b/control/RequestProcessor.php @@ -14,6 +14,10 @@ class RequestProcessor { $this->filters = $filters; } + public function setFilters($filters) { + $this->filters = $filters; + } + public function preRequest(SS_HTTPRequest $request, Session $session, DataModel $model) { foreach ($this->filters as $filter) { $res = $filter->preRequest($request, $session, $model);