Del\Common\View\Helper\AlertBox::alertBox()

Complexity

Conditions	6
Paths	13

Size

Total Lines	26
Code Lines	17

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	18
CRAP Score	6

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function alertBox(array $message)

Please use __construct() instead of a PHP4-style constructor that is named after the class.

    {
        if(!$message){return false;}

The expression $message of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

        if(count($message) < 2) {
            $class = 'info';
        } else {
            $class = array_pop($message);
        }

        $alert = '<div class="alert ';
        if($class != 'alert'){$alert .= 'alert-'.$class;}
        $alert .= '"><button type="button" class="close" data-dismiss="alert" aria-label="Close">
                      <span aria-hidden="true">&times;</span>
                    </button>';

        $num = count($message);
        $x = 1;
        foreach($message as $msg)
        {
            $alert .= $msg;
            if($x < $num){$alert .= '<br />';}
            $x ++;
        }
        $alert .= '</div>';
        return $alert;
    }
}