Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Import View Helpers from Laminas\View #66

Open
wants to merge 3 commits into
base: 2.21.x
Choose a base branch
from

Conversation

gsteel
Copy link
Member

@gsteel gsteel commented Nov 22, 2024

This patch copies the view helpers from laminas-view so that they can be deprecated and removed there.

No improvements have been made to the tests or the validators - they have been copied as-is with the exception of the namespace change, and where possible, usage of Laminas\Translator\TranslatorInterface in favour of Laminas\I18n\…

Also upgrades PHPUnit to 10.x

…iew`

Also:

- upgrades to PHPUnit 10
- adds `laminas-translator` dependency

Signed-off-by: George Steel <[email protected]>
Signed-off-by: George Steel <[email protected]>
@gsteel gsteel added this to the 2.21.0 milestone Nov 22, 2024
@gsteel gsteel requested a review from froschdesign November 22, 2024 22:59
Signed-off-by: George Steel <[email protected]>
@froschdesign
Copy link
Member

@gsteel
I have added a question here: laminas/laminas-view#259 (comment)

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

Successfully merging this pull request may close these issues.

2 participants