blocks/donat_o_meter.php ➔ b_donations_donatometer_show()

Complexity

Conditions	22
Paths	768

Size

Total Lines	195
Code Lines	150

Duplication

Lines	5
Ratio	2.56 %

Importance

Changes

0

<?php
function b_donations_donatometer_show($options)
{
    global $xoopsDB;
    $xdBlockDir = basename(dirname(__DIR__));
    $utility = new Xdonations\Utility();
    $tr_config = $utility::getConfigInfo();
    //determine the currency
    $PP_CURR_CODE = explode('|', $tr_config['pp_curr_code']); // [USD,GBP,JPY,CAD,EUR]
    $PP_CURR_CODE = $PP_CURR_CODE[0];
    $currencySign = $utility::defineCurrency($PP_CURR_CODE);

    $block = [];

    $swingd = $tr_config['swing_day'];
    if (($swingd < 0) || ($swingd > 31)) {
        $swingd = 6;
    }
    $dmshowdate = $options[1];
    $dmshowamt  = $options[2];

    if (is_numeric($options[0]) && $options[0] > 0) {
        $dmlen = $options[0];
    } elseif (is_numeric($dmlen) && 0 == $dmlen) {

The variable $dmlen seems to be never defined.

        $dmlen = -1;
    } else {
        $dmlen = 10;
    }

    // Check the current day against the swing day to execute the proper query
    if (date('d') >= $swingd) {
        $query_Recordset1 = 'SELECT count(mc_gross) AS count, sum(mc_gross) AS gross, sum('
                            . "mc_gross-mc_fee) AS net, date_format( now(),'%M') AS mon, date_format( subdate( date_format( adddate("
                            . "now(), INTERVAL 1 MONTH),'%Y-%c-1'), INTERVAL 1 DAY), '%b %e') AS due_by, date_format(now(),'%b') AS "
                            . 'mon_short FROM '
                            . $xoopsDB->prefix('donations_transactions')
                            . ' WHERE (payment_date >= date_format('
                            . "now(),'%Y-%m-"
                            . $swingd
                            . "'))";

        $query_Recordset3 = 'select custom as muser_id, option_selection1 as showname, '
                            . "date_format( payment_date, '%b-%e') as date, concat('"
                            . $currencySign
                            . "',sum(mc_gross)) as amt "
                            . 'from '
                            . $xoopsDB->prefix('donations_transactions')
                            . ' where (payment_date >= date_format( '
                            . "now(), '%Y-%m-"
                            . $swingd
                            . "')) group by txn_id order by payment_date desc";
    } else {
        $query_Recordset1 = 'select count(mc_gross) as count, sum(mc_gross) as gross, sum(mc_gross -'
                            . ' mc_fee) as net, date_format( subdate( now(), interval '
                            . $swingd
                            . " day), '%M') as mon,"
                            . " 'Now!' as due_by, date_format( subdate( now(), interval "
                            . $swingd
                            . " day), '%b') as mon_short"
                            . ' from '
                            . $xoopsDB->prefix('donations_transactions')
                            . " where (payment_date < date_format( now(), '%Y-%m-"
                            . $swingd
                            . "')"
                            . ') and payment_date > date_format( subdate( now(), interval '
                            . $swingd
                            . " day), '%Y-%m-"
                            . $swingd
                            . "')";

        $query_Recordset3 = 'select custom as muser_id, option_selection1 as showname, '
                            . "date_format( payment_date, '%b-%e') as date, concat('"
                            . $currencySign
                            . "', sum(mc_gross)) as amt "
                            . 'from '
                            . $xoopsDB->prefix('donations_transactions')
                            . ' where (payment_date < date_format(now(),'
                            . " '%Y-%m-"
                            . $swingd
                            . "')) and payment_date > date_format( subdate( now(),interval "
                            . $swingd
                            . ' '
                            . "day), '%Y-%m-"
                            . $swingd
                            . "') group by txn_id order by payment_date desc";
    }

    // Get the donation totals
    $Recordset1     = $xoopsDB->query($query_Recordset1);
    $row_Recordset1 = $xoopsDB->fetchArray($Recordset1);
    //If there are not records, then get "null" data
    if (!$row_Recordset1) {
        $query_Recordset1 = "select '0' as count, '0' as gross, '0' as net, date_format( now(),"
                            . "'%M') as mon, date_format( subdate( date_format( adddate( now(), interval 1 month), '%Y-%c-1'),"
                            . " interval 1 day), '%b %e') as due_by, date_format( now(), '%b') as mon_short from "
                            . ' '
                            . $xoopsDB->prefix('donations_transactions')
                            . '';
        $Recordset1       = $xoopsDB->query($query_Recordset1);
        $row_Recordset1   = $xoopsDB->fetchArray($Recordset1);
    }
    // Get the goal
    $query_Recordset2 = 'SELECT * FROM ' . $xoopsDB->prefix('donations_config') . " WHERE name='month_goal' AND subtype='" . $row_Recordset1['mon_short'] . "'";
    $Recordset2       = $xoopsDB->query($query_Recordset2);
    $row_Recordset2   = $xoopsDB->fetchArray($Recordset2);

    // Set our remaining template vars
    if (!$row_Recordset1['mon']) {
        $DM_MON = date('F');
    } else {
        $DM_MON = $row_Recordset1['mon'];
    }
    $difference   = $row_Recordset1['net'] - $row_Recordset2['value'];
    $DM_GOAL      = sprintf($currencySign . '%.02f', $row_Recordset2['value']);
    $DM_DUE       = $row_Recordset1['due_by'];
    $DM_NUM       = $row_Recordset1['count'];

The assignment to $DM_NUM is dead and can be removed.

    $DM_GROSS     = sprintf($currencySign . '%.02f', $row_Recordset1['gross']);
    $DM_NET       = sprintf($currencySign . '%.02f', $row_Recordset1['net']);
    $DM_LEFT      = sprintf($currencySign . '%.02f', $row_Recordset2['value'] - $row_Recordset1['net']);
    $DM_BUTTON    = $options[3];
    $DM_BUTT_DIMS = '';
    if (is_numeric($options[4])) {
        $DM_BUTT_DIMS .= 'width=' . $options[4] . ' ';
    }
    if (is_numeric($options[5])) {
        $DM_BUTT_DIMS .= 'height=' . $options[5] . ' ';
    }

    // Load the template
    $block['DM_BUTTON']    = $DM_BUTTON;
    $block['DM_BUTT_DIMS'] = $DM_BUTT_DIMS;
    $block['DM_MON']       = $DM_MON;
    $block['DM_GOAL']      = $DM_GOAL;
    $block['DM_DUE']       = $DM_DUE;
    $block['DM_GROSS']     = $DM_GROSS;
    $block['DM_NET']       = $DM_NET;
    $block['DON_URL']      = XOOPS_URL . '/modules/' . $xdBlockDir . '/index.php';

The constant XOOPS_URL was not found. Maybe you did not declare it correctly or list all dependencies?

    $show_don              = 0;
    // Do we want to display the donators?
    if (is_numeric($dmlen) && $dmlen >= 0) {
        $show_don = 1;
        // Get the list of donators
        $Recordset3 = $xoopsDB->query($query_Recordset3);

        // List all the donators
        $i   = 0;
        $var = '';
        while (($row_Recordset3 = $xoopsDB->fetchArray($Recordset3)) && ($i != $options[0])) {
            // Refunded transactions will show up with $0 amount
            if ($row_Recordset3['amt'] > '$0') {
                $dmalign = 'center';
                $var     .= "<tr><td style=\"width: 100%; text-align: {$dmalign};\" colspan=\"2\">\n";
                // Observe the user's wish regarding revealing their name
                $muser_id = $row_Recordset3['muser_id'];
                /** @var \XoopsUser $userfoin */
                if (0 == strcmp($row_Recordset3['showname'], 'Yes') && ($userfoin = $utility::getUserInfo($muser_id))) {
                    $var .= "<a href='" . XOOPS_URL . '/userinfo.php?uid=' . $userfoin->getVar('uid') . "'>" . $userfoin->getVar('uname') . "</a>\n";
                } else {
                    $var .= _MB_XDONATION_ANONYMOUS_SHORT;
                }
                $var .= '&nbsp;';
                if ($dmshowamt) {
                    $var .= '(' . $row_Recordset3['amt'] . ')';
                }
                if ($dmshowdate) {
                    $var .= $row_Recordset3['date'];
                }
                $var .= "</td></tr>\n";
            }
            ++$i;
        }
    }

    if ($difference >= 0) {
        $DM_OVERAGE          = sprintf($currencySign . '%.02f', $difference);
        $block['DM_REMAIN']  = _MB_XDONATION_SURPLUS;
        $block['DM_BALANCE'] = $DM_OVERAGE;
    } else {
        $block['DM_REMAIN']  = _MB_XDONATION_LEFT2GO;
        $block['DM_BALANCE'] = "<span style=\"color: #CC0000;\">{$DM_LEFT}</span>";
    }

    // Define language constants
    $block['DM_STAT']      = _MB_XDONATION_STAT;
    $block['DM_MONGOAL']   = _MB_XDONATION_MONGOAL;
    $block['DM_DUEDATE']   = _MB_XDONATION_DUEDATE;
    $block['DM_GROSSAMT']  = _MB_XDONATION_GROSSAMT;
    $block['DM_NETBAL']    = _MB_XDONATION_NETBAL;
    $block['DM_DONATIONS'] = _MB_XDONATION_DONATIONS;
    $block['DM_MAKEDON']   = _MB_XDONATION_MAKEDON;

    // Display block
    $block['show_don'] = $show_don;
    $block['content']  = $var;

The variable $var does not seem to be defined for all execution paths leading up to this point.

    return $block;
}