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

jquery.fileupload broken after update #3630

Closed
9 tasks done
daguiler opened this issue Mar 17, 2020 · 0 comments · Fixed by #3631
Closed
9 tasks done

jquery.fileupload broken after update #3630

daguiler opened this issue Mar 17, 2020 · 0 comments · Fixed by #3631

Comments

@daguiler
Copy link
Contributor

daguiler commented Mar 17, 2020

Description of bug

The "Add Module" dialog throws a 404 on widget.js when browsing modules that depend on jquery.fileupload.js.
This is not evident now in the DNN platform because there are no such modules advertised currently. In Evoq, however, the Visualizer module depends on this library and doesn't work anymore.

Steps to reproduce

You'll need a module that depends on jquery.fileupload to verify this bug, but it's really simple to reproduce:

  1. Create a page
  2. Click on "Add Module" at the bottom
  3. Click on < your module here >

Current behavior

A spinner is visible for some time and then a popup explains that the module could not be loaded.
The browser console shows a Javascript error (see screenshots below).

Expected behavior

Module should load without issues.

Screenshots

DNN-37504

DNN-37504-2

DNN-37504-3

Error information

require.js?cdv=111:2 GET http://daguilera-engage.dnndev.me/DesktopModules/DNNCorp/StructuredContentVisualizer/ClientScripts/jquery-ui/ui/widget.js?cdv=111 net::ERR_ABORTED 404 (Not Found)
main.js?cdv=111:3 Error: Script error for "jquery-ui/ui/widget", needed by: jquery.fileupload
http://requirejs.org/docs/errors.html#scripterror
    at makeError (require.js?cdv=111:2)
    at HTMLScriptElement.onScriptError (require.js?cdv=111:2)

Additional context

This error started to happen after #3519.

Affected version

  • 10.0.0 alpha build
  • 9.5.1 alpha build
  • 9.5.0 latest supported release

Affected browser

  • Chrome
  • Firefox
  • Safari
  • Internet Explorer 11
  • Microsoft Edge (Classic)
  • Microsoft Edge Chromium
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants