@sapui5/sap.suite.ui.generic.template 1.136.6 → 1.136.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sapui5/sap.suite.ui.generic.template",
-  "version": "1.136.6",
+  "version": "1.136.7",
   "description": "SAPUI5 Library sap.suite.ui.generic.template",
   "keywords": [
     "sapui5",

package/src/sap/suite/ui/generic/template/.library

@@ -7,7 +7,7 @@
 
 		(c) Copyright 2009-2015 SAP SE. All rights reserved
 	</copyright>
-	<version>1.136.6</version>
+	<version>1.136.7</version>
 
 	<documentation>Library with generic Suite UI templates.</documentation>
 

package/src/sap/suite/ui/generic/template/AnalyticalListPage/manifest.json

@@ -8,7 +8,7 @@
 		"i18n": "i18n/i18n.properties",
 		"applicationVersion": {
 			"__comment": "applicationVersion oder componentversion??",
-			"version": "1.136.6"
+			"version": "1.136.7"
 		},
 		"title": "{{TITLE}}",
 		"description": "{{DESCRIPTION}}",

package/src/sap/suite/ui/generic/template/Canvas/manifest.json

@@ -8,7 +8,7 @@
 		"i18n": "i18n/i18n.properties",
 		"applicationVersion": {
 			"__comment": "applicationVersion oder componentversion??",
-			"version": "1.136.6"
+			"version": "1.136.7"
 		},
 		"title": "Canvas",
 		"description": "Canvas Page",

package/src/sap/suite/ui/generic/template/ListReport/controller/IappStateHandler.js

@@ -84,12 +84,12 @@ sap.ui.define([
 		var aPageVariantControlStateWrappers = []; // controls handled from page variant management if used: smartTable, searchfield, partly multiple views
 
 		// SmartTable state: Table Settings and (in case of control level variant management) selected variant and whether it's dirty
-		// In case of multipleViews with multiple tables (mode "multi"), each tab has an own table or chart, which has an own state, so multipleViewsHandler has to take care of 
+		// In case of multipleViews with multiple tables (mode "multi"), each tab has an own table or chart, which has an own state, so multipleViewsHandler has to take care of
 		// storing/restoring their states (in its part of the appState stored in genericData.tableTabData)
-		// In this case, mutliViewsHandler exchanges oState.oPresentationControlHandler (setting it always to the current visible one), so don't rely on that to get the wrapper. 
+		// In this case, mutliViewsHandler exchanges oState.oPresentationControlHandler (setting it always to the current visible one), so don't rely on that to get the wrapper.
 		// Using the id (without providing optional parameter sQuickVariantKey) returns the smartTable only in single table case.
 		var oSmartTable = oController.byId(StableIdHelper.getStableId({type: "ListReportTable", subType: "SmartTable"}));
-		if (oSmartTable){ // in multipleViews case (with multiple tables) currently multipleViews handler handles complete state information 
+		if (oSmartTable){ // in multipleViews case (with multiple tables) currently multipleViews handler handles complete state information
 			aControlStateWrappers.push(oTemplateUtils.oCommonUtils.getControlStateWrapper(oSmartTable));
 		}
 
@@ -128,7 +128,7 @@ sap.ui.define([
 
 		// List of handlers to react on (app or adaptation) extension state changes. Only one entry expected (handler in  SFB wrapper). Currently, extensions are only possible in
 		// SFB - if extensions at other places are needed, the similar handler might differ depending on whether page variant management is used.
-		var aExtensionStateChangeHandlers = []; 
+		var aExtensionStateChangeHandlers = [];
 		function customAppStateChange() {
 			aExtensionStateChangeHandlers.forEach(function(fnHandler) {
 				fnHandler();
@@ -239,7 +239,7 @@ sap.ui.define([
 
 
 		// Wrapper to control whether data is loaded
-		
+
 		function fnGetDataLoadedWrapper(){
 			// Wrapper to control whether data is expected to be loaded - controls the state, but does not trigger loading data on restore!
 			var oTemplatePrivateModel = oTemplateUtils.oComponentUtils.getTemplatePrivateModel();
@@ -247,15 +247,15 @@ sap.ui.define([
 			// - startup with iAppState: anyway setState will be called
 			// - other startup cases: applyInitialLoadBehavior calls setState explicitly
 			// - navigation
-			
+
 			function fnSetState(bState){
 				oTemplatePrivateModel.setProperty("/generic/bDataAreShownInTable", bState);
 			}
-			
+
 			function fnGetState(){
 				return oTemplatePrivateModel.getProperty("/generic/bDataAreShownInTable");
 			}
-			
+
 			function fnSetDataShown(bDataShown, fnHandler){
 				if (bDataShown === fnGetState()){
 					return;
@@ -263,7 +263,7 @@ sap.ui.define([
 				fnSetState(bDataShown);
 				fnHandler();
 			}
-			
+
 			return {
 				getLocalId: function(){
 					return "$dataLoaded";
@@ -271,7 +271,7 @@ sap.ui.define([
 				setState: fnSetState,
 				getState: fnGetState,
 				attachStateChanged: function(fnHandler){
-					// changing from data not loaded to data loaded: 
+					// changing from data not loaded to data loaded:
 					// - when SFB triggers search
 					oState.oSmartFilterbar.attachSearch(fnSetDataShown.bind(null, true, fnHandler));
 
@@ -286,7 +286,7 @@ sap.ui.define([
 				}
 			};
 		}
-		
+
 		var oDataLoadedWrapper = fnGetDataLoadedWrapper();
 		aControlStateWrappers.push(oDataLoadedWrapper);
 //		oDataLoadedWrapper.attachStateChanged(changeIappState);
@@ -396,7 +396,7 @@ sap.ui.define([
 			if (Device.system.phone) {
 				collapseHeader();
 			}
-			// Ideally, oDataLoadedWrapper should ensure to keep its state correct. However, currently we have no means to attach to the command - i.e. oDataLoadedWrapper only 
+			// Ideally, oDataLoadedWrapper should ensure to keep its state correct. However, currently we have no means to attach to the command - i.e. oDataLoadedWrapper only
 			// attaches to SFB's search event, which is only triggered when user presses the go button. When keyboard shortcut is used instead, this handler would be called (as it
 			// is attached to the command), but SFB's search event would not be triggered. As workaround, we additionally set data loaded state to true here and inform explicitly
 			// the the state has changed.
@@ -440,7 +440,7 @@ sap.ui.define([
 				}
 			}
 		}
-		
+
 		/*
 			This method is called when an LR app is the target of an external navigation and the XAppState data contains a presentationVariant.
 			The sorting from this presentationVariant is then applied to the table.
@@ -460,13 +460,13 @@ sap.ui.define([
 			fnAdaptOtherControlsToAppState(oAppData.controlStates);
 			if (areDataShownInTable()){
 				// fnAdaptOtherControlsToAppState only (synchronously) sets the state including the information whether data should be loaded - if this is the case, the actual loading
-				// (which happens asynchronous of course) still needs to be triggered 
+				// (which happens asynchronous of course) still needs to be triggered
 				loadData();
 			} else {
 				// hide placeholder already here, if no data is to be loaded - in case data is loaded, it will be hidden in data received event
 				oTemplateUtils.oComponentUtils.hidePlaceholder();
-			} 
-			
+			}
+
 			// special case: when restoring an old app state with data loaded, but in the meantime a filter not set in that state has been changed to mandatory, SFB.search would not
 			// trigger a request (but instead only mark that filter) - thus hiding placeholder now to avoid it to stay forever
 			// TODO: refactor: Still in that case our internal data shows data are loaded - ideally, that should not be the case
@@ -603,7 +603,7 @@ sap.ui.define([
 
 		function fnAdaptToAppStateStartUpWithParameters(oAppData, oURLParameters, sPreferredQuickVariantSelectionKey){
 			handleVariantIdPassedViaURLParams(oURLParameters);
-			
+
 			var oSFBUiState = oState.oSmartFilterbar.getUiState();
 			var oSFBSemanticDates = oSFBUiState.getSemanticDates();
 			//oStartupObject to be passed to the extension where urlParameters and selectedQuickVariantSelectionKey are optional
@@ -766,23 +766,24 @@ sap.ui.define([
 			default:
 				throw new FeError(sClassName, "Invalid navigation type: " + sNavType);
 			}
-			
+
 			// common to all startup cases (except iAppState)
 			if (areDataShownInTable()){
 				// trigger search if needed
 				oState.oSmartFilterbar.search();
-				// For desktop devices, expand the header for Standard and Custom variants and for tablet and mobile devices, 
+				// For desktop devices, expand the header for Standard and Custom variants and for tablet and mobile devices,
 				// collapse the header only if search is triggered.
 				if (Device.system.desktop) {
 					oTemplateUtils.oCommonUtils.getControlStateWrapper(oController.byId(StableIdHelper.getStableId({type: "ListReportPage", subType: "DynamicPage"}))).setHeaderState(oController, true);
 				} else {
 					collapseHeader();
 				}
-			} else {
-				// if no data are loaded, place holder has to be hidden now - otherwise it will be hidden when data is received 
+			}
+			if (!areDataShownInTable() || Object.keys(oState.oSmartFilterbar.verifySearchAllowed()).length > 0) {
+				// if no data are loaded or search is not allowed, place holder has to be hidden now - otherwise it will be hidden when data is received
 				oTemplateUtils.oComponentUtils.hidePlaceholder();
 			}
-			
+
 			// ensure first iAppState is created
 			changeIappState();
 		}
@@ -999,14 +1000,14 @@ sap.ui.define([
 		}
 
 		// provide data load settings including defaulting
-		// ideally, this should be implemented in a generic way in template assembler, so that generated getters also return default values for objects if not explicitely set in 
+		// ideally, this should be implemented in a generic way in template assembler, so that generated getters also return default values for objects if not explicitely set in
 		// manifest (i.e. it should be sufficient to define the default in component - no need to individually implement defaulting!)
 		function getDataLoadSettings(){
 			// general default
 			var oDefaultDataLoadSettings = {loadDataOnAppLaunch: "ifAnyFilterExist"};
 
 			// intension is boolean, but unfortunately faulty values are (historically) not treated consistently
-			var bEnableAutoBindingMultiViews = oState.oMultipleViewsHandler.getOriginalEnableAutoBinding(); 
+			var bEnableAutoBindingMultiViews = oState.oMultipleViewsHandler.getOriginalEnableAutoBinding();
 
 			// if multiple views settings is not defined (also the case in single views case), general default is taken. Unlike other faulty values, null is treated like undefined
 			if (bEnableAutoBindingMultiViews !== undefined && bEnableAutoBindingMultiViews !== null){
@@ -1016,13 +1017,13 @@ sap.ui.define([
 
 			var oManifestDataLoadSettings = oController.getOwnerComponent().getDataLoadSettings();
 			// (historically) explicit setting empty string (not an allowed value according to definition in component!) is treated like undefined (including adopting multiple views
-			// settings. Other not allowed values (any other sting) were ignored, i.e. returning undefined from getInitialLoadBehaviourSettings, thus setting undefined to 
-			// oSmartVariantManagement.setExecuteOnStandard (which actually only sets the default) and finally using the value returned from 
+			// settings. Other not allowed values (any other sting) were ignored, i.e. returning undefined from getInitialLoadBehaviourSettings, thus setting undefined to
+			// oSmartVariantManagement.setExecuteOnStandard (which actually only sets the default) and finally using the value returned from
 			// oSmartVariantManagement.getExecuteOnStandard (only different from overall default (false), if user has explicitly set it)
 			if (oManifestDataLoadSettings && oManifestDataLoadSettings.loadDataOnAppLaunch === ""){
 				oManifestDataLoadSettings.loadDataOnAppLaunch = undefined;
 			}
-			
+
 			// explicit setting has highest priority
 			return extend(oDefaultDataLoadSettings, oManifestDataLoadSettings);
 		}
@@ -1031,7 +1032,7 @@ sap.ui.define([
 		// - sets default value for flag whether standard variant should be execute on select
 		// - determines whether we actually should load data
 		function applyInitialLoadBehavior(bDataLoadCausedByNavigation){
-			// cases definitely determining to load data initially 
+			// cases definitely determining to load data initially
 			// - worklist
 			// - livemode
 			// - master detail (i.e. bLoadListAndFirstEntryOnStartup is set)
@@ -1040,17 +1041,17 @@ sap.ui.define([
 
 			var sLoadBehaviour = getDataLoadSettings().loadDataOnAppLaunch;
 			if (!oSmartVariantManagement || oController.getOwnerComponent().getVariantManagementHidden()){
-				// No VM -> 
+				// No VM ->
 				bShouldDataBeLoaded = bShouldDataBeLoaded || sLoadBehaviour === "always";
 				bShouldDataBeLoaded = bShouldDataBeLoaded || (sLoadBehaviour === "ifAnyFilterExist" && oSmartFilterbar.getFiltersWithValues().length > 0);
 				// behavior of last patch set would translate to the following - seems not to be correct (in case of setting ifAnyFilterExist but no filters provided)
 				//				bShouldDataBeLoaded = bShouldDataBeLoaded || sLoadBehaviour !== "never";
 			} else {
-				// in case of a (visible) SVM, we need to set the default value 
-				// default value for standard variant 
+				// in case of a (visible) SVM, we need to set the default value
+				// default value for standard variant
 				// - true if one of the conditions above is fulfilled
-				// - or for any manifest setting but "never" 
-				//		(Remark: "ifAnyFilterExist" leads to mark the standard variant true, even if no filter is provided - this looks ok for startup, as the text shown to the user 
+				// - or for any manifest setting but "never"
+				//		(Remark: "ifAnyFilterExist" leads to mark the standard variant true, even if no filter is provided - this looks ok for startup, as the text shown to the user
 				//		contains the condition, but if reselecting the standard variant, it  will also select without filters!
 				//		TODO: check, how to solve that inconsistency)
 				// - but in both cases not, if a mandatory filter is missing

package/src/sap/suite/ui/generic/template/ListReport/manifest.json

@@ -8,7 +8,7 @@
 		"i18n": "i18n/i18n.properties",
 		"applicationVersion": {
 			"__comment": "applicationVersion oder componentversion??",
-			"version": "1.136.6"
+			"version": "1.136.7"
 		},
 		"title": "{{TITLE}}",
 		"description": "{{DESCRIPTION}}",

package/src/sap/suite/ui/generic/template/ObjectPage/manifest.json

@@ -6,7 +6,7 @@
 		"type": "component",
 		"i18n": "i18n/i18n.properties",
 		"applicationVersion": {
-			"version": "1.136.6"
+			"version": "1.136.7"
 		},
 		"title": "{{TITLE}}",
 		"description": "{{DESCRIPTION}}",

package/src/sap/suite/ui/generic/template/QuickCreate/manifest.json

@@ -6,7 +6,7 @@
 		"type": "component",
 		"i18n": "i18n/i18n.properties",
 		"applicationVersion": {
-			"version": "1.136.6"
+			"version": "1.136.7"
 		},
 		"title": "{{TITLE}}",
 		"description": "{{DESCRIPTION}}",

package/src/sap/suite/ui/generic/template/QuickView/manifest.json

@@ -6,7 +6,7 @@
 	"type": "component",
 	"i18n": "i18n/i18n.properties",
 	"applicationVersion": {
-		"version": "1.136.6"
+		"version": "1.136.7"
 	},
 	"title": "{{TITLE}}",
 	"description": "{{DESCRIPTION}}",

package/src/sap/suite/ui/generic/template/lib/AppComponent.js

@@ -950,7 +950,7 @@ sap.ui.define([
 	 * @public
 	 * @extends sap.ui.core.UIComponent
 	 * @author SAP SE
-	 * @version 1.136.6
+	 * @version 1.136.7
 	 * @name sap.suite.ui.generic.template.lib.AppComponent
 	 */
 	var oAppComponent = UIComponent.extend("sap.suite.ui.generic.template.lib.AppComponent", {

package/src/sap/suite/ui/generic/template/lib/navigation/NavigationController.js

@@ -3216,7 +3216,7 @@ sap.ui.define(["sap/ui/base/Object",
 	 * @param {sap.suite.ui.generic.template.lib.AppComponent} oAppComponent The AppComponent instance
 	 * @public
 	 * @extends sap.ui.base.Object
-	 * @version 1.136.6
+	 * @version 1.136.7
 	 * @since 1.30.0
 	 * @alias sap.suite.ui.generic.template.lib.NavigationController
 	 */

package/src/sap/suite/ui/generic/template/library.js

@@ -66,7 +66,7 @@ sap.ui.define([
 		interfaces: [],
 		controls: [],
 		elements: [],
-		version: "1.136.6",
+		version: "1.136.7",
 		extensions: {
 			//Configuration used for rule loading of Support Assistant
 			"sap.ui.support": {