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

Do not crash if addwiki/mediawiki-api-base is missing #8

Closed
wants to merge 1 commit into from

Conversation

DanielWTQ
Copy link
Contributor

Instead of crashing with a fatal error and a stack trace, check if one of the classes that we need is available, using that class as a proxy for the overall package. If not, uses of the parser function return a nice error message but the parsing does not fail and the rest of the page can be shown.

WIK-1250

Instead of crashing with a fatal error and a stack trace, check if one of the
classes that we need is available, using that class as a proxy for the overall
package. If not, uses of the parser function return a nice error message but
the parsing does not fail and the rest of the page can be shown.

WIK-1250
@DanielWTQ DanielWTQ requested a review from vedmaka January 22, 2024 00:44
@vedmaka vedmaka closed this Feb 2, 2024
@DanielWTQ DanielWTQ deleted the missing-deps branch November 24, 2024 07:37
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

Successfully merging this pull request may close these issues.

2 participants