From b837def5a0caa056cc2b72d27353e998d3be798b Mon Sep 17 00:00:00 2001
From: andre <andre.filipe6@gmail.com>
Date: Fri, 25 Sep 2026 21:45:08 +0100
Subject: [PATCH] refactor(horas): partilhar _form.php entre create e edit
 (audit 6.8)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Mesmo padrão de horario/: extrai dia/abertura/fecho para _form.php.
create.php formatava open/close sem substr() enquanto edit.php cortava
para "HH:MM" com substr(..., 0, 5) por vir de uma coluna TIME da BD;
unificado na versão de edit.php (sem efeito em create.php).

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
---
 views/admin/work_hours/_form.php  | 19 +++++++++++++++++++
 views/admin/work_hours/create.php | 14 +-------------
 views/admin/work_hours/edit.php   | 14 +-------------
 3 files changed, 21 insertions(+), 26 deletions(-)
 create mode 100644 views/admin/work_hours/_form.php

diff --git a/views/admin/work_hours/_form.php b/views/admin/work_hours/_form.php
new file mode 100644
index 0000000..4df8dbd
--- /dev/null
+++ b/views/admin/work_hours/_form.php
@@ -0,0 +1,19 @@
+<?php
+// Shared opening-hours form fields, rendered inside the <form> of both create.php
+// and edit.php. Expects:
+//   $data : associative array with every work_hours column (week_day, open, close)
+//   $dias : week_day value => label map
+?>
+<div class="mb-3"><label class="form-label">Dia <span class="text-danger">*</span></label>
+    <select class="form-select" name="week_day" required>
+        <option value="">Selecionar…</option>
+        <?php foreach ($dias as $val => $label): ?>
+        <option value="<?= $val ?>" <?= ($data['week_day'] ?? '') === $val ? 'selected' : '' ?>><?= $label ?></option>
+        <?php endforeach; ?>
+    </select></div>
+<div class="row g-3">
+    <div class="col-6"><label class="form-label">Abertura <span class="text-danger">*</span></label>
+        <input type="time" class="form-control" name="open" required value="<?= htmlspecialchars(substr($data['open'] ?? '09:00', 0, 5)) ?>"></div>
+    <div class="col-6"><label class="form-label">Fecho <span class="text-danger">*</span></label>
+        <input type="time" class="form-control" name="close" required value="<?= htmlspecialchars(substr($data['close'] ?? '18:00', 0, 5)) ?>"></div>
+</div>
diff --git a/views/admin/work_hours/create.php b/views/admin/work_hours/create.php
index a89c857..ed543f2 100644
--- a/views/admin/work_hours/create.php
+++ b/views/admin/work_hours/create.php
@@ -36,19 +36,7 @@ include __DIR__ . '/../../../includes/header.php';
 
     <form method="POST" class="card shadow-sm"><div class="card-body">
         <?= csrf_field() ?>
-        <div class="mb-3"><label class="form-label">Dia <span class="text-danger">*</span></label>
-            <select class="form-select" name="week_day" required>
-                <option value="">Selecionar…</option>
-                <?php foreach ($dias as $val => $label): ?>
-                <option value="<?= $val ?>" <?= $data['week_day'] === $val ? 'selected' : '' ?>><?= $label ?></option>
-                <?php endforeach; ?>
-            </select></div>
-        <div class="row g-3">
-            <div class="col-6"><label class="form-label">Abertura <span class="text-danger">*</span></label>
-                <input type="time" class="form-control" name="open" required value="<?= htmlspecialchars($data['open']) ?>"></div>
-            <div class="col-6"><label class="form-label">Fecho <span class="text-danger">*</span></label>
-                <input type="time" class="form-control" name="close" required value="<?= htmlspecialchars($data['close']) ?>"></div>
-        </div>
+        <?php include __DIR__ . '/_form.php'; ?>
         <div class="mt-4 d-flex gap-2">
             <button type="submit" class="btn btn-primary">Guardar</button>
             <a class="btn btn-outline-secondary" href="index.php">Cancelar</a>
diff --git a/views/admin/work_hours/edit.php b/views/admin/work_hours/edit.php
index 02a66b4..a8c4171 100644
--- a/views/admin/work_hours/edit.php
+++ b/views/admin/work_hours/edit.php
@@ -46,19 +46,7 @@ include __DIR__ . '/../../../includes/header.php';
 
     <form method="POST" class="card shadow-sm"><div class="card-body">
         <?= csrf_field() ?>
-        <div class="mb-3"><label class="form-label">Dia <span class="text-danger">*</span></label>
-            <select class="form-select" name="week_day" required>
-                <option value="">Selecionar…</option>
-                <?php foreach ($dias as $val => $label): ?>
-                <option value="<?= $val ?>" <?= ($data['week_day'] ?? '') === $val ? 'selected' : '' ?>><?= $label ?></option>
-                <?php endforeach; ?>
-            </select></div>
-        <div class="row g-3">
-            <div class="col-6"><label class="form-label">Abertura <span class="text-danger">*</span></label>
-                <input type="time" class="form-control" name="open" required value="<?= htmlspecialchars(substr($data['open'] ?? '09:00', 0, 5)) ?>"></div>
-            <div class="col-6"><label class="form-label">Fecho <span class="text-danger">*</span></label>
-                <input type="time" class="form-control" name="close" required value="<?= htmlspecialchars(substr($data['close'] ?? '18:00', 0, 5)) ?>"></div>
-        </div>
+        <?php include __DIR__ . '/_form.php'; ?>
         <div class="mt-4 d-flex gap-2">
             <button type="submit" class="btn btn-primary">Atualizar</button>
             <a class="btn btn-outline-secondary" href="index.php">Cancelar</a>