Removed unused call and variable.

Also Todo'd two things for someone to check
joomla · Oct 2, 2016 · 2897582 · 2897582
1 parent ceb6a45
commit 2897582
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/administrator/components/com_finder/controllers/indexer.json.php b/administrator/components/com_finder/controllers/indexer.json.php
@@ -151,6 +151,8 @@ public function batch()
 
 		// Get the HTML document.
 		$html = JDocument::getInstance('html', $attributes);
+
+		//todo: Why is this document fetched and immediately overwritten?
 		$doc = JFactory::getDocument();
 
 		// Swap the documents.
@@ -164,6 +166,8 @@ public function batch()
 
 		// Swap the app.
 		$app = JFactory::getApplication();
+
+		//todo: Why is the app fetched and immediately overwritten?
 		$app = $site;
 
 		// Start the indexer.

diff --git a/components/com_finder/models/search.php b/components/com_finder/models/search.php
@@ -1070,7 +1070,6 @@ protected function populateState($ordering = null, $direction = null)
 		$input = $app->input;
 		$params = $app->getParams();
 		$user = JFactory::getUser();
-		$filter = JFilterInput::getInstance();
 
 		$this->setState('filter.language', JLanguageMultilang::isEnabled());