ImpressCMS/impresscms

Method showRSS has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function showRSS() {
    global $icmsAdminTpl, $icmsConfigPersona;

    $rssurl = $icmsConfigPersona['rss_local'];
    $rssfile = ICMS_CACHE_PATH . '/adminnews_' . _LANGCODE . '.xml';

Missing class import via use statement (line '87', column '14').

Open

    $feed = new icms_feeds_Simplerss();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function make() {
    return new \stdClass();
}

Avoid unused local variables such as '$rssfile'.

Open

    $rssfile = ICMS_CACHE_PATH . '/adminnews_' . _LANGCODE . '.xml';

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    public function doSomething()
    {
        $i = 5; // Unused
    }
}

Suspicious type null of a variable or expression used to build a string. (Expected type to be able to cast to a string)

Open

if (!file_exists(ICMS_CACHE_PATH . '/adminmenu_' . $icmsConfig['language'] . '.php')) {

• Exclude checks
  • Disable engine
  • Disable check

Call to method set_cache_duration from undeclared class \icms_feeds_Simplerss

Open

    $feed->set_cache_duration(3600);

• Exclude checks
  • Disable engine
  • Disable check

Call to method set_autodiscovery_level from undeclared class \icms_feeds_Simplerss

Open

    $feed->set_autodiscovery_level(SIMPLEPIE_LOCATOR_NONE);

• Exclude checks
  • Disable engine
  • Disable check

Reference to instance property error from undeclared class \icms_feeds_Simplerss

Open

    if (!$feed->error) {

• Exclude checks
  • Disable engine
  • Disable check

Invalid offset "language" of array type array{startpage:,template_set:mixed}

Open

if (!file_exists(ICMS_CACHE_PATH . '/adminmenu_' . $icmsConfig['language'] . '.php')) {

• Exclude checks
  • Disable engine
  • Disable check

Call to method get_title from undeclared class \icms_feeds_Simplerss

Open

        $icmsAdminTpl->assign('admin_rss_feed_title', $feed->get_title());

• Exclude checks
  • Disable engine
  • Disable check

Call to method get_description from undeclared class \icms_feeds_Simplerss

Open

        $icmsAdminTpl->assign('admin_rss_feed_dsc', $feed->get_description());

• Exclude checks
  • Disable engine
  • Disable check

Call to method init from undeclared class \icms_feeds_Simplerss

Open

    $feed->init();

• Exclude checks
  • Disable engine
  • Disable check

Call to method set_feed_url from undeclared class \icms_feeds_Simplerss

Open

    $feed->set_feed_url($rssurl);

• Exclude checks
  • Disable engine
  • Disable check

Argument 1 (content) is array but \xoops_module_write_admin_menu() takes string defined at /code/htdocs/include/cp_functions.php:583

Open

    xoops_module_write_admin_menu(impresscms_get_adminmenu());

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_LANGCODE

Open

    $rssfile = ICMS_CACHE_PATH . '/adminnews_' . _LANGCODE . '.xml';

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_feeds_Simplerss

Open

    $feed = new icms_feeds_Simplerss();

• Exclude checks
  • Disable engine
  • Disable check

Call to method get_link from undeclared class \icms_feeds_Simplerss

Open

        $icmsAdminTpl->assign('admin_rss_feed_link', $feed->get_link());

• Exclude checks
  • Disable engine
  • Disable check

Call to method handle_content_type from undeclared class \icms_feeds_Simplerss

Open

    $feed->handle_content_type();

• Exclude checks
  • Disable engine
  • Disable check

Call to method get_items from undeclared class \icms_feeds_Simplerss

Open

        foreach ($feed->get_items() as $item) {

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \SIMPLEPIE_LOCATOR_NONE

Open

    $feed->set_autodiscovery_level(SIMPLEPIE_LOCATOR_NONE);

• Exclude checks
  • Disable engine
  • Disable check

Closing brace must be on a line by itself

Open

if (!empty($_POST['op'])) {$op = (int) ($_POST['op']);}

• Exclude checks
  • Disable engine
  • Disable check

Newline required after opening brace

Open

if (!empty($_POST['op'])) {$op = (int) ($_POST['op']);}

• Exclude checks
  • Disable engine
  • Disable check

Closing brace must be on a line by itself

Open

if (!empty($_GET['op'])) {$op = (int) ($_GET['op']);}

• Exclude checks
  • Disable engine
  • Disable check

Newline required after opening brace

Open

if (!empty($_GET['op'])) {$op = (int) ($_GET['op']);}

• Exclude checks
  • Disable engine
  • Disable check