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
Getting error array_map() must be of type array when trying to send scheduled report #19871
Comments
Hello! Can you assign me this issue to would like to solve it out |
hi @thesakshidiggikar perhaps you could discuss the approach you are considering and maybe the area of code you would look to solve it and we can discuss first before assigning the issue? |
This error is still happening and it's still failing to send the report. |
Here is the stack trace. It's an issue in tcpdf:
|
Note that I have the value it's trying to parse there but I couldn't reproduce this locally. There isn't any issue. It's basically the
see also https://3v4l.org/0iF1c. On production renamed the report title but this didn't help. Then renamed the name of the site and it did help and made it render. So there seems to some issue around that. |
Happening again many times today |
@tsteur This might be something that needs to be fixed in the vendor lib. If I shall have a look and create a PR there let me know |
How much work would that be roughly? cc @mattab that may be something good to do if it's quick. We're having basically one weekly scheduled report for a customer where this is happening. If it's not quick to do, we can maybe reach out and get them to change the name of the site. |
@tsteur The only way I'm kind of able to reproduce that problem, is when passing a non utf8 string to |
It's basically the same as #19871 (comment) which I looked up from the DB. I only replaced a few characters there. It didn't seem like it contained non-utf8 characters. Would it help to fetch the data in a certain way maybe to check if it contains non-utf8? |
@tsteur does it look normal when exporting the dataset directly from the database? |
Replied in chat 👍 |
Will try to provide a fix upstream for the tcpdf library if possible. |
This should be fixed with the latest release of tcpdf. |
Not sure if there was any change recently? Seeing this error for the first time I believe. The error happened every hour for the same report. In case more information is needed please let me know what I can do to provide more information.
Below is the report definition:
And here the custom report definition in case it's needed for reproducing it
Goal definition in case it's needed
The text was updated successfully, but these errors were encountered: