Franklinjagan / jsplumb

Automatically exported from code.google.com/p/jsplumb
0 stars 0 forks source link

params.draggable == null #2

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Create test HTML with NO jQuery.UI
2. Run it in browser (Chrome)
3. Observe error in JS console: 
Uncaught TypeError: Object #<an Object> has no method 'draggable'

Please provide any additional information below.

    var draggable = params.draggable == null ? true : params.draggable;

Is this string correct? I have params.draggable = undefined here

Original issue reported on code.google.com by aazaharo...@gmail.com on 15 Apr 2010 at 11:17

GoogleCodeExporter commented 9 years ago
the test is ok because it defaults to dragging=true.  what isn't cool is if 
jQueryUI
has not been included on the page, as you pointed out.  i changed this line:

if (draggable) {

to be:

if (draggable && self.source.draggable) {

and now no error is thrown.

this is in 1.0.3-RC1.  when we find a resolution for issue 4 we can release 
version
1.0.3.

Original comment by simon.po...@gmail.com on 21 Apr 2010 at 3:51

GoogleCodeExporter commented 9 years ago

Original comment by simon.po...@gmail.com on 9 Oct 2010 at 11:44