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

Error 500 on install #1

Closed
tbba opened this issue Feb 15, 2020 · 2 comments
Closed

Error 500 on install #1

tbba opened this issue Feb 15, 2020 · 2 comments

Comments

@tbba
Copy link

tbba commented Feb 15, 2020

ParseError syntax error, unexpected ')'
file: .../ProcessCacheControl.module:129

127: self::MODULE_PERMISSION, 128: $actionPermissionName, 129: )); 130: } else {
After removing the comma after $actionPermissionName, the Installation finished ok.

Thanks for the module. Great idea!

PHP 7.3 PW - latest dev

@MoritzLost
Copy link
Owner

Thanks for the report @tbba, I just pushed a new version that removes the trailing comma. It's allowed in PHP 7.4, I guess I forgot to test the last version I published in PHP 7.3 and below. Let me know if you have any more trouble after the update!

@tbba
Copy link
Author

tbba commented Feb 17, 2020

Thanks for the fix :-)
I close the issue now because, after my manual removing of the comma the install worked fine already.

@tbba tbba closed this as completed Feb 17, 2020
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

No branches or pull requests

2 participants