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

PHP Warnings 7.1.12 #13

Open
haietza opened this issue Dec 18, 2017 · 6 comments
Open

PHP Warnings 7.1.12 #13

haietza opened this issue Dec 18, 2017 · 6 comments

Comments

@haietza
Copy link

haietza commented Dec 18, 2017

Our logs indicate a few PHP warnings with PHP 7.1.12:
[18-Dec-2017 10:02:10 America/New_York] PHP Warning: Illegal string offset 'ID' in /var/www/html/34/blocks/export_forum/mpdf/classes/cssmgr.php on line 1076
[18-Dec-2017 10:02:10 America/New_York] PHP Warning: Cannot assign an empty string to a string offset in /var/www/html/34/blocks/export_forum/mpdf/classes/cssmgr.php on line 1076
[18-Dec-2017 10:02:10 America/New_York] PHP Warning: Illegal string offset 'ID' in /var/www/html/34/blocks/export_forum/mpdf/classes/cssmgr.php on line 1152
[18-Dec-2017 10:02:10 America/New_York] PHP Warning: Illegal string offset 'ID' in /var/www/html/34/blocks/export_forum/mpdf/classes/cssmgr.php on line 1156
[18-Dec-2017 10:02:10 America/New_York] PHP Warning: Illegal string offset 'ID' in /var/www/html/34/blocks/export_forum/mpdf/classes/cssmgr.php on line 1355
[18-Dec-2017 10:02:10 America/New_York] PHP Warning: Illegal string offset 'ID' in /var/www/html/34/blocks/export_forum/mpdf/classes/cssmgr.php on line 1359
[18-Dec-2017 10:02:10 America/New_York] PHP Warning: A non-numeric value encountered in /var/www/html/34/blocks/export_forum/mpdf/mpdf.php on line 32096

@chadmazilly
Copy link
Contributor

Hi @haietza ,

How did you get this error? I just fixed the PHP 7 compatibility issues from the other issue you posted yesterday and I'm wondering if this will still occur for you. I tested exporting (with and without anonymizing) an entire forum and a single discussion and it all worked fine for me.

If you get a chance, please let me know how you got this so I can reproduce and fix. Thanks for the feedback!

Chad

@haietza
Copy link
Author

haietza commented Dec 19, 2017

Hi @chadmazilly ,

These errors appeared in the php server logs, just from exporting a forum in the Moodle course. I just downloaded the latest code from the master branch and re-tested and these warnings are still appearing in the logs.

Thanks for your speedy response!

@chadmazilly
Copy link
Contributor

Ok got it. Did the export actually work for you?

@haietza
Copy link
Author

haietza commented Dec 19, 2017

Yes - the export worked fine. Just noticed the warnings in the logs.

@chadmazilly
Copy link
Contributor

Ok great. I'll leave the issue open but fixing the warnings from coming through may be a big task. Looks like that MPDF PHP dependency being used is a huge library, developed over 5 years ago in an older version of PHP. I'll keep you posted here.

@haietza
Copy link
Author

haietza commented Dec 20, 2017

Sounds good - just wanted to let you guys know about the warnings - since it doesn't prevent the plugin from working, it's not a major issue. Thanks so much!

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