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

Zend\Escaper should be required #2

Closed
zf2timo opened this issue Feb 17, 2016 · 2 comments
Closed

Zend\Escaper should be required #2

zf2timo opened this issue Feb 17, 2016 · 2 comments
Labels
bug

Comments

@zf2timo
Copy link
Contributor

zf2timo commented Feb 17, 2016

The Zend\Escaper Component is in suggest and require-dev section of the composer. But it is in the Zend\Debug\Debug Class as use Statement (see

zend-debug/src/Debug.php

Line 12 in 5fdb246

use Zend\Escaper\Escaper;
).
So Zend\Escaper should be a required dependency.

I can send a PR.
@Ocramius Ocramius added the bug label Feb 17, 2016
@Ocramius
Copy link
Member

Please do. It should be a dependency, yes.

@gianarb
Copy link

gianarb commented Feb 17, 2016

Yes please send it!! 👍

@gianarb gianarb mentioned this issue Feb 17, 2016
Merged
weierophinney added a commit that referenced this issue Apr 30, 2018
@weierophinney
Merge branch 'feature/13' into develop
3a6e781 
Close #13
Close #3
Close #4
Close #6
Close #7
Close #8
Close #9
Close #10
Close #12
Fixes #2
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@Ocramius @gianarb @zf2timo