Adicionar importação de reservas em massa via CSV com modal de pesquisa de IDs no painel admin by Copilot · Pull Request #152 · marpisco/ClassLink

Copilot · 2026-05-21T21:47:03Z

Este PR implementa a importação de reservas em massa via CSV no script correto da issue original: admin/reservaemmassa.php, incluindo um modal de apoio para pesquisa de IDs via API.

Alterações

Adicionado novo bloco de UI em admin/reservaemmassa.php para upload de CSV:
- link para download do modelo,
- formulário dedicado de importação.
Implementado novo fluxo action=import_csv para:
- leitura/parsing de CSV com delimitador ;,
- validação de token CSRF,
- validação de referências (SalaID, RequisitorID, TempoID),
- validação de data (YYYY-MM-DD),
- deteção e ignorar duplicados já existentes em reservas,
- inserção de reservas com aprovado = 1,
- feedback agregado de sucesso/avisos/erros.
Registo de ação administrativa no padrão existente via acaoexecutada(...).
Adicionado template de suporte assets/csvsample_reservas.csv.
Adicionado modal de pesquisa de IDs em admin/reservaemmassa.php com 3 tabs:
- requisitorID,
- tempoID,
- salaID.
Adicionados endpoints dedicados em /admin/api para suportar a pesquisa no modal:
- admin/api/requisitor_lookup.php,
- admin/api/tempo_lookup.php,
- admin/api/sala_lookup.php.
Pesquisa no modal com:
- placeholders/skeletons Bootstrap durante carregamento,
- filtro mínimo obrigatório (2 caracteres),
- limite de 10 resultados por pesquisa para reduzir carga na base de dados.

Resultado

A importação CSV fica disponível no módulo de reservas em massa (admin/reservaemmassa.php), alinhada com o escopo correto da issue, e passa a ter apoio de pesquisa rápida de IDs (requisitor, tempo e sala) para facilitar a preparação dos ficheiros CSV.

Agent-Logs-Url: https://github.com/marpisco/ClassLink/sessions/c15f61c7-4d55-40de-b428-433cbce7baa9 Co-authored-by: marpisco <162377105+marpisco@users.noreply.github.com>

marpisco

@copilot Anula, não estou a falar da importação de tempos na minha issue original, estou a referir me às importações de reservas (script atual admin/reservaemmassa.php)

Agent-Logs-Url: https://github.com/marpisco/ClassLink/sessions/3ff397cc-c4ca-405e-846d-9e183bfa8375 Co-authored-by: marpisco <162377105+marpisco@users.noreply.github.com>

Copilot · 2026-05-21T22:00:23Z

@copilot Anula, não estou a falar da importação de tempos na minha issue original, estou a referir me às importações de reservas (script atual admin/reservaemmassa.php)

Reverti totalmente essas alterações fora de escopo no commit 99cf827 (removida a importação CSV de tempos e o ficheiro de exemplo). Screenshot de verificação da UI revertida: /tmp/playwright-logs/page-2026-05-21T21-59-53-748Z.png.

Se quiseres, avanço já com a implementação correta da importação CSV nas reservas em massa (admin/reservaemmassa.php).

marpisco · 2026-05-21T22:01:11Z

@copilot Sim, podes avançar com a implementação do correto.

Agent-Logs-Url: https://github.com/marpisco/ClassLink/sessions/4b49d234-7d64-4371-9ed8-1404a6bdde04 Co-authored-by: marpisco <162377105+marpisco@users.noreply.github.com>

Agent-Logs-Url: https://github.com/marpisco/ClassLink/sessions/16198339-3d5e-4b7b-b357-4e2ab6d7ecad Co-authored-by: marpisco <162377105+marpisco@users.noreply.github.com>

Copilot · 2026-05-21T22:27:02Z

@copilot Adiciona um modal que te permita pesquisar via API (penso que consegues até reaproveitar os endpoints do requisitorID, mas usa /admin/api para criar as APIs de busca) pelo requisitorID, temp...

