Skip to content
This repository has been archived by the owner. It is now read-only.

Fix EZP-22761 - 500 Internal Server Error after request #133 from nlescure/patch-1 #134

Merged
merged 1 commit into from Apr 30, 2014

Conversation

5 participants
@pedroresende
Copy link
Contributor

pedroresende commented Apr 28, 2014

@pedroresende

This comment has been minimized.

Copy link
Contributor Author

pedroresende commented Apr 28, 2014

@bdunogier, @andrerom, @lolautruche Could you please take a look at this

@lolautruche

This comment has been minimized.

Copy link
Contributor

lolautruche commented Apr 28, 2014

Hi
Didn't you consider why @nlescure did this change ? Have you read his issue and the comments thread ?
You're fixing a symptom, not the root cause.

@lolautruche

This comment has been minimized.

Copy link
Contributor

lolautruche commented Apr 28, 2014

Besides, there is absolutely no problem. It seems that you just have a file permission issue, which then leads to a small doc issue.

So -1 from me

@pedroresende

This comment has been minimized.

Copy link
Contributor Author

pedroresende commented Apr 28, 2014

@lolautruche In that case the install.MD needs to be updated https://github.com/ezsystems/ezpublish-community/blob/master/INSTALL.md to reflect these changes

@lolautruche

This comment has been minimized.

Copy link
Contributor

lolautruche commented Apr 28, 2014

@pedroresende Feel free to update your PR 😃

@pedroresende

This comment has been minimized.

Copy link
Contributor Author

pedroresende commented Apr 28, 2014

sure ;)

@pedroresende

This comment has been minimized.

Copy link
Contributor Author

pedroresende commented Apr 28, 2014

@lolautruche updated and rebased

@mloureiro

This comment has been minimized.

Copy link
Contributor

mloureiro commented Apr 28, 2014

+1

@bdunogier

This comment has been minimized.

Copy link
Member

bdunogier commented Apr 28, 2014

Didn't we (@dpobel ?) simplify the permissions instructions ?

@pedroresende

This comment has been minimized.

Copy link
Contributor Author

pedroresende commented Apr 29, 2014

@andrerom

This comment has been minimized.

Copy link
Member

andrerom commented Apr 30, 2014

+1

andrerom added a commit that referenced this pull request Apr 30, 2014

Merge pull request #134 from pedroresende/EZP-22761
Fix EZP-22761 - 500 Internal Server Error after request #133 from nlescure/patch-1

@andrerom andrerom merged commit bea0e0f into ezsystems:master Apr 30, 2014

1 check passed

continuous-integration/travis-ci The Travis CI build passed
Details

@pedroresende pedroresende deleted the pedroresende:EZP-22761 branch Apr 30, 2014

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
You can’t perform that action at this time.