Merge pull request #9061 from bertmert/patch-54

Fixes 9054 (part 5, com_content Featured). [REGRESSION] sort order
joomla · Feb 3, 2016 · 3ebcbe0 · 3ebcbe0
2 parents f3479a9 + 0eb1f34
commit 3ebcbe0
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 2 deletions.
diff --git a/administrator/components/com_content/models/articles.php b/administrator/components/com_content/models/articles.php
@@ -74,7 +74,7 @@ public function __construct($config = array())
 	 *
 	 * @since   1.6
 	 */
-	protected function populateState($ordering = null, $direction = null)
+	protected function populateState($ordering = 'a.id', $direction = 'desc')
 	{
 		$app = JFactory::getApplication();
 
@@ -109,7 +109,7 @@ protected function populateState($ordering = null, $direction = null)
 		$this->setState('filter.tag', $tag);
 
 		// List state information.
-		parent::populateState('a.id', 'desc');
+		parent::populateState($ordering, $direction);
 
 		// Force a language
 		$forcedLanguage = $app->input->get('forcedLanguage');

diff --git a/administrator/components/com_content/models/featured.php b/administrator/components/com_content/models/featured.php
@@ -202,4 +202,21 @@ protected function getListQuery($resolveFKs = true)
 
 		return $query;
 	}
+
+	/**
+	 * Method to auto-populate the model state.
+	 *
+	 * Note. Calling getState in this method will result in recursion.
+	 *
+	 * @param   string  $ordering   An optional ordering field.
+	 * @param   string  $direction  An optional direction (asc|desc).
+	 *
+	 * @return  void
+	 *
+	 * @since   1.6
+	 */
+	protected function populateState($ordering = 'a.title', $direction = 'asc')
+	{
+		parent::populateState($ordering, $direction);
+	}
 }