Identical blocks of code found in 2 locations. Consider refactoring.

Open

                for ( $j = 0; $j < $opcount; $j ++ ) {
                    $optval = defined( $options[ $j ]->getVar( 'confop_value' ) ) ? constant( $options[ $j ]->getVar( 'confop_value' ) ) : $options[ $j ]->getVar( 'confop_value' );
                    $optkey = defined( $options[ $j ]->getVar( 'confop_name' ) ) ? constant( $options[ $j ]->getVar( 'confop_name' ) ) : $options[ $j ]->getVar( 'confop_name' );

                    $ele->addOption( $optval, $optkey . $addBr );

• Read up
  Read up

Function _readEndCentralDir has a Cognitive Complexity of 23 (exceeds 5 allowed). Consider refactoring.

Open

  public function _readEndCentralDir(&$p_central_dir)
  {
      $v_result=1;

    // ----- Go to the end of the zip file

• Read up
  Read up

Method _readEndCentralDir has 81 lines of code (exceeds 25 allowed). Consider refactoring.

Open

  public function _readEndCentralDir(&$p_central_dir)
  {
      $v_result=1;

    // ----- Go to the end of the zip file

Method altsys_oninstall_base has 80 lines of code (exceeds 25 allowed). Consider refactoring.

Open

        function altsys_oninstall_base( $module, $mydirname ) {
            // transactions on module install

            global $ret; // TODO :-D

Function draw_repost_form has a Cognitive Complexity of 22 (exceeds 5 allowed). Consider refactoring.

Open

        public function draw_repost_form(string $area = '' ) {
            // Notify which file is broken

            if ( headers_sent() ) {
                restore_error_handler();

• Read up
  Read up

Method diff has 75 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function diff( $from_lines, $to_lines ) {
        $n_from = count( $from_lines );
        $n_to   = count( $to_lines );

        $this->xchanged = $this->ychanged = [];

Function triggerEvent has a Cognitive Complexity of 21 (exceeds 5 allowed). Consider refactoring.

Open

    public function triggerEvent($mydirname, $mytrustdirname, $category, $item_id, $event, array $extra_tags = [], array $user_list = [], $omit_user_id = null)
    {
        $module_handler =& xoops_gethandler('module');

        $module =& $module_handler->getByDirname($mydirname);

• Read up
  Read up

Method _shiftBoundaries has 65 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function _shiftBoundaries( $lines, &$changed, $other_changed ) {
        $i = 0;
        $j = 0;

        assert( count( $lines ) == count( $changed ) );

Method _diag has 63 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function _diag( $xoff, $xlim, $yoff, $ylim, $nchunks ) {
        $flip = false;

        if ( $xlim - $xoff > $ylim - $yoff ) {
            /* Things seems faster (I'm not sure I understand why) when the

Method renderCell4BlockPosition has 61 lines of code (exceeds 25 allowed). Consider refactoring.

Open

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

Function _tool_PathInclusion has a Cognitive Complexity of 18 (exceeds 5 allowed). Consider refactoring.

Open

  public function _tool_PathInclusion($p_dir, $p_path)
  {
      $v_result = 1;

    // ----- Explode dir and path by directory separator

• Read up
  Read up

Method altsys_admin_in_theme_in_last has 56 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function altsys_admin_in_theme_in_last( $contents = null ) {
    global $xoops_admin_contents, $xoopsConfig, $xoopsModule, $xoopsUser, $xoopsUserIsAdmin, $xoopsLogger, $altsysModuleConfig, $altsysModuleId;

    if ( ! isset( $contents ) ) {
        while ( ob_get_level() ) {

Method list_blocks has 56 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function list_blocks() {
        global $xoopsGTicket;

        // main query
        $sql       = 'SELECT * FROM ' . $this->db->prefix( 'newblocks' ) . " WHERE mid='$this->target_mid' ORDER BY visible DESC,side,weight";

Method render has 54 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function render() {
        global $xoopsGTicket;

        // load all child ids for javascript codes
        foreach ( array_keys( $this->_itemTree ) as $item_id ) {

Method check has 54 lines of code (exceeds 25 allowed). Consider refactoring.

Open

        public function check(bool $post = true, string $area = '', bool $allow_repost = true ) {
            global $xoopsModule;

            $this->_errors = [];

Method altsys_onupdate_base has 54 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    function altsys_onupdate_base( $module, $mydirname ) {
        // transactions on module update

        global $msgs; // TODO :-D

Method read has 50 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function read($resource, $mydirname, $mytrustdirname = null, bool $read_once = true)
    {
        $d3file = XOOPS_ROOT_PATH . '/modules/' . $mydirname . '/mytrustdirname.php';

        if (empty($mytrustdirname) && is_file($d3file)) {

Method update_block has 50 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function update_block( $bid, $bside, $bweight, $bvisible, $btitle, $bcontent, $bctype, $bcachetime, $options = [] ) {
        global $xoopsConfig;

        //HACK by domifara
        if ( defined( 'XOOPS_CUBE_LEGACY' ) ) {

Similar blocks of code found in 2 locations. Consider refactoring.

Open

                    if ( ! $tplfile_handler->insert( $tplfile ) ) {
                        $msgs[] = '<span style="color:#ff0000;">ERROR: Could not insert template <b>' . htmlspecialchars( $mydirname . '_' . $file ) . '</b> to the database.</span>';
                    } else {
                        $tplid  = $tplfile->getVar( 'tpl_id' );
                        $msgs[] = 'Template <b>' . htmlspecialchars( $mydirname . '_' . $file ) . '</b> added to the database. (ID: <b>' . $tplid . '</b>)';

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

                        if ( ! $tplfile_handler->insert( $tplfile ) ) {
                            $ret[] = '<span style="color:#ff0000;">ERROR: Could not insert template <b>' . htmlspecialchars( $mydirname . '_' . $file ) . '</b> to the database.</span><br>';
                        } else {
                            $tplid = $tplfile->getVar( 'tpl_id' );
                            $ret[] = 'Template <b>' . htmlspecialchars( $mydirname . '_' . $file ) . '</b> added to the database. (ID: <b>' . $tplid . '</b>)<br>';

• Read up
  Read up