Question

I’m trying display error messages on a form but only one is displayed (the last one always). I tried using a foreach loop but I keep getting the invalid argument error. The following displays errors one by one. Code is inside a class…

public $errorContainer = '';

// ------------------------------------------------------------
// ERROR MESSAGE PROCESSING
// ------------------------------------------------------------
private function responseMessage($respBool, $respMessage) {
    $return['error'] = $respBool;
    $return['msg'] = $respMessage;
    if (isset($_POST['plAjax']) && $_POST['plAjax'] == true) {
        echo json_encode($return);
    } else {
        $this->errorContainer = $respMessage;
    }
}

The following always gives me the invalid for each argument error.

private function responseMessage($respBool, $respMessage) {
    $return['error'] = $respBool;
    $return['msg'] = $respMessage;
    if (isset($_POST['plAjax']) && $_POST['plAjax'] == true) {
        echo json_encode($return);
    } else {
        foreach ($respMessage as $value) {
            $this->errorContainer = $value;
        }
    }
}

Thank you!

Answer 1

replace your foreach() with this:

private function responseMessage($respBool, $respMessage) {
  // ...code...
  foreach ((array) $respMessage as $value) {
    $this->errorContainer .= $value;
  }
  // ...code---
}

Using type casting (array) above will make it works for both array and string type.

Edit:

Use this solution (type casting) only in your last effort. But your real problem is you’re not passing an array to the function. See this code:

// incorrect
$msg = 'This is a message';
$this->responseMessage($some_bool, $msg);

// correct
$msg = array('This is a message');
$this->responseMessage($some_bool, $msg);

// correct
$msg = array('This is a message', 'And another message');
$this->responseMessage($some_bool, $msg);

If you pass the argument correctly like above, you don’t need to cast $respMessage to array.