@sapui5/sap.cux.home 1.140.0 → 1.142.0

package/src/sap/cux/home/NewsPanel.ts

@@ -209,6 +209,7 @@ export default class NewsPanel extends BaseNewsPanel {
 	private defaultUrl!: string;
 	private _noUpdatesNewsFeed: boolean = false; // flag to check if no updates news tile is shown or not
 	private _endUserResponse!: ODataResponse;
+	private defaultNewsItems?: INewsFeed[] | null;
 
 	static readonly metadata: MetadataOptions = {
 		library: "sap.cux.home",
@@ -656,7 +657,6 @@ export default class NewsPanel extends BaseNewsPanel {
 			this.oNewsTile.removeStyleClass("sapUiSmallMarginTop");
 		} else {
 			this.oNewsTile.addStyleClass("sapUiSmallMarginTop");
-			this.getNewsWrapper().setWidth("100%");
 		}
 	}
 
@@ -676,11 +676,11 @@ export default class NewsPanel extends BaseNewsPanel {
 					let oTile;
 					if (newsInfo.getElementsByTagName("link").length > 0) {
 						oTile = new NewsItem(recycleId(`${sId}-news-item`), {
-							url: String(this.getPropertyValue(newsInfo, "link")),
-							title: String(this.getPropertyValue(newsInfo, "title")),
-							subTitle: String(this.getPropertyValue(newsInfo, "description")),
-							imageUrl: String(this.getPropertyValue(newsInfo, "imageUrl")),
-							footer: this.formatDate(String(this.getPropertyValue(newsInfo, "pubDate")))
+							url: String(this.getPropertyValue(newsInfo, "link") ?? ""),
+							title: String(this.getPropertyValue(newsInfo, "title") ?? ""),
+							subTitle: String(this.getPropertyValue(newsInfo, "description") ?? ""),
+							imageUrl: String(this.getPropertyValue(newsInfo, "imageUrl") ?? ""),
+							footer: this.formatDate(String(this.getPropertyValue(newsInfo, "pubDate") ?? ""))
 						});
 						this.addAggregation("newsItems", oTile, true);
 					} else {
@@ -688,12 +688,12 @@ export default class NewsPanel extends BaseNewsPanel {
 						let newsId = sGroupId && sGroupId.trim() ? `${sId}-newsgroup-${sGroupId}` : `${sId}-newsgroup`;
 						let subTitleNews = this.getPropertyValue(newsInfo, "subTitle") ?? "";
 						oTile = new NewsGroup(recycleId(newsId), {
-							title: String(this.getPropertyValue(newsInfo, "title")),
+							title: String(this.getPropertyValue(newsInfo, "title") ?? ""),
 							subTitle: subTitleNews || this._i18nBundle.getText("newsFeedDescription"),
-							imageUrl: String(this.getPropertyValue(newsInfo, "imageUrl")),
-							footer: String(this.getPropertyValue(newsInfo, "footer")),
+							imageUrl: String(this.getPropertyValue(newsInfo, "imageUrl") ?? ""),
+							footer: String(this.getPropertyValue(newsInfo, "footer") ?? ""),
 							priority: this.getPropertyValue(newsInfo, "priority") as Priority,
-							priorityText: String(this.getPropertyValue(newsInfo, "priorityText"))
+							priorityText: String(this.getPropertyValue(newsInfo, "priorityText") ?? "")
 						});
 						this.addAggregation("newsGroup", oTile, true);
 					}
@@ -715,7 +715,7 @@ export default class NewsPanel extends BaseNewsPanel {
 	private async extractAllImageUrls(oDocument: Document, noOfFeeds: number) {
 		for (let i = 0; i < noOfFeeds; i++) {
 			const oItemElement = oDocument?.getElementsByTagName("item")[i];
-			const sUrl: string = await this.extractImage(String(oItemElement.getElementsByTagName("link")[0].textContent));
+			const sUrl: string = await this.extractImage(String(oItemElement.getElementsByTagName("link")[0].textContent ?? ""));
 			const oImageUrl = oDocument.createElement("imageUrl");
 			oImageUrl.textContent = sUrl;
 			oItemElement.appendChild(oImageUrl);
@@ -934,30 +934,43 @@ export default class NewsPanel extends BaseNewsPanel {
 	 * @private
 	 */
 	private getDefaultNewsFeedDetails(newsResponse: ODataResponse) {
-		let newsFeedItems: INewsFeed[] = JSON.parse(JSON.stringify(newsResponse.value || [])) as INewsFeed[];
-		const defaultNews: INewsFeed[] = [];
-		const defaultFeedDict = {} as Record<string, string>;
+		const newsFeedItems: INewsFeed[] = JSON.parse(JSON.stringify(newsResponse.value || [])) as INewsFeed[];
 		this.mandatoryNewsFeed = [];
 		if (newsFeedItems?.length > 0) {
 			this.mandatoryNewsFeed = this.filterMandatoryNews(newsFeedItems).map((oFeed) => oFeed.group_id || "");
 			this.markGroupsWithCriticalArticles(newsFeedItems);
+		}
+		return newsFeedItems;
+	}
 
-			for (const feed of newsFeedItems) {
-				const title = feed.title;
-				let subTitle = "";
-				if (!defaultFeedDict[title]) {
-					subTitle = feed.subTitle || feed.description || "";
-					defaultNews.push({
-						title: title,
-						footer: feed?.footer_text,
-						imageUrl: this.getDefaultFeedImage(feed),
-						group_id: feed.group_id,
-						subTitle: subTitle,
-						priority: feed?.priority == "1" ? Priority.Medium : Priority.None,
-						priorityText: feed?.priority == "1" ? this._i18nBundle.getText("criticalNews") : ""
-					});
-					defaultFeedDict[title] = title;
-				}
+	/**
+	 * Removes expired article groups from the provided news feed items.
+	 * An article group is considered expired if it has no associated articles in _group_to_article
+	 *
+	 * @param {INewsFeed[]} newsFeedItems - An array of news feed items to filter.
+	 * @returns {INewsFeed[]} An array of news feed items that have associated articles.
+	 * @private
+	 */
+	private removeExpiredArticleGroup(newsFeedItems: INewsFeed[]): INewsFeed[] {
+		const defaultNews: INewsFeed[] = [];
+		const defaultFeedSet = new Set<string>();
+		for (const feed of newsFeedItems) {
+			const title = feed.title;
+			const group_id = feed.group_id;
+			let subTitle = "";
+			const hasArticles = (feed._group_to_article?.length ?? 0) > 0;
+			if (group_id && !defaultFeedSet.has(group_id) && hasArticles) {
+				subTitle = feed.subTitle || feed.description || "";
+				defaultNews.push({
+					title: title,
+					footer: feed?.footer_text,
+					imageUrl: this.getDefaultFeedImage(feed),
+					group_id: feed.group_id,
+					subTitle: subTitle,
+					priority: feed?.priority == "1" ? Priority.Medium : Priority.None,
+					priorityText: feed?.priority == "1" ? this._i18nBundle.getText("criticalNews") : ""
+				});
+				defaultFeedSet.add(group_id);
 			}
 		}
 		return defaultNews;
@@ -1043,6 +1056,7 @@ export default class NewsPanel extends BaseNewsPanel {
 					}
 				}
 			]);
+
 			if (supported) {
 				if (this._endUserResponse) {
 					this._defaultNewsResponse = this._endUserResponse;
@@ -1083,14 +1097,27 @@ export default class NewsPanel extends BaseNewsPanel {
 		const feedType = this.getNewsType();
 		const isDefaultNews = feedType === NewsType.Default;
 		if (isDefaultNews) {
-			await this.getDefaultNewsResponse();
-			let aDefaultgroups = this.getDefaultNewsFeedDetails(this._defaultNewsResponse);
-			return aDefaultgroups;
+			let aDefaultNewsItems = this.defaultNewsItems ?? (await this.getDefaultNewsData());
+			return this.removeExpiredArticleGroup(aDefaultNewsItems);
 		} else {
 			return this.getCustomFeedData(sFeedId, showAllPreparationRequired);
 		}
 	}
 
+	/**
+	 * Get the default news feed items.
+	 * This method fetches the default news response and processes it to extract the news feed details.
+	 * @returns {Promise<INewsFeed[]>} A promise that resolves to an array of default news feed items.
+	 * @private
+	 */
+	public async getDefaultNewsData(): Promise<INewsFeed[]> {
+		await this.getDefaultNewsResponse();
+		if (!this.defaultNewsItems) {
+			this.defaultNewsItems = this.getDefaultNewsFeedDetails(this._defaultNewsResponse);
+		}
+		return this.defaultNewsItems;
+	}
+
 	/**
 	 * Retrieves custom news feed items identified by the provided feed ID and settings.
 	 * It processes the response data and returns an array of custom news feed items.

package/src/sap/cux/home/NewsSettingsPanel.d.ts

@@ -27,6 +27,9 @@ declare module "sap/cux/home/NewsSettingsPanel" {
         private deselectedDefaultFeeds;
         private headerText;
         private title;
+        private groupTable;
+        private newsListVBox;
+        private _resetActionButton;
         /**
          * Init lifecycle method
          *
@@ -60,12 +63,18 @@ declare module "sap/cux/home/NewsSettingsPanel" {
          */
         private setTitleMessage;
         /**
-         * Returns the content for the news List
+         * Returns the content for the news List for custom news feed
          *
          * @private
          * @returns {sap.ui.core.Control} The control containing the News Settings Panel's List
          */
-        private setNewsList;
+        private createCustomNewsList;
+        /**
+         *	Creates and returns a table for managing default news group settings.
+         * @returns {Table} Table for news groups for default manage news feed
+         * @private
+         */
+        private createNewsGroupTable;
         /**
          * Checks if the custom file format is CSV based on the custom file name.
          *
@@ -80,17 +89,31 @@ declare module "sap/cux/home/NewsSettingsPanel" {
          * @private
          */
         private saveNewsFeedSettings;
-        /** Get groupId info  for the default NewsList
-         * @param {HBox} [contentBox] content Hbox
+        /**
+         * Collect selected custom feed titles from the list
+         * @returns {string[]} Array of selected custom feed titles
+         * @private
+         */
+        private collectSelectedCustomFeedTitles;
+        /**
+         * Collect deselected default group IDs from the table
+         * @returns {string[]} Array of deselected default group IDs from the table
+         * @private
+         */
+        private collectDeselectedDefaultGroupIds;
+        /**
+         * Get groupId info  for the default NewsList
+         * @param {HBox | ColumnListItem} contentBox, content Hbox or ColumnListItem
          * @returns {string} groupId
          * @private
          */
         private getDefaultGroupId;
-        /** Set items for the NewsList
+        /**
+         * Set items for the NewsList for custom news feed
          * @param {Array} [aItems] news items to be set as items aggregation
          * @private
          */
-        private setItems;
+        private setCustomNewsItems;
         /**
          * Loads news feed settings
          *
@@ -108,6 +131,12 @@ declare module "sap/cux/home/NewsSettingsPanel" {
          * If no news feed is provided, it returns undefined.
          */
         private _handleDefaultNewsFeed;
+        /**
+         * This method populates the column list items in the group table for the default news feed settings.
+         * @param aItems Array of news feed items
+         * @private
+         */
+        private setColumnListItems;
         /**
          * @param {INewsFeed[]} aNewsFeed
          * @param aPersNewsFeed
@@ -120,6 +149,13 @@ declare module "sap/cux/home/NewsSettingsPanel" {
          * It also sets the state of the show switch based on the `showAllPreparationRequired` parameter.
          */
         private _handleCustomNewsFeed;
+        /**
+         * Resets the default manage news' news settings to default by clearing
+         * the personalization data and updating the news panel.
+         * @private
+         * @returns {Promise<INewsFeed[] | undefined>} A promise that resolves to the updated news feed items or undefined.
+         */
+        private resetDefaultNewsSettings;
         /**
          * Checks if the News Settings Panel is supported based on the properties of the News Panel.
          *

package/src/sap/cux/home/NewsSettingsPanel.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"NewsSettingsPanel.d.ts","sourceRoot":"../../../../..","sources":["src/sap/cux/home/NewsSettingsPanel.ts"],"names":[],"mappings":"AAAA,OAAO,QAAQ,gCAAgC,CAAC;IAmBhD,OAAO,iBAAiB,MAAM,gCAAgC,CAAC;IAU/D,UAAU,YAAY;QACrB,KAAK,EAAE,MAAM,EAAE,CAAC;QAChB,0BAA0B,CAAC,EAAE,OAAO,CAAC;KACrC;IAED;;;;;;;;;;;;OAYG;IACH,MAAM,CAAC,OAAO,OAAO,iBAAkB,SAAQ,iBAAiB;QAC/D,OAAO,CAAC,WAAW,CAAU;QAC7B,OAAO,CAAC,wBAAwB,CAAQ;QACxC,OAAO,CAAC,KAAK,CAAQ;QACrB,OAAO,CAAC,aAAa,CAAsB;QAC3C,OAAO,CAAC,UAAU,CAAa;QAC/B,OAAO,CAAC,YAAY,CAA0B;QAC9C,OAAO,CAAC,sBAAsB,CAAY;QAC1C,OAAO,CAAC,UAAU,CAAQ;QAC1B,OAAO,CAAC,KAAK,CAAS;QAEtB;;;;;WAKG;QACI,IAAI,IAAI,IAAI;QAkBnB;;;;;WAKG;QACH,OAAO,CAAC,UAAU;QAWlB;;WAEG;gBACW,kBAAkB;QAUhC;;;;;WAKG;QACH,OAAO,CAAC,SAAS;QAYjB;;;;;WAKG;QACH,OAAO,CAAC,eAAe;QAkBvB;;;;;WAKG;QACH,OAAO,CAAC,WAAW;QA6CnB;;;;;WAKG;QACH,OAAO,CAAC,eAAe;QAIvB;;;;;WAKG;gBACW,oBAAoB;QA6ClC;;;;WAIG;QACH,OAAO,CAAC,iBAAiB;QAKzB;;;WAGG;QACH,OAAO,CAAC,QAAQ;QAgChB;;;;WAIG;gBACW,oBAAoB;QAyBlC;;;;;;;;;WASG;QACH,OAAO,CAAC,sBAAsB;QA4B9B;;;;;;;;;;WAUG;QACH,OAAO,CAAC,qBAAqB;QAyB7B;;;;WAIG;QACU,WAAW,IAAI,OAAO,CAAC,OAAO,CAAC;KAW5C;CAEA"}
+{"version":3,"file":"NewsSettingsPanel.d.ts","sourceRoot":"../../../../..","sources":["src/sap/cux/home/NewsSettingsPanel.ts"],"names":[],"mappings":"AAAA,OAAO,QAAQ,gCAAgC,CAAC;IAsBhD,OAAO,iBAAiB,MAAM,gCAAgC,CAAC;IAU/D,UAAU,YAAY;QACrB,KAAK,EAAE,MAAM,EAAE,CAAC;QAChB,0BAA0B,CAAC,EAAE,OAAO,CAAC;KACrC;IAED;;;;;;;;;;;;OAYG;IACH,MAAM,CAAC,OAAO,OAAO,iBAAkB,SAAQ,iBAAiB;QAC/D,OAAO,CAAC,WAAW,CAAU;QAC7B,OAAO,CAAC,wBAAwB,CAAQ;QACxC,OAAO,CAAC,KAAK,CAAQ;QACrB,OAAO,CAAC,aAAa,CAAsB;QAC3C,OAAO,CAAC,UAAU,CAAa;QAC/B,OAAO,CAAC,YAAY,CAA0B;QAC9C,OAAO,CAAC,sBAAsB,CAAY;QAC1C,OAAO,CAAC,UAAU,CAAQ;QAC1B,OAAO,CAAC,KAAK,CAAS;QACtB,OAAO,CAAC,UAAU,CAAS;QAC3B,OAAO,CAAC,YAAY,CAAQ;QAC5B,OAAO,CAAC,kBAAkB,CAAU;QAEpC;;;;;WAKG;QACI,IAAI,IAAI,IAAI;QA2BnB;;;;;WAKG;QACH,OAAO,CAAC,UAAU;QAWlB;;WAEG;gBACW,kBAAkB;QAUhC;;;;;WAKG;QACH,OAAO,CAAC,SAAS;QAYjB;;;;;WAKG;QACH,OAAO,CAAC,eAAe;QAkBvB;;;;;WAKG;QACH,OAAO,CAAC,oBAAoB;QA8C5B;;;;WAIG;QACH,OAAO,CAAC,oBAAoB;QA8B5B;;;;;WAKG;QACH,OAAO,CAAC,eAAe;QAIvB;;;;;WAKG;gBACW,oBAAoB;QAoClC;;;;WAIG;QACH,OAAO,CAAC,+BAA+B;QAYvC;;;;WAIG;QACH,OAAO,CAAC,gCAAgC;QAYxC;;;;;WAKG;QACH,OAAO,CAAC,iBAAiB;QAKzB;;;;WAIG;QACH,OAAO,CAAC,kBAAkB;QA2B1B;;;;WAIG;gBACW,oBAAoB;QA6BlC;;;;;;;;;WASG;QACH,OAAO,CAAC,sBAAsB;QAkC9B;;;;WAIG;QACH,OAAO,CAAC,kBAAkB;QA2B1B;;;;;;;;;;WAUG;QACH,OAAO,CAAC,qBAAqB;QA4B7B;;;;;WAKG;gBACW,wBAAwB;QAWtC;;;;WAIG;QACU,WAAW,IAAI,OAAO,CAAC,OAAO,CAAC;KAW5C;CAEA"}