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

Added new base path option specifically for console #6247

Closed
wants to merge 4 commits into from
Closed

Added new base path option specifically for console #6247

wants to merge 4 commits into from

Conversation

tldev
Copy link

@tldev tldev commented May 9, 2014

I'm currently implementing something similar in my application, and thought maybe you guys would like the idea.

Basically, for normal http requests, I want to always fallback to using the request object, for both development and production, but for console, I want to explicitly set the base_path. I need the basePath view helper on a console request so I can send emails via cronjob with the correct urls to my application. If I set the base_path with how the factory currently works, I have to now worry about setting the correct base_path no matter what the application environment is even though the request object works perfectly fine.

@Ocramius
Copy link
Member

@tldev needs a test case to demonstrate what feature is being introduced here...

@tldev
Copy link
Author

tldev commented Aug 31, 2014

@Ocramius will this suffice?

@Ocramius
Copy link
Member

@tldev I've manually merged this PR at fd7f918

Thanks @tldev

I'll ping you in an issue document this new functionality.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants