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

Wiris filter attempts explicit path load of configurationjs.php, prevents TinyMCE editor from loading #57

Closed
gavinwa725 opened this issue Jun 14, 2021 · 2 comments · Fixed by #59
Labels
state:accepted We'll work on this issue soon state:requested We're currently working on this type:bug Something isn't working

Comments

@gavinwa725
Copy link

Moodle: 3.9
Wiris: 7.26.0 2021050600

We have updated to Moodle 3.9 and Wiris 7.26.0 and have discovered that the TinyMCE editor no longer loads. We've discovered that Moodle is trying to fetch the file /filter/wiris/integration/configurationjs.php (with an explicit /filter/wiris/... path) which doesn't work on our instance as our Moodle is in a sub-directory.

The browser loads these files and other TinyMCE files fine,
/moodle3/lib/editor/tinymce/tiny_mce/3.5.11/tiny_mce.js
/moodle3/lib/editor/tinymce/tiny_mce/3.5.11/plugins/visualchars/editor_plugin.js

If I deliberately copy configurationjs.php onto the webserver into a directory were it is accessible as /filter/wiris/integration/configurationjs.php then the TinyMCE editor loads correctly.

I think Wiris' TinyMCE plugin contains an explicit path to /filter/wiris instead of filter/wiris

@dcanet-at-wiris dcanet-at-wiris added state:accepted We'll work on this issue soon type:bug Something isn't working labels Jun 14, 2021
@dcanet-at-wiris
Copy link
Contributor

Many thanks for reporting the issue, @vk6hgr
We've been able to reproduce it with the information you provided.

It seems like the last version of our plugin added this bug.

The bug will be fixed on the next release which it's planned for this very same week.

Keep you posted.

Kind regards.

@dcanet-at-wiris dcanet-at-wiris added the state:requested We're currently working on this label Jun 18, 2021
@dcanet-at-wiris dcanet-at-wiris linked a pull request Jun 18, 2021 that will close this issue
@dcanet-at-wiris
Copy link
Contributor

Last release solves this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
state:accepted We'll work on this issue soon state:requested We're currently working on this type:bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants