e107inc/e107

View on GitHub
e107_handlers/js_manager.php

Summary

Maintainability
A
3 hrs
Test Coverage
F
55%

Avoid too many return statements within this method.
Open

            return $this;
Severity: Major
Found in e107_handlers/js_manager.php - About 30 mins to fix

    Avoid too many return statements within this method.
    Open

            return $this;
    Severity: Major
    Found in e107_handlers/js_manager.php - About 30 mins to fix

      Avoid too many return statements within this method.
      Open

                      return $this;
      Severity: Major
      Found in e107_handlers/js_manager.php - About 30 mins to fix

        Avoid too many return statements within this method.
        Open

                        return $this;
        Severity: Major
        Found in e107_handlers/js_manager.php - About 30 mins to fix

          Avoid too many return statements within this method.
          Open

                  return false;
          Severity: Major
          Found in e107_handlers/js_manager.php - About 30 mins to fix

            Avoid too many return statements within this method.
            Open

                            return $this;
            Severity: Major
            Found in e107_handlers/js_manager.php - About 30 mins to fix

              Avoid too many return statements within this method.
              Open

                              return $this;
              Severity: Major
              Found in e107_handlers/js_manager.php - About 30 mins to fix

                Assigning null to property but \e_jsmanager->_dependence is string
                Open

                        $this->_dependence = null;
                Severity: Minor
                Found in e107_handlers/js_manager.php by phan

                Argument 2 (when) is void but \e_jsmanager::setLastModfied() takes int defined at /code/e107_handlers/js_manager.php:2115
                Open

                                $this->setLastModfied($mod, $this->renderFile($this->_e_jslib_theme, $external, 'Theme libraries', $mod));
                Severity: Minor
                Found in e107_handlers/js_manager.php by phan

                Returning type '' but renderFile() is declared to return void
                Open

                            return '';
                Severity: Minor
                Found in e107_handlers/js_manager.php by phan

                Argument 2 (cacheId) is 'js' but \e_jsmanager::url() takes bool|true defined at /code/e107_handlers/js_manager.php:1626
                Open

                                        $path = $this->url($path, 'js');
                Severity: Minor
                Found in e107_handlers/js_manager.php by phan

                Argument 2 (cacheId) is 'js' but \e_jsmanager::url() takes bool|true defined at /code/e107_handlers/js_manager.php:1626
                Open

                                echo "<script src='".$this->url(e_WEB_ABS."cache/".$fileName,'js',false)."' $deferCache></script>\n\n";
                Severity: Minor
                Found in e107_handlers/js_manager.php by phan

                Argument 2 (mode) is 'mix' but \e_parse::createConstants() takes int defined at /code/e107_handlers/e_parse_class.php:3255
                Open

                                $file_path = $runtime_location.$this->_sep.$tp->createConstants($file_path, 'mix').$this->_sep.$pre.$this->_sep.$post;
                Severity: Minor
                Found in e107_handlers/js_manager.php by phan

                Reference to undeclared constant \e_HTTP
                Open

                        'basePath' => e_HTTP,
                Severity: Minor
                Found in e107_handlers/js_manager.php by phan

                Argument 1 (path) is null but \e_parse::staticUrl() takes string defined at /code/e107_handlers/e_parse_class.php:2412
                Open

                            $http = $tp->staticUrl(null); // returns SITEURL or Static URL if enabled.  array('full'=>1) removed.
                Severity: Minor
                Found in e107_handlers/js_manager.php by phan

                Assigning int to property but \e_jsmanager->_zone_footer is array
                Open

                    protected $_zone_footer = 0;
                Severity: Minor
                Found in e107_handlers/js_manager.php by phan

                Assigning int to property but \e_jsmanager->_zone_header is array
                Open

                    protected $_zone_header = 0;
                Severity: Minor
                Found in e107_handlers/js_manager.php by phan

                Argument 2 (mode) is 'mix' but \e_parse::createConstants() takes int defined at /code/e107_handlers/e_parse_class.php:3255
                Open

                                $file_path = $runtime_location.$this->_sep.$tp->createConstants($file_path, 'mix').$this->_sep.$pre.$this->_sep.$post;
                Severity: Minor
                Found in e107_handlers/js_manager.php by phan

                Argument 2 (mode) is 'mix' but \e_parse::createConstants() takes int defined at /code/e107_handlers/e_parse_class.php:3255
                Open

                                    $tp->createConstants($file_path, 'mix'),
                Severity: Minor
                Found in e107_handlers/js_manager.php by phan

                Assigning null to property but \e_jsmanager->_dependence is string
                Open

                        $this->_dependence = null;
                Severity: Minor
                Found in e107_handlers/js_manager.php by phan

                Argument 2 (mode) is 'mix' but \e_parse::createConstants() takes int defined at /code/e107_handlers/e_parse_class.php:3255
                Open

                                    $tp->createConstants($file_path, 'mix'),
                Severity: Minor
                Found in e107_handlers/js_manager.php by phan

                Returning type '' but renderInline() is declared to return void
                Open

                            return '';
                Severity: Minor
                Found in e107_handlers/js_manager.php by phan

                Argument 2 (when) is void but \e_jsmanager::setLastModfied() takes int defined at /code/e107_handlers/js_manager.php:2115
                Open

                                $this->setLastModfied($mod, $this->renderFile($this->_e_jslib_plugin, $external, 'Plugin libraries', $mod));
                Severity: Minor
                Found in e107_handlers/js_manager.php by phan

                Call with 3 arg(s) to \e_jsmanager::url() which only takes 2 arg(s) defined at /code/e107_handlers/js_manager.php:1626
                Open

                                echo "<script src='".$this->url(e_WEB_ABS."cache/".$fileName,'js',false)."' $deferCache></script>\n\n";
                Severity: Info
                Found in e107_handlers/js_manager.php by phan

                Returning type 0 but renderFile() is declared to return void
                Open

                        return $lmodified;
                Severity: Minor
                Found in e107_handlers/js_manager.php by phan

                Assigning null to property but \e_jsmanager->_dependence is string
                Open

                        $this->_dependence = $dep;        
                Severity: Minor
                Found in e107_handlers/js_manager.php by phan

                Argument 2 (when) is void but \e_jsmanager::setLastModfied() takes int defined at /code/e107_handlers/js_manager.php:2115
                Open

                                $this->setLastModfied($mod, $this->renderFile($this->_e_jslib_core, $external, 'Core libraries', $mod));
                Severity: Minor
                Found in e107_handlers/js_manager.php by phan

                There are no issues that match your filters.

                Category
                Status