Merge pull request #218 from umbraco/feature/algolia-geolocation-with-content-item

Pass content item to geolocation method

Author: acoumb

src/Umbraco.Cms.Integrations.Search.Algolia/Builders/ContentRecordBuilder.cs

@@ -70,7 +70,7 @@ public ContentRecordBuilder BuildFromContent(IContent content, Func<IProperty, b
             _record.Path = content.Path.Split(',').ToList();
             _record.ContentTypeAlias = content.ContentType.Alias;
             _record.Url = _urlProvider.GetUrl(content.Id);
-            _record.GeolocationData = _algoliaGeolocationProvider.GetGeolocationAsync().ConfigureAwait(false).GetAwaiter().GetResult();
+            _record.GeolocationData = _algoliaGeolocationProvider.GetGeolocationAsync(content).ConfigureAwait(false).GetAwaiter().GetResult();
             _record.Data = new();
 
             if (content.PublishedCultures.Count() > 0)

src/Umbraco.Cms.Integrations.Search.Algolia/Services/AlgoliaIndexService.cs

@@ -11,12 +11,10 @@ namespace Umbraco.Cms.Integrations.Search.Algolia.Services
     public class AlgoliaIndexService : IAlgoliaIndexService
     {
         private readonly AlgoliaSettings _settings;
-        private readonly IAlgoliaGeolocationProvider _algoliaGeolocationProvider;
 
-        public AlgoliaIndexService(IOptions<AlgoliaSettings> options, IAlgoliaGeolocationProvider algoliaGeolocationProvider)
+        public AlgoliaIndexService(IOptions<AlgoliaSettings> options)
         {
             _settings = options.Value;
-            _algoliaGeolocationProvider = algoliaGeolocationProvider;
         }
 
         public async Task<Result> PushData(string name, List<Record> payload = null)
@@ -32,7 +30,7 @@ await index.SaveObjectsAsync(payload != null
                     : new List<Record> {
                         new Record {
                             ObjectID = Guid.NewGuid().ToString(),
-                            GeolocationData = await _algoliaGeolocationProvider.GetGeolocationAsync(),
+                            GeolocationData = null,
                             Data = new Dictionary<string, object>()}
                     }, autoGenerateObjectId: false);
 

src/Umbraco.Cms.Integrations.Search.Algolia/Services/AlgoliaNullGeolocationProvider.cs

@@ -1,9 +1,10 @@
-using Umbraco.Cms.Integrations.Search.Algolia.Models;
+using Umbraco.Cms.Core.Models;
+using Umbraco.Cms.Integrations.Search.Algolia.Models;
 
 namespace Umbraco.Cms.Integrations.Search.Algolia.Services
 {
     public class AlgoliaNullGeolocationProvider : IAlgoliaGeolocationProvider
     {
-        public async Task<List<GeolocationEntity>> GetGeolocationAsync() => null;
+        public async Task<List<GeolocationEntity>> GetGeolocationAsync(IContent content) => null;
     }
 }

src/Umbraco.Cms.Integrations.Search.Algolia/Services/IAlgoliaGeolocationProvider.cs

@@ -1,9 +1,10 @@
-using Umbraco.Cms.Integrations.Search.Algolia.Models;
+using Umbraco.Cms.Core.Models;
+using Umbraco.Cms.Integrations.Search.Algolia.Models;
 
 namespace Umbraco.Cms.Integrations.Search.Algolia.Services
 {
     public interface IAlgoliaGeolocationProvider
     {
-        Task<List<GeolocationEntity>> GetGeolocationAsync();
+        Task<List<GeolocationEntity>> GetGeolocationAsync(IContent content);
     }
 }

src/Umbraco.Cms.Integrations.Search.Algolia/Umbraco.Cms.Integrations.Search.Algolia.csproj

@@ -13,7 +13,7 @@
 		<PackageIconUrl></PackageIconUrl>
 		<PackageProjectUrl>https://github.com/umbraco/Umbraco.Cms.Integrations/tree/main/src/Umbraco.Cms.Integrations.Search.Algolia</PackageProjectUrl>
 		<RepositoryUrl>https://github.com/umbraco/Umbraco.Cms.Integrations</RepositoryUrl>
-		<Version>2.3.0</Version>
+		<Version>2.3.1</Version>
 		<Authors>Umbraco HQ</Authors>
 		<Company>Umbraco</Company>
 		<PackageTags>Umbraco;Umbraco-Marketplace</PackageTags>