-
Notifications
You must be signed in to change notification settings - Fork 153
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
Upgrade to support 2.8 & 3.0 and fixed deprecations #80
Conversation
1951265
to
97fd28e
Compare
97fd28e
to
526b4c8
Compare
"mobiledetect/mobiledetectlib": "~2.8" | ||
}, | ||
"require-dev": { | ||
"phpunit/phpunit": "~4.1" | ||
"phpunit/phpunit": "~4.1", | ||
"symfony/phpunit-bridge": "^3.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be ^2.7|3.0
(3.x requires PHP 5.5.9 or higher).
Hi, I'm trying to use this bundle with Symfony 3.0 but getting errors on composer requirements: My composer.json:
Output of
|
Can't you just check if request_stack exists and get the request from there, otherwise use the 'request' "service"? I think this option is pretty good. $requestStack = $serviceContainer->has('request_stack') ? $serviceContainer->get('request_stack') : null;
if (($requestStack && !$serviceContainer->get('request_stack')->getMasterRequest())
|| false === $serviceContainer->isScopeActive('request')) {
$this->viewType = self::VIEW_NOT_MOBILE;
return;
}
$this->request = $requestStack ? $requestStack->getMasterRequest() : $serviceContainer->get('request'); We are using this lib and we don't want to fork it because of this deprecation warning. 😢 @trq Can we agree on a solution? |
@suncat2000 Any thoughts on this? I think we should get rid of these deprecation warnings asap as well. |
@netmikey I will try on this weekend check it and fix. |
Awesome, thx! :) |
any news about this? want to upgrade to 3.0 with this bundle ;) thanks! |
sorry guys (@trq, @tsantos84, @arisro, @netmikey, @develth), did not have time to do. |
All test are passing.