New MVC GSOC 2014

https://groups.google.com/forum/#!topic/joomla-dev-cms/AH6piuw2IR0

http://gwilson.org/index.php/new-mvc
That method is intended to be overridden in the extending classes, I just put that there to show how It was supposed to work. Might have even taken direct from legacy, can't recall at the moment. 

The idea is was to prevent the setState recursion issue. If you look in setState you'll see that it sets stateIsSet to true. At least it used too, not sure if that's still intact. 

Sincerely,
Mathew Lenning

Babel-university.com

P.S. This message was sent via iPhone, so please forgive any errors

> On Aug 27, 2014, at 11:54 PM, zero-24 <notifications@github.com> wrote:
> 
> In libraries/cms/model/cms.php:
> 
> > +	 * to be called on the first call to the getState() method unless the model
> > +	 * configuration flag to ignore the request is set.
> > +	 *
> > +	 * @param string $ordering  column to order by. I.E. 'a.title'
> > +	 * @param string $direction 'ASC' or 'DESC'
> > +	 *
> > +	 * @return  void
> > +	 *
> > +	 * @note    Calling getState in this method will result in recursion.
> > +	 * @since   3.4
> > +	 */
> > +	protected function populateState($ordering = null, $direction = null)
> > +	{
> > +		if (!$this->stateIsSet)
> > +		{
> > +			//do something
> Is something missed here or is it expected? :D
> 
> \
> Reply to this email directly or view it on GitHub.