Feito no commit 45815ec (inclui os anteriores d711f8f, 3dfce7d e 5c61670): adicionei o modal com 3 tabs (requisitorID, tempoID, salaID) com pesquisa via APIs em /admin/api, skeleton placeholders durante carregamento, e limite de 10 resultados com filtro mínimo obrigatório.

Screenshot: /tmp/playwright-logs/page-2026-05-21T22-22-36-016Z.png.

… por IA)

github-actions · 2026-05-21T22:32:19Z

📝 Manual Update Required? It looks like you've modified core logic. Please check if the user manual needs a refresh! @marpisco

Copilot

Pull request overview

Implementa importação de reservas em massa via CSV no painel de admin (admin/reservaemmassa.php), com um modal para pesquisa rápida de IDs (requisitor/tempo/sala) suportado por endpoints JSON em /admin/api.

Changes:

Adiciona UI e fluxo action=import_csv para upload, parsing e importação de reservas a partir de CSV (delimitador ;), com validações e feedback agregado.
Introduz modal (tabs) para lookup de IDs e integra chamadas fetch() para endpoints dedicados.
Adiciona um ficheiro de modelo CSV para facilitar a preparação de importações.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
assets/csvsample_reservas.csv	Adiciona modelo de CSV para importação de reservas em massa.
admin/reservaemmassa.php	Implementa fluxo de importação CSV + modal de lookup de IDs no ecrã de reservas em massa.
admin/api/requisitor_lookup.php	Endpoint JSON para lookup de requisitores (cache) com filtro mínimo e limite.
admin/api/tempo_lookup.php	Endpoint JSON para lookup de tempos com filtro mínimo e limite.
admin/api/sala_lookup.php	Endpoint JSON para lookup de salas com filtro mínimo e limite.

Comments suppressed due to low confidence (1)

admin/reservaemmassa.php:444

O formulário principal de criação de reservas em massa (massReservationForm) continua sem token CSRF, apesar de já existir helper csrf_token_field() e de o novo fluxo de importação CSV o usar. Como este POST cria reservas (ação sensível de admin), deve incluir o campo CSRF e validar o token antes de processar $_POST.

<form id="massReservationForm" action="reservaemmassa.php" method="POST" class="mt-4">
    <div class="row mb-3">
        <div class="col-md-6">
            <div class="form-floating">
                <select class="form-select" id="sala" name="sala" required>

+            if (count($data) < 5) {
+                $errorCount++;
+                $errors[] = "Linha {$lineNumber} inválida (mínimo 5 colunas).";
+                continue;
+            }
+
+            $salaId = $firstColumn;
+            $requisitorId = trim($data[1]);
+            $tempoId = trim($data[2]);
+            $dataReserva = trim($data[3]);
+            $motivo = trim($data[4]);
+            $extra = isset($data[5]) ? trim($data[5]) : '';


+        $requisitoresValidos = [];
+        $stmtRequisitores = $db->prepare("SELECT id FROM cache");
+        $stmtRequisitores->execute();
+        $resultRequisitores = $stmtRequisitores->get_result();
+        while ($row = $resultRequisitores->fetch_assoc()) {
+            $requisitoresValidos[$row['id']] = true;
+        }
+        $stmtRequisitores->close();
+
+        $temposValidos = [];
+        $stmtTempos = $db->prepare("SELECT id FROM tempos");
+        $stmtTempos->execute();
+        $resultTempos = $stmtTempos->get_result();
+        while ($row = $resultTempos->fetch_assoc()) {
+            $temposValidos[$row['id']] = true;
+        }
+        $stmtTempos->close();
+


marpisco

@copilot De acordo com a revisão automática, faz o seguinte:

"O parser CSV rejeita linhas com menos de 5 colunas (count($data) < 5), mas o próprio código trata Motivo como opcional (preenche valor por omissão quando vazio) e a UI indica Extra como opcional. Assim, um CSV válido com apenas 4 colunas (sem Motivo/Extra) vai falhar desnecessariamente. Ajustar a validação para aceitar 4 colunas e tratar data[4]/data[5] como opcionais (default quando ausentes)."

Parece me correto e deve de ser corrigido.

