@openui5/sap.ui.rta 1.117.1 → 1.119.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/.eslintrc.json +25 -1
- package/THIRDPARTY.txt +2 -2
- package/package.json +6 -6
- package/src/sap/ui/rta/.library +1 -1
- package/src/sap/ui/rta/Client.js +32 -21
- package/src/sap/ui/rta/RuntimeAuthoring.js +107 -97
- package/src/sap/ui/rta/Utils.js +10 -16
- package/src/sap/ui/rta/api/startAdaptation.js +0 -1
- package/src/sap/ui/rta/api/startKeyUserAdaptation.js +0 -1
- package/src/sap/ui/rta/appVariant/AppVariantDialog.js +24 -26
- package/src/sap/ui/rta/appVariant/AppVariantManager.js +1 -3
- package/src/sap/ui/rta/appVariant/AppVariantOverviewDialog.js +5 -4
- package/src/sap/ui/rta/appVariant/AppVariantUtils.js +21 -21
- package/src/sap/ui/rta/appVariant/Feature.js +20 -30
- package/src/sap/ui/rta/appVariant/S4HanaCloudBackend.js +21 -17
- package/src/sap/ui/rta/appVariant/Utils.js +4 -3
- package/src/sap/ui/rta/appVariant/manageApps/webapp/Component.js +8 -7
- package/src/sap/ui/rta/appVariant/manageApps/webapp/controller/ManageApps.controller.js +17 -17
- package/src/sap/ui/rta/appVariant/manageApps/webapp/manifest.json +1 -1
- package/src/sap/ui/rta/appVariant/manageApps/webapp/model/models.js +1 -1
- package/src/sap/ui/rta/command/AddIFrame.js +10 -7
- package/src/sap/ui/rta/command/AddProperty.js +1 -3
- package/src/sap/ui/rta/command/AddXML.js +7 -7
- package/src/sap/ui/rta/command/AddXMLAtExtensionPoint.js +5 -5
- package/src/sap/ui/rta/command/AppDescriptorCommand.js +1 -3
- package/src/sap/ui/rta/command/BaseCommand.js +1 -3
- package/src/sap/ui/rta/command/BindProperty.js +4 -5
- package/src/sap/ui/rta/command/Combine.js +1 -3
- package/src/sap/ui/rta/command/CommandFactory.js +7 -11
- package/src/sap/ui/rta/command/CompositeCommand.js +2 -6
- package/src/sap/ui/rta/command/ControlVariantConfigure.js +2 -2
- package/src/sap/ui/rta/command/ControlVariantSave.js +1 -1
- package/src/sap/ui/rta/command/ControlVariantSaveAs.js +4 -4
- package/src/sap/ui/rta/command/ControlVariantSetTitle.js +1 -1
- package/src/sap/ui/rta/command/ControlVariantSwitch.js +4 -3
- package/src/sap/ui/rta/command/CreateContainer.js +1 -3
- package/src/sap/ui/rta/command/CustomAdd.js +1 -2
- package/src/sap/ui/rta/command/FlexCommand.js +2 -7
- package/src/sap/ui/rta/command/LREPSerializer.js +1 -3
- package/src/sap/ui/rta/command/LocalReset.js +1 -1
- package/src/sap/ui/rta/command/Move.js +1 -3
- package/src/sap/ui/rta/command/Property.js +1 -3
- package/src/sap/ui/rta/command/Remove.js +1 -3
- package/src/sap/ui/rta/command/Rename.js +1 -3
- package/src/sap/ui/rta/command/Resize.js +1 -1
- package/src/sap/ui/rta/command/Reveal.js +1 -3
- package/src/sap/ui/rta/command/Settings.js +5 -7
- package/src/sap/ui/rta/command/Split.js +1 -3
- package/src/sap/ui/rta/command/Stack.js +3 -5
- package/src/sap/ui/rta/command/appDescriptor/AddLibrary.js +1 -3
- package/src/sap/ui/rta/command/compVariant/CompVariantContent.js +1 -1
- package/src/sap/ui/rta/command/compVariant/CompVariantSaveAs.js +1 -1
- package/src/sap/ui/rta/command/compVariant/CompVariantSwitch.js +4 -3
- package/src/sap/ui/rta/command/compVariant/CompVariantUpdate.js +1 -1
- package/src/sap/ui/rta/enablement/TestDelegate.js +7 -8
- package/src/sap/ui/rta/enablement/elementActionTest.js +25 -29
- package/src/sap/ui/rta/library.js +2 -3
- package/src/sap/ui/rta/messagebundle.properties +13 -7
- package/src/sap/ui/rta/messagebundle_ar.properties +8 -8
- package/src/sap/ui/rta/messagebundle_bg.properties +20 -20
- package/src/sap/ui/rta/messagebundle_ca.properties +7 -7
- package/src/sap/ui/rta/messagebundle_cs.properties +10 -10
- package/src/sap/ui/rta/messagebundle_cy.properties +8 -8
- package/src/sap/ui/rta/messagebundle_da.properties +7 -7
- package/src/sap/ui/rta/messagebundle_de.properties +9 -9
- package/src/sap/ui/rta/messagebundle_el.properties +7 -7
- package/src/sap/ui/rta/messagebundle_en.properties +11 -8
- package/src/sap/ui/rta/messagebundle_en_GB.properties +8 -8
- package/src/sap/ui/rta/messagebundle_en_US_sappsd.properties +8 -8
- package/src/sap/ui/rta/messagebundle_en_US_saprigi.properties +7 -7
- package/src/sap/ui/rta/messagebundle_en_US_saptrc.properties +8 -8
- package/src/sap/ui/rta/messagebundle_es.properties +8 -8
- package/src/sap/ui/rta/messagebundle_es_MX.properties +13 -13
- package/src/sap/ui/rta/messagebundle_et.properties +6 -6
- package/src/sap/ui/rta/messagebundle_fi.properties +9 -9
- package/src/sap/ui/rta/messagebundle_fr.properties +9 -9
- package/src/sap/ui/rta/messagebundle_fr_CA.properties +9 -9
- package/src/sap/ui/rta/messagebundle_hi.properties +12 -12
- package/src/sap/ui/rta/messagebundle_hr.properties +9 -9
- package/src/sap/ui/rta/messagebundle_hu.properties +10 -10
- package/src/sap/ui/rta/messagebundle_id.properties +10 -10
- package/src/sap/ui/rta/messagebundle_it.properties +9 -9
- package/src/sap/ui/rta/messagebundle_iw.properties +8 -8
- package/src/sap/ui/rta/messagebundle_ja.properties +21 -21
- package/src/sap/ui/rta/messagebundle_kk.properties +9 -9
- package/src/sap/ui/rta/messagebundle_ko.properties +7 -7
- package/src/sap/ui/rta/messagebundle_lt.properties +8 -8
- package/src/sap/ui/rta/messagebundle_lv.properties +8 -8
- package/src/sap/ui/rta/messagebundle_ms.properties +11 -11
- package/src/sap/ui/rta/messagebundle_nl.properties +8 -8
- package/src/sap/ui/rta/messagebundle_no.properties +9 -9
- package/src/sap/ui/rta/messagebundle_pl.properties +9 -9
- package/src/sap/ui/rta/messagebundle_pt.properties +9 -9
- package/src/sap/ui/rta/messagebundle_pt_PT.properties +8 -8
- package/src/sap/ui/rta/messagebundle_ro.properties +9 -9
- package/src/sap/ui/rta/messagebundle_ru.properties +8 -8
- package/src/sap/ui/rta/messagebundle_sh.properties +8 -8
- package/src/sap/ui/rta/messagebundle_sk.properties +9 -9
- package/src/sap/ui/rta/messagebundle_sl.properties +9 -9
- package/src/sap/ui/rta/messagebundle_sv.properties +8 -8
- package/src/sap/ui/rta/messagebundle_th.properties +14 -14
- package/src/sap/ui/rta/messagebundle_tr.properties +8 -8
- package/src/sap/ui/rta/messagebundle_uk.properties +10 -10
- package/src/sap/ui/rta/messagebundle_vi.properties +8 -8
- package/src/sap/ui/rta/messagebundle_zh_CN.properties +9 -9
- package/src/sap/ui/rta/messagebundle_zh_TW.properties +8 -8
- package/src/sap/ui/rta/plugin/AddXMLAtExtensionPoint.js +1 -2
- package/src/sap/ui/rta/plugin/BaseCreate.js +2 -3
- package/src/sap/ui/rta/plugin/Combine.js +1 -2
- package/src/sap/ui/rta/plugin/ControlVariant.js +12 -8
- package/src/sap/ui/rta/plugin/CreateContainer.js +1 -2
- package/src/sap/ui/rta/plugin/CutPaste.js +13 -13
- package/src/sap/ui/rta/plugin/DragDrop.js +17 -16
- package/src/sap/ui/rta/plugin/EasyAdd.js +12 -10
- package/src/sap/ui/rta/plugin/EasyRemove.js +11 -9
- package/src/sap/ui/rta/plugin/LocalReset.js +4 -2
- package/src/sap/ui/rta/plugin/Plugin.js +6 -10
- package/src/sap/ui/rta/plugin/RTAElementMover.js +3 -6
- package/src/sap/ui/rta/plugin/Remove.js +8 -9
- package/src/sap/ui/rta/plugin/Rename.js +9 -9
- package/src/sap/ui/rta/plugin/RenameHandler.js +21 -22
- package/src/sap/ui/rta/plugin/Resize.js +11 -10
- package/src/sap/ui/rta/plugin/Selection.js +33 -14
- package/src/sap/ui/rta/plugin/Settings.js +11 -11
- package/src/sap/ui/rta/plugin/Split.js +1 -2
- package/src/sap/ui/rta/plugin/Stretch.js +18 -20
- package/src/sap/ui/rta/plugin/additionalElements/ActionExtractor.js +4 -9
- package/src/sap/ui/rta/plugin/additionalElements/AddElementsDialog.js +5 -9
- package/src/sap/ui/rta/plugin/additionalElements/AdditionalElementsAnalyzer.js +4 -4
- package/src/sap/ui/rta/plugin/additionalElements/AdditionalElementsPlugin.js +41 -43
- package/src/sap/ui/rta/plugin/additionalElements/AdditionalElementsUtils.js +2 -3
- package/src/sap/ui/rta/plugin/additionalElements/CommandBuilder.js +5 -6
- package/src/sap/ui/rta/plugin/iframe/AddIFrame.js +9 -7
- package/src/sap/ui/rta/plugin/iframe/AddIFrameDialog.fragment.xml +45 -3
- package/src/sap/ui/rta/plugin/iframe/AddIFrameDialog.js +22 -23
- package/src/sap/ui/rta/plugin/iframe/AddIFrameDialogController.js +58 -30
- package/src/sap/ui/rta/service/Action.js +5 -6
- package/src/sap/ui/rta/service/ControllerExtension.js +8 -9
- package/src/sap/ui/rta/service/Outline.js +13 -11
- package/src/sap/ui/rta/service/Property.js +3 -4
- package/src/sap/ui/rta/service/Receiver.js +2 -2
- package/src/sap/ui/rta/service/Selection.js +2 -3
- package/src/sap/ui/rta/service/SupportTools.js +2 -3
- package/src/sap/ui/rta/themes/base/AddIFrame.less +6 -1
- package/src/sap/ui/rta/themes/base/Toolbar.base.less +4 -0
- package/src/sap/ui/rta/toolbar/Adaptation.js +26 -19
- package/src/sap/ui/rta/toolbar/Base.js +25 -11
- package/src/sap/ui/rta/toolbar/BaseRenderer.js +2 -2
- package/src/sap/ui/rta/toolbar/FeedbackDialog.fragment.xml +4 -4
- package/src/sap/ui/rta/toolbar/Fiori.js +20 -20
- package/src/sap/ui/rta/toolbar/FioriLike.js +1 -2
- package/src/sap/ui/rta/toolbar/OverflowToolbarButton.js +5 -6
- package/src/sap/ui/rta/toolbar/Personalization.js +4 -4
- package/src/sap/ui/rta/toolbar/Standalone.js +1 -2
- package/src/sap/ui/rta/toolbar/contextBased/ManageAdaptations.js +43 -20
- package/src/sap/ui/rta/toolbar/contextBased/SaveAsAdaptation.js +16 -7
- package/src/sap/ui/rta/toolbar/translation/Translation.js +13 -14
- package/src/sap/ui/rta/toolbar/versioning/Versioning.js +31 -28
- package/src/sap/ui/rta/util/Animation.js +1 -3
- package/src/sap/ui/rta/util/PluginManager.js +1 -3
- package/src/sap/ui/rta/util/PopupManager.js +23 -19
- package/src/sap/ui/rta/util/ReloadManager.js +4 -4
- package/src/sap/ui/rta/util/ServiceEventBus.js +1 -1
- package/src/sap/ui/rta/util/changeVisualization/ChangeIndicator.js +18 -16
- package/src/sap/ui/rta/util/changeVisualization/ChangeIndicatorRegistry.js +6 -9
- package/src/sap/ui/rta/util/changeVisualization/ChangeVisualization.js +9 -10
- package/src/sap/ui/rta/util/changeVisualization/ChangeVisualizationUtils.js +1 -1
- package/src/sap/ui/rta/util/changeVisualization/commands/CombineVisualization.js +1 -1
- package/src/sap/ui/rta/util/changeVisualization/commands/CreateContainerVisualization.js +2 -2
- package/src/sap/ui/rta/util/changeVisualization/commands/MoveVisualization.js +4 -4
- package/src/sap/ui/rta/util/changeVisualization/commands/SplitVisualization.js +2 -2
- package/src/sap/ui/rta/util/hasStableId.js +1 -1
- package/src/sap/ui/rta/util/showMessageBox.js +2 -1
- package/src/sap/ui/rta/util/validateStableIds.js +1 -1
- package/src/sap/ui/rta/util/validateText.js +1 -1
- package/src/sap/ui/rta/util/StylesLoader.js +0 -31
- package/src/sap/ui/rta/util/UrlParser.js +0 -51
package/.eslintrc.json
CHANGED
|
@@ -26,6 +26,7 @@
|
|
|
26
26
|
"max-len": ["warn", {
|
|
27
27
|
"code": 140,
|
|
28
28
|
"ignorePattern": "@param|@returns|@description|<|QUnit.test",
|
|
29
|
+
"ignoreTemplateLiterals": true,
|
|
29
30
|
"ignoreUrls": true
|
|
30
31
|
}],
|
|
31
32
|
"space-before-function-paren": ["error", {
|
|
@@ -50,6 +51,29 @@
|
|
|
50
51
|
"quotes": ["error", "double", {
|
|
51
52
|
"avoidEscape": true,
|
|
52
53
|
"allowTemplateLiterals": true
|
|
53
|
-
}]
|
|
54
|
+
}],
|
|
55
|
+
"no-unsafe-optional-chaining": "error",
|
|
56
|
+
"prefer-const": ["error", { "ignoreReadBeforeAssign": true }],
|
|
57
|
+
"prefer-template": "error",
|
|
58
|
+
"rest-spread-spacing": ["error", "never"],
|
|
59
|
+
"template-curly-spacing": "error",
|
|
60
|
+
"object-shorthand": ["error", "always", { "ignoreConstructors": false, "avoidQuotes": true }],
|
|
61
|
+
"prefer-destructuring": ["error", {
|
|
62
|
+
"VariableDeclarator": {
|
|
63
|
+
"array": false,
|
|
64
|
+
"object": true
|
|
65
|
+
},
|
|
66
|
+
"AssignmentExpression": {
|
|
67
|
+
"array": true,
|
|
68
|
+
"object": false
|
|
69
|
+
}
|
|
70
|
+
}, {
|
|
71
|
+
"enforceForRenamedProperties": false
|
|
72
|
+
}
|
|
73
|
+
],
|
|
74
|
+
"prefer-rest-params": "error",
|
|
75
|
+
"prefer-spread": "error",
|
|
76
|
+
"logical-assignment-operators":["error", "always", { "enforceForIfStatements": true } ],
|
|
77
|
+
"require-await": "error"
|
|
54
78
|
}
|
|
55
79
|
}
|
package/THIRDPARTY.txt
CHANGED
|
@@ -401,7 +401,7 @@ License: MIT
|
|
|
401
401
|
License Text: https://github.com/SAP/openui5/blob/master/LICENSES/MIT.txt
|
|
402
402
|
Contained in: src/sap.ui.integration/src/sap/ui/integration/thirdparty/adaptive-expressions.js
|
|
403
403
|
|
|
404
|
-
Component: Markdown-it, version: 12.2
|
|
404
|
+
Component: Markdown-it, version: 12.3.2
|
|
405
405
|
Copyright: 2014 Vitaly Puzrin, Alex Kocharin
|
|
406
406
|
License: MIT
|
|
407
407
|
License Text: https://github.com/SAP/openui5/blob/master/LICENSES/MIT.txt
|
|
@@ -466,7 +466,7 @@ License: Apache-2.0
|
|
|
466
466
|
License Text: https://github.com/SAP/openui5/blob/master/LICENSES/Apache-2.0.txt
|
|
467
467
|
Contained in: lib/jsdoc/ui5/plugin.js
|
|
468
468
|
|
|
469
|
-
Component: SAP Theming Base Content, version: 11.
|
|
469
|
+
Component: SAP Theming Base Content, version: 11.6.8
|
|
470
470
|
Copyright: SAP SE or an SAP affiliate company and Theming Base Content contributors
|
|
471
471
|
License: Apache-2.0
|
|
472
472
|
License Text: https://github.com/SAP/openui5/blob/master/LICENSES/Apache-2.0.txt
|
package/package.json
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "@openui5/sap.ui.rta",
|
|
3
|
-
"version": "1.
|
|
3
|
+
"version": "1.119.0",
|
|
4
4
|
"description": "OpenUI5 UI Library sap.ui.rta",
|
|
5
5
|
"author": "SAP SE (https://www.sap.com)",
|
|
6
6
|
"license": "Apache-2.0",
|
|
@@ -14,10 +14,10 @@
|
|
|
14
14
|
"url": "https://github.com/SAP/openui5.git"
|
|
15
15
|
},
|
|
16
16
|
"dependencies": {
|
|
17
|
-
"@openui5/sap.m": "1.
|
|
18
|
-
"@openui5/sap.ui.core": "1.
|
|
19
|
-
"@openui5/sap.ui.dt": "1.
|
|
20
|
-
"@openui5/sap.ui.fl": "1.
|
|
21
|
-
"@openui5/sap.ui.layout": "1.
|
|
17
|
+
"@openui5/sap.m": "1.119.0",
|
|
18
|
+
"@openui5/sap.ui.core": "1.119.0",
|
|
19
|
+
"@openui5/sap.ui.dt": "1.119.0",
|
|
20
|
+
"@openui5/sap.ui.fl": "1.119.0",
|
|
21
|
+
"@openui5/sap.ui.layout": "1.119.0"
|
|
22
22
|
}
|
|
23
23
|
}
|
package/src/sap/ui/rta/.library
CHANGED
|
@@ -6,7 +6,7 @@
|
|
|
6
6
|
<copyright>OpenUI5
|
|
7
7
|
* (c) Copyright 2009-2023 SAP SE or an SAP affiliate company.
|
|
8
8
|
* Licensed under the Apache License, Version 2.0 - see LICENSE.txt.</copyright>
|
|
9
|
-
<version>1.
|
|
9
|
+
<version>1.119.0</version>
|
|
10
10
|
|
|
11
11
|
<documentation>SAPUI5 library with RTA controls.</documentation>
|
|
12
12
|
|
package/src/sap/ui/rta/Client.js
CHANGED
|
@@ -57,7 +57,7 @@ sap.ui.define([
|
|
|
57
57
|
* @alias sap.ui.rta.Client
|
|
58
58
|
* @author SAP SE
|
|
59
59
|
* @since 1.56.0
|
|
60
|
-
* @version 1.
|
|
60
|
+
* @version 1.119.0
|
|
61
61
|
* @private
|
|
62
62
|
* @ui5-restricted
|
|
63
63
|
*/
|
|
@@ -84,8 +84,9 @@ sap.ui.define([
|
|
|
84
84
|
*/
|
|
85
85
|
_bInit: false,
|
|
86
86
|
|
|
87
|
-
|
|
88
|
-
|
|
87
|
+
// eslint-disable-next-line object-shorthand
|
|
88
|
+
constructor: function(...aArgs) {
|
|
89
|
+
ManagedObject.apply(this, aArgs);
|
|
89
90
|
|
|
90
91
|
if (!this.getWindow()) {
|
|
91
92
|
throw new TypeError("sap.ui.rta.Client: window parameter is required");
|
|
@@ -175,8 +176,11 @@ sap.ui.define([
|
|
|
175
176
|
// Rejecting queued requests
|
|
176
177
|
var aRequests = this._aRequestQueue.slice();
|
|
177
178
|
this._aRequestQueue = [];
|
|
179
|
+
// eslint-disable-next-line max-nested-callbacks
|
|
178
180
|
aRequests.forEach(function(mRequest) {
|
|
179
|
-
mRequest.reject(
|
|
181
|
+
mRequest.reject(
|
|
182
|
+
new Error("sap.ui.rta.Client.getService(): connection to RuntimeAuthoring instance has been refused")
|
|
183
|
+
);
|
|
180
184
|
});
|
|
181
185
|
}, this);
|
|
182
186
|
|
|
@@ -199,14 +203,14 @@ sap.ui.define([
|
|
|
199
203
|
* After an object has been destroyed, it can no longer be used.
|
|
200
204
|
* @public
|
|
201
205
|
*/
|
|
202
|
-
Client.prototype.destroy = function() {
|
|
206
|
+
Client.prototype.destroy = function(...aArgs) {
|
|
203
207
|
this._oPostMessageBus.unsubscribe(CHANNEL_ID, "getService", this._receiverMethods, this);
|
|
204
208
|
this._oPostMessageBus.unsubscribe(CHANNEL_ID, "callMethod", this._receiverMethods, this);
|
|
205
209
|
this._oPostMessageBus.unsubscribe(CHANNEL_ID, "subscribe", this._receiverMethods, this);
|
|
206
210
|
this._oPostMessageBus.unsubscribe(CHANNEL_ID, "unsubscribe", this._receiverMethods, this);
|
|
207
211
|
this._oPostMessageBus.unsubscribe(CHANNEL_ID, "event", this._receiverEvents, this);
|
|
208
212
|
|
|
209
|
-
ManagedObject.prototype.destroy.apply(this,
|
|
213
|
+
ManagedObject.prototype.destroy.apply(this, aArgs);
|
|
210
214
|
};
|
|
211
215
|
|
|
212
216
|
/**
|
|
@@ -300,7 +304,7 @@ sap.ui.define([
|
|
|
300
304
|
var mService = merge(
|
|
301
305
|
// Create placeholders for methods
|
|
302
306
|
aMethods.reduce(function(mResult, sMethodName) {
|
|
303
|
-
mResult[sMethodName] = function() {
|
|
307
|
+
mResult[sMethodName] = function(...aArgs) {
|
|
304
308
|
return this._sendRequest(this._createRequest({
|
|
305
309
|
target: oEvent.source,
|
|
306
310
|
origin: oEvent.origin,
|
|
@@ -309,7 +313,7 @@ sap.ui.define([
|
|
|
309
313
|
data: {
|
|
310
314
|
service: sServiceName,
|
|
311
315
|
method: sMethodName,
|
|
312
|
-
arguments:
|
|
316
|
+
arguments: aArgs
|
|
313
317
|
}
|
|
314
318
|
}));
|
|
315
319
|
}.bind(this);
|
|
@@ -320,19 +324,22 @@ sap.ui.define([
|
|
|
320
324
|
);
|
|
321
325
|
|
|
322
326
|
if (Array.isArray(aEvents) && aEvents.length > 0) {
|
|
323
|
-
|
|
324
|
-
this._oServiceEventBus = new ServiceEventBus();
|
|
325
|
-
}
|
|
327
|
+
this._oServiceEventBus ||= new ServiceEventBus();
|
|
326
328
|
merge(mService, {
|
|
327
329
|
attachEvent: function(sEventName, fnCallback, oContext) {
|
|
328
330
|
if (typeof (sEventName) !== "string" || !sEventName) {
|
|
329
|
-
throw new TypeError(
|
|
331
|
+
throw new TypeError(
|
|
332
|
+
"sap.ui.rta.Client: sEventName must be a non-empty string when calling attachEvent() for a service"
|
|
333
|
+
);
|
|
330
334
|
}
|
|
331
335
|
if (typeof fnCallback !== "function") {
|
|
332
|
-
throw new TypeError(
|
|
336
|
+
throw new TypeError(
|
|
337
|
+
"sap.ui.rta.Client: fnFunction must be a function when calling attachEvent() for a service"
|
|
338
|
+
);
|
|
333
339
|
}
|
|
334
340
|
|
|
335
|
-
// 1. Check whether there are other subscribers for same event, if so,
|
|
341
|
+
// 1. Check whether there are other subscribers for same event, if so,
|
|
342
|
+
// then receiver doesn't need second notification
|
|
336
343
|
var oEventProvider = this._oServiceEventBus.getChannel(sServiceName);
|
|
337
344
|
var bShouldNotifyReceiver = !oEventProvider || !oEventProvider.hasListeners(sEventName);
|
|
338
345
|
|
|
@@ -351,7 +358,7 @@ sap.ui.define([
|
|
|
351
358
|
event: sEventName
|
|
352
359
|
}
|
|
353
360
|
})).then(function(mResponse) {
|
|
354
|
-
this._mEventHandlerIds[sServiceName
|
|
361
|
+
this._mEventHandlerIds[`${sServiceName},${sEventName}`] = mResponse.id;
|
|
355
362
|
|
|
356
363
|
// Use case when detach happens before we received response from RTA instance
|
|
357
364
|
this._checkIfEventAlive(sServiceName, sEventName);
|
|
@@ -360,10 +367,14 @@ sap.ui.define([
|
|
|
360
367
|
}.bind(this),
|
|
361
368
|
detachEvent: function(sEventName, fnCallback, oContext) {
|
|
362
369
|
if (typeof (sEventName) !== "string" || !sEventName) {
|
|
363
|
-
throw new TypeError(
|
|
370
|
+
throw new TypeError(
|
|
371
|
+
"sap.ui.rta.Client: sEventName must be a non-empty string when calling detachEvent() for a service"
|
|
372
|
+
);
|
|
364
373
|
}
|
|
365
374
|
if (typeof fnCallback !== "function") {
|
|
366
|
-
throw new TypeError(
|
|
375
|
+
throw new TypeError(
|
|
376
|
+
"sap.ui.rta.Client: fnFunction must be a function when calling detachEvent() for a service"
|
|
377
|
+
);
|
|
367
378
|
}
|
|
368
379
|
|
|
369
380
|
// 1. Unsubscribe from local EventBus
|
|
@@ -372,10 +383,10 @@ sap.ui.define([
|
|
|
372
383
|
// 2. Check and notify RTA instance if we don't want more events
|
|
373
384
|
this._checkIfEventAlive(sServiceName, sEventName);
|
|
374
385
|
}.bind(this),
|
|
375
|
-
attachEventOnce
|
|
376
|
-
function fnOnce() {
|
|
386
|
+
attachEventOnce(sEventName, fnCallback, oContext) {
|
|
387
|
+
function fnOnce(...aArgs) {
|
|
377
388
|
mService.detachEvent(sEventName, fnOnce);
|
|
378
|
-
fnCallback.apply(oContext,
|
|
389
|
+
fnCallback.apply(oContext, aArgs);
|
|
379
390
|
}
|
|
380
391
|
mService.attachEvent(sEventName, fnOnce);
|
|
381
392
|
}
|
|
@@ -405,7 +416,7 @@ sap.ui.define([
|
|
|
405
416
|
|
|
406
417
|
Client.prototype._checkIfEventAlive = function(sServiceName, sEventName) {
|
|
407
418
|
var oEventProvider = this._oServiceEventBus.getChannel(sServiceName);
|
|
408
|
-
var sEventHandlerId = this._mEventHandlerIds[sServiceName
|
|
419
|
+
var sEventHandlerId = this._mEventHandlerIds[`${sServiceName},${sEventName}`];
|
|
409
420
|
|
|
410
421
|
if (
|
|
411
422
|
(!oEventProvider || !oEventProvider.hasListeners(sEventName))
|