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.

Sign up for GitHub

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

Jump to bottom

Duplicated factory definition for Laminas\I18n\Validator\IsInt in Laminas\Validator\ValidatorPluginManager #78

Closed
devision-milen-dzhambazov opened this issue Dec 18, 2020 · 1 comment
Closed

Duplicated factory definition for Laminas\I18n\Validator\IsInt in Laminas\Validator\ValidatorPluginManager #78

devision-milen-dzhambazov opened this issue Dec 18, 2020 · 1 comment
Labels
Duplicate This issue or pull request already exists

Comments

@devision-milen-dzhambazov
Copy link

Bug Report

Q A
Version(s) 2.13.4

Summary

There is a duplicated factory definition for Laminas\I18n\Validator\IsInt in Laminas\Validator\ValidatorPluginManager on lines 368 and 373.

Current behavior

The factory definition is duplicated.

How to reproduce

Open src/ValidatorPluginManager.php and go to lines 368 and 373.

Expected behavior

There should be only one factory definition for the same class.
@devision-milen-dzhambazov devision-milen-dzhambazov added the Bug Something isn't working label Dec 18, 2020
@froschdesign
Copy link
Member

This problem is already fixed with #68 and will be released with version 2.13.5.

Thanks for your help, though.

@froschdesign froschdesign added Duplicate This issue or pull request already exists and removed Bug Something isn't working labels Dec 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Duplicate This issue or pull request already exists
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@froschdesign @devision-milen-dzhambazov