Del\Common\View\Helper\AlertBox

Complexity

Total Complexity

6

Size/Duplication

Total Lines	33
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
B	alertBox()	0	26	6

<?php

namespace Del\Common\View\Helper;

class AlertBox
{
    /**
     * @param array $message array of messages, last item in array should be alert class
     * @return bool|string
     */
    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;
    }
}