Method list_groups has 28 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function list_groups() {
        // 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 );
        $block_arr = [];

Method construct has 28 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function construct() {
        $this->db =& XoopsDatabaseFactory::getDatabaseConnection();

        $this->lang = @$GLOBALS['xoopsConfig']['language'];

Method renderCell4BlockModuleLink has 28 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function renderCell4BlockModuleLink( $block_data ) {
        $bid = (int) $block_data['bid'];

        // get selected targets
        if ( is_array( @$block_data['bmodule'] ) ) {

Function _loadAllChildItemIds has a Cognitive Complexity of 10 (exceeds 5 allowed). Consider refactoring.

Open

    public function _loadAllChildItemIds( $itemId, &$childIds ) {
        if ( ! empty( $this->_itemTree[ $itemId ]['children'] ) ) {
            $first_child = $this->_itemTree[ $itemId ]['children'];
            foreach ( $first_child as $fcid ) {
                $childIds[] = $fcid;

• Read up
  Read up

Function _tool_PathReduction has a Cognitive Complexity of 10 (exceeds 5 allowed). Consider refactoring.

Open

  public function _tool_PathReduction($p_dir)
  {
      $v_result = '';

    // ----- Look for not empty path

• Read up
  Read up

Function tplsadmin_copy_templates_db2db has a Cognitive Complexity of 10 (exceeds 5 allowed). Consider refactoring.

Open

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

    // get tplfile and tplsource

• Read up
  Read up

Function renderCell4BlockModuleLink has a Cognitive Complexity of 10 (exceeds 5 allowed). Consider refactoring.

Open

    public function renderCell4BlockModuleLink( $block_data ) {
        $bid = (int) $block_data['bid'];

        // get selected targets
        if ( is_array( @$block_data['bmodule'] ) ) {

• Read up
  Read up

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

Open

  public function _duplicate($p_archive_filename)
  {
      $v_result=1;

    // ----- Look if the $p_archive_filename exists

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

Open

function altsys_mylangadmin_get_constant_names( $langfile_unique_path, $mydirname ) {

    $constpref = '';

    $langfile_names = [];

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

Open

    public function renderCell4BlockReadGroupPerm( $block_data ) {
        $bid = (int) $block_data['bid'];

        // get selected targets
        if ( is_array( @$block_data['bgroup'] ) ) {

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

Open

    public static function getDelegateCallbackClassNames($name, bool $doRegist = true)
    {
        $names = [];

        if (!class_exists('XCube_Delegate')) {

Consider simplifying this complex logical expression.

Open

      if ((isset($p_params[ARCHIVE_ZIP_PARAM_BY_NAME]))
          && (0 != $p_params[ARCHIVE_ZIP_PARAM_BY_NAME])) {

          // ----- Look if the filename is in the list
          for ($j=0;

Consider simplifying this complex logical expression.

Open

      if ((isset($p_params[ARCHIVE_ZIP_PARAM_BY_NAME]))
          && (0 != $p_params[ARCHIVE_ZIP_PARAM_BY_NAME])) {

          // ----- Look if the filename is in the list
          for ($j=0;

Method triggerEvent has 8 arguments (exceeds 4 allowed). Consider refactoring.

Open

    public function triggerEvent($mydirname, $mytrustdirname, $category, $item_id, $event, array $extra_tags = [], array $user_list = [], $omit_user_id = null)

Function renderCell4BlockPosition has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.

Open

    public function renderCell4BlockPosition( $block_data ) {
        $bid     = (int) $block_data['bid'];
        $side    = (int) $block_data['side'];
        $visible = (int) $block_data['visible'];

• Read up
  Read up

Function delete has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.

Open

    public function delete($delete_comments = false, $delete_notifications = false)
    {
        $db =& XoopsDatabaseFactory::getDatabaseConnection();

        $ret = [];

• Read up
  Read up

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

Open

  public function _addList($p_list, &$p_result_list,
                    $p_add_dir, $p_remove_dir, $p_remove_all_dir, &$p_params)

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;

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

Open

  public function _addFileList($p_list, &$p_result_list,
                        $p_add_dir, $p_remove_dir, $p_remove_all_dir,
                        &$p_params)

Avoid deeply nested control flow statements.

Open

                  if ((strlen($v_header_list[$v_nb_extracted]['stored_filename']) > strlen($p_params[ARCHIVE_ZIP_PARAM_BY_NAME][$j]))
                      && (substr($v_header_list[$v_nb_extracted]['stored_filename'], 0, strlen($p_params[ARCHIVE_ZIP_PARAM_BY_NAME][$j])) == $p_params[ARCHIVE_ZIP_PARAM_BY_NAME][$j])) {
                      $v_found = true;
                  } elseif ((0x00000010 == ($v_header_list[$v_nb_extracted]['external'] & 0x00000010)) /* Indicates a folder */
                            && ($v_header_list[$v_nb_extracted]['stored_filename'].'/' == $p_params[ARCHIVE_ZIP_PARAM_BY_NAME][$j])) {