Ticket #1751 (closed defeito: fixed)
Lentidão na abertura da caixa dentro do ExpressoMail
Reported by: | alexandrecorreia | Owned by: | alexandrecorreia |
---|---|---|---|
Priority: | alta | Milestone: | Expresso 2.2.0-rc1 |
Component: | ExpressoMail | Version: | branch 2.2 |
Severity: | média | Keywords: | lentidão abertura caixa postal |
Cc: | WorkGroup: |
Description
Esta lentidão foi ocasionada pela revisão 3929, isto afeta diretamente tb o módulo mobile.
Attachments
Change History
comment:2 Changed 12 years ago by rafaelraymundo
Favor verificar a codificação do seu ambiente de desenvolvimento, pois o arquivo alterado apresentou inconsistências na acentuação.
comment:3 in reply to: ↑ 1 Changed 12 years ago by eduardoalex
Replying to alexandrecorreia:
Revisão [4014] - Esta correção ainda está sendo testada por Roberto Santos Junior ( Prodeb ).
alexandre,
Reveja o a codificação desse arquivo, todos os acentos estão fora de codificação.
comment:4 Changed 12 years ago by roberto.santosjunior
Alexandre,
o problema da codificação so afetou uma função, que corigi aqui para adiantar os testes. Com a sua revisão o ticket #1655 voutou a apresentar problemas. Pesquisei aqui uma nova solução, voltei a sua revisão [4014] e apliquei esta nova solução. Aqui, no nosso ambiente de desenvolvimento, esta nova solução, revisão [4022], apresentou um bom desempenho e não gerou o problema no ticket #1655.
Aguardando Teste de Alexandre Correia (CELEPAR).
comment:5 Changed 12 years ago by alexandrecorreia
Roberto !!
Isto deixa a abertura da caixa lenta, pois o range é muito grande !!!
$sort_array_msg = $this-> get_msgs($folder, $sort_box_type, $search_box_type, $sort_box_reverse,$msg_range_begin,$num_msgs);
Está em negrito, anteriormente o range da mensagem era limitado pelos parâmetros msg_range_begin e msg_range_end, parametros que são passados para este método.
Desculpe pelo commit numa codificação diferente !!!
comment:6 Changed 12 years ago by roberto.santosjunior
Observe que estou usando agora a função imap_mailboxmsginfo que deixa o desempenho bem melhor. Teste essa versão em seu ambiente.
Com o reverse a codigicação já está ok, so precisamos fechar estes dois problemas(desempenho e pesquisa correta das informações).
Apos os testes comente o que achou.
comment:7 Changed 12 years ago by alexandrecorreia
Roberto !!
Vc chegou a importar na sua caixa aquele zip que enviei pra vc com 1400 mensagens ?
comment:8 Changed 12 years ago by roberto.santosjunior
Não recebi o e-mail com o zip das mensagens, por favor envie para roberto.santosjunior@….
Changed 12 years ago by alexandrecorreia
-
attachment
Linux.tar.bz2
added
Mensagens para teste, este arquivo tem aproximadamente 1500 mensagens.
comment:10 Changed 12 years ago by alexandrecorreia
Roberto !!
Arquivo em formato tar.bz2 !!
comment:11 Changed 12 years ago by roberto.santosjunior
Alexandre,
testei em uma pasta acentuada com 2400 mensagens na VM de desenvolvimento, realmente a sua alteração gerou um resultado 4x mais rapida que a solução que implementei. Como podemos solucionar tambem o problema nas pastas acentuadas, referente ao ticket #1655?
Com a sua revisão [4014] temos um resultado rapido mais ainda incorreto para pastas acentuadas.
comment:12 Changed 12 years ago by alexandrecorreia
Opa Roberto !!
Tendo como ponto de partida aquela implementação que deixa mais rápido a abertura da caixa, podiamos a partir dele resolver o problema da acentuação !!! Qual o nome que vc está criando para as caixas ?
comment:13 Changed 12 years ago by roberto.santosjunior
A minha pasta tem o nome de "Teste de Acentuação" e o erro ocorre ao clicar nos links "Não Lidas", "Lidas", "Respondidas" e "Importantes". quando voce clica nos links o contador de mensagens apresenta null ou não mostra um numeral.
comment:14 Changed 12 years ago by alexandrecorreia
Opa Roberto !!
Acabei de reproduzir o problema aqui !! Vou resolver !
comment:15 Changed 12 years ago by alexandrecorreia
- Status changed from new to closed
- Resolution set to fixed
Revisão [4038]
comment:16 follow-up: ↓ 17 Changed 12 years ago by roberto.santosjunior
comment:17 in reply to: ↑ 16 Changed 12 years ago by rafaelraymundo
Com a correção [4150], a pré-visualização das mensagens no campo assunto passou a não funcionar.
Replying to roberto.santosjunior:
Foi criada uma nova revisão, definitiva, que garante a solução deste ticket e do ticket #1655. Revisão[4150]
comment:18 Changed 12 years ago by roberto.santosjunior
Foi criado um ticket #1805 para corrigir o problema.
Revisão [4014] - Esta correção ainda está sendo testada por Roberto Santos Junior ( Prodeb ).