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.

Sign up for GitHub

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

Jump to bottom

Incompatibilities with php 8.2 - 1.13.0 dev #507

Open
jarobase opened this issue Jul 15, 2023 · 5 comments
Open

Incompatibilities with php 8.2 - 1.13.0 dev #507

jarobase opened this issue Jul 15, 2023 · 5 comments
Assignees
@Simounet

Comments

@jarobase
Copy link

I am using Leed behind a php:8.2-apache docker container.
Leed is not the only php application running.

When I try to update the feeds, I get this error:

url: https://example.com/dir/leed/action.php?action=readAll
 Warning: Cannot modify header information - headers already sent by (output started at /var/www/html/dir/leed/Event.class.php:44) in /var/www/html/dir/leed/action.php on line 81

It is impossible to display the articles. If I remove the ?action=readAll parameter, I can see the articles with the following warnings:

Deprecated: Creation of dynamic property Event::$pubdate is deprecated in /var/www/html/dir/leed/Event.class.php on line 44
Deprecated: Creation of dynamic property Translation::$location is deprecated in /var/www/html/dir/leed/i18n.php on line 26
Deprecated: Creation of dynamic property Translation::$languages is deprecated in /var/www/html/dir/leed/i18n.php on line 30
Deprecated: Creation of dynamic property Translation::$location is deprecated in /var/www/html/dir/leed/i18n.php on line 26
Deprecated: Creation of dynamic property Translation::$languages is deprecated in /var/www/html/dir/leed/i18n.php on line 30
Deprecated: Creation of dynamic property Event::$pubdate is deprecated in /var/www/html/dir/leed/Event.class.php on line 44
Deprecated: Creation of dynamic property Event::$pubdate is deprecated in /var/www/html/dir/leed/Event.class.php on line 44
Deprecated: Creation of dynamic property Event::$pubdate is deprecated in /var/www/html/dir/leed/Event.class.php on line 44
Deprecated: Creation of dynamic property Event::$pubdate is deprecated in /var/www/html/dir/leed/Event.class.php on line 44
Deprecated: Creation of dynamic property Event::$pubdate is deprecated in /var/www/html/dir/leed/Event.class.php on line 44
@Simounet Simounet self-assigned this Jul 15, 2023
@Simounet
Copy link
Member

Hi @jarobase thanks a lot for your issue. Can you try this branch https://github.com/LeedRSS/Leed/tree/fix/php82-deprecations ?

@jarobase
Copy link
Author

Hi @jarobase thanks a lot for your issue. Can you try this branch https://github.com/LeedRSS/Leed/tree/fix/php82-deprecations ?

It is working ! All the warnings disappeared. Thanks a lot.

@jarobase
Copy link
Author

jarobase commented Jul 16, 2023 via email

@Simounet
Copy link
Member

Yep, it's on my list. I will take care of this as soon as possible.

@Simounet
Copy link
Member

Can you try it with my last commit, please?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Simounet Simounet

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Simounet @jarobase