Fix tests, fix places job

src/Client/Client/Controls/RatingControl.cs

@@ -102,7 +102,7 @@ private Label CreateLabel(State state, int i)
 
 		if (AllowRating)
 		{
-			label.GestureRecognizers.Add(new TapGestureRecognizer()
+			label.GestureRecognizers.Add(new TapGestureRecognizer
 			{
 				NumberOfTapsRequired = 1,
 				Command = new Command(() => Value = i)

src/Client/Client/Framework/BaseContentPage.cs

@@ -1,5 +1,6 @@
 namespace Client.Framework;
 
+using CommunityToolkit.Maui.Behaviors;
 using Microsoft.Maui.Controls.PlatformConfiguration;
 using Microsoft.Maui.Controls.PlatformConfiguration.iOSSpecific;
 
@@ -12,7 +13,7 @@ protected BaseContentPage(T viewModel)
 		On<iOS>().SetUseSafeArea(true);
 		if (OperatingSystem.IsAndroid() || OperatingSystem.IsOSPlatformVersionAtLeast("iOS", 15))
 		{
-			Behaviors.Add(new CommunityToolkit.Maui.Behaviors.StatusBarBehavior
+			Behaviors.Add(new StatusBarBehavior
 			{
 				StatusBarColor = BackgroundColor
 			});

src/Client/Client/MauiProgram.cs

@@ -76,14 +76,16 @@ public static MauiApp CreateMauiApp()
 #endif
 		});
 
-		builder.UseMauiCommunityToolkit(x =>
-		{
+		builder.UseMauiCommunityToolkit(
 #if !DEBUG
+			x =>
+		{
 			x.SetShouldSuppressExceptionsInConverters(true);
 			x.SetShouldSuppressExceptionsInBehaviors(true);
 			x.SetShouldSuppressExceptionsInAnimations(true);
+		}
 #endif
-		});
+			);
 
 		builder.Services.AddSingleton<INavigationService, NavigationService>();
 

src/Modules/Places/WorldExplorer.Modules.Places.Application/Places/DeletePlaces/DeletePlacesCommandHandler.cs

@@ -1,10 +1,10 @@
 namespace WorldExplorer.Modules.Places.Application.Places.DeletePlaces;
 
+using Common.Application.EventBus;
 using Common.Application.Messaging;
 using Common.Domain;
 using Domain.Places;
 using IntegrationEvents;
-using WorldExplorer.Common.Application.EventBus;
 
 internal sealed class DeletePlacesCommandHandler(IPlaceRepository placeRepository, IEventBus eventBus)
 	: ICommandHandler<DeletePlacesCommand>

src/Modules/Places/WorldExplorer.Modules.Places.Domain/Places/Place.cs

@@ -13,18 +13,20 @@ private Place()
 
 	public static Place Create(string name, Point location)
 	{
-		var place = new Place()
+		var place = new Place
 		{
 			Id = Guid.CreateVersion7(),
 			Name = name,
-			Location = location
+			Location = location,
+			CreatedAt = DateTimeOffset.UtcNow
 		};
 
 		place.Raise(new PlaceCreatedDomainEvent(place.Id));
 		return place;
 	}
 
 	public Guid Id { get; private init; }
+	public DateTimeOffset CreatedAt { get; private init; }
 	public string Name { get; private set; }
 	public Point Location { get; private set; }
 	public string? Description { get; private set; }

src/Modules/Places/WorldExplorer.Modules.Places.Infrastructure.UnitTests/PointJsonConverterTests.cs

@@ -7,7 +7,7 @@
 
 public class PointJsonConverterTests
 {
-	private readonly JsonSerializerOptions jsonSerializerOptions = new JsonSerializerOptions()
+	private readonly JsonSerializerOptions jsonSerializerOptions = new JsonSerializerOptions
 	{
 		Converters =
 		{

src/Modules/Places/WorldExplorer.Modules.Places.Infrastructure/AI/AiService.cs

@@ -29,7 +29,7 @@ The output must contain only JSON because I will parse it later. Do not include
 		                      }
 		                      """;
 
-		var options = new ChatOptions()
+		var options = new ChatOptions
 		{
 			ResponseFormat = ChatResponseFormat.Json
 		};
@@ -54,7 +54,7 @@ The output must contain only the detailed information because I will parse it la
 		                     Dmytro Yavornytsky National Historical Museum of Dnipro is a museum, established in Dnipro (Ukraine) in 1848 by Andriy Fabr, local governor. Its permanent collection consists of 283 thousand objects from ancient Paleolithic implements to display units of World War II. Among its notable objects are the Kurgan stelae, Kernosivsky idol and vast collection of cossack's antiquities.
 		                     """;
 
-		return GetResponse(generalPrompt, new ChatOptions() { ResponseFormat = ChatResponseFormat.Text }, cancellationToken);
+		return GetResponse(generalPrompt, new ChatOptions { ResponseFormat = ChatResponseFormat.Text }, cancellationToken);
 	}
 
 	private async Task<string?> GetResponse(string request,

src/Modules/Places/WorldExplorer.Modules.Places.Infrastructure/Database/Migrations/20241229210934_Places.cs → src/Modules/Places/WorldExplorer.Modules.Places.Infrastructure/Database/Migrations/20241230000644_Places.cs

@@ -53,6 +53,7 @@ protected override void Up(MigrationBuilder migrationBuilder)
                 columns: table => new
                 {
                     Id = table.Column<Guid>(type: "uniqueidentifier", nullable: false),
+                    CreatedAt = table.Column<DateTimeOffset>(type: "datetimeoffset", nullable: false),
                     Name = table.Column<string>(type: "nvarchar(max)", nullable: false),
                     Location = table.Column<Point>(type: "geography", nullable: false),
                     Description = table.Column<string>(type: "nvarchar(max)", nullable: true)

src/Modules/Places/WorldExplorer.Modules.Places.Infrastructure/Database/Migrations/PlacesDbContextModelSnapshot.cs

@@ -77,6 +77,9 @@ protected override void BuildModel(ModelBuilder modelBuilder)
                         .ValueGeneratedOnAdd()
                         .HasColumnType("uniqueidentifier");
 
+                    b.Property<DateTimeOffset>("CreatedAt")
+                        .HasColumnType("datetimeoffset");
+
                     b.Property<string>("Description")
                         .HasColumnType("nvarchar(max)");
 

src/Modules/Places/WorldExplorer.Modules.Places.Infrastructure/Outbox/ConfigureProcessOutboxJob.cs

@@ -3,21 +3,22 @@
 using Microsoft.Extensions.Options;
 using Quartz;
 
-internal sealed class ConfigureProcessOutboxJob(IOptions<OutboxOptions> outboxOptions)
-	: IConfigureOptions<QuartzOptions>
+internal sealed class ConfigureProcessOutboxJob(IOptions<OutboxOptions> outboxOptions) : IConfigureOptions<QuartzOptions>
 {
-	private readonly OutboxOptions _outboxOptions = outboxOptions.Value;
-
 	public void Configure(QuartzOptions options)
 	{
 		var jobName = typeof(ProcessOutboxJob).FullName!;
 
 		options.AddJob<ProcessOutboxJob>(configure => configure.WithIdentity(jobName))
-		       .AddTrigger(configure =>
-			                   configure.ForJob(jobName)
-			                            .WithSimpleSchedule(schedule =>
-				                                                schedule.WithIntervalInSeconds(
-					                                                        _outboxOptions.IntervalInSeconds)
-				                                                        .RepeatForever()));
+			   .AddTrigger(configure =>
+			   {
+				   configure.ForJob(jobName)
+				            .WithSimpleSchedule(schedule =>
+				            {
+					            schedule
+						            .WithIntervalInSeconds(outboxOptions.Value.IntervalInSeconds)
+						            .RepeatForever();
+				            });
+			   });
 	}
 }

src/Modules/Places/WorldExplorer.Modules.Places.Infrastructure/Places/ConfigurePlacesJob.cs

@@ -3,24 +3,42 @@
 using Microsoft.Extensions.Options;
 using Quartz;
 
-internal sealed class ConfigurePlacesJob : IConfigureOptions<QuartzOptions>
+internal sealed class ConfigurePlacesJob(IOptions<PlacesJobOptions> placesJobOptions) : IConfigureOptions<QuartzOptions>
 {
 	public void Configure(QuartzOptions options)
 	{
 		var placeLookupJobName = typeof(PlacesLookupJob).FullName!;
 
 		options.AddJob<PlacesLookupJob>(configure => configure.WithIdentity(placeLookupJobName))
-		       .AddTrigger(configure => configure.ForJob(placeLookupJobName)
-		                                         .WithSimpleSchedule(schedule =>
-			                                                             schedule.WithIntervalInSeconds(15)
-				                                                             .RepeatForever()));
+			   .AddTrigger(configure =>
+			   {
+				   configure.ForJob(placeLookupJobName)
+				            .WithSimpleSchedule(schedule =>
+				            {
+					            schedule
+						            .WithIntervalInSeconds(placesJobOptions.Value.IntervalInSeconds)
+						            .RepeatForever();
+				            });
+			   });
 
 		var placeConfigurationJobName = typeof(PlaceDetailsJob).FullName!;
 
 		options.AddJob<PlaceDetailsJob>(configure => configure.WithIdentity(placeConfigurationJobName))
-		       .AddTrigger(configure => configure.ForJob(placeConfigurationJobName)
-		                                         .WithSimpleSchedule(schedule =>
-			                                                             schedule.WithIntervalInSeconds(15)
-				                                                             .RepeatForever()));
+			   .AddTrigger(configure =>
+			   {
+				   configure.ForJob(placeConfigurationJobName)
+				            .WithSimpleSchedule(schedule =>
+				            {
+					            schedule
+						            .WithIntervalInSeconds(placesJobOptions.Value.IntervalInSeconds)
+						            .RepeatForever();
+				            });
+			   });
 	}
+}
+
+internal sealed class PlacesJobOptions
+{
+	public int IntervalInSeconds { get; init; }
+	public int BatchSize { get; init; }
 }

src/Modules/Places/WorldExplorer.Modules.Places.Infrastructure/Places/PlaceDetailsJob.cs

@@ -7,6 +7,7 @@
 using Image;
 using Microsoft.EntityFrameworkCore;
 using Microsoft.Extensions.Logging;
+using Microsoft.Extensions.Options;
 using Quartz;
 
 [DisallowConcurrentExecution]
@@ -15,6 +16,7 @@ internal sealed class PlaceDetailsJob(
 	IUnitOfWork unitOfWork,
 	IAiService aiService,
 	IImageSearchService imageSearchService,
+	IOptions<PlacesJobOptions> placeJobOptions,
 	ILogger<PlaceDetailsJob> logger) : IJob
 {
 	private const string ModuleName = "Places";
@@ -27,7 +29,8 @@ public async Task Execute(IJobExecutionContext context)
 											 .AsTracking()
 											 .Include(x => x.Images)
 											 .Where(x => x.Images.Count == 0 || string.IsNullOrEmpty(x.Description))
-											 .OrderBy(x => x.Images.Count)
+											 .OrderBy(x => x.CreatedAt)
+											 .Take(placeJobOptions.Value.BatchSize)
 											 .ToListAsync(context.CancellationToken);
 
 		if (notFilledPlaces.Count == 0)

src/Modules/Places/WorldExplorer.Modules.Places.Infrastructure/PlacesModule.cs

@@ -55,11 +55,11 @@ private static void AddInfrastructure(this IHostApplicationBuilder builder)
 		builder.Services.AddHttpClient("ImageSearch");
 		builder.Services.AddSingleton<IImageSearchService, ImageSearchService>();
 
-		builder.Services.ConfigureOptions<ConfigurePlacesJob>();
 		builder.Services.Configure<PlacesSettings>(builder.Configuration);
+		builder.Services.Configure<PlacesJobOptions>(builder.Configuration.GetSection("Places:Jobs"));
+		builder.Services.ConfigureOptions<ConfigurePlacesJob>();
 
 		builder.Services.Configure<OutboxOptions>(builder.Configuration.GetSection("Places:Outbox"));
-
 		builder.Services.ConfigureOptions<ConfigureProcessOutboxJob>();
 
 		SerializerSettings.ConfigureJsonSerializerOptionsInstance([new PointJsonConverter()]);

src/Modules/Places/WorldExplorer.Modules.Places.Infrastructure/PointJsonConverter.cs

@@ -1,9 +1,10 @@
 namespace WorldExplorer.Modules.Places.Infrastructure;
 
 using System.Text.Json;
+using System.Text.Json.Serialization;
 using NetTopologySuite.Geometries;
 
-public class PointJsonConverter : System.Text.Json.Serialization.JsonConverter<Point>
+public class PointJsonConverter : JsonConverter<Point>
 {
 	public override Point Read(ref Utf8JsonReader reader, Type typeToConvert, JsonSerializerOptions options)
 	{

src/Modules/Places/WorldExplorer.Modules.Places.Presentation/LocationInfoRequests/GetPlaceDetails.cs

@@ -1,13 +1,13 @@
 namespace WorldExplorer.Modules.Places.Presentation.LocationInfoRequests;
 
+using Application.LocationInfoRequests.GetLocationInfoRequest;
+using Common.Infrastructure.Authorization;
+using Common.Presentation.Endpoints;
+using Common.Presentation.Results;
 using MediatR;
 using Microsoft.AspNetCore.Builder;
 using Microsoft.AspNetCore.Http;
 using Microsoft.AspNetCore.Routing;
-using WorldExplorer.Common.Infrastructure.Authorization;
-using WorldExplorer.Common.Presentation.Endpoints;
-using WorldExplorer.Common.Presentation.Results;
-using WorldExplorer.Modules.Places.Application.LocationInfoRequests.GetLocationInfoRequest;
 
 internal sealed class GetLocationInfoRequest : IEndpoint
 {

src/Modules/Travellers/WorldExplorer.Modules.Travellers/Application/Travellers/DeleteTraveller/DeleteTravellerCommand.cs

@@ -1,5 +1,5 @@
 namespace WorldExplorer.Modules.Travellers.Application.Travellers.DeleteTraveller;
 
-using WorldExplorer.Common.Application.Messaging;
+using Common.Application.Messaging;
 
 public sealed record DeleteTravellerCommand(Guid TravellerId) : ICommand;

src/Modules/Travellers/WorldExplorer.Modules.Travellers/Application/Travellers/DeleteTraveller/DeleteTravellerCommandHandler.cs

@@ -1,8 +1,8 @@
 namespace WorldExplorer.Modules.Travellers.Application.Travellers.DeleteTraveller;
 
-using WorldExplorer.Common.Application.Messaging;
-using WorldExplorer.Common.Domain;
-using WorldExplorer.Modules.Travellers.Abstractions.Data;
+using Abstractions.Data;
+using Common.Application.Messaging;
+using Common.Domain;
 
 internal sealed class DeleteTravellerCommandHandler(ITravellerRepository travellerRepository, IUnitOfWork unitOfWork)
 	: ICommandHandler<DeleteTravellerCommand>

src/Modules/Travellers/WorldExplorer.Modules.Travellers/Application/Visits/CreateVisit/CreateVisitHandler.cs

@@ -6,13 +6,13 @@ public class CreateVisitHandler(TravellersDbContext context)
 {
 	public async Task<Visit> CreateVisit(VisitRequest request, CancellationToken ct = default)
 	{
-		var visit = new Visit()
+		var visit = new Visit
 		{
 			Id = Guid.CreateVersion7(),
 			PlaceId = request.PlaceId,
 			TravellerId = request.TravellerId,
 			VisitDate = DateTime.UtcNow,
-			Review = new Review()
+			Review = new Review
 			{
 				Id = Guid.CreateVersion7(),
 				Comment = request.Comment,

src/Modules/Travellers/WorldExplorer.Modules.Travellers/Infrastructure/Inbox/ConfigureProcessInboxJob.cs

@@ -3,20 +3,22 @@
 using Microsoft.Extensions.Options;
 using Quartz;
 
-internal sealed class ConfigureProcessInboxJob(IOptions<InboxOptions> outboxOptions) : IConfigureOptions<QuartzOptions>
+internal sealed class ConfigureProcessInboxJob(IOptions<InboxOptions> inboxOptions) : IConfigureOptions<QuartzOptions>
 {
-	private readonly InboxOptions _inboxOptions = outboxOptions.Value;
-
 	public void Configure(QuartzOptions options)
 	{
 		var jobName = typeof(ProcessInboxJob).FullName!;
 
 		options.AddJob<ProcessInboxJob>(configure => configure.WithIdentity(jobName))
 		       .AddTrigger(configure =>
-			                   configure.ForJob(jobName)
-			                            .WithSimpleSchedule(schedule =>
-				                                                schedule.WithIntervalInSeconds(
-					                                                        _inboxOptions.IntervalInSeconds)
-				                                                        .RepeatForever()));
+		       {
+			       configure.ForJob(jobName)
+			                .WithSimpleSchedule(schedule =>
+			                {
+				                schedule
+					                .WithIntervalInSeconds(inboxOptions.Value.IntervalInSeconds)
+					                .RepeatForever();
+			                });
+		       });
 	}
 }