Skip to content

Forward port of the output buffering check fix to 3.9#19612

Merged
wilsonge merged 7 commits intojoomla:3.10-devfrom
zero-24:3.9_output_buffering
May 15, 2018
Merged

Forward port of the output buffering check fix to 3.9#19612
wilsonge merged 7 commits intojoomla:3.10-devfrom
zero-24:3.9_output_buffering

Conversation

@zero-24
Copy link
Copy Markdown
Contributor

@zero-24 zero-24 commented Feb 8, 2018

Pull Request for Issue #19033 & #19611

Summary of Changes

In case output_buffering is set to Off we should show it set off currently we show it to be enabled.

Testing Instructions

review. As i can't remember how to trigger this page. But it is just a forward port of the same code used in the 3.8 PR.

Expected result

Showed as off

Actual result

showed as on

Documentation Changes Required

none

@brianteeman
Copy link
Copy Markdown
Contributor

@zero-24 shouldnt this be against the 3.9 branch and not staging?

@zero-24 zero-24 changed the base branch from staging to 3.9-dev February 8, 2018 10:02
@zero-24
Copy link
Copy Markdown
Contributor Author

zero-24 commented Feb 8, 2018

Yes thanks @brianteeman fixed

@zero-24 zero-24 added PR-3.9-dev and removed Language Change This is for Translators PR-staging labels Feb 8, 2018
@zero-24
Copy link
Copy Markdown
Contributor Author

zero-24 commented Feb 26, 2018

I have just implemted that changes provided by @ggppdk from here #19784

@ggppdk
Copy link
Copy Markdown
Contributor

ggppdk commented Feb 26, 2018

Should be like PR for staging branch
(int) ini_get('output_buffering') !== 0

@zero-24
Copy link
Copy Markdown
Contributor Author

zero-24 commented Feb 26, 2018

Fixed.

@zero-24
Copy link
Copy Markdown
Contributor Author

zero-24 commented Mar 13, 2018

@wilsonge as this is just a forward port of the 3.8 change (#19791) can you consider merging this on review?

@zero-24 zero-24 added this to the Joomla 3.9.0 milestone Mar 13, 2018
@wilsonge wilsonge merged commit ac5cf4c into joomla:3.10-dev May 15, 2018
@zero-24 zero-24 deleted the 3.9_output_buffering branch May 15, 2018 16:45
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.

6 participants