Return void in dispatcher-Forward in controller...

[borisdelev]

I want to know something that bothers me. Its about forward in controller- by documentation i must be aware that forward is not like HTTP redirect- thats great. That is how i expect to work. But found that snippet in MVC/Controller:

     public function saveAction()
      {
          // Forwards flow to the index action
          return $this->dispatcher->forward(
              [
                  "controller" => "people",
                  "action"     => "index",
              ]
          );
      }

and... forward return void... and... this is working, yes, but is somehow wrong. Must be something like that for example:

     public function saveAction()
      {
          // Forwards flow to the index action
          $this->dispatcher->forward(
              [
                  "controller" => "people",
                  "action"     => "index",
              ]
          );
          return true;
      }

So to be clear- my question is about what is right- can we return in our action: return dispatcher->forward or make return after forward?

[niden]

The second is correct. Forward returns void so no "return" from a controller.

[niden]

This has been fixed phalcon/documentation@fba5821

Thank you @borisdelev

[borisdelev]

Thank you!!!!