@sapui5/sap.ushell 1.138.0 → 1.139.0

package/src/main/js/sap/ushell/components/tiles/action/ActionTile.controller.js

@@ -6,27 +6,27 @@ sap.ui.define([
     "sap/ui/model/json/JSONModel",
     "sap/ui/thirdparty/jquery",
     "sap/base/Log"
-], function (Controller, tileUtils, JSONModel, jQuery, Log) {
+], (Controller, tileUtils, JSONModel, jQuery, Log) => {
     "use strict";
 
     return Controller.extend("sap.ushell.components.tiles.action.ActionTile", {
         onInit: function () {
-            var oView = this.getView();
-            var oViewData = oView.getViewData();
-            var oResourceModel = tileUtils.getResourceBundleModel();
-            var oTileApi = oViewData.chip; // instance
-            var oConfig = tileUtils.getActionConfiguration(oTileApi);
-            var that = this;
+            const oView = this.getView();
+            const oViewData = oView.getViewData();
+            const oResourceModel = tileUtils.getResourceBundleModel();
+            const oTileApi = oViewData.chip; // instance
+            const oConfig = tileUtils.getActionConfiguration(oTileApi);
+            const that = this;
 
             function formatDisplayText (sSemanticObject, sSemanticAction) {
-                var oBundle = tileUtils.getResourceBundle();
-                var sResult = oBundle.getText("configuration.semantic_object") + ":\n" + sSemanticObject + "\n\n"
-                    + oBundle.getText("configuration.semantic_action") + ":\n" + sSemanticAction;
+                const oBundle = tileUtils.getResourceBundle();
+                const sResult = `${oBundle.getText("configuration.semantic_object")}:\n${sSemanticObject}\n\n${
+                    oBundle.getText("configuration.semantic_action")}:\n${sSemanticAction}`;
                 return sResult;
             }
 
             oView.setModel(oResourceModel, "i18n");
-            var oModel = new JSONModel({
+            const oModel = new JSONModel({
                 config: oConfig,
                 displayText: formatDisplayText(oConfig.semantic_object, oConfig.semantic_action)
             });
@@ -35,11 +35,11 @@ sap.ui.define([
             // implement configurationUi contract: setup configuration UI
             if (oTileApi.configurationUi.isEnabled()) {
                 // attach configuration UI provider, which is essentially a components.tiles.action.Configuration
-                oTileApi.configurationUi.setAsyncUiProvider(function () {
+                oTileApi.configurationUi.setAsyncUiProvider(() => {
                     return tileUtils.getConfigurationUi(
                         that.getView(),
                         "sap.ushell.components.tiles.action.Configuration"
-                    ).then(function (oConfigurationUi) {
+                    ).then((oConfigurationUi) => {
                         oTileApi.configurationUi.attachCancel(that.onCancelConfiguration.bind(null, oConfigurationUi));
                         oTileApi.configurationUi.attachSave(that.onSaveConfiguration.bind(null, oConfigurationUi, formatDisplayText));
                         return oConfigurationUi;
@@ -51,7 +51,7 @@ sap.ui.define([
 
         onPress: function (oEvent) {
             // trigger to show the configuration UI if the tile is pressed in Admin mode
-            var oTileApi = this.getView().getViewData().chip;
+            const oTileApi = this.getView().getViewData().chip;
             if (oTileApi.configurationUi.isEnabled()) {
                 oTileApi.configurationUi.display();
             }
@@ -61,11 +61,11 @@ sap.ui.define([
         // The target mapping tile is enhanced with mapping_signature and supported form_factors properties.
         onSaveConfiguration: function (oConfigurationView, fnFormatDisplayText) {
             // the deferred object required from the configurationUi contract
-            var oDeferred = jQuery.Deferred();
-            var oModel = oConfigurationView.getModel();
+            const oDeferred = jQuery.Deferred();
+            let oModel = oConfigurationView.getModel();
             // tile model placed into configuration model by getConfigurationUi
-            var oTileModel = oModel.getProperty("/tileModel");
-            var oTileApi = oConfigurationView.getViewData().chip;
+            const oTileModel = oModel.getProperty("/tileModel");
+            const oTileApi = oConfigurationView.getViewData().chip;
 
             // error handler
             function logErrorAndReject (oError) {
@@ -75,18 +75,18 @@ sap.ui.define([
 
             // If the input fields icon, semantic object and action are failing the input validations, then through an error message requesting the
             // user to enter/correct those fields
-            var bReject = tileUtils.checkTMInputOnSaveConfig(oConfigurationView);
+            const bReject = tileUtils.checkTMInputOnSaveConfig(oConfigurationView);
             if (bReject) {
                 oDeferred.reject("mandatory_fields_missing");
                 return oDeferred.promise();
             }
             // Before saving the model data, check if Mapping signature table contains duplicate parameter names
             // in this case the save will fail and all the data will be lost as this is the designer behavior.
-            var oBundle = tileUtils.getResourceBundle();
+            const oBundle = tileUtils.getResourceBundle();
             if (tileUtils.tableHasDuplicateParameterNames(oModel.getProperty("/config/rows"))) {
                 oDeferred.reject(oBundle.getText("configuration.signature.uniqueParamMessage.text"));
             } else if (!tileUtils.tableHasInvalidSapPrelaunchOperationValue(oModel.getProperty("/config/rows"))) {
-                var sSapPrelaunchOperationsTemplate = JSON.stringify([{
+                const sSapPrelaunchOperationsTemplate = JSON.stringify([{
                     type: "split",
                     source: "p1",
                     target: ["p2", "p3"]
@@ -99,11 +99,11 @@ sap.ui.define([
             } else { // only if the data is valid proceed with the save operation
                 // Decide according to special flag if the setting in form factor are default
                 // if so , the configuration should not be saved - this is crucial for the backend checks
-                var oFormFactor = oModel.getProperty("/config/formFactorConfigDefault") ? undefined : tileUtils.buildFormFactorsObject(oModel);
-                var sMappingSignature = tileUtils.getMappingSignatureString(oModel.getProperty("/config/rows"), oModel.getProperty("/config/isUnknownAllowed"));
-                var oMappingSignature = tileUtils.getMappingSignature(oModel.getProperty("/config/rows"), oModel.getProperty("/config/isUnknownAllowed"));
+                const oFormFactor = oModel.getProperty("/config/formFactorConfigDefault") ? undefined : tileUtils.buildFormFactorsObject(oModel);
+                const sMappingSignature = tileUtils.getMappingSignatureString(oModel.getProperty("/config/rows"), oModel.getProperty("/config/isUnknownAllowed"));
+                const oMappingSignature = tileUtils.getMappingSignature(oModel.getProperty("/config/rows"), oModel.getProperty("/config/isUnknownAllowed"));
                 // get the configuration to save from the model
-                var configToSave = {
+                const configToSave = {
                     semantic_object: (oModel.getProperty("/config/semantic_object") || "").trim(),
                     semantic_action: (oModel.getProperty("/config/semantic_action") || "").trim(),
                     display_title_text: (oModel.getProperty("/config/display_title_text") || "").trim(),
@@ -129,37 +129,37 @@ sap.ui.define([
                     signature: oMappingSignature
                 };
                 // use bag in order to store translatable properties
-                var tilePropertiesBag = oTileApi.bag.getBag("tileProperties");
+                const tilePropertiesBag = oTileApi.bag.getBag("tileProperties");
                 tilePropertiesBag.setText("display_title_text", configToSave.display_title_text);
 
                 // use configuration contract to write parameter values
                 oTileApi.writeConfiguration.setParameterValues({
                     tileConfiguration: JSON.stringify(configToSave)
                 },
-                    // success handler
-                    function () {
-                        var oConfigurationConfig = tileUtils.getActionConfiguration(oTileApi, false);
-                        var oTileConfig = tileUtils.getActionConfiguration(oTileApi, true);
-                        // switching the model under the tile -> keep the tile model
-                        oModel = new JSONModel({
-                            config: oConfigurationConfig,
-                            tileModel: oTileModel
-                        });
-                        oConfigurationView.setModel(oModel);
-                        // update model (no merge)
-                        oTileModel.setData({
-                            config: oTileConfig,
-                            displayText: fnFormatDisplayText(oTileConfig.semantic_object, oTileConfig.semantic_action)
-                        }, false);
-                        // Added for new LPD_CUST implementation
-                        tilePropertiesBag.save(
-                            // success handler
-                            function () {
-                                Log.debug("property bag 'tileProperties' saved successfully");
-                                oDeferred.resolve();
-                            }, logErrorAndReject // error handler
-                        );
-                    }, logErrorAndReject // error handler
+                // success handler
+                () => {
+                    const oConfigurationConfig = tileUtils.getActionConfiguration(oTileApi, false);
+                    const oTileConfig = tileUtils.getActionConfiguration(oTileApi, true);
+                    // switching the model under the tile -> keep the tile model
+                    oModel = new JSONModel({
+                        config: oConfigurationConfig,
+                        tileModel: oTileModel
+                    });
+                    oConfigurationView.setModel(oModel);
+                    // update model (no merge)
+                    oTileModel.setData({
+                        config: oTileConfig,
+                        displayText: fnFormatDisplayText(oTileConfig.semantic_object, oTileConfig.semantic_action)
+                    }, false);
+                    // Added for new LPD_CUST implementation
+                    tilePropertiesBag.save(
+                        // success handler
+                        () => {
+                            Log.debug("property bag 'tileProperties' saved successfully");
+                            oDeferred.resolve();
+                        }, logErrorAndReject // error handler
+                    );
+                }, logErrorAndReject // error handler
                 );
             }
             return oDeferred.promise();
@@ -168,12 +168,12 @@ sap.ui.define([
         // configuration cancel handler
         onCancelConfiguration: function (oConfigurationView) {
             // reload old configuration and display
-            var oViewData = oConfigurationView.getViewData();
-            var oModel = oConfigurationView.getModel();
+            const oViewData = oConfigurationView.getViewData();
+            const oModel = oConfigurationView.getModel();
             // tile model placed into configuration model by getConfigurationUi
-            var oTileModel = oModel.getProperty("/tileModel");
-            var oTileApi = oViewData.chip;
-            var oCurrentConfig = tileUtils.getActionConfiguration(oTileApi, false);
+            const oTileModel = oModel.getProperty("/tileModel");
+            const oTileApi = oViewData.chip;
+            const oCurrentConfig = tileUtils.getActionConfiguration(oTileApi, false);
             oConfigurationView.getModel().setData({
                 config: oCurrentConfig,
                 tileModel: oTileModel

package/src/main/js/sap/ushell/components/tiles/action/Configuration.controller.js

@@ -6,12 +6,11 @@ sap.ui.define([
     "sap/ushell/components/tiles/utils",
     "sap/ui/core/library",
     "sap/m/MessageBox"
-], function (Element, Controller, tileUtils, coreLibrary, MessageBox) {
+], (Element, Controller, tileUtils, coreLibrary, MessageBox) => {
     "use strict";
 
     // shortcut for sap.ui.core.ValueState
-    var ValueState = coreLibrary.ValueState;
-
+    const ValueState = coreLibrary.ValueState;
 
     return Controller.extend("sap.ushell.components.tiles.action.Configuration", {
         sEnterValuePlaceHolder: "",
@@ -33,14 +32,14 @@ sap.ui.define([
             tileUtils.checkTMInput(this.getView(), oControlEvent);
         },
         onInit: function () {
-            var oView = this.getView();
-            var oSemanticObjectSelector = oView.byId("semantic_objectInput");
-            var oRoleSelector = oView.byId("navigation_provider_roleInput");
-            var oInstanceSelector = oView.byId("navigation_provider_instanceInput");
-            var oAliasSelector = oView.byId("target_application_aliasInput");
-            var oActionSelector = oView.byId("semantic_actionInput");
-            var oTargetTypeSelector = oView.byId("targetTypeInput");
-            var oResourceModel = tileUtils.getResourceBundleModel();
+            const oView = this.getView();
+            const oSemanticObjectSelector = oView.byId("semantic_objectInput");
+            const oRoleSelector = oView.byId("navigation_provider_roleInput");
+            const oInstanceSelector = oView.byId("navigation_provider_instanceInput");
+            const oAliasSelector = oView.byId("target_application_aliasInput");
+            const oActionSelector = oView.byId("semantic_actionInput");
+            const oTargetTypeSelector = oView.byId("targetTypeInput");
+            const oResourceModel = tileUtils.getResourceBundleModel();
 
             oView.setModel(oResourceModel, "i18n");
             oView.setViewName("sap.ushell.components.tiles.action.Configuration");
@@ -51,25 +50,25 @@ sap.ui.define([
             tileUtils.createActionModel(this, oActionSelector);
             tileUtils.createNavigationProviderModel(this, oTargetTypeSelector);
             // make sure that the chose object is written back to the configuration
-            oSemanticObjectSelector.attachChange(function (oControlEvent) {
-                var sValue = oControlEvent.getSource().getValue();
+            oSemanticObjectSelector.attachChange((oControlEvent) => {
+                const sValue = oControlEvent.getSource().getValue();
                 oView.getModel().setProperty("/config/semantic_object", sValue);
             });
-            oRoleSelector.attachChange(function (oControlEvent) {
-                var sValue = oControlEvent.getSource().getValue();
+            oRoleSelector.attachChange((oControlEvent) => {
+                const sValue = oControlEvent.getSource().getValue();
                 oView.getModel().setProperty("/config/navigation_provider_role", sValue);
                 tileUtils.updateAliasModel(oView, oAliasSelector);
             });
-            oInstanceSelector.attachChange(function (oControlEvent) {
-                var sValue = oControlEvent.getSource().getValue();
+            oInstanceSelector.attachChange((oControlEvent) => {
+                const sValue = oControlEvent.getSource().getValue();
                 oView.getModel().setProperty("/config/navigation_provider_instance", sValue);
             });
-            oAliasSelector.attachChange(function (oControlEvent) {
-                var sValue = oControlEvent.getSource().getValue();
+            oAliasSelector.attachChange((oControlEvent) => {
+                const sValue = oControlEvent.getSource().getValue();
                 oView.getModel().setProperty("/config/target_application_alias", sValue);
             });
 
-            var oBundle = tileUtils.getResourceBundle();
+            const oBundle = tileUtils.getResourceBundle();
             this.sEnterValuePlaceHolder = oBundle.getText("configuration.signature.table.valueFieldLbl");
             this.sDuplicateErrorMsg = oBundle.getText("configuration.signature.uniqueParamMessage.text");
             this.sDuplicateErrorTitle = oBundle.getText("configuration.signature.uniqueParamMessage.title");
@@ -79,10 +78,10 @@ sap.ui.define([
         // This function applies table logic for the mapping signature structure according to the Mandatory check-box:
         // if mandatory is unselected: Value and isRegularExpression fields should be disabled and vice versa...
         handleMandatoryChange: function (oMandatoryCheckBox) {
-            var sId = oMandatoryCheckBox.getParameter("id");
-            var aParentCells = Element.getElementById(sId).getParent().getCells();
+            const sId = oMandatoryCheckBox.getParameter("id");
+            const aParentCells = Element.getElementById(sId).getParent().getCells();
 
-            var bIsMandatory = oMandatoryCheckBox.getParameter("selected");
+            const bIsMandatory = oMandatoryCheckBox.getParameter("selected");
 
             if (bIsMandatory) {
                 aParentCells[2].setEnabled(true); // Value field
@@ -103,28 +102,28 @@ sap.ui.define([
         },
 
         addRow: function () {
-            var oView = this.getView();
-            var oModel = oView.getModel();
-            var rows = oModel.getProperty("/config/rows");
+            const oView = this.getView();
+            const oModel = oView.getModel();
+            const rows = oModel.getProperty("/config/rows");
 
             // Init a row template for adding new empty row to the params table (mapping signature)
-            var newParamRow = tileUtils.getEmptyRowObj();
+            const newParamRow = tileUtils.getEmptyRowObj();
             rows.push(newParamRow);
             oModel.setProperty("/config/rows", rows);
         },
 
         deleteRow: function () {
-            var oView = this.getView();
-            var oModel = oView.getModel();
-            var rows = oModel.getProperty("/config/rows");
+            const oView = this.getView();
+            const oModel = oView.getModel();
+            const rows = oModel.getProperty("/config/rows");
 
-            var table = oView.byId("mappingSignatureTable");
-            var aSelectedItemsIndexes = table.getSelectedIndices();
-            var aSortedDescending = aSelectedItemsIndexes.sort(function (a, b) {
+            const table = oView.byId("mappingSignatureTable");
+            const aSelectedItemsIndexes = table.getSelectedIndices();
+            const aSortedDescending = aSelectedItemsIndexes.sort((a, b) => {
                 return b - a;
             }).slice();
 
-            for (var i = 0; i < aSortedDescending.length; i++) {
+            for (let i = 0; i < aSortedDescending.length; i++) {
                 table.removeSelectionInterval(aSortedDescending[i], aSortedDescending[i]);// Make sure to turn off the selection or it will pass to
                 // the next row.
                 rows.splice(aSortedDescending[i], 1); // There is a major assumption here that the index in the model is identical to the index in
@@ -135,11 +134,10 @@ sap.ui.define([
         // Will be called on change event of the name column in Parameters table.
         // 2 parameters cannot have the same name. (in case the user decide to ignore the error message, there is a second validation on the save)
         checkDuplicateNames: function (changeEvent) {
-
-            var oModel = this.getView().getModel();
-            var rows = oModel.getProperty("/config/rows");
-            var nameCol = Element.getElementById(changeEvent.getParameter("id"));
-            var sNewValue = changeEvent.getParameter("newValue");
+            const oModel = this.getView().getModel();
+            const rows = oModel.getProperty("/config/rows");
+            const nameCol = Element.getElementById(changeEvent.getParameter("id"));
+            const sNewValue = changeEvent.getParameter("newValue");
 
             if (sNewValue !== "" && !(/^[\w-/]+$/.test(sNewValue))) {
                 nameCol.setValueState(ValueState.Error); MessageBox.alert(this.sInvalidParmMsg, this.focusNameField.bind(nameCol), this.sDuplicateErrorTitle);
@@ -180,13 +178,13 @@ sap.ui.define([
             tileUtils.applicationAliasSelectOnValueHelpRequest(this, oEvent);
         },
         onFormFactorChange: function () {
-            var oModel = this.getView().getModel();
+            const oModel = this.getView().getModel();
             oModel.setProperty("/config/formFactorConfigDefault", false);
         },
 
         // Will be called on change event of the application type radio buttons.
         onApplicationTypeChange: function (oEvent) {
-            var oParameter = oEvent.getParameters();
+            const oParameter = oEvent.getParameters();
             if (oParameter.selectedItem) {
                 tileUtils.displayApplicationTypeFields(oParameter.selectedItem.getKey(), this.getView());
             }

package/src/main/js/sap/ushell/components/tiles/applauncher/Component.js

@@ -3,10 +3,10 @@
 sap.ui.define([
     "sap/ui/core/mvc/View",
     "sap/ui/core/UIComponent"
-], function (
+], (
     View,
     UIComponent
-) {
+) => {
     "use strict";
 
     return UIComponent.extend("sap.ushell.components.tiles.applauncher", {
@@ -15,7 +15,7 @@ sap.ui.define([
         },
 
         createContent: function () {
-            var oComponentData = this.getComponentData();
+            const oComponentData = this.getComponentData();
 
             return View.create({
                 viewName: "module:sap/ushell/components/tiles/applauncher/StaticTile.view",

package/src/main/js/sap/ushell/components/tiles/applauncher/Configuration.controller.js

@@ -4,10 +4,9 @@ sap.ui.define([
     "sap/ui/core/mvc/Controller",
     "sap/ushell/components/tiles/utils",
     "sap/ui/core/ListItem"
-], function (Controller, tileUtils, ListItem) {
+], (Controller, tileUtils, ListItem) => {
     "use strict";
 
-
     return Controller.extend("sap.ushell.components.tiles.applauncher.Configuration", {
         // checks given inputs
         onConfigurationInputChange: function (oControlEvent) {
@@ -16,22 +15,22 @@ sap.ui.define([
         // default semantic objects for static applauncher: blank
         aDefaultObjects: [{ obj: "", name: "" }],
         onInit: function () {
-            var oView = this.getView(),
-                oTargetUrl = oView.byId("targetUrl"),
-                oSemanticObjectSelector = oView.byId("navigation_semantic_objectInput"),
-                oActionSelector = oView.byId("navigation_semantic_actionInput"),
-                oResourceModel = tileUtils.getResourceBundleModel();
+            const oView = this.getView();
+            const oTargetUrl = oView.byId("targetUrl");
+            const oSemanticObjectSelector = oView.byId("navigation_semantic_objectInput");
+            const oActionSelector = oView.byId("navigation_semantic_actionInput");
+            const oResourceModel = tileUtils.getResourceBundleModel();
 
             oView.setModel(oResourceModel, "i18n");
-            var oBundle = tileUtils.getResourceBundle();
+            const oBundle = tileUtils.getResourceBundle();
             // set view name for identification in utils
             oView.setViewName("sap.ushell.components.tiles.applauncher.Configuration");
             tileUtils.createSemanticObjectModel(this, oSemanticObjectSelector, this.aDefaultObjects);
             tileUtils.createActionModel(this, oActionSelector);
 
             // make sure that the chose object is written back to the configuration
-            oSemanticObjectSelector.attachChange(function (oControlEvent) {
-                var sValue = oControlEvent.getSource().getValue();
+            oSemanticObjectSelector.attachChange((oControlEvent) => {
+                const sValue = oControlEvent.getSource().getValue();
                 oView.getModel().setProperty("/config/navigation_semantic_object", sValue);
             });
 
@@ -43,8 +42,8 @@ sap.ui.define([
                 formatter: targetUrlVisibilityFormatter,
                 path: "/config/navigation_use_semantic_object"
             });
-            //Adding list items URL and Intent to the Target Type in Tile Actions section
-            var oItem = new ListItem({ key: "URL", text: oBundle.getText("configuration.tile_actions.table.target_type.url") });
+            // Adding list items URL and Intent to the Target Type in Tile Actions section
+            let oItem = new ListItem({ key: "URL", text: oBundle.getText("configuration.tile_actions.table.target_type.url") });
             oView.byId("targetTypeCB").addItem(oItem);
             oItem = new ListItem({ key: "INT", text: oBundle.getText("configuration.tile_actions.table.target_type.intent") });
             oView.byId("targetTypeCB").addItem(oItem);
@@ -56,26 +55,26 @@ sap.ui.define([
 
         // forward semantic object value helper request to utils
         onValueHelpRequest: function (oEvent) {
-            //Third parameter is to differentiate whether it's Tile Actions icon field or general icon field. If it's true, then it's tile actions icon field, else general icon field.
+            // Third parameter is to differentiate whether it's Tile Actions icon field or general icon field. If it's true, then it's tile actions icon field, else general icon field.
             tileUtils.objectSelectOnValueHelpRequest(this, oEvent, false);
         },
         // forward semantic action value helper request to utils
         onActionValueHelpRequest: function (oEvent) {
-            //Third parameter is to differentiate whether it's Tile Actions icon field or general icon field. If it's true, then it's tile actions icon field, else general icon field.
+            // Third parameter is to differentiate whether it's Tile Actions icon field or general icon field. If it's true, then it's tile actions icon field, else general icon field.
             tileUtils.actionSelectOnValueHelpRequest(this, oEvent, false);
         },
         // change handler for check box
         onCheckBoxChange: function (oEvent) {
-            var oView = this.getView(),
-                oSemanticObjectSelector = oView.byId("navigation_semantic_objectInput"),
-                oModel = oSemanticObjectSelector.getModel(),
-                value = oEvent.getSource().getSelected();
+            const oView = this.getView();
+            const oSemanticObjectSelector = oView.byId("navigation_semantic_objectInput");
+            const oModel = oSemanticObjectSelector.getModel();
+            const value = oEvent.getSource().getSelected();
             oModel.setProperty("/enabled", value);
             tileUtils.checkInput(this.getView(), oEvent);
         },
         // forward icon value help request to utils
         onIconValueHelpRequest: function (oEvent) {
-            //Third parameter is to differentiate whether it's Tile Actions icon field or general icon field. If it's true, then it's tile actions icon field, else general icon field.
+            // Third parameter is to differentiate whether it's Tile Actions icon field or general icon field. If it's true, then it's tile actions icon field, else general icon field.
             tileUtils.iconSelectOnValueHelpRequest(this, oEvent, false);
         },
         // forward icon close request to utils
@@ -86,28 +85,28 @@ sap.ui.define([
         onSelectIconOk: function () {
             tileUtils.onSelectIconOk(this.getView());
         },
-        //This function applies table logic for the Action according to the Target Type:
-        //if Taregt Type is URL, then Action field should be disabled else if it's Intent, then the Action field should be enabled.
+        // This function applies table logic for the Action according to the Target Type:
+        // if Taregt Type is URL, then Action field should be disabled else if it's Intent, then the Action field should be enabled.
         handleTargetTypeChange: function (oTargetTypeComboBox) {
             tileUtils.onTargetTypeChange(oTargetTypeComboBox);
         },
 
-        //forward tile actions semantic object value helper request to utils
+        // forward tile actions semantic object value helper request to utils
         onTileActionValueHelp: function (oEvent) {
-            //Third parameter is to differentiate whether it's Tile Actions icon field or general icon field. If it's true, then it's tile actions icon field, else general icon field.
+            // Third parameter is to differentiate whether it's Tile Actions icon field or general icon field. If it's true, then it's tile actions icon field, else general icon field.
             tileUtils.objectSelectOnValueHelpRequest(this, oEvent, true);
         },
 
-        //forward icon value help request to utils
+        // forward icon value help request to utils
         onTileActionIconValueHelp: function (oEvent) {
-            //Third parameter is to differentiate whether it's Tile Actions icon field or general icon field. If it's true, then it's tile actions icon field, else general icon field.
+            // Third parameter is to differentiate whether it's Tile Actions icon field or general icon field. If it's true, then it's tile actions icon field, else general icon field.
             tileUtils.iconSelectOnValueHelpRequest(this, oEvent, true);
         },
-        //adds new row in the tile actions table
+        // adds new row in the tile actions table
         addRow: function () {
             tileUtils.addTileActionsRow(this.getView());
         },
-        //delets row in the tile actions table
+        // delets row in the tile actions table
         deleteRow: function () {
             tileUtils.deleteTileActionsRow(this.getView());
         }