Skip to content

Commit ec40577

Browse files
andreaskienastgeorgringer
authored andcommitted
[TASK] Remove unused FormEngine.legacyFieldChangedCb()
The method `FormEngine.legacyFieldChangedCb()` was used back in the days by legacy FormEngine which passed a callback function via the PHP property `TBE_EDITOR_fieldChanged_func`. Later on, the configuration `TYPO3.settings.FormEngine.legacyFieldChangedCb` was used. Neither TYPO3 core, nor any extension available in TER use this relict. It is therefore safe to assume we can remove this method without being a breaking change. Resolves: #106278 Releases: main, 13.4 Change-Id: I6c47998edf5da042f0952c9dd3f28a2b879641af Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/88407 Reviewed-by: Garvin Hicking <gh@faktor-e.de> Tested-by: Garvin Hicking <gh@faktor-e.de> Tested-by: core-ci <typo3@b13.com> Tested-by: Georg Ringer <georg.ringer@gmail.com> Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
1 parent b7a4a3f commit ec40577

File tree

5 files changed

+2
-13
lines changed

5 files changed

+2
-13
lines changed

Build/Sources/TypeScript/backend/form-engine.ts

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,6 @@ type FormEngineType = {
5050
formName: string,
5151
formElement: HTMLFormElement,
5252
openedPopupWindow: Window | null,
53-
legacyFieldChangedCb: () => void,
5453
browserUrl: string,
5554
doSaveFieldName: string,
5655
};
@@ -152,11 +151,6 @@ export default (function() {
152151
formName: TYPO3.settings.FormEngine.formName,
153152
formElement: undefined,
154153
openedPopupWindow: null,
155-
legacyFieldChangedCb: function() {
156-
if ($.isFunction(TYPO3.settings.FormEngine.legacyFieldChangedCb)) {
157-
TYPO3.settings.FormEngine.legacyFieldChangedCb();
158-
}
159-
},
160154
browserUrl: '',
161155
doSaveFieldName: ''
162156
};
@@ -320,8 +314,6 @@ export default (function() {
320314
// set the hidden field
321315
FormEngine.updateHiddenFieldValueFromSelect(fieldEl, originalFieldEl);
322316

323-
// execute the phpcode from $FormEngine->TBE_EDITOR_fieldChanged_func
324-
FormEngine.legacyFieldChangedCb();
325317
FormEngine.Validation.markFieldAsChanged(originalFieldEl);
326318
FormEngine.Validation.validateField(fieldEl);
327319
FormEngine.Validation.validateField(availableFieldEl);

Build/Sources/TypeScript/backend/form-engine/element/abstract-sortable-select-items.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -138,7 +138,6 @@ export abstract class AbstractSortableSelectItems {
138138
}
139139

140140
FormEngine.updateHiddenFieldValueFromSelect(fieldElement, relatedField);
141-
FormEngine.legacyFieldChangedCb();
142141
FormEngineValidation.markFieldAsChanged(relatedAvailableValuesField);
143142
FormEngineValidation.validateField(relatedAvailableValuesField);
144143
});
@@ -178,7 +177,6 @@ export abstract class AbstractSortableSelectItems {
178177
}
179178
if (e.defaultPrevented) {
180179
FormEngine.updateHiddenFieldValueFromSelect(fieldElement, relatedField);
181-
FormEngine.legacyFieldChangedCb();
182180
FormEngineValidation.markFieldAsChanged(relatedAvailableValuesField);
183181
FormEngineValidation.validateField(relatedAvailableValuesField);
184182
}

Build/types/TYPO3/index.d.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,6 @@ declare namespace TYPO3 {
5353
export const formName: string;
5454

5555
export const doSaveFieldName: string;
56-
export const legacyFieldChangedCb: () => void;
5756
}
5857
export namespace FormEngineInline {
5958
export const config: {

0 commit comments

Comments
 (0)