New MVC GSOC 2014

https://groups.google.com/forum/#!topic/joomla-dev-cms/AH6piuw2IR0

http://gwilson.org/index.php/new-mvc
There is a fallback in the constructor, but I'd agree if it was doing some checks.

Sincerely,
Mathew Lenning

Babel-university.com

P.S. This message was sent via iPhone, so please forgive any errors

> On Aug 22, 2014, at 9:43 PM, Michael Babker <notifications@github.com> wrote:
> 
> In libraries/cms/view/html/cms.php:
> 
> > +	}
> > +
> > +	/**
> > +	 * Method to render the view.
> > +	 *
> > +	 * @return  string  The rendered view.
> > +	 *
> > +	 * @since   3.4
> > +	 * @throws  RuntimeException
> > +	 */
> > +	public function render()
> > +	{
> > +		// Add toolbar items as necessary
> > +		$this->addToolbar();
> > +
> > +		return $this->getRenderer()->render($this->getLayout(), $this->getData());
> Personal preference - I try to refer to get/set methods when they exist instead of directly accessing class variables in the off chance those variables change names. Also, if the get method has an error handling mechanism in it, that's preferable too (what if the renderer isn't set for some odd reason?).
> 
> \
> Reply to this email directly or view it on GitHub.