Ainda persiste erro em navegação anterior/próximo do resultado da busca

[niltonneto]

Os problemas relacionados à navegação (anterior/próxima) entre as mensagens retornadas, na aba que lista o resultado da busca ainda persistem, mesmo após as correções feitas em #624. Alguém já corrigiu isso?

[amuller]

Sim, eu já corrigi muitas vezes.

[niltonneto]

Detectei esse problema no Branch 1.2, Branch 2.0 e Trunk. O mais estranho é não termos verificado isso antes. Ele se perde na navegação, abrindo mensagens que não fazem parte do resultado e sim da pasta pertecente à mensagem que foi clicada.

[diogenesduarte]

O problema é exatamente o que estava descrito nos comentários do ticket #624. Os tr's da busca possuem id correspondendo ao id da mensagem no imap, tanto na busca quanto na lista de mensagens da pasta atual. Quando uma busca é feita e a mensagem também está listada na pasta aberta, o expresso pega a linha da mensagem na primeira aba, e a próxima mensagem vira a próxima da pasta aberta e não da busca. Não consegui rastrear em commits antigos uma solução para esse problema, então adotei a seguinte estratégia para resolvê-lo:

• Todas as linhas no resultado da busca deverão possuir um indicador em seu id, indicando que é referente a uma aba de busca.
  
• O indicador seguirá no padrão [id da mensagem no imap]+"_s"+[número para diferenciar diferentes resultados na busca].
  

A estratégia descrita acima foi commitada no trunk na revisão [1969].

[diogenesduarte]

Replicado no branch 2.1 na revisão [1970].

[niltonneto]

Este ticket foi fechado sem aplicar a correção no Branch 2.0.

[eduardoalex]

Replicar no branch 2.0 e verificar no branch 2.2

[niltonneto]

Diogenes, por favor, verifique e comite no Branch 2.2. Assim que estiver ok lá, eu replico na 2.0.

[diogenesduarte]

Na verdade isso já tinha sido resolvido há muito tempo na revisão [1969], mas deixou de funcionar devido a uma alteração na revisão [3380] no arquivo search.js, revertendo a linha de uma das alterações feitas para correção desse problema. Colocamos novamente a linha corrigida na revisão [3659]. Nilton, agora é só replicar na 2.0, mas acredito que deva ser replicada a revisão [1969] e talvez depois a [3659].

[niltonneto]

Problema de navegação anterior/próxima no resultado da pesquisa

[niltonneto]

Diogenes, isso ainda não funciona direito na última revisão do Branch 2.2. Portanto, só vou poder aplicar na 2.0 quando lá estiver ok. Para reproduzir o erro que detectei, siga o modelo da imagem em anexo neste ticket. Você irá verificar que a navegação anterior/próximo não funciona corretamente ainda, quando há mensagens na Caixa de entrada e em outras pastas.

[diogenesduarte]

Nilton, tem certeza? Para não ter dúvidas, eu fiz checkout do branch 2.2, limpei todos os caches e cookies e testei com inúmeros resultados de várias pastas, e funcionou perfeito, sem nenhum erro e sem se perder hora alguma... Será que no seu teste você não tinha algum cache armazenado com código com problema? até porque a modificação feita foi em um arquivo .js.

Replying to niltonneto:

Diogenes, isso ainda não funciona direito na última revisão do Branch 2.2. Portanto, só vou poder aplicar na 2.0 quando lá estiver ok. Para reproduzir o erro que detectei, siga o modelo da imagem em anexo neste ticket. Você irá verificar que a navegação anterior/próximo não funciona corretamente ainda, quando há mensagens na Caixa de entrada e em outras pastas.

[niltonneto]

Em relação a esse teste em específico que mandei, tenho absoluta certeza. Estou com a última revisão, cache limpo, etc... Cara, está bem estranho. Pesquisa com acentuação (ex. Integração) não acha nada. Testa aí. Agora teve uma pesquisa que funcionou belezinha. Está bem intermitente o problema.

[diogenesduarte]

Com acentuação eu já tinha percebido há algum tempo que ele não acha mesmo. Você precisa pesquisar pela palavra sem acento, aí ele acha, tipo integracao. Quanto a navegação, eu tou tentando aqui de todos os jeitos e não acontece de jeito nenhum... Tentei buscar mensagens em pastas com acento e até achei um outro erro, mas a navegação entre as mensagens funcionou perfeito em todos os meus testes... Estranho demais isso, tem como você me mandar as mensagens que dão erro contigo para ver se consigo reproduzir?

Replying to niltonneto:

Em relação a esse teste em específico que mandei, tenho absoluta certeza. Estou com a última revisão, cache limpo, etc... Cara, está bem estranho. Pesquisa com acentuação (ex. Integração) não acha nada. Testa aí. Agora teve uma pesquisa que funcionou belezinha. Está bem intermitente o problema.

[niltonneto]

Tem sim. Vou te mandar. Deixe algumas na Caixa de Entrada, e as demais em outra pasta qualquer.

[niltonneto]

Você não acredita!! Exportei as mensagens, deletei da minha caixa, e depois importei elas novamente. Agora funcionou certinho...... Não sei o que houve, na boa, chefe... Há algo conspirando contra mim. Vou testar na 2.0 novamente......

[niltonneto]

Correção replicada em [3676] no Branch 2.0.