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

Fix inverted command verbosity for PropelBundle 1.4/1.5/1.6 #453

Merged
merged 1 commit into from
Nov 9, 2017
Merged

Fix inverted command verbosity for PropelBundle 1.4/1.5/1.6 #453

merged 1 commit into from
Nov 9, 2017

Conversation

xelan
Copy link

@xelan xelan commented Jul 4, 2017

Q A
Branch? 1.4/1.5/1.6
Bug fix? yes
New feature? no
BC breaks? possibly (if users rely on the incorrect behaviour)
Deprecations? no
Tests pass? yes
Fixed tickets -
License MIT

For the 1.x branches of PropelBundle, the verbosity is apparently inverted to the expected behaviour.

$bufferPhingOutput is initially set to Symfony's kernel.debug parameter.
However, the conditional for checking the buffer works the other way round:
If buffering is enabled, the buffer is discarded, otherwise it is output.

This commit fixes that issue. Now if $bufferPhingOutput is true,
the detailed output of the command is displayed.

$bufferPhingOutput is initially set to Symfony's kernel.debug parameter.
However, the conditional for checking the buffer works the other way round:
If buffering is enabled, the buffer is discarded, otherwise it is output.

This commit fixes that issue. Now if $bufferPhingOutput is true,
the detailed output of the command is displayed.
@jaugustin jaugustin added the 1.x label Nov 9, 2017
@jaugustin jaugustin merged commit ddaba19 into propelorm:1.4 Nov 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants