e107inc/e107

Consider simplifying this complex logical expression.

Open

        if(defined('e_PAGE_LANGUAGE') && ($detect_language = $this->isValid(e_PAGE_LANGUAGE))) // page specific override.
        {
            $doNothing = '';
            // Do nothing as $detect_language is set.
        }

Avoid too many return statements within this method.

Open

            return false;

Default value for \str $type can't be string

Open

    function installed($type='english')

• Exclude checks
  • Disable engine
  • Disable check

Returning type false but convert() is declared to return string

Open

            return (isset($this->list[$data])) ? $this->list[$data] : false;

• Exclude checks
  • Disable engine
  • Disable check

Parameter $type has undeclared type \str

Open

    function installed($type='english')

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared property \language->_select_array

Open

        if($force ||null === $this->_select_array)

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \e_PAGE_LANGUAGE

Open

        if(defined('e_PAGE_LANGUAGE') && ($detect_language = $this->isValid(e_PAGE_LANGUAGE))) // page specific override.

• Exclude checks
  • Disable engine
  • Disable check

Argument 2 (value) is false but \setcookie() takes string

Open

                setcookie('e107_language', $this->detect, time() + 86400, e_HTTP);

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared property \language->_select_array

Open

                $this->_select_array[$this->convert($lan)] = $this->toNative($lan);

• Exclude checks
  • Disable engine
  • Disable check

Default value for object $lang can't be string

Open

    function isValid($lang='')

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared property \language->_select_array

Open

            $this->_select_array = array();

• Exclude checks
  • Disable engine
  • Disable check

Returning type object|string but isValid() is declared to return false

Open

            return $lang;    

• Exclude checks
  • Disable engine
  • Disable check

Returning type int but installed() is declared to return array

Open

                return count($this->lanlist);

• Exclude checks
  • Disable engine
  • Disable check

Returning type false but convert() is declared to return string

Open

            return isset($tmp[$data]) ? $tmp[$data] : false;

• Exclude checks
  • Disable engine
  • Disable check

Argument 1 (lang) is false but \language::isValid() takes object|string defined at /code/e107_handlers/language_class.php:278

Open

            if(!$session->has('e_language') || (($session->get('e_language') != $this->detect) && $this->isValid($this->detect)))

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared property \language->_select_array

Open

        return $this->_select_array;

• Exclude checks
  • Disable engine
  • Disable check