foundation-mvc-cms icon indicating copy to clipboard operation
foundation-mvc-cms copied to clipboard

ISettingsService fails when none of SiteDefinition.SiteUrl is valid

Open giangnb opened this issue 3 years ago • 1 comments

Steps to reproduce:

  1. Set up site, runs at localhost:5000 (default)
  2. Go to CMS admin, manage websites and change all host name to a non-existing domain, except wildcard host
  3. Back to edit mode at localhost:5000

Observed error: System.NullReferenceException: Object reference not set to an instance of an object. at Foundation.Features.Settings.ButtonBackgroundColorSelectionFactory.GetSelections(ExtendedMetadata metadata) in ...\Src\Foundation\Features\Settings\CollectionSettings.cs:line 122 at EPiServer.Shell.UI.Rest.DefaultMetadataStoreModelCreator.GetSelections(ExtendedMetadata metadata, MetadataStoreModel metadataDTO, ISelectionFactory[] selectionFactories) at EPiServer.Shell.UI.Rest.DefaultMetadataStoreModelCreator.CreateFromModelMetadata(ExtendedMetadata metadata, LocalizationService localizationService, UIDescriptorRegistry uiDescriptorRegistry, ISelectionFactory[] selectionFactories, Boolean alwaysCreateChildProperties) at EPiServer.Shell.UI.Rest.DefaultMetadataStoreModelCreator.ExtractMetadataFromProperties(ExtendedMetadata metadata, MetadataStoreModel MetadataStoreModel, LocalizationService localizationService, UIDescriptorRegistry uiDescriptorRegistry, ISelectionFactory[] selectionFactories) at EPiServer.Shell.UI.Rest.DefaultMetadataStoreModelCreator.CreateFromModelMetadata(ExtendedMetadata metadata, LocalizationService localizationService, UIDescriptorRegistry uiDescriptorRegistry, ISelectionFactory[] selectionFactories, Boolean alwaysCreateChildProperties) at EPiServer.Shell.UI.Rest.DefaultMetadataStoreModelCreator.Create(ExtendedMetadata metadata) at EPiServer.Shell.UI.Rest.MetadataStore.Get(String id, String modelAccessor) at lambda_method814(Closure , Object , Object[] ) Suggestion: Needs null check or fault-tolerance logic.

image

giangnb avatar Feb 16 '22 07:02 giangnb

@giangnb Could you please confirm this solves the issue for you (and close this if so)? Thanks!

daniel-isaacs avatar Feb 19 '22 01:02 daniel-isaacs