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

Misleading error message on upload #86

Open
t-schroeder opened this issue Jul 19, 2019 · 0 comments
Open

Misleading error message on upload #86

t-schroeder opened this issue Jul 19, 2019 · 0 comments

Comments

@t-schroeder
Copy link

t-schroeder commented Jul 19, 2019

When you don't have the capability "mod/hvp:updatelibraries" but try to upload a new Content type, you get a message like this:

Missing required library H5P.GreetingCard 1.0

... although that library was included in the Content type you uploaded. The message seems to come from here:

$this->core->h5pF->getMessages('error')

I noticed that in addition to that "error" message there is also an "info" message being set in PHP but not shown to the user. It reads:

Note that the libraries may exist in the file you uploaded, but you're not allowed to upload new libraries. Contact the site administrator about this.

The latter is a much more helpful message but isn't shown to the user. The one that is shown is misleading because the "missing" library is (or might be) in the uploaded .h5p file. Why is the "info" message not displayed?

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

1 participant