# Fornecedores — Regras e Portabilidade > Regras de negócio não óbvias, acoplamentos e armadilhas do módulo. --- ## 1. Regras de negócio não óbvias ### 1.1 "Ativo" é um booleano nullable, não uma restrição `suppliers.fornecedor_ativo` é `boolean` nullable. A Query de exibição é `->where('fornecedor_ativo', true)->first()`. Como nada impede dois fornecedores ativos, e o `first()` não tem ordenação, **qualquer um pode ser exibido** — de forma não determinística. ### 1.2 A troca de ativo **desativa** o anterior, mesmo sendo o mesmo `switchFornecedorAtivo` sempre desativa o ativo anterior e só reativa o novo se for **diferente**. Ou seja: clicar no fornecedor que já está ativo **deixa o sistema sem fornecedor ativo** (a página de Suporte e o rodapé ficam vazios). > Na **v5** a lógica era um *toggle*: clicar no fornecedor ativo o > desativava, e clicar em outro desativava o anterior e ativava o novo. > A v6 mudou para "sempre ativar o clicado". Ver `08-GAPS-E-DIVIDAS.md`. ### 1.3 O `slug` é derivado do `name` a cada gravação `$request->merge(["slug" => Str::slug($request->name)])` acontece em **todo** `upInsert` — inclusive na edição. Renomear o fornecedor muda o slug. Não há índice único na coluna. ### 1.4 Telefone e WhatsApp são gravados só com dígitos `preg_replace('/\D/', '', ...)`. O valor **validado** (com máscara) pode ter 13 caracteres e o **gravado** tem 10. A exibição usa `changeToFone()` para reintroduzir a máscara. ### 1.5 `fornecedor_ativo` nunca é gravado pelo formulário A lista de campos do `SupplierRepository::structureUpInsert` **não** inclui `fornecedor_ativo`. O status só muda pela rota de alternância. ### 1.6 O `image` e o `icon` guardam o **caminho relativo**, não a URL `'img/images_fornecedores/' . $nomeDoArquivo`. A view monta a URL como `/storage/{caminho}`. Se o disk `public` mudar de link simbólico, as imagens quebram. ### 1.7 A listagem do CMS **não** usa o model Usa a view SQL `fornecedores`, que expõe `phone` como `telefone` e `address` como `endereco` — e **não** expõe `slug`, `locale` nem `whatsapp`. O WhatsApp só aparece na página de Suporte. ### 1.8 A exclusão é soft delete, mas os arquivos ficam `Supplier` usa `SoftDeletes`; a view SQL filtra `deleted_at IS NULL`. Os arquivos em `img/images_fornecedores/` e `img/icons_fornecedores/` **permanecem no disco**. --- ## 2. Acoplamentos ### 2.1 Dependências de saída | Dependência | Tipo | Onde | |---|---|---| | `ISupplierRepository` / `SupplierRepository` | interno | controller | | `AppTrait` / `FileTrait` | interno | repository | | `Str` (slug), `Str::slug` | Laravel | repository | | `Storage` (disk `public`) | Laravel | `FileTrait::saveUpload` | | `Intervention\Image` (compressão) | externo | `FileTrait::compress` | | `modulos` (id 6) | interno | menu | ### 2.2 Dependências de entrada | Consumidor | Dependência | Risco | |---|---|---| | Menu do AdminLTE | `route('supplier.index')` + `$modulos['fornecedores']` | item some | | **Módulo `suporte`** | `Supplier` (ativo) | página de contato vazia | | Rodapé do AdminLTE | `Supplier` (ativo) | rodapé sem fornecedor | | `Prefecture\DatatableController` | view SQL `fornecedores` | — | | `resources` | **nenhum** recurso `supplier` | permissões inacessíveis | > O módulo `fornecedores` é a **dependência de dados** do módulo > `suporte`: se o fornecedor ativo não existir, a página de Suporte > fica em branco. Alterações aqui têm efeito direto lá. --- ## 3. Armadilhas ### 3.1 [CRÍTICO] As rotas de escrita não têm autenticação `supplier.store`, `supplier.update`, `supplier.destroy` e `/cms/fornecedores_switch` **não** declaram `check.logged`, e o `SupplierController::__construct()` não registra `check.route.permission` (nem chama `parent::__construct()`, que é quem o registraria). Resultado: **qualquer pessoa com um token CSRF válido** — inclusive um visitante que leia o token de uma página pública, ou um operador com perfil mínimo — pode criar, alterar, excluir e ativar fornecedores. Mitigação: aplicar `check.logged` nas quatro rotas **e** registrar `check.route.permission` no construtor (ou usar Policy). ### 3.2 [ALTO] O módulo não tem recurso em `resources` Não há `resources.slug = 'supplier'`. Logo: - `has_access(['supplier.index'])` e `has_access(['supplier.create'])` são falsos para qualquer perfil que não seja `super-admin`; - o item de menu **só** aparece para `super-admin`; - os botões de ação da linha (editar/excluir/alternar) ficam desabilitados para todos os demais. O módulo está, na prática, restrito ao `super-admin` — apesar de estar "ligado" em `modulos.status`. ### 3.3 [ALTO] `Repository::upInsert` faz `dd($th)` `v6/app/Repositories/Repository.php:64-71` captura `Throwable` e chama `dd($th)` — a exceção é **vazada na página** em vez de retornar erro. Com o `SupplierRequest` validando quase tudo, o caso mais provável é o `phone`/`whatsapp` vazio virando `null` numa coluna NOT NULL. ### 3.4 [ALTO] Telefone vazio vira `null` numa coluna NOT NULL `filter_request` converte valores falsos em `null`: ```php if (!$valor) $filter[$campo] = null; else $filter[$campo] = $valor; ``` Um `phone` que, após o `preg_replace`, fique `""` (por exemplo, o usuário digitou só letras) vira `null` -> violação de `NOT NULL` -> `dd($th)`. ### 3.5 [MÉDIO] `switchFornecedorAtivo` não é transacional e não valida o id - Desativa o anterior e depois ativa o novo, **sem transação**: se a segunda parte falhar, o sistema fica sem fornecedor ativo. - `find($request->contents_id)->update(...)` com id inexistente -> `null->update()` -> **erro 500**. - O parâmetro se chama `contents_id` (herança de outros módulos) em vez de `supplier_id` — confuso. ### 3.6 [MÉDIO] Clicar no fornecedor já ativo o desativa Como a v6 Activation substituiu o toggle da v5, o efeito é: o operador não consegue mais **desativar** um fornecedor pela interface. Precisa ativar outro. ### 3.7 [MÉDIO] `fornecedor_ativo` é gravado como `null`, não `false` `$oldSulplier->update(['fornecedor_ativo' => null])`. Funciona (o `where('fornecedor_ativo', true)` não encontra `null`), mas deixa a coluna com três estados (`true`, `null`) e nenhum `false`. A coluna é nullable, então funciona — mas é um estado implícito. ### 3.8 [MÉDIO] `slug` recalculado a cada edição, sem unicidade Trocar o nome do fornecedor muda o slug silenciosamente. Não há índice único, então dois fornecedores com nomes iguais geram slugs iguais. ### 3.9 [MÉDIO] Sem auditoria Nenhuma chamada a `activity()`: criar, editar, excluir e ativar fornecedores **não** deixa rastro em `activity_log` — apesar de o módulo controlar quem é o responsável técnico pelo portal. ### 3.10 [MÉDIO] Arquivos órfãos no disco A exclusão (soft) não remove a imagem nem o ícone; trocar a imagem não remove o arquivo anterior (`saveUpload` remove apenas o arquivo de **mesmo nome**, e o nome é aleatório). ### 3.11 [BAIXO] `Supplier::$fillable` com colunas inexistentes `telefone` e `endereco` não existem em `suppliers`. O `SupplierRepository::structureUpInsert` limita os campos, o que evita o erro — mas o model está errado. ### 3.12 [BAIXO] `locale` sem validação de URL O campo é o `src` de um `