Don't define AMD module with a name to make it more portable

[arryon]

The current AMD module implementation explicitly defines a name for the buzz module, which makes it less portable, see http://requirejs.org/docs/api.html#modulename. If name definition is omitted, this makes the module easier to import into RequireJS.

Can create a pull request if necessary.

[stephanebachelier]

@arryon I think you can close this issue.

[jaysalvat]

Done :)

[norn]

How should I use it with require.js?
I've got "Uncaught ReferenceError: buzz is not defined" error if list buzz as dependency.

[stephanebachelier]

@norn have you configure your path correctly ? Show us some code.

[norn]

@stephanebachelier here it is
var require = {
baseUrl: '/static/js',
shim: {
bustime: {
deps: ['autobahn6.min', 'jquery', 'jquery-ui', {%if us.sound%}'jquery.mb.audio.min', 'buzz', {%endif%}'nbusstops-{{us.city.id}}b']
},
"jquery-ui": {
deps: ['jquery']
}
},
paths: {
'jquery': '//ajax.googleapis.com/ajax/libs/jquery/1.10.2/jquery.min',
'jquery-ui':'//ajax.googleapis.com/ajax/libs/jqueryui/1.10.3/jquery-ui.min',
// 'buzz':'//jaysalvat.github.io/buzz/releases/latest/buzz.min',
'bustime': 'bustime-281'
},
deps: ['jquery', 'autobahn6.min', {%if us.sound%}'howler.min', 'jquery.mb.audio.min', 'buzz', {%endif%}'jquery-ui', 'openapi.min', 'nbusstops-{{us.city.id}}b', 'bustime', 'addtohomescreen-ru.min'],
};

[stephanebachelier]

@norn why have you commented the line 'buzz':'//jaysalvat.github.io/buzz/releases/latest/buzz.min'` ?

[norn]

I have tried to use the local one on the server and commented the path. But non of them are worked.