Skip to content
This repository has been archived by the owner on Jan 31, 2020. It is now read-only.

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

Open
popovserhii opened this issue Apr 6, 2018 · 4 comments
Open

Comments

@popovserhii
Copy link

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.

@popovserhii popovserhii changed the title Zend\View\Helper\FlashMessenger isn't compatible with Expressive app Zend\View\Helper\FlashMessenger isn't compatible with Expressive Apr 6, 2018
@michalbundyra
Copy link
Member

@popovserhii
Copy link
Author

@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
Copy link
Contributor

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
Copy link
Member

This repository has been closed and moved to laminas/laminas-view; a new issue has been opened at laminas/laminas-view#8.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants