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 de Paginação - SkyHub\Api\Handler\Request\Catalog\ProductHandler #14

Open
renanconstancio opened this issue Sep 10, 2019 · 3 comments

Comments

@renanconstancio
Copy link

renanconstancio commented Sep 10, 2019

Linha 114

  • public function products($status = null)
  • {
  •     $query = [
    
  •         'status' => $status
    
  •     ];
    
  • public function products($inputs = null)
  • {
    
  •   $defaults = [
    
  •   	'status' => null,
    
  •   	'page' => 1,
    
  •   	'per_page' => 100
    
  •   ];
    
  •   $query = array_merge($defaults, $inputs); 
    
@renanconstancio
Copy link
Author

Há uma correção para nova atualização.
Api\Handler\Request\Catalog\ProductHandler.php
linha 163

  • $query[$key] = $value;
  • $query['filters'][$key] = $value;

Apenas fiz uma união para a busca de dados

@tiagosampaio
Copy link
Contributor

Olá @renanconstancio, tudo bem?

Não entendi muito bem o problema e se já foi corrigido. Poderia explicar melhor?

Abraços.

@renanconstancio
Copy link
Author

Olá @tiagosampaio .
Tenho uma pequena busca do meu sistema, para a api, sendo que a busca vem a gerar por chaves na api, ex: name, sku, etc, sendo unida a filters.
Citado acima, qualquer coisa estou disposto a ajudar ou explicar melhor essa funcionalidade.
O problema foi corrido, mas na hora de fazer essa pesquisa simples em meu sistema, me deparei a isso.
Vi que fizeram uma pequena verificação simples, porém bem valiosa, mais as chaves não se ajustaram a chamada na hora da pesquisa desse dados.

At.te:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants