feat: vylepšení Pizza day

2026-02-10 23:59:58 +01:00
parent f13cd4ffa9
commit ac6727efa5
5 changed files with 175 additions and 36 deletions
--- a/client/src/App.tsx
+++ b/client/src/App.tsx
@@ -175,6 +175,11 @@ function App() {
    }
  }, [auth?.login, data?.pizzaDay?.orders])

+  // Kontrola, zda má uživatel vybranou volbu PIZZA
+  const userHasPizzaChoice = useMemo(() => {
+    return auth?.login ? data?.choices?.PIZZA?.[auth.login] != null : false;
+  }, [data?.choices?.PIZZA, auth?.login]);
+
  useEffect(() => {
    if (choiceRef?.current?.value && choiceRef.current.value !== "") {
      const locationKey = choiceRef.current.value as LunchChoice;
@@ -226,10 +231,65 @@ function App() {
    }
  }, [auth?.login, easterEgg?.duration, easterEgg?.url, eggImage]);

+  // Pomocná funkce pro kontrolu a potvrzení změny volby při existujícím Pizza day
+  const checkPizzaDayBeforeChange = async (newLocationKey: LunchChoice): Promise<boolean> => {
+    if (!auth?.login || !data) return false;
+
+    // Kontrola, zda uživatel má vybranou PIZZA a mění na něco jiného
+    const hasPizzaChoice = data.choices?.PIZZA?.[auth.login] != null;
+    const isCreator = data.pizzaDay?.creator === auth.login;
+    const isPizzaDayCreated = data.pizzaDay?.state === PizzaDayState.CREATED;
+
+    // Pokud není vybraná PIZZA nebo přepínáme na PIZZA, není potřeba kontrolovat
+    if (!hasPizzaChoice || newLocationKey === LunchChoice.PIZZA) {
+      return true;
+    }
+
+    // Pokud uživatel není zakladatel Pizza day, není potřeba dialogu
+    if (!isCreator || !data?.pizzaDay) {
+      return true;
+    }
+
+    // Uživatel je zakladatel Pizza day a mění volbu z PIZZA
+    if (!isPizzaDayCreated) {
+      // Pizza day není ve stavu CREATED, nelze změnit volbu
+      alert(`Nelze změnit volbu. Pizza day je ve stavu "${data.pizzaDay.state}" a musí být nejprve dokončen.`);
+      return false;
+    }
+
+    // Pizza day je CREATED, zobrazit potvrzovací dialog
+    const confirmed = window.confirm(
+      'Jsi zakladatel aktivního Pizza day. Změna volby smaže celý Pizza day včetně všech objednávek. Pokračovat?'
+    );
+
+    if (!confirmed) {
+      return false;
+    }
+
+    // Uživatel potvrdil, smazat Pizza day
+    try {
+      await deletePizzaDay();
+      return true;
+    } catch (error: any) {
+      alert(`Chyba při mazání Pizza day: ${error.message || error}`);
+      return false;
+    }
+  };
+
  const doAddClickFoodChoice = async (location: LunchChoice, foodIndex?: number) => {
    if (document.getSelection()?.type !== 'Range') { // pouze pokud se nejedná o výběr textu
      if (canChangeChoice && auth?.login) {
-        await addChoice({ body: { locationKey: location, foodIndex, dayIndex } });
+        // Kontrola Pizza day před změnou volby
+        const canProceed = await checkPizzaDayBeforeChange(location);
+        if (!canProceed) {
+          return;
+        }
+
+        try {
+          await addChoice({ body: { locationKey: location, foodIndex, dayIndex } });
+        } catch (error: any) {
+          alert(`Chyba při změně volby: ${error.message || error}`);
+        }
      }
    }
  }
@@ -237,11 +297,32 @@ function App() {
  const doAddChoice = async (event: React.ChangeEvent<HTMLSelectElement>) => {
    const locationKey = event.target.value as LunchChoice;
    if (canChangeChoice && auth?.login) {
-      await addChoice({ body: { locationKey, dayIndex } });
-      if (foodChoiceRef.current?.value) {
-        foodChoiceRef.current.value = "";
+      // Kontrola Pizza day před změnou volby
+      const canProceed = await checkPizzaDayBeforeChange(locationKey);
+      if (!canProceed) {
+        // Uživatel zrušil akci nebo došlo k chybě, reset výběru zpět na PIZZA
+        if (choiceRef.current) {
+          choiceRef.current.value = LunchChoice.PIZZA;
+        }
+        return;
+      }
+
+      try {
+        await addChoice({ body: { locationKey, dayIndex } });
+        if (foodChoiceRef.current?.value) {
+          foodChoiceRef.current.value = "";
+        }
+        choiceRef.current?.blur();
+      } catch (error: any) {
+        alert(`Chyba při změně volby: ${error.message || error}`);
+        // Reset výběru zpět
+        const hasPizzaChoice = data?.choices?.PIZZA?.[auth.login] != null;
+        if (choiceRef.current && hasPizzaChoice) {
+          choiceRef.current.value = LunchChoice.PIZZA;
+        } else if (choiceRef.current) {
+          choiceRef.current.value = "";
+        }
      }
-      choiceRef.current?.blur();
    }
  }

@@ -556,28 +637,28 @@ function App() {
                                        <div className="user-actions">
                                          {login === auth.login && canChangeChoice && locationKey === LunchChoice.OBJEDNAVAM && <span title='Označit/odznačit se jako objednávající'>
                                            <FontAwesomeIcon onClick={() => {
-                                            markAsBuyer();
-                                          }} icon={faBasketShopping} className={isBuyer ? 'buyer-icon' : 'action-icon'} style={{cursor: 'pointer'}} />
+                                              markAsBuyer();
+                                            }} icon={faBasketShopping} className={isBuyer ? 'buyer-icon' : 'action-icon'} style={{ cursor: 'pointer' }} />
                                          </span>}
                                          {login !== auth.login && locationKey === LunchChoice.OBJEDNAVAM && isBuyer && <span title='Objednávající'>
                                            <FontAwesomeIcon onClick={() => {
-                                            copyNote(userPayload.note!);
-                                          }} icon={faBasketShopping} className='buyer-icon' />
+                                              copyNote(userPayload.note!);
+                                            }} icon={faBasketShopping} className='buyer-icon' />
                                          </span>}
                                          {login !== auth.login && canChangeChoice && userPayload?.note?.length && <span title='Převzít poznámku'>
                                            <FontAwesomeIcon onClick={() => {
-                                            copyNote(userPayload.note!);
-                                          }} className='action-icon' icon={faComment} />
+                                              copyNote(userPayload.note!);
+                                            }} className='action-icon' icon={faComment} />
                                          </span>}
                                          {login === auth.login && canChangeChoice && <span title='Upravit poznámku'>
                                            <FontAwesomeIcon onClick={() => {
-                                            setNoteModalOpen(true);
-                                          }} className='action-icon' icon={faNoteSticky} />
+                                              setNoteModalOpen(true);
+                                            }} className='action-icon' icon={faNoteSticky} />
                                          </span>}
                                          {login === auth.login && canChangeChoice && <span title={`Odstranit volbu ${locationName}, včetně případných zvolených jídel`}>
                                            <FontAwesomeIcon onClick={() => {
-                                            doRemoveChoices(key as LunchChoice);
-                                          }} className='action-icon' icon={faTrashCan} />
+                                              doRemoveChoices(key as LunchChoice);
+                                            }} className='action-icon' icon={faTrashCan} />
                                          </span>}
                                        </div>
                                      </div>
