CDPP / AMDA_IHM

Commit ecac355e11e008e3cf9b4445eba0d8e2ab5c8c02

Authored by Benjamin Renard
1 parent 72884bb3
Exists in master and in 110 other branches 10098, 10178, 5390, 6629, 6728, 6920, 7494, 7497, 7616, 8459, 8528, 8928, 9014, 9016Bis, 9017, 9049, 9084, 9259, 9326, 9345, 9505, 9660, 9750, 9750.bck, 9800, 9980, FER_10285, FER_10400, FER_10435, FER_10557, FER_10663, FER_10765, FER_10828, FER_10828_2, FER_10829, FER_10836, FER_10852, FER_10934, FER_11100, FER_11242, FER_11478, FER_11500, FER_11537, FER_11586, FER_11588, FER_5119, FER_5165, FER_6265, FER_6860, FER_6899, FER_6951, FER_7618, FER_7862, FER_8455, FER_9629, FER_9660, FER_9915, FER_CR41, FER_CR42, FER_CR43, FER_TFCat, Histo1D2D, MAZ_10936, MAZ_11362, MAZ_11427, MAZ_6018, MAZ_9016_New, MAZ_FIX_9016, New_ColorMaps, SpeasyGet, alias, amdadev, amdadev_test, anr_tempete, catalog-histo, create-user, cu2022, data-activities, density_maps, develop, epn-tap, expression-parser-bre, frames-transformation, hapi, homePage, install-rework, master_vge, minimize, mlplasmas, multiplot, new-save-plot-request, openam, plot-request, plot-request-update, plot-tt-catalog, remoteData, rm_5118, rm_5776, rm_6912, rm_6912_bis, rm_7926, rm_8932, save-other-requests, save-plot, save-plot-request, shared-objects-updater, special-params, sum-components-indexes, test-merge, ws_get_catalog

Destroy plot elements when they are unregistered

Showing 3 changed files with 11 additions and 9 deletions   Show diff stats
Inline Side-by-side
js/app/views/PlotComponents/PlotElementPanel.js
  Diff comments   View file @ecac355
... ... @@ -23,7 +23,8 @@ Ext.define('amdaPlotComp.PlotElementPanel', {
23 23 destroy: function() {
24 24 this.elementFormsManager.each(function (key, value, length) {
25 25 this.elementFormsManager.unregister(value);
26   - }, this);
  26 + value.destroy();
  27 + }, this);
27 28 this.callParent();
28 29 },
29 30  
... ... @@ -188,4 +189,4 @@ Ext.define('amdaPlotComp.PlotElementPanel', {
188 189  
189 190 Ext.apply (this , Ext.apply (arguments, myConf));
190 191 }
191   -});
192 192 \ No newline at end of file
  193 +});
... ...
js/app/views/PlotComponents/PlotLayoutForm.js
  Diff comments   View file @ecac355
... ... @@ -23,8 +23,9 @@ Ext.define('amdaPlotComp.PlotLayoutForm', {
23 23  
24 24 destroy: function() {
25 25 this.layoutOptionsFormsManager.each(function (key, value, length) {
26   - this.layoutOptionsFormsManager.unregister(value);
27   - }, this);
  26 + this.layoutOptionsFormsManager.unregister(value);
  27 + value.destroy();
  28 + }, this);
28 29 this.callParent();
29 30 },
30 31  
... ... @@ -99,4 +100,4 @@ Ext.define('amdaPlotComp.PlotLayoutForm', {
99 100 this.layoutOptionsContainer
100 101 ];
101 102 }
102   -});
103 103 \ No newline at end of file
  104 +});
... ...
js/app/views/PlotComponents/PlotParamForm.js
  Diff comments   View file @ecac355
... ... @@ -30,8 +30,9 @@ Ext.define('amdaPlotComp.PlotParamForm', {
30 30  
31 31 destroy: function() {
32 32 this.drawingOptionsFormsManager.each(function (key, value, length) {
33   - this.drawingOptionsFormsManager.unregister(value);
34   - }, this);
  33 + this.drawingOptionsFormsManager.unregister(value);
  34 + value.destroy();
  35 + }, this);
35 36 this.callParent();
36 37 },
37 38  
... ... @@ -49,7 +50,6 @@ Ext.define('amdaPlotComp.PlotParamForm', {
49 50 formId = type;
50 51  
51 52 formId += '-drawing-options-form';
52   -
53 53 if (!this.drawingOptionsFormsManager.get(formId))
54 54 {
55 55 switch(type)
... ... @@ -143,4 +143,4 @@ Ext.define('amdaPlotComp.PlotParamForm', {
143 143 this.drawingOptionsContainer
144 144 ];
145 145 }
146   -});
147 146 \ No newline at end of file
  147 +});
... ...