Separating the controller and the view

[eluser]CroNiX[/eluser]
Views are typically used for presenting HTML to the user. I don't think most people would consider a downloaded file, a "view". Also, the HTML views are usually separated so a designer can work just on the html and the developer can work on the code.

There's nothing really wrong with how you're doing it. But, if you are doing this often, in multiple places, it might be better to create a library so all of your PHPWord code is self contained in a single file instead of spread across several controllers/methods. Much easier for maintenance if the PHPWord class ever gets upgraded and things change.