[Solved] Return $data question

When I var dump my my $message variable it displays the multiple messages which is  correct.

But the return only shows one as shown here.



How can I produce the same on return

PHP Code:

public function display_error_messages($open_tag = '<p>', $close_tag = '</p>') {

if (isset($this->error_message)) {

  foreach($this->error_message as $msg) {

    $message = $open_tag . $msg . $close_tag;
    var_dump($message);

  }


}

return $message;

} 


There's only one rule - please don't tell anyone to go and read the manual.  Sometimes the manual just SUCKS!

You recreate (and overwrite) the $message variable instead of appending the second and third messages to it. Try:

PHP Code:

$message .= $open_tag . $msg . $close_tag; 


You recreate (and overwrite) the $message variable instead of appending the second and third messages to it. Try:

PHP Code:

$message .= $open_tag . $msg . $close_tag; 


I did this now but no getting undefined variable $message

PHP Code:

public function display_error_messages($open_tag = '<p>', $close_tag = '</p>') {

        if (isset($this->error_message)) {
            foreach($this->error_message as $msg) {
               $message .= $open_tag . $msg . $close_tag;
               var_dump($message);
            }


        }

        return $message;

    } 


There's only one rule - please don't tell anyone to go and read the manual.  Sometimes the manual just SUCKS!

Define $message='' before entering the foreach loop.

Website: http://avenir.ro

Define $message='' before entering the foreach loop.

Is This Correct

PHP Code:

public function display_error_messages($open_tag = '<p>', $close_tag = '</p>') {

        if (isset($this->error_message)) {

            $message = "";

            foreach($this->error_message as $msg) {
                $message .= $open_tag . $msg . $close_tag;
            }


        }

        return $message;

    } 


There's only one rule - please don't tell anyone to go and read the manual.  Sometimes the manual just SUCKS!

if you defined the $message depending on the if statement, then you should also return the $message from inside the if statement...

Code:

public function display_error_messages($open_tag = '<p>', $close_tag = '</p>') {

        if (isset($this->error_message)) {

            $message = "";

            foreach($this->error_message as $msg) {
                $message .= $open_tag . $msg . $close_tag;
            }
            return $message;


        }
        return FALSE; // or whatever you want to return in case there are no error messages...

        

    } 

But it is you who should tell me if it's correct or not

Website: http://avenir.ro

if you defined the $message depending on the if statement, then you should also return the $message from inside the if statement...

Code:

public function display_error_messages($open_tag = '<p>', $close_tag = '</p>') {

        if (isset($this->error_message)) {

            $message = "";

            foreach($this->error_message as $msg) {
                $message .= $open_tag . $msg . $close_tag;
            }
           return $message;


        }
       return FALSE; // or whatever you want to return in case there are no error messages...

        

    } 

But it is you who should tell me if it's correct or not

Thank you for help it all works now. And thank you to every body else to.

There's only one rule - please don't tell anyone to go and read the manual.  Sometimes the manual just SUCKS!