J2.5 talk

Supporting SEF URLs in your component

From Joomla! Documentation

Revision as of 08:01, 3 October 2008 by Batch1211 (talk | contribs) (New page: Hi there, for me the following statement does not really makes sense: <source lang="php"> function check() { jimport( 'joomla.filter.output' ); $alias = JFilterOutput::strin...)
(diff) ← Older revision | Latest revision (diff) | Newer revision → (diff)

Hi there, for me the following statement does not really makes sense:

function check()
{
       jimport( 'joomla.filter.output' );
       $alias = JFilterOutput::stringURLSafe( $this-­>title );
       if (empty( $this-­>alias ) || $this-­>alias === $alias ) {
               $this->alias = $alias;
       }
       /* All your other checks */
       return true;
}

As I read it, the filtered $alias is only asigned to the table property, if $table->alias is empty OR $table->alias is exactly the same. So in case $this->alias is not empty, the property will not even be checked or reasigned.

What I find more logical is the method used in the Joomla components

if(empty($this->alias)) {
	$this->alias = $this->title;
}
$this->alias = JFilterOutput::stringURLSafe($this->alias);