Skip to content

Commit

Permalink
Failsafe requirejs unset/reset
Browse files Browse the repository at this point in the history
Introduced in 5c75aa8 it might happen,
that there is no `define` or `require` defined. handle this gracefully.
  • Loading branch information
petschki committed Jun 23, 2017
1 parent f65ddfb commit 18bc1fe
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 4 deletions.
3 changes: 3 additions & 0 deletions CHANGES.rst
Expand Up @@ -4,6 +4,9 @@ Changelog
1.10 (unreleased)
-----------------

- failsafe Unset and reset ``define`` and ``require``
[petschki]

- PEP 8.
[thet]

Expand Down
4 changes: 2 additions & 2 deletions plone/app/widgets/static/requirejs-reset.js
@@ -1,2 +1,2 @@
var define = _old_define;
var require = _old_require;
if(typeof _old_define !== 'undefined') { var define = _old_define; }
if(typeof _old_require !== 'undefined') { var require = _old_require; }
4 changes: 2 additions & 2 deletions plone/app/widgets/static/requirejs-unset.js
@@ -1,4 +1,4 @@
var _old_define = define;
var _old_require = require;
if(typeof define !== 'undefined') { var _old_define = define; }
if(typeof require !== 'undefined') { var _old_require = require; }
define = undefined;
require = undefined;

0 comments on commit 18bc1fe

Please sign in to comment.