@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.
Files changed (177) hide show
  1. package/.eslintrc.json +25 -1
  2. package/THIRDPARTY.txt +2 -2
  3. package/package.json +6 -6
  4. package/src/sap/ui/rta/.library +1 -1
  5. package/src/sap/ui/rta/Client.js +32 -21
  6. package/src/sap/ui/rta/RuntimeAuthoring.js +107 -97
  7. package/src/sap/ui/rta/Utils.js +10 -16
  8. package/src/sap/ui/rta/api/startAdaptation.js +0 -1
  9. package/src/sap/ui/rta/api/startKeyUserAdaptation.js +0 -1
  10. package/src/sap/ui/rta/appVariant/AppVariantDialog.js +24 -26
  11. package/src/sap/ui/rta/appVariant/AppVariantManager.js +1 -3
  12. package/src/sap/ui/rta/appVariant/AppVariantOverviewDialog.js +5 -4
  13. package/src/sap/ui/rta/appVariant/AppVariantUtils.js +21 -21
  14. package/src/sap/ui/rta/appVariant/Feature.js +20 -30
  15. package/src/sap/ui/rta/appVariant/S4HanaCloudBackend.js +21 -17
  16. package/src/sap/ui/rta/appVariant/Utils.js +4 -3
  17. package/src/sap/ui/rta/appVariant/manageApps/webapp/Component.js +8 -7
  18. package/src/sap/ui/rta/appVariant/manageApps/webapp/controller/ManageApps.controller.js +17 -17
  19. package/src/sap/ui/rta/appVariant/manageApps/webapp/manifest.json +1 -1
  20. package/src/sap/ui/rta/appVariant/manageApps/webapp/model/models.js +1 -1
  21. package/src/sap/ui/rta/command/AddIFrame.js +10 -7
  22. package/src/sap/ui/rta/command/AddProperty.js +1 -3
  23. package/src/sap/ui/rta/command/AddXML.js +7 -7
  24. package/src/sap/ui/rta/command/AddXMLAtExtensionPoint.js +5 -5
  25. package/src/sap/ui/rta/command/AppDescriptorCommand.js +1 -3
  26. package/src/sap/ui/rta/command/BaseCommand.js +1 -3
  27. package/src/sap/ui/rta/command/BindProperty.js +4 -5
  28. package/src/sap/ui/rta/command/Combine.js +1 -3
  29. package/src/sap/ui/rta/command/CommandFactory.js +7 -11
  30. package/src/sap/ui/rta/command/CompositeCommand.js +2 -6
  31. package/src/sap/ui/rta/command/ControlVariantConfigure.js +2 -2
  32. package/src/sap/ui/rta/command/ControlVariantSave.js +1 -1
  33. package/src/sap/ui/rta/command/ControlVariantSaveAs.js +4 -4
  34. package/src/sap/ui/rta/command/ControlVariantSetTitle.js +1 -1
  35. package/src/sap/ui/rta/command/ControlVariantSwitch.js +4 -3
  36. package/src/sap/ui/rta/command/CreateContainer.js +1 -3
  37. package/src/sap/ui/rta/command/CustomAdd.js +1 -2
  38. package/src/sap/ui/rta/command/FlexCommand.js +2 -7
  39. package/src/sap/ui/rta/command/LREPSerializer.js +1 -3
  40. package/src/sap/ui/rta/command/LocalReset.js +1 -1
  41. package/src/sap/ui/rta/command/Move.js +1 -3
  42. package/src/sap/ui/rta/command/Property.js +1 -3
  43. package/src/sap/ui/rta/command/Remove.js +1 -3
  44. package/src/sap/ui/rta/command/Rename.js +1 -3
  45. package/src/sap/ui/rta/command/Resize.js +1 -1
  46. package/src/sap/ui/rta/command/Reveal.js +1 -3
  47. package/src/sap/ui/rta/command/Settings.js +5 -7
  48. package/src/sap/ui/rta/command/Split.js +1 -3
  49. package/src/sap/ui/rta/command/Stack.js +3 -5
  50. package/src/sap/ui/rta/command/appDescriptor/AddLibrary.js +1 -3
  51. package/src/sap/ui/rta/command/compVariant/CompVariantContent.js +1 -1
  52. package/src/sap/ui/rta/command/compVariant/CompVariantSaveAs.js +1 -1
  53. package/src/sap/ui/rta/command/compVariant/CompVariantSwitch.js +4 -3
  54. package/src/sap/ui/rta/command/compVariant/CompVariantUpdate.js +1 -1
  55. package/src/sap/ui/rta/enablement/TestDelegate.js +7 -8
  56. package/src/sap/ui/rta/enablement/elementActionTest.js +25 -29
  57. package/src/sap/ui/rta/library.js +2 -3
  58. package/src/sap/ui/rta/messagebundle.properties +13 -7
  59. package/src/sap/ui/rta/messagebundle_ar.properties +8 -8
  60. package/src/sap/ui/rta/messagebundle_bg.properties +20 -20
  61. package/src/sap/ui/rta/messagebundle_ca.properties +7 -7
  62. package/src/sap/ui/rta/messagebundle_cs.properties +10 -10
  63. package/src/sap/ui/rta/messagebundle_cy.properties +8 -8
  64. package/src/sap/ui/rta/messagebundle_da.properties +7 -7
  65. package/src/sap/ui/rta/messagebundle_de.properties +9 -9
  66. package/src/sap/ui/rta/messagebundle_el.properties +7 -7
  67. package/src/sap/ui/rta/messagebundle_en.properties +11 -8
  68. package/src/sap/ui/rta/messagebundle_en_GB.properties +8 -8
  69. package/src/sap/ui/rta/messagebundle_en_US_sappsd.properties +8 -8
  70. package/src/sap/ui/rta/messagebundle_en_US_saprigi.properties +7 -7
  71. package/src/sap/ui/rta/messagebundle_en_US_saptrc.properties +8 -8
  72. package/src/sap/ui/rta/messagebundle_es.properties +8 -8
  73. package/src/sap/ui/rta/messagebundle_es_MX.properties +13 -13
  74. package/src/sap/ui/rta/messagebundle_et.properties +6 -6
  75. package/src/sap/ui/rta/messagebundle_fi.properties +9 -9
  76. package/src/sap/ui/rta/messagebundle_fr.properties +9 -9
  77. package/src/sap/ui/rta/messagebundle_fr_CA.properties +9 -9
  78. package/src/sap/ui/rta/messagebundle_hi.properties +12 -12
  79. package/src/sap/ui/rta/messagebundle_hr.properties +9 -9
  80. package/src/sap/ui/rta/messagebundle_hu.properties +10 -10
  81. package/src/sap/ui/rta/messagebundle_id.properties +10 -10
  82. package/src/sap/ui/rta/messagebundle_it.properties +9 -9
  83. package/src/sap/ui/rta/messagebundle_iw.properties +8 -8
  84. package/src/sap/ui/rta/messagebundle_ja.properties +21 -21
  85. package/src/sap/ui/rta/messagebundle_kk.properties +9 -9
  86. package/src/sap/ui/rta/messagebundle_ko.properties +7 -7
  87. package/src/sap/ui/rta/messagebundle_lt.properties +8 -8
  88. package/src/sap/ui/rta/messagebundle_lv.properties +8 -8
  89. package/src/sap/ui/rta/messagebundle_ms.properties +11 -11
  90. package/src/sap/ui/rta/messagebundle_nl.properties +8 -8
  91. package/src/sap/ui/rta/messagebundle_no.properties +9 -9
  92. package/src/sap/ui/rta/messagebundle_pl.properties +9 -9
  93. package/src/sap/ui/rta/messagebundle_pt.properties +9 -9
  94. package/src/sap/ui/rta/messagebundle_pt_PT.properties +8 -8
  95. package/src/sap/ui/rta/messagebundle_ro.properties +9 -9
  96. package/src/sap/ui/rta/messagebundle_ru.properties +8 -8
  97. package/src/sap/ui/rta/messagebundle_sh.properties +8 -8
  98. package/src/sap/ui/rta/messagebundle_sk.properties +9 -9
  99. package/src/sap/ui/rta/messagebundle_sl.properties +9 -9
  100. package/src/sap/ui/rta/messagebundle_sv.properties +8 -8
  101. package/src/sap/ui/rta/messagebundle_th.properties +14 -14
  102. package/src/sap/ui/rta/messagebundle_tr.properties +8 -8
  103. package/src/sap/ui/rta/messagebundle_uk.properties +10 -10
  104. package/src/sap/ui/rta/messagebundle_vi.properties +8 -8
  105. package/src/sap/ui/rta/messagebundle_zh_CN.properties +9 -9
  106. package/src/sap/ui/rta/messagebundle_zh_TW.properties +8 -8
  107. package/src/sap/ui/rta/plugin/AddXMLAtExtensionPoint.js +1 -2
  108. package/src/sap/ui/rta/plugin/BaseCreate.js +2 -3
  109. package/src/sap/ui/rta/plugin/Combine.js +1 -2
  110. package/src/sap/ui/rta/plugin/ControlVariant.js +12 -8
  111. package/src/sap/ui/rta/plugin/CreateContainer.js +1 -2
  112. package/src/sap/ui/rta/plugin/CutPaste.js +13 -13
  113. package/src/sap/ui/rta/plugin/DragDrop.js +17 -16
  114. package/src/sap/ui/rta/plugin/EasyAdd.js +12 -10
  115. package/src/sap/ui/rta/plugin/EasyRemove.js +11 -9
  116. package/src/sap/ui/rta/plugin/LocalReset.js +4 -2
  117. package/src/sap/ui/rta/plugin/Plugin.js +6 -10
  118. package/src/sap/ui/rta/plugin/RTAElementMover.js +3 -6
  119. package/src/sap/ui/rta/plugin/Remove.js +8 -9
  120. package/src/sap/ui/rta/plugin/Rename.js +9 -9
  121. package/src/sap/ui/rta/plugin/RenameHandler.js +21 -22
  122. package/src/sap/ui/rta/plugin/Resize.js +11 -10
  123. package/src/sap/ui/rta/plugin/Selection.js +33 -14
  124. package/src/sap/ui/rta/plugin/Settings.js +11 -11
  125. package/src/sap/ui/rta/plugin/Split.js +1 -2
  126. package/src/sap/ui/rta/plugin/Stretch.js +18 -20
  127. package/src/sap/ui/rta/plugin/additionalElements/ActionExtractor.js +4 -9
  128. package/src/sap/ui/rta/plugin/additionalElements/AddElementsDialog.js +5 -9
  129. package/src/sap/ui/rta/plugin/additionalElements/AdditionalElementsAnalyzer.js +4 -4
  130. package/src/sap/ui/rta/plugin/additionalElements/AdditionalElementsPlugin.js +41 -43
  131. package/src/sap/ui/rta/plugin/additionalElements/AdditionalElementsUtils.js +2 -3
  132. package/src/sap/ui/rta/plugin/additionalElements/CommandBuilder.js +5 -6
  133. package/src/sap/ui/rta/plugin/iframe/AddIFrame.js +9 -7
  134. package/src/sap/ui/rta/plugin/iframe/AddIFrameDialog.fragment.xml +45 -3
  135. package/src/sap/ui/rta/plugin/iframe/AddIFrameDialog.js +22 -23
  136. package/src/sap/ui/rta/plugin/iframe/AddIFrameDialogController.js +58 -30
  137. package/src/sap/ui/rta/service/Action.js +5 -6
  138. package/src/sap/ui/rta/service/ControllerExtension.js +8 -9
  139. package/src/sap/ui/rta/service/Outline.js +13 -11
  140. package/src/sap/ui/rta/service/Property.js +3 -4
  141. package/src/sap/ui/rta/service/Receiver.js +2 -2
  142. package/src/sap/ui/rta/service/Selection.js +2 -3
  143. package/src/sap/ui/rta/service/SupportTools.js +2 -3
  144. package/src/sap/ui/rta/themes/base/AddIFrame.less +6 -1
  145. package/src/sap/ui/rta/themes/base/Toolbar.base.less +4 -0
  146. package/src/sap/ui/rta/toolbar/Adaptation.js +26 -19
  147. package/src/sap/ui/rta/toolbar/Base.js +25 -11
  148. package/src/sap/ui/rta/toolbar/BaseRenderer.js +2 -2
  149. package/src/sap/ui/rta/toolbar/FeedbackDialog.fragment.xml +4 -4
  150. package/src/sap/ui/rta/toolbar/Fiori.js +20 -20
  151. package/src/sap/ui/rta/toolbar/FioriLike.js +1 -2
  152. package/src/sap/ui/rta/toolbar/OverflowToolbarButton.js +5 -6
  153. package/src/sap/ui/rta/toolbar/Personalization.js +4 -4
  154. package/src/sap/ui/rta/toolbar/Standalone.js +1 -2
  155. package/src/sap/ui/rta/toolbar/contextBased/ManageAdaptations.js +43 -20
  156. package/src/sap/ui/rta/toolbar/contextBased/SaveAsAdaptation.js +16 -7
  157. package/src/sap/ui/rta/toolbar/translation/Translation.js +13 -14
  158. package/src/sap/ui/rta/toolbar/versioning/Versioning.js +31 -28
  159. package/src/sap/ui/rta/util/Animation.js +1 -3
  160. package/src/sap/ui/rta/util/PluginManager.js +1 -3
  161. package/src/sap/ui/rta/util/PopupManager.js +23 -19
  162. package/src/sap/ui/rta/util/ReloadManager.js +4 -4
  163. package/src/sap/ui/rta/util/ServiceEventBus.js +1 -1
  164. package/src/sap/ui/rta/util/changeVisualization/ChangeIndicator.js +18 -16
  165. package/src/sap/ui/rta/util/changeVisualization/ChangeIndicatorRegistry.js +6 -9
  166. package/src/sap/ui/rta/util/changeVisualization/ChangeVisualization.js +9 -10
  167. package/src/sap/ui/rta/util/changeVisualization/ChangeVisualizationUtils.js +1 -1
  168. package/src/sap/ui/rta/util/changeVisualization/commands/CombineVisualization.js +1 -1
  169. package/src/sap/ui/rta/util/changeVisualization/commands/CreateContainerVisualization.js +2 -2
  170. package/src/sap/ui/rta/util/changeVisualization/commands/MoveVisualization.js +4 -4
  171. package/src/sap/ui/rta/util/changeVisualization/commands/SplitVisualization.js +2 -2
  172. package/src/sap/ui/rta/util/hasStableId.js +1 -1
  173. package/src/sap/ui/rta/util/showMessageBox.js +2 -1
  174. package/src/sap/ui/rta/util/validateStableIds.js +1 -1
  175. package/src/sap/ui/rta/util/validateText.js +1 -1
  176. package/src/sap/ui/rta/util/StylesLoader.js +0 -31
  177. 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.0
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.4.2
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.117.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.117.1",
18
- "@openui5/sap.ui.core": "1.117.1",
19
- "@openui5/sap.ui.dt": "1.117.1",
20
- "@openui5/sap.ui.fl": "1.117.1",
21
- "@openui5/sap.ui.layout": "1.117.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
  }
