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

bug: ErrorException: Warning: Invalid argument supplied for foreach() - in GFMAlerts handler #49

Closed
sentry-io bot opened this issue Dec 9, 2024 · 0 comments · Fixed by #50
Closed

bug: ErrorException: Warning: Invalid argument supplied for foreach() - in GFMAlerts handler #49

sentry-io bot opened this issue Dec 9, 2024 · 0 comments · Fixed by #50
Assignees
@mcdurdin @Meng-Heng
Labels
bug Something isn't working
Milestone
A18S17

Comments

@sentry-io
Copy link

sentry-io bot commented Dec 9, 2024 •
edited by mcdurdin
Loading

Sentry Issue: HELP-KEYMAN-COM-DH8

ErrorException: Warning: Invalid argument supplied for foreach()
  File "/var/www/html/_common/MarkdownHost.php", line 104, in Keyman\Site\Common\MarkdownHost::__construct
    $ParsedownAndAlerts->text($contents) .
  File "/mdhost.php", line 10
    $md = new MarkdownHost($_REQUEST['file']);
...
(1 additional frame(s) were not displayed)

More detail:

Warning: chop() expects parameter 1 to be string, array given in /var/www/vendor/erusev/parsedown/Parsedown.php on line 150

Warning: strpos() expects parameter 1 to be string, array given in /var/www/vendor/erusev/parsedown/Parsedown.php on line 160

Warning: explode() expects parameter 2 to be string, array given in /var/www/vendor/erusev/parsedown/Parsedown.php on line 162

Notice: Trying to access array offset on value of type null in /var/www/vendor/erusev/parsedown/Parsedown.php on line 164

Warning: Invalid argument supplied for foreach() in /var/www/vendor/erusev/parsedown/Parsedown.php on line 168

Notice: Trying to access array offset on value of type null in /var/www/vendor/erusev/parsedown/Parsedown.php on line 213
@keymanapp-test-bot keymanapp-test-bot bot added the bug Something isn't working label Dec 9, 2024
@mcdurdin mcdurdin added this to the A18S17 milestone Dec 9, 2024
@mcdurdin mcdurdin transferred this issue from keymanapp/keyman Dec 9, 2024
mcdurdin added a commit that referenced this issue Dec 9, 2024
@mcdurdin
fix: crash in GFMAlerts when falling back to blockquote
3d97cf1 
The code paths when a normal blockquote was encountered were not quite
right. Also tidied up a few other bits and pieces for robustness.

Fixes: #49
@mcdurdin mcdurdin mentioned this issue Dec 9, 2024
Merged
mcdurdin added a commit that referenced this issue Dec 9, 2024
@mcdurdin
fix: crash in GFMAlerts when falling back to blockquote
9a20ed4 
The code paths when a normal blockquote was encountered were not quite
right. Also tidied up a few other bits and pieces for robustness.

Fixes: #49
Fixes: HELP-KEYMAN-COM-DH8
mcdurdin added a commit that referenced this issue Dec 9, 2024
@mcdurdin
fix: crash in GFMAlerts when falling back to blockquote (#50)
f6374e7 
The code paths when a normal blockquote was encountered were not quite
right. Also tidied up a few other bits and pieces for robustness.

Fixes: #49
Fixes: HELP-KEYMAN-COM-DH8
@github-project-automation github-project-automation bot moved this to Done in Keyman Dec 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mcdurdin mcdurdin

@Meng-Heng Meng-Heng

Labels
bug Something isn't working
Projects
Keyman
Archived in project
Milestone
A18S17
Development

Successfully merging a pull request may close this issue.

fix: crash in GFMAlerts when falling back to blockquote keymanapp/shared-sites
2 participants
@mcdurdin @Meng-Heng