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

Support for composer v2 and PHP 8.0 - any plans? #1002

Closed
vladaman opened this issue Jan 15, 2021 · 6 comments
Closed

Support for composer v2 and PHP 8.0 - any plans? #1002

vladaman opened this issue Jan 15, 2021 · 6 comments

Comments

@vladaman
Copy link

I wonder if there is any plan to support composer v2 and PHP 8.0 since PHP v 7.3 is currently end of life, 7.4 will end by end of 2021.

#616 #432 #831

@freescout-helpdesk
Copy link
Contributor

freescout-helpdesk commented Jan 15, 2021

Both - 1-2 years.

@freescout-helpdesk
Copy link
Contributor

PHP 8 support added. Now minimum supported PHP version is 7.1
https://github.com/freescout-helpdesk/freescout/releases/tag/1.7.0

When using composer and PHP 8, composer should be executed with additional flag: composer install --ignore-platform-reqs

@tiredofit
@nebulade

@nebulade
Copy link
Contributor

Thanks for the heads up, we have updated our package to 1.7.0 now.

@amanpatel
Copy link

amanpatel commented May 19, 2021

I'm using this on PHP 8 (via the docker pkg from "tiredofit"). On some email messages I get the whoops message:

[2021-05-19 18:27:56] production.ERROR: Undefined property: DOMComment::$data (View: /www/html/resources/views/conversations/partials/thread.blade.php) (View: /www/html/resources/views/conversations/partials/thread.blade.php) (View: /www/html/resources/views/conversations/partials/thread.blade.php) {"userId":1,"email":"[email protected]","exception":"[object] (ErrorException(code: 0): Undefined property: DOMComment::$data (View: /www/html/resources/views/conversations/partials/thread.blade.php) (View: /www/html/resources/views/conversations/partials/thread.blade.php) (View: /www/html/resources/views/conversations/partials/thread.blade.php) at /www/html/vendor/ezyang/htmlpurifier/library/HTMLPurifier/Lexer/DOMLex.php:216, ErrorException(code: 0): Undefined property: DOMComment::$data (View: /www/html/resources/views/conversations/partials/thread.blade.php) (View: /www/html/resources/views/conversations/partials/thread.blade.php) at /www/html/vendor/ezyang/htmlpurifier/library/HTMLPurifier/Lexer/DOMLex.php:216, ErrorException(code: 0): Undefined property: DOMComment::$data (View: /www/html/resources/views/conversations/partials/thread.blade.php) at /www/html/vendor/ezyang/htmlpurifier/library/HTMLPurifier/Lexer/DOMLex.php:216, ErrorException(code: 0): Undefined property: DOMComment::$data at /www/html/vendor/ezyang/htmlpurifier/library/HTMLPurifier/Lexer/DOMLex.php:216)

Updating the DomLex.php file to check for properties via isset() or ?? operator seems to fix it.

I don't think this is related to PHP 8

@freescout-helpdesk
Copy link
Contributor

@amanpatel Could you please attach patched file.

@amanpatel
Copy link

@freescout-helpdesk Great question. I created PR in that package.

ezyang/htmlpurifier#292

However, I am not sure if there are similar issues in other places in this package. But this seems to have alleviated for me. I just setup my Freescout install about 2-3 days ago and have tested with approximately 8 or 9 emails. If it breaks again, I'll update the PR.

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

4 participants