zendframework / zend-view

View component from Zend Framework
BSD 3-Clause "New" or "Revised" License
49 stars 61 forks source link

Zend\View\Helper\FlashMessenger isn't compatible with Expressive #151

Open popovserhii opened 6 years ago

popovserhii commented 6 years ago

In our project, we use Zend\Expressive as core and Zend\View as template engine. And we come across with an issue that FlashMessenger only works with Zend\Mvc\Controller\Plugin\FlashMessenger or Zend\Mvc\Plugin\FlashMessenger\FlashMessenger. None of them is available in Expressive and add dependencies on Zend\Mvc isn't a good idea.

There are different Flash implementations and will be great that Zend\View\Helper\FlashMessenger depend on some FlashInterface instead of Zend\Mvc\Plugin\FlashMessenger\FlashMessenger. This feature will allow integrating Zend\View\Helper\FlashMessenger with many Flash Messanger libraries.

I've written a module for this but for now FlashInterface is dirty and require improvement.

michalbundyra commented 6 years ago

@popovserhii Have you seen https://github.com/zendframework/zend-expressive-flash ?

popovserhii commented 6 years ago

@webimpress Yes, but this module isn't working correctly. The problem is not in zend-expressive-flash but in zend-expressive-session-ext.

Аfter composer require zendframework/zend-expressive-flash and modifying config/pipeline.php, authorization to my admin stops working. I was trying to understand but everything is very confusing. That why I've created a custom module.

But even if zend-expressive-flash will work I will not be able to use a flashMessanger helper which helps renders html markup.

samsonasik commented 6 years ago

the zend-expressive-flash should work, I created a blog post about it some time ago https://samsonasik.wordpress.com/2018/02/11/using-view-helper-for-accessing-zend-expressive-flash-messages-in-expressive-3/

weierophinney commented 4 years ago

This repository has been closed and moved to laminas/laminas-view; a new issue has been opened at https://github.com/laminas/laminas-view/issues/8.