From fa56351ccec1a140058d69f849b74b0b520d80eb Mon Sep 17 00:00:00 2001 From: Jacob Overgaard <752371+iOvergaard@users.noreply.github.com> Date: Thu, 23 Feb 2023 17:16:34 +0100 Subject: [PATCH] add flag to test if packages have been loaded once (is there a better way?) --- .../src/backoffice/packages/repository/package.repository.ts | 4 ++++ .../src/backoffice/packages/repository/package.store.ts | 3 +++ 2 files changed, 7 insertions(+) diff --git a/src/Umbraco.Web.UI.Client/src/backoffice/packages/repository/package.repository.ts b/src/Umbraco.Web.UI.Client/src/backoffice/packages/repository/package.repository.ts index dbc01537ff..212696603b 100644 --- a/src/Umbraco.Web.UI.Client/src/backoffice/packages/repository/package.repository.ts +++ b/src/Umbraco.Web.UI.Client/src/backoffice/packages/repository/package.repository.ts @@ -30,6 +30,10 @@ export class UmbPackageRepository { * @private */ async #requestRootItems(store: UmbPackageStore) { + if (store.isPackagesLoaded) { + return; + } + const { data } = await this.#packageSource.getRootItems(); if (data) { diff --git a/src/Umbraco.Web.UI.Client/src/backoffice/packages/repository/package.store.ts b/src/Umbraco.Web.UI.Client/src/backoffice/packages/repository/package.store.ts index f7bd27f54a..3884ff9fa7 100644 --- a/src/Umbraco.Web.UI.Client/src/backoffice/packages/repository/package.store.ts +++ b/src/Umbraco.Web.UI.Client/src/backoffice/packages/repository/package.store.ts @@ -26,6 +26,8 @@ export class UmbPackageStore extends UmbStoreBase { extensions = this.#extensions.asObservable(); + isPackagesLoaded = false; + /** * Creates an instance of PackageStore. * @param {UmbControllerHostInterface} host @@ -40,6 +42,7 @@ export class UmbPackageStore extends UmbStoreBase { */ appendItems(packages: Array) { this.#packages.next(packages); + this.isPackagesLoaded = true; } appendExtensions(extensions: ManifestBase[]) {