Skip to content

change variable name to a better fitting one #3715

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

Merged
merged 1 commit into from
Mar 26, 2014

Conversation

xabbuh
Copy link
Member

@xabbuh xabbuh commented Mar 24, 2014

Q A
Doc fix? yes
New docs? no
Applies to all
Fixed tickets

As mentioned by @weaverryan in #3706 since Nginx is supported the name $APACHEUSER doesn't any more match its usage really properly. Thus, I renamed it to $HTTPDUSER.

@xabbuh xabbuh changed the title Issue 3706 change variable name to a better fitting one Mar 24, 2014
@weaverryan
Copy link
Member

Awesome, thanks for running with my comment - that rocks!

weaverryan added a commit that referenced this pull request Mar 26, 2014
This PR was merged into the 2.3 branch.

Discussion
----------

change variable name to a better fitting one

| Q             | A
| ------------- | ---
| Doc fix?      | yes
| New docs?     | no
| Applies to    | all
| Fixed tickets |

As mentioned by @weaverryan in #3706 since Nginx is supported the name ``$APACHEUSER`` doesn't any more match its usage really properly. Thus, I renamed it to ``$HTTPDUSER``.

Commits
-------

2f99c90 change variable name to a better fitting one
@weaverryan weaverryan merged commit 2f99c90 into symfony:2.3 Mar 26, 2014
@xabbuh xabbuh deleted the issue-3706 branch March 26, 2014 09:19
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.

2 participants