small fixes

app/Http/Controllers/RulesController.php

@@ -31,7 +31,7 @@ public function getRules($country, $city): array
             ->where("city_id", $city->id)
             ->first();
 
-        if(!$rules || !$rules->rulesENG || !$rules->rulesPL){
+        if(!$rules || $rules->rulesENG === null || $rules->rulesPL === null){
             $cityData = [
                 "city_id" => $city->id,
                 "country_id" => $country_id,

app/Services/OpenAIService.php

@@ -21,7 +21,7 @@ public function __construct()
 
     public function importRulesForAllCities(bool $force): void
     {
-        $cities = City::query()->whereHas('cityProviders')->get();
+        $cities = City::query()->whereHas('cityProviders')->orderBy('country.name')->get();
         foreach ($cities as $city) {
             $cityData = [
                 "city_id" => $city->id,
@@ -56,7 +56,7 @@ public function importRulesForCity(array $cityData, bool $force): array
         $country_name = $cityData["country_name"];
 
         $promptENG = "Act as a helpful assistant. Explain what are the legal limitations for riding electric scooters in $city_name, $country_name? Contain information about: max speed, helmet requirements, allowed ABV, passengers, other relevant details. Be formal, speak English. Don't include city name in your response. If you don't have information answering the question, write 'null'";
-        $promptPL = "Zachowuj się jako pomocny asystent, wyjaśnij jakie są prawa dotyczące jazdy na hulajnogach elektrycznych w $city_name, $country_name? Zawrzyj informacje o: maksymalnej prędkości, potrzebie kasku, dozwolonym alkoholu we krwi, pasażerach, inne. Bądź formalny, mów po polsku. Nie zawieraj nazwy miasta w odpowiedzi. Jeśli nie masz informacji odpowiadających na pytanie, napisz 'null'";
+        $promptPL = "Zachowuj się jako pomocny asystent. wyjaśnij jakie są prawa dotyczące jazdy na hulajnogach elektrycznych w $city_name, $country_name? Zawrzyj informacje o: maksymalnej prędkości, potrzebie kasku, dozwolonym alkoholu we krwi, pasażerach, inne. Bądź formalny, mów po polsku. Nie zawieraj nazwy miasta w odpowiedzi. Jeśli nie masz informacji odpowiadających na pytanie, napisz 'null'";
 
         if (!$force) {
             $currentRules = Rules::query()->where("city_id", $city_id)->where("country_id", $country_id)->first();
@@ -90,6 +90,9 @@ public function importRulesForCity(array $cityData, bool $force): array
             ];
         }
 
+        $rulesENG = str_replace("\n", "<br>", $rulesENG);
+        $rulesPL = str_replace("\n", "<br>", $rulesPL);
+
         Rules::query()->updateOrCreate([
             "city_id" => $city_id,
             "country_id" => $country_id,

routes/web.php

@@ -47,7 +47,7 @@
         Route::delete("/delete-city-without-assigned-country/{city}", [CityWithoutAssignedCountryController::class, "destroy"]);
         Route::post("/delete-all-cities-without-assigned-country", [CityWithoutAssignedCountryController::class, "destroyAll"]);
 
-        Route::get("/import-rules/{force}", [\App\Services\OpenAIService::class, "importRules"]);
+        Route::get("/import-rules/{force}", [\App\Services\OpenAIService::class, "importRulesForAllCities"]);
     });
 });