Function smarty_resource_db_tplinfo has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

function smarty_resource_db_tplinfo( $tpl_name ) {
    static $cache = [];
    global $xoopsConfig;

    if ( isset( $cache[ $tpl_name ] ) ) {

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

                if ( ! $tpl_updated && $config->getVar( 'conf_catid' ) == XOOPS_CONF && $config->getVar( 'conf_name' ) == 'template_set' ) {
                    // clear cached/compiled files and regenerate them if default theme has been changed
                    if ( $xoopsConfig['template_set'] != $_POST[ $config->getVar( 'conf_name' ) ] ) {
                        $newtplset = $_POST[ $config->getVar( 'conf_name' ) ];

Function processPost has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

    public function processPost() {
        // Ticket Check
        if ( ! $GLOBALS['xoopsGTicket']->check( true, 'myblocksadmin' ) ) {
            redirect_header( XOOPS_URL . '/', 3, $GLOBALS['xoopsGTicket']->getErrors() );
        }

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

                    if ( '.' == substr( $file, 0, 1 ) ) {
                        continue;
                    }

Function tplsadmin_import_data has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

function tplsadmin_import_data( $tplset, $tpl_file, $tpl_source, $lastmodified = 0 ) {

    $db =& XoopsDatabaseFactory::getDatabaseConnection();

    // check the file is valid template

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

                if ( ! $theme_updated && $config->getVar( 'conf_catid' ) == XOOPS_CONF && $config->getVar( 'conf_name' ) == 'theme_set' ) {
                    $member_handler =& xoops_gethandler( 'member' );
                    $member_handler->updateUsersByField( 'theme', $_POST[ $config->getVar( 'conf_name' ) ] );
                    $theme_updated = true;
                }

Avoid deeply nested control flow statements.

Open

                    if ( is_file( $file_path ) ) {
                        $mtime   = (int) @filemtime( $file_path );
                        $tplfile = $tplfile_handler->create();
                        $tplfile->setVar( 'tpl_source', file_get_contents( $file_path ), true );
                        $tplfile->setVar( 'tpl_refid', $mid );

Avoid deeply nested control flow statements.

Open

                if ( ! $startmod_updated && $new_value != '--' && $config->getVar( 'conf_catid' ) == XOOPS_CONF && $config->getVar( 'conf_name' ) == 'startpage' ) {
                    $member_handler     =& xoops_gethandler( 'member' );
                    $groups             =& $member_handler->getGroupList();
                    $moduleperm_handler =& xoops_gethandler( 'groupperm' );
                    $module_handler     =& xoops_gethandler( 'module' );

Avoid deeply nested control flow statements.

Open

                if ( ! $lang_updated && $config->getVar( 'conf_catid' ) == XOOPS_CONF && $config->getVar( 'conf_name' ) == 'language' ) {
                    // regenerate admin menu file
                    $xoopsConfig['language'] = $_POST[ $config->getVar( 'conf_name' ) ];
                    xoops_module_write_admin_menu( xoops_module_get_admin_menu() );
                    $lang_updated = true;

Function list_groups has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

    public function list_groups() {
        $handler = null;
  // query for getting blocks
        $sql       = 'SELECT * FROM ' . $this->db->prefix( 'newblocks' ) . " WHERE mid='$this->target_mid' ORDER BY visible DESC,side,weight";
        $result    = $this->db->query( $sql );

• Read up
  Read up

Method __construct has 6 arguments (exceeds 4 allowed). Consider refactoring.

Open

    public function __construct($mydirname, $table_body, $primary_key, $cols, $page_name, $action_base_hiddens)

Method _block has 5 arguments (exceeds 4 allowed). Consider refactoring.

Open

    public function _block( $xbeg, $xlen, $ybeg, $ylen, $edits ) {

Method _diag has 5 arguments (exceeds 4 allowed). Consider refactoring.

Open

    public function _diag( $xoff, $xlim, $yoff, $ylim, $nchunks ) {

Function tplsadmin_copy_templates_f2db has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

function tplsadmin_copy_templates_f2db( $tplset_to, $whr_append = '1' ) {
    global $db;

    // get tplsource
    //$result = $db->query('SELECT * FROM ' . $db->prefix('tplfile') . "  WHERE tpl_tplset='default' AND ($whr_append)") ;

• Read up
  Read up

Function altsys_message_append_oninstall has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

        function altsys_message_append_oninstall( &$module_obj, &$log ) {
            if ( is_array( @$GLOBALS['ret'] ) ) {
                foreach ( $GLOBALS['ret'] as $message ) {
                    $log->add( strip_tags( $message ) );
                }

• Read up
  Read up

Function do_edit has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

    public function do_edit( $bid ) {
        $bid = (int) $bid;

        if ( $bid <= 0 ) {
            // new custom block

• Read up
  Read up

Function _renderOptionTree has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

    public function _renderOptionTree( &$tree, $option, $prefix, $parentIds = [] ) {
        $tree .= $prefix . '<input type="checkbox" name="' . $this->getName() . '[groups][' . $this->_groupId . '][' . $option['id'] . ']" id="' . $this->getName() . '[groups][' . $this->_groupId . '][' . $option['id'] . ']" onclick="';
        // If there are parent elements, add javascript that will
        // make them selected when this element is checked to make
        // sure permissions to parent items are added as well.

• Read up
  Read up

Function altsys_message_append_onuninstall has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

        function altsys_message_append_onuninstall(&$module_obj, &$log)
        {
            if (is_array(@$GLOBALS['ret'])) {
                foreach ($GLOBALS['ret'] as $message) {
                    $log->add(strip_tags($message));

• Read up
  Read up

Function isInstalledXclHtmleditor has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

    public static function isInstalledXclHtmleditor(): bool
    {
        if (defined('LEGACY_BASE_VERSION') && version_compare(LEGACY_BASE_VERSION, '2.2', '>=')) {
            $cNames = self::getDelegateCallbackClassNames('Site.TextareaEditor.HTML.Show');
            if ($cNames) {

• Read up
  Read up

Function list_blocks has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

    public function list_blocks() {
        $handler = null;
  global $xoopsGTicket;

        // main query

• Read up
  Read up