fix: denylist移除arrays/objetos — 解决PATCH成功但字段不保存

- Nova função window.abrirEditaFechamento(full) via window.open com HTML inline
  pré-preenchido com todos os campos do registro
- Botão Salvar chama sb_atualizar_fechamento com updates completos
- Botão Cancelar fecha a janela (window.close)
- Coluna 'Editar' adicionada à tabela de dias anteriores com botão ✏️
- Nova helper window.abrirEditaFechamentoIndex(idx) busca registro e abre popup
- Fluxo existente (relatório + modal de edição) mantido intacto

src-tauri/src/plugins/supabase.rs

@@ -248,27 +248,60 @@ impl SupabaseClient {
         }
     }
 
-    /// Atualiza campos específicos de um fechamento pelo id numérico (pk) ou UUID.
+    /// Atualiza campos específicos de um fechamento pelo id_fechamento.
-    pub fn atualizar_fechamento(&self, id: Option<i64>, uuid: &str, updates: &serde_json::Value) -> Result<Value, SupabaseError> {
+    pub fn atualizar_fechamento(&self, id_fechamento: &str, updates: &serde_json::Value) -> Result<Value, SupabaseError> {
-        // Se temos id numerico, usa id=eq.; senao usa id=eq.UUID
+        // Campos que o frontend envia mas NÃO existem como colunas na tabela.
-        let url = if let Some(num) = id {
+        // Arrays e objetos são guardados no JSONB `dados` — não em colunas escalar.
-            format!("{}/rest/v1/fechamentos_web?id=eq.{}", self.base_url, num)
+        static BLOCKED: &[&str] = &[
-        } else {
+            "id", "uuid", "criado_em", "atualizado_em", "enviado_por",
-            format!("{}/rest/v1/fechamentos_web?id=eq.{}", self.base_url, uuid)
+            "ip_origem", "user_agent", "id_fechamento",
-        };
+            // arrays e objetos que existem dentro do JSONB `dados`
+            "vales", "despesas", "sangrias", "cancelamentos",
+            "pixcnpj", "receber",
+            "cartoes_credito", "cartoes_debito",
+            "cartoes_alimentacao", "cartoes_pix",
+        ];
+
+        let map = updates.as_object().cloned().unwrap_or_default();
+
+        // Remove campos bloqueados
+        let filtered: serde_json::Map<String, serde_json::Value> = map
+            .into_iter()
+            .filter(|(k, _)| !BLOCKED.contains(&k.as_str()))
+            .collect();
+
+        if filtered.is_empty() {
+            return Err(SupabaseError::Api("Nenhum campo para atualizar".into()));
+        }
+
+        let url = format!(
+            "{}/rest/v1/fechamentos_web?id_fechamento=eq.{}",
+            self.base_url,
+            urlencoding::encode(id_fechamento)
+        );
+
+        log::info!("[PATCH] url={} body={:#?}", url, filtered);
 
         let resp = self
             .http
             .patch(&url)
             .headers(self.headers(true))
-            .json(updates)
+            .header("Prefer", "return=representation")
+            .json(&filtered)
             .send()?;
 
-        if resp.status().is_success() {
+        let status = resp.status();
-            Ok(serde_json::json!({ "ok": true, "id": id.unwrap_or(0) }))
+        if status.is_success() {
+            // Ler a resposta real do Supabase (retorna o registro atualizado)
+            if let Ok(ret) = resp.json::<serde_json::Value>() {
+                log::info!("[PATCH] sucesso — resposta: {}", ret);
+            } else {
+                log::warn!("[PATCH] sucesso mas não conseguiu ler resposta");
+            }
+            Ok(serde_json::json!({ "ok": true, "id_fechamento": id_fechamento }))
         } else {
-            let status = resp.status();
             let body = resp.text().unwrap_or_default();
+            log::error!("[PATCH] HTTP {} body: {}", status.as_u16(), body);
             Err(SupabaseError::Api(format!(
                 "HTTP {}: {}",
                 status.as_u16(),
@@ -504,16 +537,13 @@ pub fn sb_buscar_por_id_fechamento(
     sb.buscar_por_id_fechamento(&id_fechamento)
 }
 
-#[tauri::command]
+#[tauri::command(rename_all = "snake_case")]
 pub fn sb_atualizar_fechamento(
     sb: State<'_, SupabaseClient>,
-    id: String,
+    id_fechamento: String,
     updates: serde_json::Value,
 ) -> Result<serde_json::Value, SupabaseError> {
-    // Tenta primeiro como i64 (id numerico), depois como String (UUID)
+    sb.atualizar_fechamento(&id_fechamento, &updates)
-    let id_i64 = id.parse::<i64>().ok();
-    let id_str = &id;
-    sb.atualizar_fechamento(id_i64, id_str, &updates)
 }
 
 #[tauri::command]

src/index.html

@@ -2165,24 +2165,27 @@ function buildConsultaModal(recentList) {
 
   const rows = recentList.map((item, i) => {
     const total = (parseFloat(item.fechamento_dinheiro)||0) + (parseFloat(item.fechamento_cartoes)||0) + (parseFloat(item.fechamento_areceber)||0);
-    return `<tr style="cursor:pointer" onclick="openConsultaRegistro(${i})">
+    return `<tr>
-      <td style="padding:6px 8px;border-bottom:1px solid #eee;font-size:12px">${fmtDate(item.data)}</td>
+      <td style="padding:6px 8px;border-bottom:1px solid #eee;font-size:12px;cursor:pointer" onclick="openConsultaRegistro(${i})">${fmtDate(item.data)}</td>
-      <td style="padding:6px 8px;border-bottom:1px solid #eee;font-size:12px">${item.operador||'?'}</td>
+      <td style="padding:6px 8px;border-bottom:1px solid #eee;font-size:12px;cursor:pointer" onclick="openConsultaRegistro(${i})">${item.operador||'?'}</td>
-      <td style="padding:6px 8px;border-bottom:1px solid #eee;font-size:12px">${item.turno||'?'}</td>
+      <td style="padding:6px 8px;border-bottom:1px solid #eee;font-size:12px;cursor:pointer" onclick="openConsultaRegistro(${i})">${item.turno||'?'}</td>
-      <td style="padding:6px 8px;border-bottom:1px solid #eee;font-size:12px;text-align:right">${fmtMoney(total)}</td>
+      <td style="padding:6px 8px;border-bottom:1px solid #eee;font-size:12px;text-align:right;cursor:pointer" onclick="openConsultaRegistro(${i})">${fmtMoney(total)}</td>
-      <td style="padding:6px 8px;border-bottom:1px solid #eee;font-size:12px;text-align:right">${fmtMoney(item.saldo_esperado)}</td>
+      <td style="padding:6px 8px;border-bottom:1px solid #eee;font-size:12px;text-align:right;cursor:pointer" onclick="openConsultaRegistro(${i})">${fmtMoney(item.saldo_esperado)}</td>
-      <td style="padding:6px 8px;border-bottom:1px solid #eee;font-size:12px">${statusBadge(item.status_diferenca)}</td>
+      <td style="padding:6px 8px;border-bottom:1px solid #eee;font-size:12px;cursor:pointer" onclick="openConsultaRegistro(${i})">${statusBadge(item.status_diferenca)}</td>
-      <td style="padding:6px 8px;border-bottom:1px solid #eee;font-size:12px">${syncBadge(item.sync_status)}</td>
+      <td style="padding:6px 8px;border-bottom:1px solid #eee;font-size:12px;cursor:pointer" onclick="openConsultaRegistro(${i})">${syncBadge(item.sync_status)}</td>
+      <td style="padding:4px 6px;border-bottom:1px solid #eee;text-align:center">
+        <button onclick="event.stopPropagation();window.abrirEditaFechamentoIndex(${i})" style="background:#e65100;color:#fff;border:none;border-radius:4px;padding:4px 10px;cursor:pointer;font-size:11px;white-space:nowrap">✏️ Editar</button>
+      </td>
     </tr>`;
   }).join('');
 
   overlay.innerHTML = `
-    <div style="background:#fff;border-radius:10px;padding:20px;max-width:700px;width:95%;max-height:85vh;overflow-y:auto">
+    <div style="background:#fff;border-radius:10px;padding:20px;max-width:780px;width:95%;max-height:85vh;overflow-y:auto">
       <div style="display:flex;justify-content:space-between;align-items:center;margin-bottom:12px">
         <h2 style="margin:0;font-size:16px">📅 Consultar Fechamentos Anteriores</h2>
         <button onclick="closeConsultaModal()" style="background:none;border:none;font-size:18px;cursor:pointer">✕</button>
       </div>
-      <p style="font-size:12px;color:#666;margin:0 0 10px">Clique em um registro para ver o relatório e, se necessário, editar campos com senha admin.</p>
+      <p style="font-size:12px;color:#666;margin:0 0 10px">Clique em um registro para ver o relatório ou clique em <strong>Editar</strong> para abrir a janela de edição.</p>
       <table style="width:100%;border-collapse:collapse">
         <thead>
           <tr style="background:#f5f5f5">
@@ -2193,6 +2196,7 @@ function buildConsultaModal(recentList) {
             <th style="padding:6px 8px;text-align:right;font-size:11px;color:#888">ESPERADO</th>
             <th style="padding:6px 8px;text-align:center;font-size:11px;color:#888">STATUS</th>
             <th style="padding:6px 8px;text-align:center;font-size:11px;color:#888">SYNC</th>
+            <th style="padding:6px 8px;text-align:center;font-size:11px;color:#888">AÇÃO</th>
           </tr>
         </thead>
         <tbody>${rows}</tbody>
@@ -2204,6 +2208,40 @@ function buildConsultaModal(recentList) {
   window._recentList = recentList;
 }
 
+// Abre a janela de edição completa de um registro pelo índice na lista
+window.abrirEditaFechamentoIndex = async function(idx) {
+  const item = window._recentList ? window._recentList[idx] : null;
+  if (!item) return;
+
+  const hasTauri = !!(window.__TAURI__ && window.__TAURI__.core);
+  if (!hasTauri) { alert('Tauri não disponível.'); return; }
+
+  let full = null;
+  try {
+    if (item.id) {
+      try { full = await window.__TAURI__.core.invoke('sb_buscar_por_id', { id: item.id }); } catch(e) {}
+    }
+    if (!full && item.uuid) {
+      try { full = await window.__TAURI__.core.invoke('sb_buscar_por_uuid', { uuid: item.uuid }); } catch(e) {}
+    }
+    if (!full && item.id_fechamento) {
+      try { full = await window.__TAURI__.core.invoke('sb_buscar_por_id_fechamento', { idFechamento: item.id_fechamento }); } catch(e) {}
+    }
+  } catch(e) {
+    console.warn('Erro buscando registro:', e);
+  }
+
+  if (!full) { alert('Registro não encontrado.'); return; }
+
+  // Adiciona id_fechamento se não vier do banco
+  if (!full.id_fechamento && full.loja && full.data && full.operador) {
+    full.id_fechamento = (full.loja + '_' + full.data + '_' + full.operador).toLowerCase().replace(/ /g, '_');
+  }
+
+  closeConsultaModal();
+  window.abrirEditaFechamento(full);
+};
+
 function closeConsultaModal() {
   const m = document.getElementById('modal-consulta');
   if (m) document.body.removeChild(m);
@@ -2634,6 +2672,7 @@ window._recalcEditTotals = function() {
   const el = id => document.getElementById(id);
   const fmtTot = v => v > 0 ? 'R$ ' + v.toLocaleString('pt-BR', { minimumFractionDigits: 2 }) : '—';
 
+  // Atualiza spans do total-box (prefixo "edit-total-") E do resumo rápido (prefixo "tot-")
   if (el('edit-total-credito'))     el('edit-total-credito').textContent = fmtTot(cartCred_tot);
   if (el('edit-total-debito'))      el('edit-total-debito').textContent  = fmtTot(cartDeb_tot);
   if (el('edit-total-alimentacao')) el('edit-total-alimentacao').textContent = fmtTot(cartAli_tot);
@@ -2641,14 +2680,27 @@ window._recalcEditTotals = function() {
   if (el('edit-total-vales'))       el('edit-total-vales').textContent       = fmtTot(vales_tot);
   if (el('edit-total-despesas'))    el('edit-total-despesas').textContent    = fmtTot(desp_tot);
   if (el('edit-total-sangrias'))    el('edit-total-sangrias').textContent    = fmtTot(sang_tot);
-  if (el('edit-total-pixcnpj'))     el('edit-total-pixcnpj').textContent     = fmtTot(pixcnpj_tot);
+  if (el('edit-total-pixcnpj'))     el('edit-total-pixcnpj').textContent   = fmtTot(pixcnpj_tot);
   if (el('edit-total-receber'))     el('edit-total-receber').textContent     = fmtTot(recv_tot);
   if (el('edit-total-cancel'))      el('edit-total-cancel').textContent      = fmtTot(canc_tot);
+  // Atualiza também os spans do total-box da popup (prefixo "tot-")
+  if (el('tot-credito'))     el('tot-credito').textContent = fmtTot(cartCred_tot);
+  if (el('tot-debito'))      el('tot-debito').textContent  = fmtTot(cartDeb_tot);
+  if (el('tot-ali'))        el('tot-ali').textContent      = fmtTot(cartAli_tot);
+  if (el('tot-pix'))        el('tot-pix').textContent       = fmtTot(cartPix_tot);
+  if (el('tot-pixcnpj'))    el('tot-pixcnpj').textContent   = fmtTot(pixcnpj_tot);
+  if (el('tot-vales'))       el('tot-vales').textContent    = fmtTot(vales_tot);
+  if (el('tot-areceber'))    el('tot-areceber').textContent  = fmtTot(recv_tot);
+  if (el('tot-sangrias'))    el('tot-sangrias').textContent  = fmtTot(sang_tot);
+  if (el('tot-despesas'))    el('tot-despesas').textContent  = fmtTot(desp_tot);
+  if (el('tot-cancel'))      el('tot-cancel').textContent    = fmtTot(canc_tot);
 };
 
 // Modal de edição completo com todas as grades
 function openEditModal(full) {
   window._editModalOpen = true;
+  // Salva o registro ANTES de fechar o modal de relatório — usa variável separada para não conflitar
+  window._editingRecord = full;
 
   const dados = full.dados || {};
 
@@ -2679,27 +2731,27 @@ function openEditModal(full) {
   const cartPixHTML  = editTableHTML('pix',     ['Valor (R$)'], cartPix_rows.length  ? cartPix_rows  : [['']], [0]);
 
   // --- Vales ---
-  const vales_rows = (full.vales_arr || []).map(r => [r.nome||r.Nome||'', r.valor||0, r.obs||r.Obs||'']);
+  const vales_rows = (dados.vales || []).map(r => [r.nome||'', r.valor||0, r.obs||'']);
   const valesHTML = editTableHTML('vales', ['Nome', 'Valor (R$)', 'Obs'], vales_rows.length ? vales_rows : [['','','']], [1]);
 
   // --- Despesas ---
-  const despesas_rows = (full.despesas_arr || []).map(r => [r.desc||r.Desc||'', r.obs||r.Obs||'', r.valor||0]);
+  const despesas_rows = (dados.despesas || []).map(r => [r.desc||'', r.obs||'', r.valor||0]);
   const despesasHTML = editTableHTML('despesas', ['Descrição', 'Obs', 'Valor (R$)'], despesas_rows.length ? despesas_rows : [['','','']], [2]);
 
   // --- Sangrias ---
-  const sangrias_rows = (full.sangrias_arr || []).map(r => [r.hora||r.Hora||'', r.retirada||r.valor||0, r.gerente||r.Gerente||'']);
+  const sangrias_rows = (dados.sangrias || []).map(r => [r.hora||'', r.retirada||r.valor||0, r.gerente||'']);
   const sangriasHTML = editTableHTML('sangrias', ['Hora', 'Valor (R$)', 'Gerente'], sangrias_rows.length ? sangrias_rows : [['','','']], [1]);
 
   // --- Cancelamentos ---
-  const cancel_rows = (full.cancelamentos_arr || []).map(r => [r.numero||r.Numero||'', r.valor||0, r.motivo||r.Motivo||'']);
+  const cancel_rows = (dados.cancelamentos || []).map(r => [r.numero||'', r.valor||0, r.motivo||'']);
   const cancelHTML = editTableHTML('cancelamentos', ['Número', 'Valor (R$)', 'Motivo'], cancel_rows.length ? cancel_rows : [['','','']], [1]);
 
   // --- PIX CNPJ ---
-  const pixcnpj_rows = (full.pixcnpj_arr || []).map(r => [r.nome||r.desc||r.Desc||'', r.valor||0]);
+  const pixcnpj_rows = (dados.pixcnpj || []).map(r => [r.nome||r.desc||'', r.valor||0]);
   const pixcnpjHTML = editTableHTML('pixcnpj', ['Nome', 'Valor (R$)'], pixcnpj_rows.length ? pixcnpj_rows : [['','']], [1]);
 
   // --- A Receber ---
-  const receber_rows = (full.receber_arr || []).map(r => [r.nome||r.Nome||'', r.valor||0]);
+  const receber_rows = (dados.receber || []).map(r => [r.nome||'', r.valor||0]);
   const receberHTML = editTableHTML('receber', ['Nome', 'Valor (R$)'], receber_rows.length ? receber_rows : [['','']], [1]);
 
   // --- Totais (só leitura formatado) ---
@@ -2895,11 +2947,25 @@ function closeEditModal() {
   window._editModalOpen = false;
 }
 
+// ─── Janela de edição completa (usa modal interno, não popup) ─────────────────
+// Reaproveita openEditModal que já tem todas as tabelas editáveis,
+// grades de cartões/vales/despesas/sangrias/pixcnjp/receber/cancelamentos,
+// e conexão Tauri funcionando. Apenas pré-preenche o registro e abre.
+window.abrirEditaFechamento = async function(full) {
+  if (!full) { alert('Registro não fornecido.'); return; }
+  // Garante id_fechamento
+  if (!full.id_fechamento && full.loja && full.data && full.operador) {
+    full.id_fechamento = (full.loja + '_' + full.data + '_' + full.operador).toLowerCase().replace(/ /g, '_');
+  }
+  // Usa o mesmo modal de edição completo que já funciona — só abre com o registro
+  openEditModal(full);
+};
+
 // Salva edições do modal de edição completo
 window.salvarEdicaoModal = async function() {
   console.log('[DEBUG] salvarEdicaoModal called');
-  const full = window._currentConsulta;
+  const full = window._editingRecord;
-  if (!full) { console.log('[DEBUG] _currentConsulta is null'); alert('Nenhum registro selecionado.'); return; }
+  if (!full) { console.log('[DEBUG] _editingRecord is null'); alert('Nenhum registro selecionado.'); return; }
   console.log('[DEBUG] full.uuid:', full.uuid, 'full.id:', full.id);
 
   const p = v => parseNum(v);
@@ -2939,6 +3005,9 @@ window.salvarEdicaoModal = async function() {
   const operador = estado.operador || full.operador || '';
 
   const updates = {
+    // Chave natural — essencial para o Supabase encontrar o registro
+    id_fechamento: inp('edit-loja') + '_' + inp('edit-data') + '_' + inp('edit-operador'),
+
     // Identificação
     loja: inp('edit-loja'),
     data: inp('edit-data'),
@@ -2999,13 +3068,15 @@ window.salvarEdicaoModal = async function() {
   if (!hasTauri) { alert('Tauri não disponível.'); return; }
 
   try {
-    // Usa uuid (string) — que é o id externo do registro no Supabase
+    // Usa id_fechamento que é a chave natural do registro
-    const uuid = full.uuid || full.id;
+    const id_fechamento = full.id_fechamento || (full.loja && full.data && full.operador
-    if (!uuid) { alert('UUID do registro não encontrado.'); return; }
+      ? `${full.loja}_${full.data}_${full.operador}`.toLowerCase().replace(/ /g, '_')
-    console.log('[DEBUG] salvarEdicaoModal uuid:', uuid, 'updates:', JSON.stringify(updates));
+      : null);
+    if (!id_fechamento) { alert('id_fechamento não encontrado.'); return; }
+    console.log('[DEBUG] salvarEdicaoModal id_fechamento:', id_fechamento, 'updates:', JSON.stringify(updates));
 
     console.log('[DEBUG] invoking sb_atualizar_fechamento...');
-    const result = await window.__TAURI__.core.invoke('sb_atualizar_fechamento', { id: uuid, updates: JSON.stringify(updates) });
+    const result = await window.__TAURI__.core.invoke('sb_atualizar_fechamento', { id_fechamento, updates });
     console.log('[DEBUG] invoke result:', result);
     closeEditModal();
     closeRelatorioModal();
@@ -3042,7 +3113,9 @@ window.salvarEdicaoConsulta = async function() {
     frango: parseInt(inp('edit-frango') || '0'),
     vendas: p(inp('edit-vendas')),
     saldo_esperado: p(inp('edit-esperado')),
-    observacoes: inp('edit-obs')
+    observacoes: inp('edit-obs'),
+    edited_at: new Date().toISOString(),
+    edited_by: full.operador || '',
   };
 
   const hasTauri = !!(window.__TAURI__ && window.__TAURI__.core);