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

prevent symfony/validator v5 from installing #38

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

PavelJurasek
Copy link

Version 5 was released 5 days ago (https://github.com/symfony/validator/releases) and breaks this package with error Nette\DI\ServiceCreationException: Class Symfony\Component\Validator\Mapping\Cache\DoctrineCache used in service 'validator.cache' not found.

Thanks for merge and release of new version.

@PavelJurasek
Copy link
Author

ping @enumag just in case you don't receive notifications on new PRs :)

@PavelJurasek
Copy link
Author

oops, this version restriction might not be enough.

@enumag
Copy link
Member

enumag commented Nov 26, 2019

Hmm... what's the proper fix to make this package v5 compatible? (I don't really use Nette or Symfony too much these days.)

And thanks for the ping. I indeed don't get notifications otherwise.

@PavelJurasek
Copy link
Author

Forcepushed change with restriction to <4.4 (4.4 causes method signature incompatibility)

Honestly, I didn't investigate the incompatible changes now since I depend on 3rd party package using this package and I just need to get my app running 😇

@Majkl578
Copy link
Member

Majkl578 commented Nov 26, 2019

(4.4 causes method signature incompatibility)

Could you provide upstream issue about this BC break?

@enumag
Copy link
Member

enumag commented Nov 26, 2019

I just need to get my app running

You can always do that by adding a conflict with SF validator 4.4+ or adding a dependency on SF validator 4.3.

@PavelJurasek
Copy link
Author

@Majkl578 it's caused by symfony/translation-contracts being installed at version 2.0 if symfony/validator v4.4 is installed. where should I report it? In validator repository?

Declaration of Symfony\Component\Translation\TranslatorInterface::setLocale($locale) must be compatible with Symfony\Contracts\Translation\LocaleAwareInterface::setLocale(string $locale)

@enumag sure, I did that :)

@enumag
Copy link
Member

enumag commented Nov 27, 2019

All issues should be reported in symfony/symfony. This looks like symfony/translation doesn't have correct version constraint for symfony/translation-contracts maybe.

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.

3 participants