#14550 closed defect (fixed)
dojox/io/OAuth.js will not build - easy fix
Reported by: | kbenjamin | Owned by: | Adam Peller |
---|---|---|---|
Priority: | high | Milestone: | 1.7.2 |
Component: | Dojox | Version: | 1.7.1 |
Keywords: | build oauth | Cc: | |
Blocked By: | Blocking: |
Description
Downloading the current dojo-boilerplate and doing a build fails on dojox/io/OAuth.js with an error on line 7 (SHA1):
define([ "dojo/_base/kernel", // dojo "dojo/_base/lang", // mixin "dojo/_base/array", // isArray, map "dojo/_base/xhr", // formToObject, queryToObject, xhr "dojo/dom", // byId "dojox/encoding/digests/SHA1", // SHA1 ], function(dojo, lang, array, xhr, dom, SHA1){
The issue is that the define dependency array ends with a comma and that is not supported by Internet Explorer (according to the builder).
Editing the array to remove the trailing comma after SHA1 fixes the problem and allows the build to complete:
define([ "dojo/_base/kernel", // dojo "dojo/_base/lang", // mixin "dojo/_base/array", // isArray, map "dojo/_base/xhr", // formToObject, queryToObject, xhr "dojo/dom", // byId "dojox/encoding/digests/SHA1" // SHA1 ], function(dojo, lang, array, xhr, dom, SHA1){
Change History (3)
comment:1 Changed 9 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
comment:2 Changed 9 years ago by
Milestone: | tbd → 1.7.2 |
---|---|
severity: | blocker → critical |
Note: See
TracTickets for help on using
tickets.
In [27481]: