ReferenceError: result is not defined

[wjjr]

This issue was introduced in #216 as result of minifying the dist/browser.js file. See babel/minify#594.

ERROR ReferenceError: result is not defined
    at Object.randomString (browser.js:1)
    at Object.header (browser.js:1)
    [...]

Relevant part of the dist/browser.js file prettified:

// [...]
hawk.utils = {
  // [...]
  randomString: function randomString(c) {
    for (var d = 'abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789', e = d.length, f = [], g = 0; g < c; ++g)
      f[g] = d[Math.floor(Math.random() * e)];
    return result.join('')
  },
  // [...]
};
// [...]

[steffenmllr]

Same Error here. Since I'm using babel anyway import Hawk from 'hawk/lib/browser'; fixed the issue for me.

[tkh44]

This bug has been fixed in babel/minify#713 and needs to be released. Depending on their schedule I can change the build setup to use Uglify.

[edmorley]

This broke my project too, after updating from Hawk 6.0.2 to 7.0.5.

Reading the Babel issue/PR it seems the bug is limited to just the babel-plugin-transform-merge-sibling-variables plugin.

As a workaround for now (that's less drastic than switching to Uglify), perhaps just that plugin should be excluded via .babelrc?

[edmorley]

Hi! This issue still exists in v7.0.6 (comparing dist/browser.js in 7.0.5 and 7.0.6 shows them to be identical).

Please can it be reopened?

[hueniverse]

@tkh44 what needs to happen to fix it?

[lock]

This thread has been automatically locked due to inactivity. Please open a new issue for related bugs or questions following the new issue template instructions.