From 7c0ff89adf24ca41ad2555f40f7b6887dec18f37 Mon Sep 17 00:00:00 2001 From: Marco Hermo Date: Sat, 29 Apr 2017 01:31:53 +1200 Subject: [PATCH] Removed @todo as SearchLogFactory interface is injected via Yaml config --- code/solr/tasks/Solr_BuildTask.php | 4 ---- 1 file changed, 4 deletions(-) diff --git a/code/solr/tasks/Solr_BuildTask.php b/code/solr/tasks/Solr_BuildTask.php index 7107137..9e4e3c9 100644 --- a/code/solr/tasks/Solr_BuildTask.php +++ b/code/solr/tasks/Solr_BuildTask.php @@ -25,8 +25,6 @@ class Solr_BuildTask extends BuildTask */ public function getLogger() { - //@todo left commented after a confusing merge conflict. Revisit if further testing is required - //return Injector::inst()->get('Logger'); return Injector::inst()->get(LoggerInterface::class); } @@ -45,8 +43,6 @@ class Solr_BuildTask extends BuildTask */ protected function getLoggerFactory() { - //@todo left commented after a confusing merge conflict. Revisit if further testing is required - //return Injector::inst()->get('SearchLogFactory'); return Injector::inst()->get(SearchLogFactory::class); }