Agent-Logs-Url: https://github.com/marpisco/ClassLink/sessions/7ee34890-325a-4694-af70-b9c8c4869a66 Co-authored-by: marpisco <162377105+marpisco@users.noreply.github.com>

Copilot · 2026-05-21T22:46:18Z

@copilot De acordo com a revisão automática, faz o seguinte:

"O parser CSV rejeita linhas com menos de 5 colunas (count($data) < 5), mas o próprio código trata Motivo como opcional (preenche valo...

Corrigido no commit 9035d51: o parser agora aceita linhas com mínimo de 4 colunas e trata data[4] (Motivo) e data[5] (Extra) como opcionais quando ausentes. Validei com php -l admin/reservaemmassa.php.

marpisco

@copilot Fiz no Excel e saiu me um ficheiro com ,. Modifica a lógica do /admin/materiais.php para ler ficheiros com , (porque isto é suposto ser feito no Excel), modifica o csvsample.csv para ser csvsample_materiais.csv (e ser em virgulas), e modifica o csvsample_reservas.csv para ser também com vírgulas e faz com a importação das reservas em massa sejam em vírgulas (e não em ponto e vírgula, refiro a apenas virgula quando falo em ficheiros com virgulas neste pedido).

Agent-Logs-Url: https://github.com/marpisco/ClassLink/sessions/41bc349a-24b5-4d23-a36a-79d2398775f9 Co-authored-by: marpisco <162377105+marpisco@users.noreply.github.com>

Initial plan

545d6e3

Copilot AI assigned Copilot and marpisco May 21, 2026

Copilot AI requested review from Copilot and removed request for Copilot May 21, 2026 21:47

Copilot started work on behalf of marpisco May 21, 2026 21:47 View session

Copilot AI linked an issue May 21, 2026 that may be closed by this pull request

Horários importação por CSV #150

Open

feat: add CSV import for tempos in admin panel

63a07d3

Agent-Logs-Url: https://github.com/marpisco/ClassLink/sessions/c15f61c7-4d55-40de-b428-433cbce7baa9 Co-authored-by: marpisco <162377105+marpisco@users.noreply.github.com>

Copilot AI requested review from Copilot and removed request for Copilot May 21, 2026 21:48

fix: add csrf validation to tempos CSV import form

f6b35ff

Agent-Logs-Url: https://github.com/marpisco/ClassLink/sessions/c15f61c7-4d55-40de-b428-433cbce7baa9 Co-authored-by: marpisco <162377105+marpisco@users.noreply.github.com>

Copilot AI requested review from Copilot and removed request for Copilot May 21, 2026 21:50

Copilot AI changed the title ~~[WIP] Add horarios importação por CSV~~ Adicionar importação de horários (tempos) via CSV no painel admin May 21, 2026

Copilot finished work on behalf of marpisco May 21, 2026 21:51

Copilot AI requested a review from marpisco May 21, 2026 21:51

marpisco reviewed May 21, 2026

View reviewed changes

Copilot started work on behalf of marpisco May 21, 2026 21:58 View session

revert: remove tempos CSV import changes

99cf827

Agent-Logs-Url: https://github.com/marpisco/ClassLink/sessions/3ff397cc-c4ca-405e-846d-9e183bfa8375 Co-authored-by: marpisco <162377105+marpisco@users.noreply.github.com>

Copilot AI requested review from Copilot and removed request for Copilot May 21, 2026 22:00

Copilot AI changed the title ~~Adicionar importação de horários (tempos) via CSV no painel admin~~ Reverter importação de horários (tempos) via CSV no painel admin May 21, 2026

Copilot finished work on behalf of marpisco May 21, 2026 22:00

Copilot AI requested a review from marpisco May 21, 2026 22:00

Copilot started work on behalf of marpisco May 21, 2026 22:01 View session

feat: add CSV import for reservas em massa

f93b8fc

Agent-Logs-Url: https://github.com/marpisco/ClassLink/sessions/4b49d234-7d64-4371-9ed8-1404a6bdde04 Co-authored-by: marpisco <162377105+marpisco@users.noreply.github.com>

Copilot AI requested review from Copilot and removed request for Copilot May 21, 2026 22:04

