Add auto escape method to Zend\View\Helper\FlashMessenger

[Martin-P]

Added methods setAutoEscape and getAutoEscape to Zend\View\Helper\FlashMessenger as proposed in issue #5600.

[mwillbanks]

I think this should also be defined out on the render portion of the object so that you can set it easily within one line or rather than using an initializer.

[Martin-P]

What exactly do you mean? As a third parameter for the method render()? You don't need an initializer to set the auto escape, you can do this when calling the view helper:

echo $this->flashMessenger()->setAutoEscape(false)->render();

[mwillbanks]

@Martin-P yes; understand that. but many people like to utilize it as follows:

echo $this->flashMessenger()->render('MyNamespace', array('alert alert-success'));

I am ultimately saying that one might want to leverage a boolean flag as the third option.

[weierophinney]

Agreed with @mwillbanks -- escaping is a rendering issue, and it should be possible to pass the flag to render(). The workflow would be as follows:

• User can write a factory to set it via setter: $flash->setAutoEscape(true)
• User can set the flag in a view script (or elsewhere): $flash->setAutoEscape(true)
• User can set the flag for the current invocation only: $flash('MyNamespace', array('alert'), true) - In this case, if the flag is null, the value of the $autoEscape property is used.

Make sense, @Martin-P ?

[Martin-P]

Thanks for the clarification, I updated the code accordingly.

[Ocramius]

This PR needs a rebase

[rodrigotavio91]

I'm trying to put a link into a flashmessage... this working? Cause in my project, this function don't have this parameter $autoEscape.

I'm sorry, just now I saw that it will be available in ZF 2.4.0.