fix: tri par nom, redirect lieux, filtre status côté query, retry tick

Author: BlackAngel242

src/lib/place-service.ts

@@ -39,7 +39,6 @@ export async function fetchPaginatedPlaces(params: {
   if (category && category !== "all") query = query.eq("category", category);
 
   const { data, count, error } = await query
-    .order("created_at", { ascending: false })
     .order("name", { ascending: true })
     .range(from, to);
 

src/pages/place-create-page.tsx

@@ -107,7 +107,7 @@ export function PlaceCreatePage() {
       const latitude = data.latitude ?? coords?.latitude ?? 0;
       const longitude = data.longitude ?? coords?.longitude ?? 0;
 
-      const newId = await createPlace({
+      await createPlace({
         name: data.name,
         category: data.category,
         description: data.description ?? "",
@@ -122,7 +122,7 @@ export function PlaceCreatePage() {
       });
 
       toast.success("Lieu proposé ! Il sera visible après validation.");
-      navigate(`/lieux/${newId}`);
+      navigate("/lieux");
     } catch {
       toast.error("Erreur lors de la proposition du lieu");
     }
@@ -140,7 +140,7 @@ export function PlaceCreatePage() {
           Proposer un lieu
         </h1>
         <p className="mt-1 text-sm text-muted-foreground">
-          Les lieux proposés sont vérifiés avant publication. Vous serez redirigé vers la fiche après envoi.
+          Les lieux proposés sont vérifiés avant publication.
         </p>
       </div>
 
@@ -227,7 +227,7 @@ export function PlaceCreatePage() {
                   <Label className="text-xs text-muted-foreground uppercase tracking-wider">Adresse / repère</Label>
                   <Input {...field} placeholder="Ex: Avenue…, quartier… ou repère visible" className="bg-white/5 border-white/10" />
                   <p className="text-[11px] text-muted-foreground/60">
-                    À Brazzaville, décrivez un repère visible (ex : "En face du marché Total Poto-Poto")
+                    À Brazzaville, décrivez un repère visible (ex : « En face du marché Total Poto-Poto »)
                   </p>
                   {errors.address && <p className="text-xs text-destructive">{errors.address.message}</p>}
                 </div>

src/pages/place-detail-page.tsx

@@ -59,31 +59,25 @@ export function PlaceDetailPage() {
       setIsLoading(true);
       setFetchError(false);
       try {
-        const { data, error } = await supabase
-          .from("places")
-          .select("*")
-          .eq("id", id)
-          .maybeSingle();
+        const baseQuery = supabase.from("places").select("*").eq("id", id);
+        const { data, error } = await (
+          isAdmin ? baseQuery : baseQuery.eq("status", "approved")
+        ).maybeSingle();
 
         if (error) throw error;
 
         if (!cancelled) {
           const p = data as Place | null;
-          // Non-admin users cannot view pending or rejected places
-          if (p && p.status !== "approved" && !isAdmin) {
-            setPlace(null);
-          } else {
-            setPlace(p);
-            if (p) {
-              const { data: related } = await supabase
-                .from("places")
-                .select("id,name,category,city")
-                .eq("status", "approved")
-                .or(`category.eq.${p.category},city.eq.${p.city ?? ""}`)
-                .neq("id", p.id)
-                .limit(4);
-              if (!cancelled) setRelatedPlaces((related ?? []) as Place[]);
-            }
+          setPlace(p);
+          if (p) {
+            const { data: related } = await supabase
+              .from("places")
+              .select("id,name,category,city")
+              .eq("status", "approved")
+              .or(`category.eq.${p.category},city.eq.${p.city ?? ""}`)
+              .neq("id", p.id)
+              .limit(4);
+            if (!cancelled) setRelatedPlaces((related ?? []) as Place[]);
           }
         }
       } catch {

src/pages/places-page.tsx

@@ -62,6 +62,7 @@ export function PlacesPage() {
   const [category, setCategory] = useState<string>("all");
   const [mobileView, setMobileView] = useState<"map" | "list">("map");
   const [focusedPlaceId, setFocusedPlaceId] = useState<string | undefined>(undefined);
+  const [retryTick, setRetryTick] = useState(0);
   const leafletMapRef = useRef<L.Map | null>(null);
 
   const hasActiveFilters = search !== "" || city !== "all" || category !== "all";
@@ -105,7 +106,7 @@ export function PlacesPage() {
       cancelled = true;
       window.clearTimeout(t);
     };
-  }, [search, city, category]);
+  }, [search, city, category, retryTick]);
 
   const lastPlacesLengthRef = useRef(places.length);
 
@@ -228,7 +229,10 @@ export function PlacesPage() {
                   <AlertCircle className="mx-auto h-6 w-6 text-destructive/70" />
                   <p className="text-sm text-muted-foreground">{fetchError}</p>
                   <button
-                    onClick={resetFilters}
+                    onClick={() => {
+                      resetFilters();
+                      setRetryTick((v) => v + 1);
+                    }}
                     className="text-xs text-tertiary hover:underline"
                   >
                     Réessayer