possible fix for cytoscape#819 w/ dagre and re-used layout

NeveHanter · Feb 12, 2015 · 6e4647c · 6e4647c
1 parent 08d2ad5
commit 6e4647c
Showing 1 changed file with 9 additions and 1 deletion.
diff --git a/src/core-layout.js b/src/core-layout.js
@@ -3,8 +3,16 @@
   $$.fn.core({
 
     layout: function( params ){
-      var layout = this._private.prevLayout = ( params == null ? this._private.prevLayout : this.initLayout( params ) );
+      var layout;
 
+      // always use a new layout w/ init opts; slightly different backwards compatibility
+      // but fixes layout reuse issues like dagre #819 
+      if( params == null ){ 
+        params = $$.util.extend({}, this._private.options.layout);
+        params.eles = this.$();
+      }
+
+      layout = this.initLayout( params );
       layout.run();
 
       return this; // chaining