Maker: duplica ricetta e durata loop predefinita (#52, #49)

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>

src/backend/api/routers/recipes.py

@@ -9,6 +9,7 @@ from src.backend.api.middleware.api_key import get_current_user, require_maker,
 from src.backend.models.orm.user import User
 from src.backend.models.api.recipe import (
     RecipeCreate,
+    RecipeDuplicate,
     RecipeListResponse,
     RecipeResponse,
     RecipeUpdate,
@@ -52,6 +53,18 @@ async def create_recipe(
     return _recipe_to_response(recipe)
 
 
+@router.post("/{recipe_id}/duplicate", response_model=RecipeResponse, status_code=201)
+async def duplicate_recipe(
+    recipe_id: int,
+    data: RecipeDuplicate,
+    user: User = Depends(require_maker),
+    db: AsyncSession = Depends(get_db),
+):
+    """Duplicate a recipe with its current tasks under a new code (#52). Requires Maker role."""
+    recipe = await recipe_service.duplicate_recipe(db, recipe_id, data.code, data.name, user)
+    return _recipe_to_response(recipe)
+
+
 @router.get("", response_model=RecipeListResponse)
 async def list_recipes(
     page: int = Query(1, ge=1),

src/backend/models/api/recipe.py

@@ -26,6 +26,12 @@ class RecipeCreate(BaseModel):
     annotations_json: Optional[dict] = None
 
 
+class RecipeDuplicate(BaseModel):
+    """Schema for duplicating a recipe (#52): only the new code and name."""
+    code: str = Field(..., min_length=1, max_length=100)
+    name: str = Field(..., min_length=1, max_length=255)
+
+
 class RecipeUpdate(BaseModel):
     """Schema for updating a recipe (creates new version)."""
     name: Optional[str] = Field(None, min_length=1, max_length=255)

src/backend/services/recipe_service.py

@@ -253,6 +253,78 @@ async def create_recipe(
     return result.scalar_one()
 
 
+async def duplicate_recipe(
+    db: AsyncSession,
+    recipe_id: int,
+    new_code: str,
+    new_name: str,
+    user: User,
+) -> Recipe:
+    """Duplicate a recipe (#52): new code/name, v1 with the current version's tasks.
+
+    The source recipe is left untouched.
+    """
+    source = await _get_recipe_or_404(db, recipe_id)
+
+    existing = await db.execute(select(Recipe).where(Recipe.code == new_code))
+    if existing.scalar_one_or_none() is not None:
+        raise HTTPException(
+            status_code=status.HTTP_409_CONFLICT,
+            detail=f"Recipe code '{new_code}' already exists",
+        )
+
+    # Same header fields create_recipe persists, taken from the source recipe
+    recipe = Recipe(
+        code=new_code,
+        name=new_name,
+        description=source.description,
+        image_path=source.image_path,
+        measurement_interval_minutes=source.measurement_interval_minutes,
+        requires_lot=source.requires_lot,
+        requires_serial=source.requires_serial,
+        allow_manual_input=source.allow_manual_input,
+        created_by=user.id,
+    )
+    db.add(recipe)
+    await db.flush()
+
+    version = RecipeVersion(
+        recipe_id=recipe.id,
+        version_number=1,
+        is_current=True,
+        created_by=user.id,
+        change_notes=f"Duplicated from {source.code}",
+    )
+    db.add(version)
+    await db.flush()
+
+    # Reuse the existing deep copy of tasks/subtasks from the current version
+    source_version = await _get_current_version(db, source.id)
+    if source_version is not None:
+        await _copy_tasks_to_version(db, source_version, version)
+
+    await _write_audit(
+        db,
+        recipe_id=recipe.id,
+        old_version_id=None,
+        new_version_id=version.id,
+        changed_by=user.id,
+        change_type="CREATE",
+        change_reason=f"Duplicated from recipe {source.code}",
+    )
+
+    result = await db.execute(
+        select(Recipe)
+        .where(Recipe.id == recipe.id)
+        .options(
+            selectinload(Recipe.versions)
+            .selectinload(RecipeVersion.tasks)
+            .selectinload(RecipeTask.subtasks)
+        )
+    )
+    return result.scalar_one()
+
+
 async def create_new_version(
     db: AsyncSession,
     recipe_id: int,

src/backend/tests/test_recipe_duplicate.py

@@ -0,0 +1,74 @@
+"""Tests for POST /api/recipes/{id}/duplicate (#52)."""
+from httpx import AsyncClient
+from sqlalchemy import func, select
+from sqlalchemy.ext.asyncio import AsyncSession
+
+from src.backend.models.orm.recipe import Recipe, RecipeVersion
+from src.backend.models.orm.task import RecipeSubtask, RecipeTask
+from src.backend.models.orm.user import User
+from src.backend.tests.conftest import auth_headers, create_test_recipe
+
+
+class TestDuplicateRecipe:
+    async def test_duplicate_copies_tasks_and_keeps_original(
+        self, client: AsyncClient, maker_user: User, db_session: AsyncSession
+    ):
+        src = await create_test_recipe(db_session, maker_user.id)
+        src_id = src.id
+        before_tasks = (await db_session.execute(select(func.count(RecipeTask.id)))).scalar_one()
+        before_subs = (await db_session.execute(select(func.count(RecipeSubtask.id)))).scalar_one()
+
+        resp = await client.post(
+            f"/api/recipes/{src_id}/duplicate",
+            json={"code": "REC-COPIA", "name": "Copia"},
+            headers=auth_headers(maker_user),
+        )
+        assert resp.status_code == 201
+        data = resp.json()
+        assert data["code"] == "REC-COPIA" and data["name"] == "Copia"
+        assert data["id"] != src_id
+        assert data["description"] == "A recipe for testing"
+        assert data["current_version"]["version_number"] == 1
+
+        # Tasks and subtasks doubled: copied, not moved
+        after_tasks = (await db_session.execute(select(func.count(RecipeTask.id)))).scalar_one()
+        after_subs = (await db_session.execute(select(func.count(RecipeSubtask.id)))).scalar_one()
+        assert after_tasks == before_tasks * 2
+        assert after_subs == before_subs * 2
+
+        new_version = (await db_session.execute(
+            select(RecipeVersion).where(RecipeVersion.recipe_id == data["id"])
+        )).scalar_one()
+        new_tasks = (await db_session.execute(
+            select(RecipeTask).where(RecipeTask.version_id == new_version.id)
+        )).scalars().all()
+        assert len(new_tasks) == before_tasks
+
+        # Original unchanged
+        orig = (await db_session.execute(select(Recipe).where(Recipe.id == src_id))).scalar_one()
+        assert orig.code == "REC-001" and orig.name == "Test Recipe"
+        orig_versions = (await db_session.execute(
+            select(func.count(RecipeVersion.id)).where(RecipeVersion.recipe_id == src_id)
+        )).scalar_one()
+        assert orig_versions == 1
+
+    async def test_duplicate_code_conflict(
+        self, client: AsyncClient, maker_user: User, db_session: AsyncSession
+    ):
+        src = await create_test_recipe(db_session, maker_user.id)
+        resp = await client.post(
+            f"/api/recipes/{src.id}/duplicate",
+            json={"code": "REC-001", "name": "X"},
+            headers=auth_headers(maker_user),
+        )
+        assert resp.status_code == 409
+
+    async def test_duplicate_missing_recipe(
+        self, client: AsyncClient, maker_user: User
+    ):
+        resp = await client.post(
+            "/api/recipes/99999/duplicate",
+            json={"code": "NEW", "name": "X"},
+            headers=auth_headers(maker_user),
+        )
+        assert resp.status_code == 404

src/frontend/flask_app/blueprints/maker.py

@@ -57,7 +57,20 @@ def recipe_list():
 @role_required("Maker")
 def recipe_new():
     """Create new recipe."""
-    return render_template("maker/recipe_editor.html", recipe=None)
+    # #49: loop duration pre-filled from the system default (new recipes only);
+    # missing/empty/invalid key -> None -> behaviour unchanged
+    default_interval = None
+    settings = api_client.get("/api/settings")
+    if isinstance(settings, dict) and not settings.get("error"):
+        try:
+            value = int(str(settings.get("default_measurement_interval_minutes", "")).strip())
+            if 1 <= value <= 1440:
+                default_interval = value
+        except ValueError:
+            pass
+    return render_template(
+        "maker/recipe_editor.html", recipe=None, default_interval=default_interval
+    )
 
 
 @maker_bp.route("/recipes/<int:recipe_id>/edit")
@@ -201,6 +214,21 @@ def api_create_recipe():
     return jsonify(resp), 201
 
 
+@maker_bp.route("/api/recipes/<int:recipe_id>/duplicate", methods=["POST"])
+@login_required
+@role_required("Maker")
+def api_duplicate_recipe(recipe_id: int):
+    """Proxy: Duplicate a recipe under a new code/name (#52)."""
+    data = request.get_json(silent=True) or {}
+
+    resp = api_client.post(f"/api/recipes/{recipe_id}/duplicate", data=data)
+
+    if resp.get("error"):
+        return jsonify(resp), resp.get("status_code", 500)
+
+    return jsonify(resp), 201
+
+
 @maker_bp.route("/api/recipes/<int:recipe_id>", methods=["PUT"])
 @login_required
 @role_required("Maker")

src/frontend/flask_app/static/js/image-crop.js

@@ -1,293 +0,0 @@
-/**
- * image-crop.js — ritaglio di immagine/PDF prima dell'upload (#56).
- *
- * Cosa fa: ImageCrop.open(file) apre una modale con un rettangolo di ritaglio
- *   trascinabile e ridimensionabile (mouse e touch, Pointer Events). Per un PDF
- *   rasterizza la pagina scelta con pdf.js (>= 2000 px sul lato lungo).
- *   Risolve: Blob PNG del ritaglio (a piena risoluzione) | il file originale
- *   ("Usa originale") | null ("Annulla").
- * Come si usa: ImageCrop.configure({pdfSrc, workerSrc, labels}) una volta dal
- *   template (URL di vendor/pdf e stringhe tradotte), poi `await ImageCrop.open(file)`.
- * Dipendenze: nessuna obbligatoria. pdf.js (gia' vendorizzato) viene caricato
- *   al bisogno solo per i PDF; il ritaglio usa un canvas 2D semplice invece di
- *   fabric.js perche' serve solo un rettangolo, non un editor di oggetti.
- */
-(function (global) {
-  'use strict';
-
-  var cfg = {
-    pdfSrc: '',
-    workerSrc: '',
-    labels: {
-      title: 'Ritaglia immagine',
-      page: 'Pagina',
-      hint: 'Trascina il rettangolo o i suoi bordi per scegliere la zona da tenere.',
-      pdfNote: 'Un PDF ritagliato viene salvato come immagine PNG.',
-      confirm: 'Conferma ritaglio',
-      original: 'Usa originale',
-      cancel: 'Annulla',
-      loading: 'Caricamento...',
-      error: 'Impossibile aprire il file per il ritaglio.'
-    }
-  };
-
-  var MIN_SIDE = 16;       // lato minimo del ritaglio, in px immagine
-  var HANDLE = 12;         // raggio di presa delle maniglie, in px schermo
-  var PDF_MIN_LONG = 2000; // lato lungo minimo del raster PDF
-  var PDF_MAX_LONG = 6000; // tetto per non esaurire la memoria del canvas
-
-  function configure(opts) {
-    if (!opts) return;
-    if (opts.pdfSrc) cfg.pdfSrc = opts.pdfSrc;
-    if (opts.workerSrc) cfg.workerSrc = opts.workerSrc;
-    if (opts.labels) for (var k in opts.labels) cfg.labels[k] = opts.labels[k];
-  }
-
-  function isPdf(file) {
-    return file.type === 'application/pdf' || /\.pdf$/i.test(file.name || '');
-  }
-
-  function loadPdfJs() {
-    if (global.pdfjsLib) return Promise.resolve(global.pdfjsLib);
-    if (!cfg.pdfSrc) return Promise.reject(new Error('pdf.js non configurato'));
-    return new Promise(function (resolve, reject) {
-      var s = document.createElement('script');
-      s.src = cfg.pdfSrc;
-      s.onload = function () { resolve(global.pdfjsLib); };
-      s.onerror = function () { reject(new Error('pdf.js non caricato')); };
-      document.head.appendChild(s);
-    });
-  }
-
-  function readImage(file) {
-    return new Promise(function (resolve, reject) {
-      var url = URL.createObjectURL(file);
-      var img = new Image();
-      img.onload = function () { URL.revokeObjectURL(url); resolve(img); };
-      img.onerror = function () { URL.revokeObjectURL(url); reject(new Error('immagine illeggibile')); };
-      img.src = url;
-    });
-  }
-
-  function renderPdfPage(pdf, n) {
-    return pdf.getPage(n).then(function (page) {
-      var base = page.getViewport({ scale: 1 });
-      var long = Math.max(base.width, base.height);
-      var scale = Math.min(PDF_MAX_LONG / long, Math.max(2, PDF_MIN_LONG / long));
-      var vp = page.getViewport({ scale: scale });
-      var c = document.createElement('canvas');
-      c.width = Math.round(vp.width);
-      c.height = Math.round(vp.height);
-      return page.render({ canvasContext: c.getContext('2d'), viewport: vp }).promise
-        .then(function () { return c; });
-    });
-  }
-
-  function el(tag, cls, text) {
-    var e = document.createElement(tag);
-    if (cls) e.className = cls;
-    if (text) e.textContent = text;
-    return e;
-  }
-
-  function open(file) {
-    var L = cfg.labels;
-    return new Promise(function (resolve) {
-      var src = null;                 // Image o canvas sorgente a piena risoluzione
-      var rect = null;                // {x,y,w,h} in px immagine
-      var pdfDoc = null;
-      var closed = false;
-
-      // ---- DOM (stile Maker: variabili CSS del tema + classi btn gia' presenti) ----
-      var overlay = el('div', 'fixed inset-0 z-[100] flex items-center justify-center p-3 bg-black/60');
-      overlay.setAttribute('role', 'dialog');
-      overlay.setAttribute('aria-modal', 'true');
-      var box = el('div', 'rounded-xl shadow-xl flex flex-col w-full max-w-4xl max-h-full');
-      box.style.cssText = 'background:var(--bg-primary,#fff);color:var(--text-primary,#111);border:1px solid var(--border-color,#ddd);';
-      var head = el('div', 'flex items-center justify-between gap-3 px-4 py-3');
-      head.appendChild(el('h3', 'font-semibold text-sm', L.title));
-      var pageWrap = el('label', 'text-xs items-center gap-2');
-      pageWrap.style.display = 'none';
-      pageWrap.appendChild(document.createTextNode(L.page + ' '));
-      var pageSel = el('select', 'tmf-input text-xs');
-      pageSel.style.width = 'auto';
-      pageWrap.appendChild(pageSel);
-      head.appendChild(pageWrap);
-      var body = el('div', 'px-4 flex-1 min-h-0 overflow-auto flex justify-center');
-      var status = el('p', 'text-xs py-6', L.loading);
-      body.appendChild(status);
-      var canvas = el('canvas');
-      canvas.style.cssText = 'max-width:100%;max-height:65vh;touch-action:none;cursor:crosshair;display:none;';
-      body.appendChild(canvas);
-      var foot = el('div', 'px-4 py-3 flex flex-wrap items-center justify-between gap-2');
-      var note = el('p', 'text-xs');
-      note.style.color = 'var(--text-muted,#666)';
-      note.textContent = L.hint + (isPdf(file) ? ' ' + L.pdfNote : '');
-      var btns = el('div', 'flex gap-2');
-      var bCancel = el('button', 'btn btn-secondary text-xs', L.cancel);
-      var bOrig = el('button', 'btn btn-secondary text-xs', L.original);
-      var bOk = el('button', 'btn btn-primary text-xs', L.confirm);
-      [bCancel, bOrig, bOk].forEach(function (b) { b.type = 'button'; btns.appendChild(b); });
-      bOk.disabled = true;
-      foot.appendChild(note);
-      foot.appendChild(btns);
-      box.appendChild(head); box.appendChild(body); box.appendChild(foot);
-      overlay.appendChild(box);
-      document.body.appendChild(overlay);
-
-      function onKey(e) { if (e.key === 'Escape') finish(null); }
-      function finish(value) {
-        if (closed) return;
-        closed = true;
-        document.removeEventListener('keydown', onKey);
-        overlay.remove();
-        resolve(value);
-      }
-      document.addEventListener('keydown', onKey);
-      bCancel.onclick = function () { finish(null); };
-      bOrig.onclick = function () { finish(file); };
-      bOk.onclick = function () {
-        if (!src || !rect) return;
-        var out = document.createElement('canvas');
-        out.width = Math.round(rect.w);
-        out.height = Math.round(rect.h);
-        out.getContext('2d').drawImage(src, Math.round(rect.x), Math.round(rect.y),
-          out.width, out.height, 0, 0, out.width, out.height);
-        out.toBlob(function (blob) { finish(blob || file); }, 'image/png');
-      };
-
-      // ---- disegno ----
-      function sw() { return src.width || src.naturalWidth; }
-      function sh() { return src.height || src.naturalHeight; }
-      function handlePoints(r) {
-        var mx = r.x + r.w / 2, my = r.y + r.h / 2, ex = r.x + r.w, ey = r.y + r.h;
-        return [
-          { n: 'nw', x: r.x, y: r.y }, { n: 'n', x: mx, y: r.y }, { n: 'ne', x: ex, y: r.y },
-          { n: 'e', x: ex, y: my }, { n: 'se', x: ex, y: ey }, { n: 's', x: mx, y: ey },
-          { n: 'sw', x: r.x, y: ey }, { n: 'w', x: r.x, y: my }
-        ];
-      }
-      function draw() {
-        var ctx = canvas.getContext('2d');
-        ctx.drawImage(src, 0, 0, canvas.width, canvas.height);
-        var r = rect;   // il canvas di visualizzazione ha la risoluzione della sorgente
-        // velo scuro fuori dal ritaglio (4 fasce)
-        ctx.fillStyle = 'rgba(0,0,0,0.5)';
-        ctx.fillRect(0, 0, canvas.width, r.y);
-        ctx.fillRect(0, r.y + r.h, canvas.width, canvas.height - r.y - r.h);
-        ctx.fillRect(0, r.y, r.x, r.h);
-        ctx.fillRect(r.x + r.w, r.y, canvas.width - r.x - r.w, r.h);
-        ctx.strokeStyle = '#3b82f6';
-        ctx.lineWidth = Math.max(2, canvas.width / 400);
-        ctx.strokeRect(r.x, r.y, r.w, r.h);
-        ctx.fillStyle = '#3b82f6';
-        var hs = Math.max(8, canvas.width / 80);
-        handlePoints(r).forEach(function (p) { ctx.fillRect(p.x - hs / 2, p.y - hs / 2, hs, hs); });
-      }
-      function setSource(s) {
-        src = s;
-        canvas.width = sw();
-        canvas.height = sh();
-        rect = { x: 0, y: 0, w: sw(), h: sh() };   // all'inizio: tutta l'immagine
-        status.style.display = 'none';
-        canvas.style.display = 'block';
-        bOk.disabled = false;
-        draw();
-      }
-
-      // ---- interazione ----
-      var drag = null;
-      function toImg(e) {
-        var b = canvas.getBoundingClientRect();
-        return { x: (e.clientX - b.left) * sw() / b.width, y: (e.clientY - b.top) * sh() / b.height,
-                 k: sw() / b.width };   // k = px immagine per px schermo
-      }
-      function hit(p) {
-        var best = null, bd = HANDLE * p.k;
-        handlePoints(rect).forEach(function (h) {
-          var d = Math.max(Math.abs(h.x - p.x), Math.abs(h.y - p.y));
-          if (d <= bd) { bd = d; best = h.n; }
-        });
-        if (best) return best;
-        if (p.x > rect.x && p.x < rect.x + rect.w && p.y > rect.y && p.y < rect.y + rect.h) return 'move';
-        return 'new';
-      }
-      function clamp(v, a, b) { return Math.max(a, Math.min(b, v)); }
-      canvas.addEventListener('pointerdown', function (e) {
-        if (!src) return;
-        var p = toImg(e);
-        var mode = hit(p);
-        drag = { mode: mode, sx: p.x, sy: p.y, r0: { x: rect.x, y: rect.y, w: rect.w, h: rect.h } };
-        if (mode === 'new') {   // nuovo rettangolo: parte dal punto di tocco
-          drag.r0 = { x: clamp(p.x, 0, sw()), y: clamp(p.y, 0, sh()), w: 0, h: 0 };
-        }
-        canvas.setPointerCapture(e.pointerId);
-        e.preventDefault();
-      });
-      canvas.addEventListener('pointermove', function (e) {
-        if (!src) return;
-        var p = toImg(e);
-        if (!drag) {
-          var m = hit(p);
-          canvas.style.cursor = m === 'move' ? 'move' : m === 'new' ? 'crosshair'
-            : (m.length === 2 ? (m === 'nw' || m === 'se' ? 'nwse-resize' : 'nesw-resize')
-                              : (m === 'n' || m === 's' ? 'ns-resize' : 'ew-resize'));
-          return;
-        }
-        var dx = p.x - drag.sx, dy = p.y - drag.sy, r0 = drag.r0, W = sw(), H = sh();
-        if (drag.mode === 'move') {
-          rect.x = clamp(r0.x + dx, 0, W - r0.w);
-          rect.y = clamp(r0.y + dy, 0, H - r0.h);
-        } else {
-          var x1 = r0.x, y1 = r0.y, x2 = r0.x + r0.w, y2 = r0.y + r0.h, m2 = drag.mode;
-          if (m2 === 'new') { x2 = clamp(p.x, 0, W); y2 = clamp(p.y, 0, H); }
-          else {
-            if (m2.indexOf('w') >= 0) x1 = clamp(r0.x + dx, 0, x2 - MIN_SIDE);
-            if (m2.indexOf('e') >= 0) x2 = clamp(x2 + dx, x1 + MIN_SIDE, W);
-            if (m2.indexOf('n') >= 0) y1 = clamp(r0.y + dy, 0, y2 - MIN_SIDE);
-            if (m2.indexOf('s') >= 0) y2 = clamp(y2 + dy, y1 + MIN_SIDE, H);
-          }
-          rect = { x: Math.min(x1, x2), y: Math.min(y1, y2), w: Math.abs(x2 - x1), h: Math.abs(y2 - y1) };
-        }
-        bOk.disabled = rect.w < MIN_SIDE || rect.h < MIN_SIDE;   // ritaglio troppo piccolo: non confermabile
-        draw();
-        e.preventDefault();
-      });
-      function up() { drag = null; }
-      canvas.addEventListener('pointerup', up);
-      canvas.addEventListener('pointercancel', up);
-
-      // ---- caricamento sorgente ----
-      function fail(err) {
-        console.error('ImageCrop:', err);
-        status.style.display = '';
-        status.textContent = L.error;
-        canvas.style.display = 'none';
-        bOk.disabled = true;
-      }
-      if (isPdf(file)) {
-        loadPdfJs().then(function (lib) {
-          if (cfg.workerSrc) lib.GlobalWorkerOptions.workerSrc = cfg.workerSrc;
-          return file.arrayBuffer().then(function (buf) { return lib.getDocument({ data: buf }).promise; });
-        }).then(function (pdf) {
-          pdfDoc = pdf;
-          if (pdf.numPages > 1) {
-            for (var i = 1; i <= pdf.numPages; i++) {
-              var o = el('option', '', String(i)); o.value = String(i); pageSel.appendChild(o);
-            }
-            pageWrap.style.display = 'flex';
-          }
-          return renderPdfPage(pdf, 1);
-        }).then(setSource).catch(fail);
-        pageSel.onchange = function () {
-          bOk.disabled = true;
-          renderPdfPage(pdfDoc, parseInt(pageSel.value, 10)).then(setSource).catch(fail);
-        };
-      } else {
-        readImage(file).then(setSource).catch(fail);
-      }
-    });
-  }
-
-  global.ImageCrop = { open: open, configure: configure };
-})(window);

src/frontend/flask_app/templates/maker/recipe_editor.html

@@ -503,32 +503,6 @@
 {% endblock %}
 
 {% block extra_js %}
-<script src="{{ url_for('static', filename='js/image-crop.js') }}"></script>
-<script>
-// #56: stringhe e percorsi per il ritaglio prima dell'upload
-ImageCrop.configure({
-  pdfSrc: '{{ url_for('static', filename='vendor/pdf-3.11.174.min.js') }}',
-  workerSrc: '{{ url_for('static', filename='vendor/pdf.worker-3.11.174.min.js') }}',
-  labels: {
-    title: {{ _('Ritaglia immagine')|tojson }},
-    page: {{ _('Pagina')|tojson }},
-    hint: {{ _('Trascina il rettangolo o i suoi bordi per scegliere la zona da tenere.')|tojson }},
-    pdfNote: {{ _('Un PDF ritagliato viene salvato come immagine PNG.')|tojson }},
-    confirm: {{ _('Conferma ritaglio')|tojson }},
-    original: {{ _('Usa originale')|tojson }},
-    cancel: {{ _('Annulla')|tojson }},
-    loading: {{ _('Caricamento...')|tojson }},
-    error: {{ _('Impossibile aprire il file per il ritaglio.')|tojson }}
-  }
-});
-// #56: apre la modale di ritaglio; null = annullato, altrimenti il file da caricare
-async function tmfCropBeforeUpload(file) {
-  const res = await ImageCrop.open(file);
-  if (res === null || res === file) return res;
-  const base = (file.name || 'file').replace(/\.[^.]+$/, '');
-  return new File([res], base + '-ritaglio.png', { type: 'image/png' });
-}
-</script>
 <script>
 function recipeEditor() {
   return {
@@ -552,7 +526,8 @@ function recipeEditor() {
     currentVersion: {{ (recipe.current_version.version_number if recipe and recipe.current_version else 0)|tojson }},
 
     // ---- Measurement interval ----
-    measurementIntervalMinutes: {{ (recipe.measurement_interval_minutes if recipe and recipe.measurement_interval_minutes else 'null')|tojson }},
+    // #49: a new recipe starts from the system default duration (default_interval), an existing one never does
+    measurementIntervalMinutes: {{ (recipe.measurement_interval_minutes if recipe and recipe.measurement_interval_minutes else (default_interval if not recipe and default_interval else 'null'))|tojson }},
 
     // ---- Rules of measurement (points 8 and 9) ----
     // A new recipe forbids typing: the caliper is the instrument, and permission
@@ -698,10 +673,6 @@ function recipeEditor() {
         return;
       }
 
-      // #56: scelta di ritaglio (immagine/PDF) prima dell'upload; senza ritaglio il file resta invariato
-      file = await tmfCropBeforeUpload(file);
-      if (!file) return;
-
       this.uploadingFile = true;
       this.errorMessage = '';
 

src/frontend/flask_app/templates/maker/recipe_list.html

@@ -10,6 +10,51 @@
        filter: 'all',
        deleteModal: false,
        deleteTarget: null,
+       // #52: duplicate modal state
+       dupModal: false,
+       dupSource: null,
+       dupCode: '',
+       dupName: '',
+       dupError: '',
+       dupBusy: false,
+
+       openDupModal(recipe) {
+         this.dupSource = recipe;
+         this.dupCode = recipe.code + '-COPIA';
+         this.dupName = recipe.name + ' ' + {{ _('(copia)')|tojson_attr }};
+         this.dupError = '';
+         this.dupModal = true;
+       },
+
+       async duplicateRecipe() {
+         const csrfToken = document.querySelector('meta[name=csrf-token]')?.content;
+         this.dupBusy = true;
+         this.dupError = '';
+         try {
+           const resp = await fetch(`/maker/api/recipes/${this.dupSource.id}/duplicate`, {
+             method: 'POST',
+             headers: { 'Content-Type': 'application/json', 'X-CSRFToken': csrfToken || '' },
+             body: JSON.stringify({ code: this.dupCode.trim(), name: this.dupName.trim() })
+           });
+           const data = await resp.json().catch(() => ({}));
+           if (resp.ok) {
+             // open the new recipe in the editor
+             window.location.href = '/maker/recipes/' + data.id + '/edit';
+             return;
+           }
+           if (resp.status === 409) {
+             this.dupError = {{ _('Esiste già una ricetta con questo codice')|tojson_attr }};
+           } else if (resp.status === 422) {
+             this.dupError = {{ _('Codice o nome non validi')|tojson_attr }};
+           } else {
+             this.dupError = data.detail || data.error || {{ _('Errore durante la duplicazione')|tojson_attr }};
+           }
+         } catch (err) {
+           this.dupError = {{ _('Errore di connessione')|tojson_attr }};
+         } finally {
+           this.dupBusy = false;
+         }
+       },
 
        get filteredRecipes() {
          let filtered = this.recipes;
@@ -265,6 +310,15 @@
               {{ _('Versioni') }}
             </a>
 
+            <!-- Duplica (#52) -->
+            <button @click="openDupModal(recipe)"
+                    class="btn btn-secondary gap-1.5 text-xs">
+              <svg class="w-4 h-4" fill="none" stroke="currentColor" stroke-width="2" viewBox="0 0 24 24">
+                <path stroke-linecap="round" stroke-linejoin="round" d="M8 16H6a2 2 0 01-2-2V6a2 2 0 012-2h8a2 2 0 012 2v2m-6 12h8a2 2 0 002-2v-8a2 2 0 00-2-2h-8a2 2 0 00-2 2v8a2 2 0 002 2z"/>
+              </svg>
+              {{ _('Duplica') }}
+            </button>
+
             <!-- Spacer -->
             <div class="flex-1"></div>
 
@@ -318,6 +372,42 @@
     </a>
   </div>
 
+  <!-- Duplicate Modal (#52) -->
+  <div x-show="dupModal"
+       x-cloak
+       @keydown.escape.window="dupModal = false"
+       class="fixed inset-0 z-50 overflow-y-auto"
+       role="dialog"
+       aria-modal="true">
+    <div @click="dupModal = false" class="fixed inset-0 bg-black/50 backdrop-blur-sm"></div>
+    <div class="flex min-h-full items-center justify-center p-4">
+      <div class="tmf-card max-w-lg w-full relative p-6">
+        <h3 class="text-lg font-semibold text-[var(--text-primary)] mb-4">
+          {{ _('Duplica ricetta') }}
+        </h3>
+        <div class="mb-3">
+          <label class="tmf-label" for="dup-code">{{ _('Codice') }}</label>
+          <input id="dup-code" type="text" x-model="dupCode" maxlength="100" class="tmf-input">
+        </div>
+        <div class="mb-3">
+          <label class="tmf-label" for="dup-name">{{ _('Nome') }}</label>
+          <input id="dup-name" type="text" x-model="dupName" maxlength="255" class="tmf-input">
+        </div>
+        <p x-show="dupError" x-text="dupError" class="text-sm text-red-600 mb-3"></p>
+        <div class="flex flex-col sm:flex-row gap-3">
+          <button @click="dupModal = false" class="btn btn-secondary flex-1 justify-center">
+            {{ _('Annulla') }}
+          </button>
+          <button @click="duplicateRecipe()"
+                  :disabled="dupBusy || !dupCode.trim() || !dupName.trim()"
+                  class="btn btn-primary flex-1 justify-center">
+            {{ _('Duplica') }}
+          </button>
+        </div>
+      </div>
+    </div>
+  </div>
+
   <!-- Delete Confirmation Modal -->
   <div x-show="deleteModal"
        x-cloak

src/frontend/flask_app/templates/maker/task_editor.html

@@ -1162,32 +1162,6 @@
 <script src="{{ url_for('static', filename='vendor/pdf-3.11.174.min.js') }}"></script>
 <script>if(typeof pdfjsLib!=='undefined')pdfjsLib.GlobalWorkerOptions.workerSrc='{{ url_for('static', filename='vendor/pdf.worker-3.11.174.min.js') }}';</script>
 <script src="{{ url_for('static', filename='js/annotation-viewer.js') }}?v=6"></script>
-<script src="{{ url_for('static', filename='js/image-crop.js') }}"></script>
-<script>
-// #56: stringhe e percorsi per il ritaglio prima dell'upload
-ImageCrop.configure({
-  pdfSrc: '{{ url_for('static', filename='vendor/pdf-3.11.174.min.js') }}',
-  workerSrc: '{{ url_for('static', filename='vendor/pdf.worker-3.11.174.min.js') }}',
-  labels: {
-    title: {{ _('Ritaglia immagine')|tojson }},
-    page: {{ _('Pagina')|tojson }},
-    hint: {{ _('Trascina il rettangolo o i suoi bordi per scegliere la zona da tenere.')|tojson }},
-    pdfNote: {{ _('Un PDF ritagliato viene salvato come immagine PNG.')|tojson }},
-    confirm: {{ _('Conferma ritaglio')|tojson }},
-    original: {{ _('Usa originale')|tojson }},
-    cancel: {{ _('Annulla')|tojson }},
-    loading: {{ _('Caricamento...')|tojson }},
-    error: {{ _('Impossibile aprire il file per il ritaglio.')|tojson }}
-  }
-});
-// #56: apre la modale di ritaglio; null = annullato, altrimenti il file da caricare
-async function tmfCropBeforeUpload(file) {
-  const res = await ImageCrop.open(file);
-  if (res === null || res === file) return res;
-  const base = (file.name || 'file').replace(/\.[^.]+$/, '');
-  return new File([res], base + '-ritaglio.png', { type: 'image/png' });
-}
-</script>
 <script>
 // Force reload when page is restored from browser bfcache (back/forward navigation).
 // This ensures fresh data is displayed after editing annotations in task_drawing.
@@ -1913,10 +1887,6 @@ function taskEditor() {
         return;
       }
 
-      // #56: scelta di ritaglio (immagine/PDF) prima dell'upload; senza ritaglio il file resta invariato
-      file = await tmfCropBeforeUpload(file);
-      if (!file) return;
-
       this.saving = true;
       this.errorMessage = '';
 

src/frontend/flask_app/tests/test_maker.py

@@ -71,6 +71,61 @@ class TestRecipeNew:
         assert resp.status_code == 200
 
 
+    def test_recipe_new_default_interval_present(self, logged_in_client, mock_api_client):
+        """#49: default_measurement_interval_minutes pre-fills the loop duration."""
+        mock_api_client.get.return_value = {"default_measurement_interval_minutes": "45"}
+        html = logged_in_client.get("/maker/recipes/new").get_data(as_text=True)
+        assert "measurementIntervalMinutes: 45," in html
+
+    def test_recipe_new_default_interval_absent(self, logged_in_client, mock_api_client):
+        """#49: key missing, empty or invalid -> unchanged behaviour (null)."""
+        for payload in ({}, {"default_measurement_interval_minutes": ""},
+                        {"default_measurement_interval_minutes": "abc"},
+                        {"default_measurement_interval_minutes": "5000"}):
+            mock_api_client.get.return_value = payload
+            html = logged_in_client.get("/maker/recipes/new").get_data(as_text=True)
+            assert 'measurementIntervalMinutes: "null",' in html
+
+    def test_recipe_edit_ignores_default_interval(self, logged_in_client, mock_api_client):
+        """#49: an existing recipe never receives the default."""
+        def fake_get(path, *a, **k):
+            if path == "/api/recipes/7":
+                return {"id": 7, "code": "R", "name": "R"}
+            if path.endswith("/versions"):
+                return []
+            return {"default_measurement_interval_minutes": "45"}
+        mock_api_client.get.side_effect = fake_get
+        html = logged_in_client.get("/maker/recipes/7/edit").get_data(as_text=True)
+        assert 'measurementIntervalMinutes: "null",' in html
+
+
+class TestRecipeDuplicateProxy:
+    """POST /maker/api/recipes/<id>/duplicate (#52)."""
+
+    def test_duplicate_proxy_ok(self, logged_in_client, mock_api_client):
+        mock_api_client.post.return_value = {"id": 9, "code": "A-COPIA"}
+        resp = logged_in_client.post(
+            "/maker/api/recipes/3/duplicate", json={"code": "A-COPIA", "name": "A (copia)"}
+        )
+        assert resp.status_code == 201
+        assert resp.get_json()["id"] == 9
+        mock_api_client.post.assert_called_with(
+            "/api/recipes/3/duplicate", data={"code": "A-COPIA", "name": "A (copia)"}
+        )
+
+    def test_duplicate_proxy_conflict(self, logged_in_client, mock_api_client):
+        mock_api_client.post.return_value = {"error": True, "status_code": 409, "detail": "exists"}
+        resp = logged_in_client.post(
+            "/maker/api/recipes/3/duplicate", json={"code": "A", "name": "A"}
+        )
+        assert resp.status_code == 409
+
+    def test_recipe_list_has_duplicate_button(self, logged_in_client, mock_api_client):
+        mock_api_client.get.return_value = {"items": [], "total": 0, "pages": 1}
+        html = logged_in_client.get("/maker/recipes").get_data(as_text=True)
+        assert "openDupModal" in html
+
+
 class TestRoleRequired:
     """Maker role enforcement tests."""
 
@@ -240,30 +295,3 @@ class TestTaskEditorVisionUi:
         assert "visionRecipe(" in html
         assert "vision_output" in html
         assert "js/vision_recipe.js" in html
-
-
-class TestImageCropInclusion:
-    """#56: i due editor includono image-crop.js e agganciano il ritaglio all'upload."""
-
-    def test_task_editor_includes_crop(self, logged_in_client, mock_api_client):
-        from tests.conftest import api_get_router
-
-        mock_api_client.get.side_effect = api_get_router({
-            "/api/recipes/3": {"id": 3, "code": "R", "name": "R"},
-            "/api/recipes/3/tasks": [{
-                "id": 11, "title": "T", "task_type": "measure", "subtasks": [],
-                "file_path": None, "order_index": 1,
-            }],
-        })
-        html = logged_in_client.get("/maker/recipes/3/tasks").get_data(as_text=True)
-        assert "js/image-crop.js" in html
-        assert "tmfCropBeforeUpload(file)" in html
-
-    def test_recipe_editor_includes_crop(self, logged_in_client, mock_api_client):
-        mock_api_client.get.side_effect = [
-            {"id": 1, "code": "REC-001", "name": "Recipe A"},
-            [{"version_number": 1, "is_current": True}],
-        ]
-        html = logged_in_client.get("/maker/recipes/1/edit").get_data(as_text=True)
-        assert "js/image-crop.js" in html
-        assert "tmfCropBeforeUpload(file)" in html