Fix several TODOs in code

joomla · Dec 6, 2018 · be52cb0 · be52cb0
1 parent 8caf7e9
commit be52cb0
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 7 deletions.
diff --git a/libraries/src/Filesystem/FilesystemHelper.php b/libraries/src/Filesystem/FilesystemHelper.php
@@ -22,7 +22,7 @@ class FilesystemHelper
 	/**
 	 * Remote file size function for streams that don't support it
 	 *
-	 * @param   string  $url  TODO Add text
+	 * @param   string  $url  Link identifier
 	 *
 	 * @return  mixed
 	 *

diff --git a/libraries/src/Pagination/Pagination.php b/libraries/src/Pagination/Pagination.php
@@ -737,11 +737,7 @@ protected function _buildDataObject()
 
 		if ($this->pagesCurrent > 1)
 		{
-			$page = ($this->pagesCurrent - 2) * $this->limit;
-
-			// Set the empty for removal from route
-			// @todo remove code: $page = $page == 0 ? '' : $page;
-
+			$page                 = ($this->pagesCurrent - 2) * $this->limit;
 			$data->start->base    = '0';
 			$data->start->link    = Route::_($params . '&' . $this->prefix . 'limitstart=0');
 			$data->previous->base = $page;

diff --git a/libraries/src/Updater/Adapter/ExtensionAdapter.php b/libraries/src/Updater/Adapter/ExtensionAdapter.php
@@ -103,7 +103,6 @@ protected function _endElement($parser, $name)
 	{
 		array_pop($this->stack);
 
-		// @todo remove code: echo 'Closing: '. $name .'<br>';
 		switch ($name)
 		{
 			case 'UPDATE':