@openui5/sap.ui.mdc 1.111.6 → 1.112.1

package/src/sap/ui/mdc/p13n/PropertyHelper.js

@@ -23,7 +23,7 @@ sap.ui.define([
 	 * @extends sap.ui.mdc.util.PropertyHelper
 	 *
 	 * @author SAP SE
-	 * @version 1.111.6
+	 * @version 1.112.1
 	 *
 	 * @private
 	 * @experimental

package/src/sap/ui/mdc/p13n/UIManager.js

@@ -33,7 +33,7 @@ sap.ui.define([
 	 * @extends sap.ui.base.Object
 	 *
 	 * @author SAP SE
-	 * @version 1.111.6
+	 * @version 1.112.1
 	 *
 	 * @private
 	 * @ui5-restricted sap.ui.mdc

package/src/sap/ui/mdc/p13n/panels/FilterPanel.js

@@ -39,7 +39,7 @@ sap.ui.define([
 	 * @extends sap.m.p13n.QueryPanel
 	 *
 	 * @author SAP SE
-	 * @version 1.111.6
+	 * @version 1.112.1
 	 *
 	 * @public
 	 * @experimental Since 1.107.

package/src/sap/ui/mdc/p13n/subcontroller/AdaptFiltersController.js

@@ -73,9 +73,11 @@ sap.ui.define([
     };
 
     AdaptFiltersController.prototype.update = function(oPropertyHelper) {
-        var oAdaptationData = this.mixInfoAndState(oPropertyHelper);
-        this._oPanel.setP13nData(oAdaptationData);
-        this.getAdaptationControl().getInbuiltFilter().createFilterFields();
+        if (this._oPanel) {
+            var oAdaptationData = this.mixInfoAndState(oPropertyHelper);
+            this._oPanel.setP13nData(oAdaptationData);
+            this.getAdaptationControl().getInbuiltFilter().createFilterFields();
+        }
     };
 
     AdaptFiltersController.prototype.mixInfoAndState = function(oPropertyHelper) {

package/src/sap/ui/mdc/p13n/subcontroller/ColumnWidthController.js

@@ -6,7 +6,7 @@
 
 sap.ui.define([
 	"./SelectionController",
-	"sap/ui/mdc/p13n/FlexUtil",
+	"sap/m/p13n/FlexUtil",
     'sap/m/p13n/modules/xConfigAPI',
 	"sap/base/util/merge"
 ], function (BaseController, FlexUtil, xConfigAPI, merge) {

package/src/sap/ui/mdc/p13n/subcontroller/FilterController.js

@@ -5,8 +5,8 @@
  */
 
 sap.ui.define([
-	'sap/ui/mdc/enum/ProcessingStrategy', 'sap/ui/mdc/condition/FilterOperatorUtil', './SelectionController', 'sap/ui/mdc/p13n/P13nBuilder', 'sap/ui/mdc/p13n/FlexUtil', 'sap/base/Log', 'sap/base/util/merge', 'sap/base/util/UriParameters'
-], function (ProcessingStrategy, FilterOperatorUtil, BaseController, P13nBuilder, FlexUtil, Log, merge, SAPUriParameters) {
+	'sap/ui/mdc/enum/ProcessingStrategy', 'sap/ui/mdc/condition/FilterOperatorUtil', './SelectionController', 'sap/ui/mdc/p13n/P13nBuilder', 'sap/base/Log', 'sap/base/util/merge', 'sap/base/util/deepEqual'
+], function (ProcessingStrategy, FilterOperatorUtil, BaseController, P13nBuilder, Log, merge, deepEqual) {
 	"use strict";
 
     var FilterController = BaseController.extend("sap.ui.mdc.p13n.subcontroller.FilterController", {
@@ -109,6 +109,7 @@ sap.ui.define([
         return this.getAdaptationControl().retrieveInbuiltFilter().then(function(oAdaptationFilterBar){
             oAdaptationFilterBar.setP13nData(oAdaptationData);
             oAdaptationFilterBar.setLiveMode(false);
+            oAdaptationFilterBar.setProperty("_useFixedWidth", false);
             oAdaptationFilterBar.getTitle = function() {
                 return sap.ui.getCore().getLibraryResourceBundle("sap.ui.mdc").getText("filter.PERSONALIZATION_DIALOG_TITLE");
             };
@@ -121,14 +122,16 @@ sap.ui.define([
     };
 
     FilterController.prototype.update = function(oPropertyHelper){
-        var oAdaptationData = this.mixInfoAndState(oPropertyHelper);
-        this._oPanel.setP13nData(oAdaptationData);
-        var oAdaptationControl = this.getAdaptationControl();
-        var oInbuiltFilter = oAdaptationControl && oAdaptationControl.getInbuiltFilter();
-
-        if (oInbuiltFilter) {
-            oInbuiltFilter.createFilterFields();
+        if (this._oPanel) {
+            var oAdaptationData = this.mixInfoAndState(oPropertyHelper);
+            this._oPanel.setP13nData(oAdaptationData);
+            var oAdaptationControl = this.getAdaptationControl();
+            var oInbuiltFilter = oAdaptationControl && oAdaptationControl.getInbuiltFilter();
+            if (oInbuiltFilter) {
+                oInbuiltFilter.createFilterFields();
+            }
         }
+
     };
 
     FilterController.prototype.getDelta = function(mPropertyBag) {
@@ -139,9 +142,153 @@ sap.ui.define([
                 }
             });
         }
-        return FlexUtil.getConditionDeltaChanges(mPropertyBag);
+        return getConditionDeltaChanges(mPropertyBag);
     };
 
+    /**
+    * Generates a set of changes based on the given arrays for a specified control
+    *
+    * @public
+    *
+    * @param {object} mDeltaInfo Map containing the necessary information to calculate the diff as change objects
+    * @param {array} mDeltaInfo.existingState An array describing the control state before a adaptation
+    * @param {array} mDeltaInfo.changedState An array describing the control state after a certain adaptation
+    * @param {object} mDeltaInfo.control Control instance which is being used to generate the changes
+    * @param {object} mDeltaInfo.changeOperations Map containing the changeOperations for the given Control instance
+    * @param {string} mDeltaInfo.changeOperations.add Name of the control specific 'add' changehandler
+    * @param {boolean} mDeltaInfo.applyAbsolute Indicates whether the appliance should also implicitly remove entries in case they are not provided in the new state
+    * @param {string} mDeltaInfo.changeOperations.remove Name of the control specific 'remove' changehandler
+    * @param {string} [mDeltaInfo.changeOperations.move] Name of the control specific 'move' changehandler
+    * @param {string} [mDeltaInfo.generator] Name of the change generator (E.g. the namespace of the UI creating the change object)
+    *
+    * @returns {array} Array containing the delta based created changes
+    */
+    var getConditionDeltaChanges = function(mDeltaInfo) {
+        var aConditionChanges = [];
+
+        var mNewConditionState = mDeltaInfo.changedState;
+        var mPreviousConditionState = mDeltaInfo.existingState;
+        var oAdaptationControl = mDeltaInfo.control;
+        var bAbsoluteAppliance = mDeltaInfo.hasOwnProperty("applyAbsolute") ? mDeltaInfo.applyAbsolute : true;
+        var aPropertyInfo = mDeltaInfo.propertyInfo;
+
+        for (var sFieldPath in mNewConditionState) {
+            var bValidProperty = _hasProperty(aPropertyInfo, sFieldPath);
+            if (!bValidProperty && oAdaptationControl.isA("sap.ui.mdc.Control") && oAdaptationControl.isPropertyHelperFinal()) {
+                Log.warning("property '" + sFieldPath + "' not supported");
+                continue;
+            }
+
+            var aFilterConditionChanges = _diffConditionPath(sFieldPath, mNewConditionState[sFieldPath], mPreviousConditionState[sFieldPath], oAdaptationControl, bAbsoluteAppliance);
+            aConditionChanges = aConditionChanges.concat(aFilterConditionChanges);
+        }
+
+        return aConditionChanges;
+    };
+
+    var _hasProperty = function(aPropertyInfo, sName) {
+        return aPropertyInfo.some(function(oProperty){
+            //First check unique name
+            var bValid = oProperty.name === sName || sName == "$search";
+
+            //Use path as Fallback
+            bValid = bValid ? bValid : oProperty.path === sName;
+
+            return bValid;
+        });
+    };
+
+    var createConditionChange = function(sChangeType, oControl, sFieldPath, oCondition) {
+        delete oCondition.filtered;
+        var oConditionChange = {
+            selectorElement: oControl,
+            changeSpecificData: {
+                changeType: sChangeType,
+                content: {
+                    name: sFieldPath,
+                    condition: oCondition
+                }
+            }
+        };
+
+        return oConditionChange;
+    };
+
+    /**
+    * Generates a set of changes based on the given conditions
+    *
+    * @public
+    * @param {array} sFieldPath The relevant fieldPath
+    * @param {array} aConditions The conditions after they have been changed
+    * @param {function} aOrigShadowConditions The conditions before they have been changed
+    * @param {object} oControl Control instance which is being used to generate the changes
+    * @param {boolean} [bAbsoluteAppliance] Indicates whether the appliance should also implicitly remove entries in case they are not provided in the new state
+    *
+    * @returns {array} Array containing the delta based created changes
+    */
+    var _diffConditionPath = function(sFieldPath, aConditions, aOrigShadowConditions, oControl, bAbsoluteAppliance){
+        var oChange, aChanges = [];
+        var aOrigConditions = merge([], aConditions);
+        var aShadowConditions = aOrigShadowConditions ? merge([], aOrigShadowConditions) : [];
+
+
+        if (deepEqual(aConditions, aShadowConditions)) {
+            return aChanges;
+        }
+
+        var fnRemoveSameConditions = function(aConditions, aShadowConditions){
+            var bRunAgain;
+
+            do  {
+                bRunAgain = false;
+
+                for (var i = 0; i < aConditions.length; i++) {
+
+                    var oNewCondition = aConditions[i];
+                    var nConditionIdx = FilterOperatorUtil.indexOfCondition(oNewCondition, aShadowConditions);
+                    if (nConditionIdx > -1) {
+
+                        aConditions.splice(i, 1);
+
+                        if (bAbsoluteAppliance) {
+                            aShadowConditions.splice(nConditionIdx, 1);
+                        }
+
+                        bRunAgain = true;
+                        break;
+                    }
+                }
+            }  while (bRunAgain);
+        };
+
+        fnRemoveSameConditions(aConditions, aShadowConditions);
+
+        if ((aConditions.length > 0) || (aShadowConditions.length > 0)) {
+
+            aShadowConditions.forEach(function(oCondition) {
+                //In case of absolute appliance always remove, in case of explicit appliance only remove if explicitly given in the new state via filtered=false
+                var iNewCondition = FilterOperatorUtil.indexOfCondition(oCondition, aOrigConditions);
+                var bNewConditionExplicitlyRemoved = iNewCondition > -1 && aOrigConditions[iNewCondition].filtered === false;
+                if (bAbsoluteAppliance || bNewConditionExplicitlyRemoved) {
+                    oChange = createConditionChange("removeCondition", oControl, sFieldPath, oCondition);
+                    aChanges.push(oChange);
+                }
+            });
+
+            aConditions.forEach(function(oCondition) {
+                if (bAbsoluteAppliance || (!oCondition.hasOwnProperty("filtered") || oCondition.filtered !== false)) {
+                    oChange = createConditionChange("addCondition", oControl, sFieldPath, oCondition);
+                    aChanges.push(oChange);
+                }
+            });
+
+        }
+
+        return aChanges;
+    };
+
+
+
     FilterController.prototype.model2State = function() {
         var oItems = {},
             oFilter = this.getCurrentState();
@@ -167,7 +314,7 @@ sap.ui.define([
         });
 
         P13nBuilder.sortP13nData({
-            visible: new SAPUriParameters(window.location.search).getAll("sap-ui-xx-filterQueryPanel")[0] === "true" ? "active" : null,//FIXME: remove with URL parameter
+            visible: "active",
             position: undefined
         }, oP13nData.items);
 

package/src/sap/ui/mdc/p13n/subcontroller/LinkPanelController.js

@@ -88,10 +88,10 @@ sap.ui.define([
 
         var oLinkItem = sap.ui.getCore().byId(sLinkItemId);
 
-        var aChanges = [];
+        var oAddRemoveChange;
 
         if (!oLinkItem) {
-            aChanges.push({
+            oAddRemoveChange = {
                 selectorElement: oControl,
                 changeSpecificData: {
                     changeType: "createItem",
@@ -99,18 +99,18 @@ sap.ui.define([
                         selector: sLinkItemId
                     }
                 }
-            });
+            };
         } else {
-            aChanges.push({
+            oAddRemoveChange = {
                 selectorElement: oLinkItem,
                 changeSpecificData: {
                     changeType: vOperations === "hideItem" ? "hideItem" : "revealItem",
                     content: {}
                 }
-            });
+            };
         }
 
-        return aChanges;
+        return oAddRemoveChange;
     };
 
     LinkPanelController.prototype.mixInfoAndState = function(oPropertyHelper) {
@@ -145,7 +145,7 @@ sap.ui.define([
         return oP13nData;
     };
 
-    LinkPanelController.prototype._createMoveChange = function(sId, sPropertyName, iNewIndex, sMoveOperation, oControl, bPersistId) {
+    LinkPanelController.prototype._createMoveChange = function(sPropertyName, iNewIndex, sMoveOperation, oControl) {
         return {
             selectorElement: oControl,
             changeSpecificData: {

package/src/sap/ui/mdc/p13n/subcontroller/SelectionController.js

@@ -57,7 +57,7 @@ sap.ui.define([
 		return SelectionController.prototype.getDelta.apply(this, arguments);
 	};
 
-	MDCSelectionController.prototype._createMoveChange = function(sId, sPropertykey, iNewIndex, sMoveOperation, oControl, bPersistId){
+	MDCSelectionController.prototype._createMoveChange = function(sPropertykey, iNewIndex, sMoveOperation, oControl){
 		var oMoveChange =  {
 			selectorElement: oControl,
 			changeSpecificData: {

package/src/sap/ui/mdc/table/ColumnSettings.js

@@ -19,7 +19,7 @@ sap.ui.define([
 	 *
 	 * @class The table type info class for the metadata-driven table.
 	 * @extends sap.ui.core.Element
-	 * @version 1.111.6
+	 * @version 1.112.1
 	 * @author SAP SE
 	 * @constructor
 	 * @experimental

package/src/sap/ui/mdc/table/CreationRow.js

@@ -21,7 +21,7 @@ sap.ui.define([
 	 * {@link sap.ui.mdc.table.Column#getCreationTemplate creationTemplate} aggregation of the {@link sap.ui.mdc.table.Column}.
 	 * @extends sap.ui.core.Element
 	 * @author SAP SE
-	 * @version 1.111.6
+	 * @version 1.112.1
 	 * @constructor
 	 * @experimental
 	 * @private

package/src/sap/ui/mdc/table/GridTableType.js

@@ -58,6 +58,16 @@ sap.ui.define([
 				 * Number of indices which can be selected in a range.
 				 * Accepts positive integer values. If set to 0, the selection limit is disabled, and the Select All checkbox appears instead of the
 				 * Deselect All button.
+				 *
+				 * <b>Note:</b> To avoid severe performance problems, the limit should only be set to 0 in the following cases:
+				 * <ul>
+				 *   <li>With client-side models</li>
+				 *   <li>With server-side models if they are used in client mode</li>
+				 *   <li>If the entity set is small</li>
+				 * </ul>
+				 *
+				 * In other cases, we recommend to set the limit to at least double the value of the {@link sap.ui.mdc.Table#getThreshold threshold}
+				 * property of the table.
 				 */
 				selectionLimit: {
 					type: "int",
@@ -69,6 +79,13 @@ sap.ui.define([
 				showHeaderSelector: {
 					type: "boolean",
 					defaultValue: true
+				},
+				/**
+				 * Defines the number of fixed columns in the inner table.
+				 */
+				fixedColumnCount: {
+					type: "int",
+					defaultValue: 0
 				}
 			}
 		}
@@ -116,6 +133,8 @@ sap.ui.define([
 			if (oMultiSelectionPlugin) {
 				oMultiSelectionPlugin.setShowHeaderSelector(vValue);
 			}
+		} else if (sProperty === "fixedColumnCount") {
+			oGridTable.setFixedColumnCount(vValue);
 		}
 	};
 

package/src/sap/ui/mdc/table/PropertyHelper.js

@@ -108,7 +108,7 @@ sap.ui.define([
 	 * @extends sap.ui.mdc.util.PropertyHelper
 	 *
 	 * @author SAP SE
-	 * @version 1.111.6
+	 * @version 1.112.1
 	 *
 	 * @private
 	 * @experimental

package/src/sap/ui/mdc/table/ResponsiveColumnSettings.js

@@ -24,7 +24,7 @@ sap.ui.define([
 	 *
 	 * @class The table type info class for the metadata-driven table.
 	 * @extends sap.ui.mdc.table.ColumnSettings
-	 * @version 1.111.6
+	 * @version 1.112.1
 	 * @author SAP SE
 	 * @private
 	 * @constructor

package/src/sap/ui/mdc/table/RowActionItem.js

@@ -24,7 +24,7 @@ sap.ui.define([
 	 * The <code>RowActionItem</code> control represents a action for a row.
 	 * This control can only be used in the context of <code>sap.ui.mdc.Table</code> control to define row actions.
 	 * @extends sap.ui.core.Element
-	 * @version 1.111.6
+	 * @version 1.112.1
 	 *
 	 * @constructor
 	 * @experimental

package/src/sap/ui/mdc/table/RowSettings.js

@@ -21,7 +21,7 @@ sap.ui.define([
 	 * The <code>RowSettings</code> control is used to configure a row.
 	 * This control can only be used in the context of the <code>sap.ui.mdc.Table</code> control to define row settings.
 	 * @extends sap.ui.core.Element
-	 * @version 1.111.6
+	 * @version 1.112.1
 	 *
 	 * @constructor
 	 * @experimental

package/src/sap/ui/mdc/table/V4AnalyticsPropertyHelper.js

@@ -80,7 +80,7 @@ sap.ui.define([
 	 * @extends sap.ui.mdc.table.PropertyHelper
 	 *
 	 * @author SAP SE
-	 * @version 1.111.6
+	 * @version 1.112.1
 	 *
 	 * @private
 	 * @experimental

package/src/sap/ui/mdc/table/menu/Item.js

@@ -5,9 +5,11 @@
  */
 sap.ui.define([
 	"sap/ui/core/Core",
+	"sap/ui/Device",
 	"sap/m/table/columnmenu/Item"
 ], function(
 	Core,
+	Device,
 	ItemBase
 ) {
 	"use strict";
@@ -37,6 +39,10 @@ sap.ui.define([
 		}
 
 		return oTable.getEngine().uimanager.create(oTable, [sKey]).then(function(aPanels) {
+			if (!Device.system.phone) {
+				aPanels[0].setProperty("_useFixedWidth", true);
+			}
+
 			this.setContent(aPanels[0]);
 			this.setLabel(oController.getUISettings().title);
 

package/src/sap/ui/mdc/themes/base/Chart.less

@@ -40,4 +40,4 @@
 .sapUiMDCChart .sapUiMDCChartTempInnerText {
     justify-content: center;
     align-content: center;
-}
+}

package/src/sap/ui/mdc/themes/base/ValueHelpDialog.less

@@ -1,3 +1,7 @@
+.sapMdcValueHelpPanel {
+    background-color: @sapUiBaseBG;
+}
+
 // give all panels on the valueHelpPanel a left/right padding
 .sapMdcValueHelpPanel .sapMPanel {
 	padding-top: 0.5rem;
@@ -165,7 +169,7 @@
 	height: 200px !important;
 }
 
-.sapMdcValueHelpPanelTableBox .sapUiMdcTable .sapMTitleTB {
+.sapMdcValueHelpPanelTableBox .sapUiMdcTable .sapMTB > .sapMTitle {
 	font-size: 1rem;
 }