e107inc/e107

View on GitHub
e107_handlers/mailout_admin_class.php

Summary

Maintainability
A
0 mins
Test Coverage
F
11%

Returning type array<int meaningful data for email>|array{}</int> but checkEmailPost() is declared to return true
Open

            return $errList;
Severity: Minor
Found in e107_handlers/mailout_admin_class.php by phan

Suspicious array access to false
Open

                if (count($ourTemplate->lastTemplateData['email_overrides']))
Severity: Minor
Found in e107_handlers/mailout_admin_class.php by phan

Variable $text was undeclared, but it is being used as the left-hand side of an assignment operation
Open

     $text .= '<tr><td>'.LAN_MAILOUT_173.'</td><td>'.($mailData['mail_togo_count'] +

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

                            $line = "Count update for {$saveRow['mail_source_id']} - {$saveRow['mail_togo_count']}, {$saveRow['mail_sent_count']}, {$saveRow['mail_fail_count']}, {$saveRow['mail_bounce_count']} => ";
Severity: Minor
Found in e107_handlers/mailout_admin_class.php by phan

Call to deprecated function \e_parse::text_truncate() defined at /code/e107_handlers/e_parse_class.php:1286
Open

                        $text .= e107::getParser()->text_truncate($val, 200, '...');
Severity: Minor
Found in e107_handlers/mailout_admin_class.php by phan

Variable $text was undeclared, but it is being used as the left-hand side of an assignment operation
Open

     $text .= $this->showMailDetail($mailData, 'basic');

Variable $counters is undeclared
Open

                        foreach ($counters as $k => $v)
Severity: Minor
Found in e107_handlers/mailout_admin_class.php by phan

Variable $text was undeclared, but it is being used as the left-hand side of an assignment operation
Open

     $text .=

Variable $text was undeclared, but it is being used as the left-hand side of an assignment operation
Open

     $text .= "

Variable $text was undeclared, but it is being used as the left-hand side of an assignment operation
Open

     $text .= "</tbody></table>\n</fieldset>";

Invalid offset "mail_fail_count" of array type array{}
Open

                            $line = "Count update for {$saveRow['mail_source_id']} - {$saveRow['mail_togo_count']}, {$saveRow['mail_sent_count']}, {$saveRow['mail_fail_count']}, {$saveRow['mail_bounce_count']} => ";
Severity: Minor
Found in e107_handlers/mailout_admin_class.php by phan

Argument 2 (key) is false but \e107::getCoreTemplate() takes null|string defined at /code/e107_handlers/e107_class.php:3348
Open

        $templates = e107::getCoreTemplate('email', false, 'front', false);
Severity: Minor
Found in e107_handlers/mailout_admin_class.php by phan

Suspicious array access to false
Open

                    $email['mail_overrides'] = $ourTemplate->lastTemplateData['email_overrides'];
Severity: Minor
Found in e107_handlers/mailout_admin_class.php by phan

Variable $text is undeclared
Open

     $ns-> tablerender(ADLAN_136.SEP.LAN_MAILOUT_171, $mes->render().$text);
Severity: Minor
Found in e107_handlers/mailout_admin_class.php by phan

Invalid offset "mail_source_id" of array type array{}
Open

                            $line = "Count update for {$saveRow['mail_source_id']} - {$saveRow['mail_togo_count']}, {$saveRow['mail_sent_count']}, {$saveRow['mail_fail_count']}, {$saveRow['mail_bounce_count']} => ";
Severity: Minor
Found in e107_handlers/mailout_admin_class.php by phan

Invalid offset "mail_togo_count" of array type array{}
Open

                            $line = "Count update for {$saveRow['mail_source_id']} - {$saveRow['mail_togo_count']}, {$saveRow['mail_sent_count']}, {$saveRow['mail_fail_count']}, {$saveRow['mail_bounce_count']} => ";
Severity: Minor
Found in e107_handlers/mailout_admin_class.php by phan

Returning type array<int send format error>|array<int sender email address specified>|array<int sender name specified>|array<int subject specified>|array<int text in email body>|array<int>|array{}</int></int></int></int></int></int> but checkEmailPost() is declared to return true
Open

        return $errList;
Severity: Minor
Found in e107_handlers/mailout_admin_class.php by phan

Call to deprecated function \e_parse::text_truncate() defined at /code/e107_handlers/e_parse_class.php:1286
Open

                $text .= ($v > 1) ? $tp->text_truncate($val, $v, '...') : $val;
Severity: Minor
Found in e107_handlers/mailout_admin_class.php by phan

Variable $text was undeclared, but it is being used as the left-hand side of an assignment operation
Open

     $text .= "<div class='buttons-bar center'>

Argument 2 (replace) is int|true but \str_replace() takes array|string
Open

                $results[] = str_replace('[x]', $res, LAN_MAILOUT_226);
Severity: Minor
Found in e107_handlers/mailout_admin_class.php by phan

Returning type ?'\x3cspan class=\'label label-warning\'\x3eNo \'email address sources\' selected in Preferences\x3c/span\x3e'|?string but emailSelector() is declared to return string
Open

            return $text;
Severity: Minor
Found in e107_handlers/mailout_admin_class.php by phan

Variable $text is undeclared
Open

     $ns->tablerender(ADLAN_136.SEP.LAN_MAILOUT_171, $text);
Severity: Minor
Found in e107_handlers/mailout_admin_class.php by phan

Invalid offset "mail_sent_count" of array type array{}
Open

                            $line = "Count update for {$saveRow['mail_source_id']} - {$saveRow['mail_togo_count']}, {$saveRow['mail_sent_count']}, {$saveRow['mail_fail_count']}, {$saveRow['mail_bounce_count']} => ";
Severity: Minor
Found in e107_handlers/mailout_admin_class.php by phan

Invalid offset "mail_bounce_count" of array type array{}
Open

                            $line = "Count update for {$saveRow['mail_source_id']} - {$saveRow['mail_togo_count']}, {$saveRow['mail_sent_count']}, {$saveRow['mail_fail_count']}, {$saveRow['mail_bounce_count']} => ";
Severity: Minor
Found in e107_handlers/mailout_admin_class.php by phan

Return type of loadMailHandlers() is undeclared type \number
Open

    public function loadMailHandlers($options = 'all')
Severity: Minor
Found in e107_handlers/mailout_admin_class.php by phan

Returning type 0 but loadMailHandlers() is declared to return \number
Open

        return $ret;
Severity: Minor
Found in e107_handlers/mailout_admin_class.php by phan

Argument 2 (datestamp) is string but \e_form::datepicker() takes bool|false|int defined at /code/e107_handlers/form_handler.php:1712
Open

        return e107::getForm()->datepicker($calName, $calVal, $options);
Severity: Minor
Found in e107_handlers/mailout_admin_class.php by phan

Returning type 0 but loadMailHandlers() is declared to return \number
Open

            return $ret;
Severity: Minor
Found in e107_handlers/mailout_admin_class.php by phan

Variable $counters was undeclared, but array fields are being added to it.
Open

            $counters['add'] = $mailData['mail_togo_count'];

Variable $counters is undeclared
Open

                            $line .= implode(', ', $counters);
Severity: Minor
Found in e107_handlers/mailout_admin_class.php by phan

There are no issues that match your filters.

Category
Status