@@ -589,11 +670,11 @@ function App() {
                                            return <div key={foodIndex} className="food-choice-item">
                                              <span className="food-choice-name">{foodName}</span>
                                              {login === auth.login && canChangeChoice &&
-                                              <span title={`Odstranit ${foodName}`}>
-                                                <FontAwesomeIcon onClick={() => {
-                                                doRemoveFoodChoice(restaurantKey, foodIndex);
-                                              }} className='action-icon' icon={faTrashCan} />
-                                              </span>}
+                                                <span title={`Odstranit ${foodName}`}>
+                                                  <FontAwesomeIcon onClick={() => {
+                                                    doRemoveFoodChoice(restaurantKey, foodIndex);
+                                                  }} className='action-icon' icon={faTrashCan} />
+                                                </span>}
                                            </div>
                                          })}
                                        </div>
@@ -613,7 +694,7 @@ function App() {
                : <div className='no-votes mt-4'>Zatím nikdo nehlasoval...</div>
              }
            </div>
-            {dayIndex === data.todayDayIndex &&
+            {dayIndex === data.todayDayIndex && userHasPizzaChoice &&
              <div className='pizza-section fade-in'>
                {!data.pizzaDay &&
                  <>
--- a/server/src/index.ts
+++ b/server/src/index.ts
@@ -6,7 +6,7 @@ import dotenv from 'dotenv';
 import path from 'path';
 import { getQr } from "./qr";
 import { generateToken, getLogin, verify } from "./auth";
-import { getIsWeekend, InsufficientPermissions, parseToken } from "./utils";
+import { getIsWeekend, InsufficientPermissions, PizzaDayConflictError, parseToken } from "./utils";
 import { getPendingQrs } from "./pizza";
 import { initWebsocket } from "./websocket";
 import pizzaDayRoutes from "./routes/pizzaDayRoutes";
@@ -56,7 +56,7 @@ app.get("/api/whoami", (req, res) => {
    if (!HTTP_REMOTE_USER_ENABLED) {
        res.status(403).json({ error: 'Není zapnuté přihlášení z hlaviček' });
    }
-    if(process.env.ENABLE_HEADERS_LOGGING === 'yes'){
+    if (process.env.ENABLE_HEADERS_LOGGING === 'yes') {
        delete req.headers["cookie"]
        console.log(req.headers)
    }
@@ -68,7 +68,7 @@ app.post("/api/login", (req, res) => {
        // Autentizace pomocí trusted headers
        const remoteUser = req.header(HTTP_REMOTE_USER_HEADER_NAME);
        //const remoteName = req.header('remote-name');
-        if (remoteUser && remoteUser.length > 0 ) {
+        if (remoteUser && remoteUser.length > 0) {
            res.status(200).json(generateToken(Buffer.from(remoteUser, 'latin1').toString(), true));
        } else {
            throw Error("Je zapnuto přihlášení přes hlavičky, ale nepřišla hlavička nebo ??");
@@ -106,7 +106,7 @@ app.use("/api/", (req, res, next) => {
    if (HTTP_REMOTE_USER_ENABLED) {
        // Autentizace pomocí trusted headers
        const remoteUser = req.header(HTTP_REMOTE_USER_HEADER_NAME);
-        if(process.env.ENABLE_HEADERS_LOGGING === 'yes'){
+        if (process.env.ENABLE_HEADERS_LOGGING === 'yes') {
            delete req.headers["cookie"]
            console.log(req.headers)
        }
@@ -168,6 +168,8 @@ app.use(express.static('public'));
 app.use((err: any, req: any, res: any, next: any) => {
    if (err instanceof InsufficientPermissions) {
        res.status(403).send({ error: err.message })
+    } else if (err instanceof PizzaDayConflictError) {
+        res.status(409).send({ error: err.message })
    } else {
        res.status(500).send({ error: err.message })
    }
--- a/server/src/pizza.ts
+++ b/server/src/pizza.ts
@@ -113,6 +113,36 @@ export async function addPizzaOrder(login: string, pizza: Pizza, size: PizzaSize
    return clientData;
 }

+/**
+ * Odstraní všechny pizzy uživatele (celou jeho objednávku).
+ * Pokud Pizza day neexistuje nebo není ve stavu CREATED, neudělá nic.
+ *
+ * @param login login uživatele
+ * @param date datum, pro které se objednávka odstraňuje (výchozí je dnešek)
+ * @returns aktuální data pro klienta
+ */
+export async function removeAllUserPizzas(login: string, date?: Date) {
+    const usedDate = date ?? getToday();
+    const today = formatDate(usedDate);
+    const clientData = await getClientData(usedDate);
+
+    if (!clientData.pizzaDay) {
+        return clientData; // Pizza day neexistuje, není co mazat
+    }
+
+    if (clientData.pizzaDay.state !== PizzaDayState.CREATED) {
+        return clientData; // Pizza day není ve stavu CREATED, nelze mazat
+    }
+
+    const orderIndex = clientData.pizzaDay.orders!.findIndex(o => o.customer === login);
+    if (orderIndex >= 0) {
+        clientData.pizzaDay.orders!.splice(orderIndex, 1);
+        await storage.setData(today, clientData);
+    }
+
+    return clientData;
+}
+
 /**
 * Odstraní danou objednávku pizzy.
 *
--- a/server/src/service.ts
+++ b/server/src/service.ts
@@ -1,8 +1,9 @@
-import { InsufficientPermissions, formatDate, getDayOfWeekIndex, getFirstWorkDayOfWeek, getHumanDate, getIsWeekend, getWeekNumber } from "./utils";
+import { InsufficientPermissions, PizzaDayConflictError, formatDate, getDayOfWeekIndex, getFirstWorkDayOfWeek, getHumanDate, getIsWeekend, getWeekNumber } from "./utils";
 import getStorage from "./storage";
 import { getMenuSladovnicka, getMenuTechTower, getMenuZastavkaUmichala, getMenuSenkSerikova } from "./restaurants";
 import { getTodayMock } from "./mock";
-import { ClientData, DepartureTime, LunchChoice, Restaurant, RestaurantDayMenu, WeekMenu } from "../../types/gen/types.gen";
+import { removeAllUserPizzas } from "./pizza";
+import { ClientData, DepartureTime, LunchChoice, PizzaDayState, Restaurant, RestaurantDayMenu, WeekMenu } from "../../types/gen/types.gen";

 const storage = getStorage();
 const MENU_PREFIX = 'menu';
@@ -405,6 +406,29 @@ export async function addChoice(login: string, trusted: boolean, locationKey: Lu
    let data = await getClientData(usedDate);
    validateTrusted(data, login, trusted);
    await validateFoodIndex(locationKey, foodIndex, date);
+
+    // Pokud uživatel měl vybranou PIZZA a mění na něco jiného
+    const hadPizzaChoice = data.choices.PIZZA && login in data.choices.PIZZA;
+    if (hadPizzaChoice && locationKey !== LunchChoice.PIZZA && foodIndex == null) {
+        // Kontrola, zda existuje Pizza day a uživatel je jeho zakladatel
+        if (data.pizzaDay && data.pizzaDay.creator === login) {
+            // Pokud Pizza day není ve stavu CREATED, nelze změnit volbu
+            if (data.pizzaDay.state !== PizzaDayState.CREATED) {
+                throw new PizzaDayConflictError(
+                    `Nelze změnit volbu. Pizza day je ve stavu "${data.pizzaDay.state}" a musí být nejprve dokončen nebo smazán.`
+                );
+            }
+            // Pizza day je ve stavu CREATED - bude smazán frontendem po potvrzení uživatelem
+            // (frontend volá nejprve deletePizzaDay, pak teprve addChoice)
+        }
+
+        // Smažeme pizzy uživatele (pokud Pizza day nebyl založen tímto uživatelem,
+        // nebo byl již smazán frontendem)
+        await removeAllUserPizzas(login, usedDate);
+        // Znovu načteme data, protože removeAllUserPizzas je upravila
+        data = await getClientData(usedDate);
+    }
+
    // Pokud měníme pouze lokaci, mažeme případné předchozí
    if (foodIndex == null) {
        data = await removeChoiceIfPresent(login, usedDate);
--- a/server/src/utils.ts
+++ b/server/src/utils.ts
@@ -114,6 +114,8 @@ export const checkBodyParams = (req: any, paramNames: string[]) => {
 // TODO umístit do samostatného souboru
 export class InsufficientPermissions extends Error { }

+export class PizzaDayConflictError extends Error { }
+
 export const getUsersByLocation = (choices: LunchChoices, login?: string): string[] => {
    const result: string[] = [];