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

O plugin está fazendo duas requisições de uma só vez a cada chamada do scroll infinito #2

Open
guilhermemarconi opened this issue Apr 4, 2016 · 3 comments

Comments

@guilhermemarconi
Copy link
Contributor

Reparei que, ao usar esse plugin, mesmo removendo o outro Infinity Scroll da página, ele faz duas vezes a mesma requisição quando deveria fazer somente uma. Olhei no Network do Chrome e percebi os dois requests juntos:

captura de tela 2016-04-04 as 15 54 24

E isso causou esse tipo de erro, que são as mesmas uls dessa requisição sendo retornadas duas vezes:

captura de tela 2016-04-04 as 15 51 59

Utilizei as seguintes configurações do plugin:

$('.search-navigator-wrap').find('input[type="checkbox"]').vtexSmartResearch({
  elemLoading: '<i class="shelf__loading"></i>',
  returnTopText:'',
  ajaxCallback: function() {
    shelfPriceAtacado();
  },
  shelfCallback: function() {
    shelfPriceAtacado();
  }
});

Onde a função shelfPriceAtacado somente manipula dados dos produtos, sem nenhuma requisição AJAX.

Alguma noção do que pode ser? Estou a disposição para ajudar no que puder!


EDIT

Obs.: Reparei que esse erro só acontece se eu não tiver feito nenhum filtro. A partir do momento que começo a filtrar, o scroll infinito funciona corretamente, mas volta a não funcionar se atualizar a página e remover os filtros.

@cvscarlos
Copy link
Owner

@guilhermemarconi normalmente esse erro é causado quando você chama duas vezes o plugin, talvez você tenha o InfinityScroll e o SmartResearch juntos.

Pode ser que seja algum outro erro também, porém esse plugin não recebe mais atualizações e então não sei te afirmar isso.

@guilhermemarconi
Copy link
Contributor Author

@caljp13 encontrei o erro. Ele é causado por uma instância própria do plugin que é feita no final do código fonte, a partir da linha 470 até a linha 510, como pode ser visto aqui: https://github.com/caljp13/VTEX-Smart-Research/blob/master/vtex-smartResearch.js#L470-L510

Comentando (ou apagando) as linhas que citei, esse problema é resolvido e o plugin continua funcionando normalmente.

Reparei um comportamento estranho porque o plugin já se instanciava sozinho em todas as páginas, mesmo quando eu não iniciava o plugin.

Se quiser, posso fazer um pull request, pelo menos para que outras pessoas não tenham esse problema. 😁

@cvscarlos
Copy link
Owner

@guilhermemarconi

kkkk aquilo realmente não faz o menor sentido ali. Pode mandar o pull-request sim.

Obrigado

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