fix: harden lookup fetch handling and simplify API filters

2c8886e

Agent-Logs-Url: https://github.com/marpisco/ClassLink/sessions/16198339-3d5e-4b7b-b357-4e2ab6d7ecad Co-authored-by: marpisco <162377105+marpisco@users.noreply.github.com>

Copilot AI requested review from Copilot and removed request for Copilot May 21, 2026 22:24

fix: use relative lookup API endpoints in admin modal

5c61670

Agent-Logs-Url: https://github.com/marpisco/ClassLink/sessions/16198339-3d5e-4b7b-b357-4e2ab6d7ecad Co-authored-by: marpisco <162377105+marpisco@users.noreply.github.com>

Copilot AI requested review from Copilot and removed request for Copilot May 21, 2026 22:25

perf: defer modal lookup fetch until modal is opened

45815ec

Agent-Logs-Url: https://github.com/marpisco/ClassLink/sessions/16198339-3d5e-4b7b-b357-4e2ab6d7ecad Co-authored-by: marpisco <162377105+marpisco@users.noreply.github.com>

Copilot AI requested review from Copilot and removed request for Copilot May 21, 2026 22:26

Copilot AI changed the title ~~Adicionar importação de reservas em massa via CSV no painel admin~~ Adicionar importação de reservas em massa via CSV com modal de pesquisa de IDs no painel admin May 21, 2026

Copilot finished work on behalf of marpisco May 21, 2026 22:27

fix: remover texto de (limitado a 10 para reduzir carga, texto gerado…

efbe4c5

… por IA)

Copilot AI review requested due to automatic review settings May 21, 2026 22:32

Copilot started reviewing on behalf of marpisco May 21, 2026 22:32 View session

Copilot AI reviewed May 21, 2026

View reviewed changes

marpisco reviewed May 21, 2026

View reviewed changes

Copilot started work on behalf of marpisco May 21, 2026 22:44 View session

fix: accept 4-column rows in reserva CSV import

9035d51

Agent-Logs-Url: https://github.com/marpisco/ClassLink/sessions/7ee34890-325a-4694-af70-b9c8c4869a66 Co-authored-by: marpisco <162377105+marpisco@users.noreply.github.com>

Copilot finished work on behalf of marpisco May 21, 2026 22:46

Copilot AI requested a review from marpisco May 21, 2026 22:46

marpisco reviewed May 21, 2026

View reviewed changes

Copilot started work on behalf of marpisco May 21, 2026 22:59 View session

Changes before error encountered

7b54cbd

Agent-Logs-Url: https://github.com/marpisco/ClassLink/sessions/41bc349a-24b5-4d23-a36a-79d2398775f9 Co-authored-by: marpisco <162377105+marpisco@users.noreply.github.com>

Copilot AI requested review from Copilot and removed request for Copilot May 21, 2026 23:02

Copilot AI requested a review from marpisco May 21, 2026 23:02

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Adicionar importação de reservas em massa via CSV com modal de pesquisa de IDs no painel admin#152

Adicionar importação de reservas em massa via CSV com modal de pesquisa de IDs no painel admin#152
Copilot wants to merge 18 commits into
mainfrom
copilot/importar-horarios-por-csv

Copilot AI commented May 21, 2026 •

edited

Loading

Uh oh!

marpisco left a comment

Uh oh!

Copilot AI commented May 21, 2026

Uh oh!

marpisco commented May 21, 2026

Uh oh!

Copilot AI commented May 21, 2026

Uh oh!

github-actions Bot commented May 21, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

marpisco left a comment

Uh oh!

Copilot AI commented May 21, 2026

Uh oh!

marpisco left a comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

Copilot AI commented May 21, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Alterações

Resultado

Uh oh!

marpisco left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI commented May 21, 2026

Uh oh!

marpisco commented May 21, 2026

Uh oh!

Copilot AI commented May 21, 2026

Uh oh!

github-actions Bot commented May 21, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

marpisco left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI commented May 21, 2026

Uh oh!

marpisco left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Copilot AI commented May 21, 2026 •

edited

Loading