# Implementacao frontend — bipagem-cota-minima ## Loop de correcao pos-review (06-review.md) — Ciclo 3 ### B4 — `buildScanCotaDataForLuz`: `find($id)` sem `->fetch()` corrigido **Arquivo**: `componentes/Missoes/Logistica/sinais/App/Handlers/SeparacaoHandler.php` **Metodo**: `buildScanCotaDataForLuz` (linha ~585) **Diagnostico**: `$mModel->find($missaoId)` passava um `int` diretamente ao `ModelInteligencia::find()`, que e um query builder e retorna `$this` sem executar o SELECT. `$this->data` nunca e populado, entao `$missao->configuracoes` vinha `null`. Consequencia: `MissaoPresenter::montarScanCotaFront($missao)` recebia `configuracoes=null` → retornava `ativo=false` → todo card re-renderizado apos a 1a bipagem saía com `data-scan-cota-ativo="0"`, desligando a cota silenciosamente do 2o card em diante. **Fix** (1 linha): substituído por WHERE explicito + fetch(), espelhando o padrao correto ja usado em `:189-192` do mesmo arquivo: ```php // antes (errado): $missao = $mModel->find($missaoId); // depois (correto): $missao = $mModel->find("id = :id", "id={$missaoId}")->fetch(); ``` `fetch()` sem `$all=true` retorna o objeto hydratado diretamente (nao em array), portanto nenhum desembrulho necessario — diferente do `findById()` que embrulha em `[$response]`. **Escopo**: os dois `reencaminheLuz` (`:519` apos bipagem de item e `:563` via observatorio) e o bloco de `endereco_alocacao` em `Luz_SeparacaoItemCard` (`:386-394`) todos chamam `buildScanCotaDataForLuz` — os tres caminhos sao corrigidos por esta unica mudanca. **`endereco_alocacao` (`:386-394`)**: NAO era um `find()` independente — e o proprio `buildScanCotaDataForLuz` chamado via `$scanCotaData = $this->buildScanCotaDataForLuz(...)`. Portanto ja estava coberto pelo fix acima; sem alteracao adicional necessaria. **php -l**: limpo. --- ## Loop de correcao pos-review (06-review.md) — Ciclo 2 ### B1 (ciclo 2) — Fila única: fix movido para o caminho real de renderizacao (Luz) **Diagnóstico do ciclo 1**: o fix anterior foi aplicado no bloco `else` do `modoFilaUnica` em `separacao/item.php` (linhas 518-530). Esse bloco é código morto: o handler força `modoFilaUnica => true` (SeparacaoHandler.php:217), então a separacao sempre entra no `if ($modoFilaUnica)` e renderiza a Luz `SeparacaoItemCard`. O `else` nunca executa. Resultado: `data-scan-cota-ativo="0"` → `ExecucaoCard.initScanCota()` saía com `disponivel = Infinity` → bloqueio de tempo real inerte na separação. **Fix ciclo 2** — 3 arquivos alterados: **`separacao/item.php`** (bloco `if ($modoFilaUnica)`, linha ~415): - Adicionado `'missao' => $missao` e `'jsItensData' => $jsItensData ?? []` na chamada `$galaxia->luz('SeparacaoItemCard', [...])`, para que a view da Luz receba os dados necessários para calcular `scan_cota.disponivel`. - Removido o bloco `scan_cota` morto do `else` (modo livre), que nunca era executado. **`luz/separacao/item_card.php`** (view da Luz): - Adicionadas as declarações de segurança `$missao = $missao ?? null` e `$jsItensData = $jsItensData ?? []` no topo (logo após `$tipoCard`). - Adicionado bloco `$scanCotaFila` (closure IIFE) que chama `MissaoPresenter::montarScanCotaFront($missao)` e reconstrói `disponivel = manuais - count(bipaveis manchados)` a partir dos itens com `bipavel && manchado`. - Passado `'scan_cota' => $scanCotaFila` no array do `execucaoCard()`. **`SeparacaoHandler.php`** (handler da Luz): - Adicionado `use GalaxiaConstelacoes\Logistica\MissaoLogistica\Presenters\MissaoPresenter`. - Adicionado método privado `buildScanCotaDataForLuz(int $missaoId): array` que busca `$missao` via `MissaoLogistica::find()` e `$jsItensData` via `MissoesExecucao::lerItens` + `MissaoPresenter::montarJsonFrontendBase()`. - `Luz_SeparacaoItemCard`: refatorado para usar `buildScanCotaDataForLuz` em vez da query manual de gondola; agora popula `$data['missao']` e `$data['jsItensData']` para qualquer re-render via autoload da Luz (após navegação/refresh). - `biparItemSeparacao` (reencaminheLuz após bipagem completa): `array_merge` com `buildScanCotaDataForLuz` para que o card seguinte também receba `scan_cota` correto com `disponivel` já decrementado. - `handleObservatorioMissoesBipagem` (observador do `biparItem`): idem — `array_merge` com `buildScanCotaDataForLuz`. ### B2 — Decremento manual bloqueava/consumia cota indevidamente **Fix**: Gate de cota restrito a `delta > 0` nos tres fluxos: - `execucao_card.js::decrementarQuantidade()` — removido todo o bloco de cota; decremento vai direto para `_executarDecrementoManual()`. - `carregamento/item.js::ajustarQuantidade()` — `isManualAction` renomeado para `isManualIncremento = (codigo === 'MANUAL' && delta > 0)`. - `separacao/item.js::ajustarQuantidade()` — idem. ### Nits - **m3**: removidos todos os `console.log('[ScanCota]...')` dos 3 arquivos JS. - **n1**: `mostrarAvísoCotaBloqueada` renomeado para `mostrarAvisoCotaBloqueada` (sem acento, typo corrigido) e todas as chamadas atualizadas nos 3 arquivos. - **n2**: modal de `mostrarModalUltimaCota` em `carregamento/item.js` refatorado para usar `var(--bg-card)`, `var(--color-primary)`, `var(--color-warning)`, `var(--text-primary)`, `var(--text-muted)`, `var(--border-color)` — alinhado com o padrao de `separacao/item.js`. --- ## Views / componentes modificados Nenhuma view nova foi criada. O trabalho foi inteiramente de modificacao de componentes compartilhados ja existentes e de correcao de uma pendencia apontada pelo backend. | Arquivo modificado | O que mudou | |---|---| | `componentes/Missoes/Logistica/sinais/App/html/shared/components/v2/execucao_bipagem.php` | Adicionado bloco `