Closed tushargugnani closed 4 years ago
Hi @tushargugnani
Thanks for raising this one, we will change.
@tushargugnani i have send a PR to fix this , but i couldn't find any $_SERVER['REQUEST_URI'] instance in the following files
packages/Webkul/Shop/src/Http/Middleware/Currency.php packages/Webkul/Shop/src/Http/Middleware/Locale.php
@ghermans It's there in the version that I have on my local, check this.
oh, hmm It looks like this has already been fixed in the master branch
@prashant-webkul Off topic. How can I use the latest (master) code of bagisto instead of v0.1.0 on my local?
@prashant-webkul Off topic. How can I use the latest (master) code of bagisto instead of v0.1.0 on my local?
Can't you update to latest version, v0.1.0? So many new features and security fixes have been done since then.
@prashant-webkul I am on version v.0.1.0 , was looking if I can ahead to the master branch.
I installed bagisto and tried to run the tests. Looks like you can't run the feature test without spinning up the server, since there are instances of $_SERVER['REQUEST_URI'] in many files. I suggest it to replace with $request->getRequestUri().
I replaced this in the following files and was able to run the test without spinning up the server.
packages/Webkul/Admin/src/Exceptions/Handler.php packages/Webkul/Shop/src/Http/Middleware/Currency.php packages/Webkul/Shop/src/Http/Middleware/Locale.php Shop -> footer.blade.php Shop -> header.blade.php