Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Erro com produtos com Estrutura vazias / Chamada de Modulo #20

Closed
crodriguesmg opened this issue Aug 20, 2024 · 7 comments
Closed

Erro com produtos com Estrutura vazias / Chamada de Modulo #20

crodriguesmg opened this issue Aug 20, 2024 · 7 comments
Assignees
Labels
priority::high Prioridade alta type::bug Something isn't working

Comments

@crodriguesmg
Copy link

crodriguesmg commented Aug 20, 2024

Quando um produto está sendo solicitado via "find" se o nó do array estrutura estiver com os dados vazio ele retorna erro, ou seja, seria como um KIT ou combo, se não tiver dados ele retorna erro no Entities/Produtos/Enum -> Situacao.php , TipoEstoque.php, lancamentoEstoque.php

Situacao.php ->Adicionar o case EXCLUIDO = 'E';
TipoEstoque.php e lancamentoEstoque.php -> case UNDEFINED = ''; // Caso para tratar valores vazios

definir esses case resolveram o problema me meu codigo

Outro problema está no arquivo
Bling.php -> o modulo Canais de venda não foi adicionado.

Adicionar a seguinte linha na função __get
'CanaisDeVenda' => $this->getModule(CanaisDeVenda::class),

@AlexandreBellas
Copy link
Owner

Obrigado pelo feedback! Até este fim de semana estarei implementando as correções que você pontuou.
Caso você precise das correções mais cedo, você pode abrir um PR com elas, o que você preferir. 😄

@AlexandreBellas AlexandreBellas added type::bug Something isn't working priority::high Prioridade alta labels Aug 20, 2024
@AlexandreBellas AlexandreBellas self-assigned this Aug 20, 2024
AlexandreBellas added a commit that referenced this issue Aug 25, 2024
AlexandreBellas added a commit that referenced this issue Aug 25, 2024
@AlexandreBellas
Copy link
Owner

Lancei a versão v1.14.1 que adiciona as correções mencionadas.

Me avise caso alguma outra correção seja necessária. Se não houver, fecharei a issue. 🙏

@crodriguesmg
Copy link
Author

crodriguesmg commented Aug 29, 2024 via email

@AlexandreBellas
Copy link
Owner

AlexandreBellas commented Sep 3, 2024

Fico feliz que a biblioteca está sendo útil para a sua empresa! 😄

Vou dar uma olhada neste último bug esta semana. Te dou um feedback assim que possível.
Uma pena que a documentação do Bling não é tão clara quanto à possibilidade de vir string vazia na resposta. 🫠

Você chegou a ter esse erro em outros objetos também?

@AlexandreBellas
Copy link
Owner

Pelo o que vi no código, o atributo ordem de um objeto do array midia->imagens->internas não está vindo como parte do $attributes de BaseResponseObject, representando que a API não está retornando o valor de ordem preenchido. Ou seja, ele pode ser null.

Vou deixá-lo nullable para resolver seu problema. Uma pena que não podemos confiar tanto assim na documentação. rs

image

AlexandreBellas added a commit that referenced this issue Sep 3, 2024
feat: corrigir novos bugs da issue #20
@AlexandreBellas
Copy link
Owner

Lancei a release v1.14.2 que torna os atributos nullable. Faça um teste com a nova versão e veja se isso corrige o seu caso. 🙏

@crodriguesmg
Copy link
Author

Perfeito, inicialmente problema resolvido.

A grande questão do bling é que temos 2 pontos,

  1. Produto principal ou único que ja vem com esses dados tudo certinho,
  2. outro é por exemplo um kit, que acesso os produtos que estão na estrutura do produto principal,
    então o problema não estava dando no produto principal (pai), mas sim no produtos da composição.

Perfeito, obrigado!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority::high Prioridade alta type::bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants