Browse Source

Remove unwanted paramter from datavariables

pull/6351/head
mohamedsalem401 2 years ago
parent
commit
b1c80f0cf9
  1. 12
      packages/core/src/data_sources/model/DataVariableListenerManager.ts
  2. 1
      packages/core/src/data_sources/model/conditional_variables/DataCondition.ts
  3. 1
      packages/core/src/data_sources/view/ComponentDataVariableView.ts
  4. 1
      packages/core/src/domain_abstract/model/StyleableModel.ts
  5. 2
      packages/core/src/trait_manager/model/Trait.ts

12
packages/core/src/data_sources/model/DataVariableListenerManager.ts

@ -3,13 +3,11 @@ import { stringToPath } from '../../utils/mixins';
import { Model } from '../../common';
import EditorModel from '../../editor/model/Editor';
import DataVariable, { DataVariableType } from './DataVariable';
import ComponentView from '../../dom_components/view/ComponentView';
import { DynamicValue } from '../types';
import { DataCondition, ConditionalVariableType } from './conditional_variables/DataCondition';
import ComponentDataVariable from './ComponentDataVariable';
export interface DynamicVariableListenerManagerOptions {
model: Model | ComponentView;
em: EditorModel;
dataVariable: DynamicValue;
updateValueFromDataVariable: (value: any) => void;
@ -18,13 +16,12 @@ export interface DynamicVariableListenerManagerOptions {
export default class DynamicVariableListenerManager {
private dataListeners: DataVariableListener[] = [];
private em: EditorModel;
private model: Model | ComponentView;
private dynamicVariable: DynamicValue;
private updateValueFromDynamicVariable: (value: any) => void;
private model = new Model();
constructor(options: DynamicVariableListenerManagerOptions) {
this.em = options.em;
this.model = options.model;
this.dynamicVariable = options.dataVariable;
this.updateValueFromDynamicVariable = options.updateValueFromDataVariable;
@ -37,7 +34,7 @@ export default class DynamicVariableListenerManager {
};
listenToDynamicVariable() {
const { em, dynamicVariable, model } = this;
const { em, dynamicVariable } = this;
this.removeListeners();
// @ts-ignore
@ -51,7 +48,7 @@ export default class DynamicVariableListenerManager {
dataListeners = this.listenToConditionalVariable(dynamicVariable as DataCondition, em);
break;
}
dataListeners.forEach((ls) => model.listenTo(ls.obj, ls.event, this.onChange));
dataListeners.forEach((ls) => this.model.listenTo(ls.obj, ls.event, this.onChange));
this.dataListeners = dataListeners;
}
@ -81,8 +78,7 @@ export default class DynamicVariableListenerManager {
}
private removeListeners() {
const { model } = this;
this.dataListeners.forEach((ls) => model.stopListening(ls.obj, ls.event, this.onChange));
this.dataListeners.forEach((ls) => this.model.stopListening(ls.obj, ls.event, this.onChange));
this.dataListeners = [];
}

1
packages/core/src/data_sources/model/conditional_variables/DataCondition.ts

@ -94,7 +94,6 @@ export class DataCondition extends Model<DataConditionType> {
dataVariables.forEach((variable) => {
const variableInstance = new DataVariable(variable, { em: this.em });
const listener = new DynamicVariableListenerManager({
model: this as any,
em: this.em!,
dataVariable: variableInstance,
updateValueFromDataVariable: (() => {

1
packages/core/src/data_sources/view/ComponentDataVariableView.ts

@ -8,7 +8,6 @@ export default class ComponentDataVariableView extends ComponentView<ComponentDa
initialize(opt = {}) {
super.initialize(opt);
this.dynamicVariableListener = new DynamicVariableListenerManager({
model: this,
em: this.em!,
dataVariable: this.model,
updateValueFromDataVariable: () => this.postRender(),

1
packages/core/src/domain_abstract/model/StyleableModel.ts

@ -168,7 +168,6 @@ export default class StyleableModel<T extends ObjectHash = any> extends Model<T>
this.dynamicVariableListeners[styleProp].listenToDynamicVariable();
} else {
this.dynamicVariableListeners[styleProp] = new DynamicVariableListenerManager({
model: this,
em: this.em!,
dataVariable: dataVar,
updateValueFromDataVariable: () => this.updateView(),

2
packages/core/src/trait_manager/model/Trait.ts

@ -77,7 +77,6 @@ export default class Trait extends Model<TraitProperties> {
const dv = this.dynamicVariable.getDataValue();
this.set({ value: dv });
this.dynamicVariableListener = new DynamicVariableListenerManager({
model: this,
em: this.em,
dataVariable: this.dynamicVariable,
updateValueFromDataVariable: this.updateValueFromDataVariable.bind(this),
@ -384,6 +383,7 @@ export default class Trait extends Model<TraitProperties> {
}
getInitValue() {
if (this.hasDynamicValue()) return this.getDynamicValue();
const { component } = this;
const name = this.getName();
let value;

Loading…
Cancel
Save