@@ -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.117.1</version>
9
+ <version>1.119.0</version>
10
10
 
11
11
  <documentation>SAPUI5 library with RTA controls.</documentation>
12
12
 
@@ -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.117.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
- constructor: function() {
88
- ManagedObject.apply(this, arguments);
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(new Error("sap.ui.rta.Client.getService(): connection to RuntimeAuthoring instance has been refused"));
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, arguments);
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: Array.prototype.slice.call(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
- if (!this._oServiceEventBus) {
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("sap.ui.rta.Client: sEventName must be a non-empty string when calling attachEvent() for a service");
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("sap.ui.rta.Client: fnFunction must be a function when calling attachEvent() for a service");
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, then receiver doesn't need second notification
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 + "," + sEventName] = mResponse.id;
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("sap.ui.rta.Client: sEventName must be a non-empty string when calling detachEvent() for a service");
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("sap.ui.rta.Client: fnFunction must be a function when calling detachEvent() for a service");
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: function(sEventName, fnCallback, oContext) {
376
- function fnOnce() {
386
+ attachEventOnce(sEventName, fnCallback, oContext) {
387
+ function fnOnce(...aArgs) {
377
388
  mService.detachEvent(sEventName, fnOnce);
378
- fnCallback.apply(oContext, arguments);
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 + "," + sEventName];
419
+ var sEventHandlerId = this._mEventHandlerIds[`${sServiceName},${sEventName}`];
409
420
 
410
421
  if (
411
422
  (!oEventProvider || !oEventProvider.hasListeners(sEventName))