salvataggio immagini

api.py

@@ -10,6 +10,8 @@ from logging.handlers import TimedRotatingFileHandler
 from pathlib import Path
 from typing import Any
 
+import cloudinary
+import cloudinary.uploader
 import chromadb
 import httpx
 from dotenv import load_dotenv
@@ -41,6 +43,19 @@ def _default_plants_db_path() -> str:
 
 
 PLANTS_SQLITE_PATH = os.getenv("PLANTS_SQLITE_PATH", _default_plants_db_path())
+
+# Cloudinary configuration (optional - photo upload disabled if not set)
+CLOUDINARY_CLOUD_NAME = os.getenv("CLOUDINARY_CLOUD_NAME", "")
+CLOUDINARY_API_KEY = os.getenv("CLOUDINARY_API_KEY", "")
+CLOUDINARY_API_SECRET = os.getenv("CLOUDINARY_API_SECRET", "")
+if CLOUDINARY_CLOUD_NAME and CLOUDINARY_API_KEY and CLOUDINARY_API_SECRET:
+    cloudinary.config(
+        cloud_name=CLOUDINARY_CLOUD_NAME,
+        api_key=CLOUDINARY_API_KEY,
+        api_secret=CLOUDINARY_API_SECRET,
+        secure=True,
+    )
+
 GOOGLE_CLIENT_IDS = [
     value.strip()
     for value in os.getenv("GOOGLE_CLIENT_ID", "").split(",")
@@ -272,10 +287,17 @@ def ensure_user_plants_table(conn: sqlite3.Connection) -> None:
             user_given_name TEXT NOT NULL,
             user_id TEXT NOT NULL,
             user_email TEXT,
+            user_photo_url TEXT,
             created_at TEXT NOT NULL
         )
         """
     )
+    # Add user_photo_url column to existing databases (migration)
+    try:
+        conn.execute("ALTER TABLE user_plants ADD COLUMN user_photo_url TEXT")
+        conn.commit()
+    except Exception:
+        pass  # Column already exists
     conn.commit()
 
 
@@ -285,6 +307,7 @@ def _user_plant_row_to_payload(row: sqlite3.Row) -> dict[str, Any]:
         "plant_name": row["plant_name"],
         "user_given_name": row["user_given_name"],
         "user": row["user_email"] or row["user_id"],
+        "user_photo_url": row["user_photo_url"] if "user_photo_url" in row.keys() else None,
         "created_at_iso": row["created_at"],
         "created_at": _format_datetime_display(row["created_at"]),
     }
@@ -803,6 +826,75 @@ def update_user_plant_first_watering_date(
     return JSONResponse(content={"updated": updated})
 
 
+@app.post("/user/plants/{plant_id}/photo")
+async def upload_user_plant_photo(
+    plant_id: int,
+    file: UploadFile = File(...),
+    authorization: str | None = Header(default=None),
+):
+    """Upload a user photo for a saved plant, store it on Cloudinary."""
+    user = _get_google_user_from_authorization(authorization)
+    if not user:
+        raise HTTPException(status_code=401, detail="Accedi con Google per caricare una foto.")
+
+    if not (CLOUDINARY_CLOUD_NAME and CLOUDINARY_API_KEY and CLOUDINARY_API_SECRET):
+        raise HTTPException(status_code=503, detail="Servizio foto non configurato.")
+
+    if not file.content_type or not file.content_type.startswith("image/"):
+        raise HTTPException(status_code=400, detail="Il file caricato non è un'immagine valida.")
+
+    user_id = str(user.get("sub") or "").strip()
+    if not user_id:
+        raise HTTPException(status_code=401, detail="Utente non valido.")
+
+    # Verify the plant belongs to this user
+    with get_plants_db_connection() as conn:
+        ensure_user_plants_table(conn)
+        row = conn.execute(
+            "SELECT id FROM user_plants WHERE id = ? AND user_id = ? LIMIT 1",
+            (plant_id, user_id),
+        ).fetchone()
+    if row is None:
+        raise HTTPException(status_code=404, detail="Pianta non trovata.")
+
+    suffix = os.path.splitext(file.filename or "")[1] or ".jpg"
+    with tempfile.NamedTemporaryFile(delete=False, suffix=suffix) as tmp:
+        tmp.write(await file.read())
+        tmp_path = tmp.name
+
+    try:
+        result = cloudinary.uploader.upload(
+            tmp_path,
+            folder="green-assistant/user-plants",
+            public_id=f"plant_{plant_id}_user_{user_id[:16]}",
+            overwrite=True,
+            resource_type="image",
+            transformation=[{"width": 1200, "crop": "limit", "quality": "auto:good"}],
+        )
+        photo_url = result.get("secure_url", "")
+    except Exception as e:
+        raise HTTPException(status_code=500, detail=f"Errore upload foto: {e}")
+    finally:
+        if os.path.exists(tmp_path):
+            os.remove(tmp_path)
+
+    # Save URL to DB
+    with get_plants_db_connection() as conn:
+        conn.execute(
+            "UPDATE user_plants SET user_photo_url = ? WHERE id = ? AND user_id = ?",
+            (photo_url, plant_id, user_id),
+        )
+        conn.commit()
+        updated_row = conn.execute(
+            "SELECT id, plant_name, user_given_name, user_id, user_email, user_photo_url, created_at "
+            "FROM user_plants WHERE id = ?",
+            (plant_id,),
+        ).fetchone()
+
+    _log_api("/user/plants/{plant_id}/photo", "uploaded", {"plant_id": plant_id})
+    return JSONResponse(content={"updated": _user_plant_row_to_payload(updated_row)})
+
+
 @app.get("/health")
 def health():
     status = get_search_backend_status()

pwa-app/src/App.jsx

@@ -14,6 +14,7 @@ import {
   getSpeciesPreviews,
   searchPlantImage,
   updateMyPlantFirstWaterDate,
+  uploadMyPlantPhoto,
   verifyGoogleToken,
   toAbsoluteImage,
   toOptimizedImage
@@ -145,6 +146,9 @@ export default function App({ googleClientIdConfigured = false }) {
   const [isEditingFirstWaterDate, setIsEditingFirstWaterDate] = useState(false);
   const [firstWaterDateInput, setFirstWaterDateInput] = useState("");
   const [deletingPlantId, setDeletingPlantId] = useState(null);
+  const [uploadingPhotoId, setUploadingPhotoId] = useState(null);
+  const plantPhotoInputRef = useRef(null);
+  const plantPhotoTargetIdRef = useRef(null);
 
   const [busy, setBusy] = useState({
     search: false,
@@ -778,10 +782,50 @@ export default function App({ googleClientIdConfigured = false }) {
     setError("");
   }
 
+  function openPlantPhotoDialog(plantId) {
+    plantPhotoTargetIdRef.current = plantId;
+    plantPhotoInputRef.current?.click();
+  }
+
+  async function handlePlantPhotoFileChange(event) {
+    const photoFile = event.target.files?.[0] || null;
+    event.target.value = "";
+    if (!photoFile || !plantPhotoTargetIdRef.current) {
+      return;
+    }
+    const targetId = plantPhotoTargetIdRef.current;
+    plantPhotoTargetIdRef.current = null;
+    setUploadingPhotoId(targetId);
+    setError("");
+    try {
+      const data = await uploadMyPlantPhoto(targetId, photoFile);
+      const updated = data.updated;
+      if (updated) {
+        setMyPlants((prev) => prev.map((item) => (item.id === updated.id ? updated : item)));
+        if (selectedMyPlant?.id === updated.id) {
+          setSelectedMyPlant(updated);
+        }
+      }
+    } catch (err) {
+      setError(err.message || "Errore durante l'upload della foto.");
+    } finally {
+      setUploadingPhotoId(null);
+    }
+  }
+
   const googleCalendarUrl = buildGoogleCalendarRecurringUrl();
 
   return (
     <main className="page">
+      {/* Hidden input for user plant photo upload */}
+      <input
+        ref={plantPhotoInputRef}
+        type="file"
+        accept="image/*"
+        capture="environment"
+        style={{ display: "none" }}
+        onChange={handlePlantPhotoFileChange}
+      />
       <section className="hero">
         <div className="hero-inner">
           <div className="hero-brand">
@@ -1125,8 +1169,23 @@ export default function App({ googleClientIdConfigured = false }) {
                       {deletingPlantId === item.id ? "Elimino..." : "Elimina"}
                     </button>
                   </div>
+                  {item.user_photo_url && (
+                    <img
+                      src={item.user_photo_url}
+                      alt={`Foto di ${item.user_given_name}`}
+                      className="my-plant-item-photo"
+                    />
+                  )}
                   <p>Specie: {item.plant_name}</p>
                   <p>Inserita: {item.created_at}</p>
+                  <button
+                    type="button"
+                    className="btn-secondary btn-small btn-upload-photo"
+                    onClick={(event) => { event.stopPropagation(); openPlantPhotoDialog(item.id); }}
+                    disabled={uploadingPhotoId === item.id}
+                  >
+                    {uploadingPhotoId === item.id ? "Carico..." : (item.user_photo_url ? "📷 Cambia foto" : "📷 Aggiungi foto")}
+                  </button>
                 </article>
               ))}
             </div>
@@ -1145,6 +1204,13 @@ export default function App({ googleClientIdConfigured = false }) {
             {selectedMyPlant?.user_given_name && (
               <p>Il tuo nome: {selectedMyPlant.user_given_name}</p>
             )}
+            {selectedMyPlant?.user_photo_url && (
+              <img
+                src={selectedMyPlant.user_photo_url}
+                alt={`La tua foto di ${selectedMyPlant.user_given_name}`}
+                className="my-plant-detail-photo"
+              />
+            )}
             {myPlantCard.common_name && <p>Nome comune: {myPlantCard.common_name}</p>}
 
             {!!myPlantProfileEntries.length && (

pwa-app/src/api.js

@@ -133,6 +133,16 @@ export async function updateMyPlantFirstWaterDate(plantId, firstWateringDate) {
   return parseResponse(response);
 }
 
+export async function uploadMyPlantPhoto(plantId, file) {
+  const formData = new FormData();
+  formData.append("file", file);
+  const response = await apiFetch(`/user/plants/${plantId}/photo`, {
+    method: "POST",
+    body: formData,
+  });
+  return parseResponse(response);
+}
+
 export function toAbsoluteImage(urlOrPath) {
   if (!urlOrPath) {
     return "";

pwa-app/src/styles.css

@@ -612,6 +612,29 @@ pre {
   margin: 0.2rem 0 0;
 }
 
+.my-plant-item-photo {
+  width: 100%;
+  max-height: 180px;
+  object-fit: cover;
+  border-radius: 6px;
+  margin: 0.5rem 0 0.25rem;
+}
+
+.btn-upload-photo {
+  display: block;
+  width: 100%;
+  margin-top: 0.4rem;
+  text-align: center;
+}
+
+.my-plant-detail-photo {
+  width: 100%;
+  max-height: 260px;
+  object-fit: cover;
+  border-radius: 8px;
+  margin: 0.6rem 0;
+}
+
 .my-plant-item-head {
   display: flex;
   align-items: center;

requirements.txt

@@ -14,3 +14,4 @@ python-dotenv
 chromadb>=1.5.0
 sentence-transformers>=5.0.0
 requests>=2.28.0
+cloudinary