Ticket #1056 (closed defeito: fixed)

Opened 12 years ago

Last modified 12 years ago

Criação de filtros que apagam todas as regras

Reported by: amuller Owned by: amuller
Priority: grave Milestone: Expresso 2.0.6
Component: ExpressoMail Version: branch 2.0
Severity: Keywords:
Cc: niltonneto WorkGroup:

Description

A criação de filtros continua com problema, mesmo tendo sido alterada em #615. A correção deve tratar melhor os erros.

Attachments

sieve_filters.diff Download (2.7 KB) - added by amuller 12 years ago.
common.diff Download (435 bytes) - added by amuller 12 years ago.

Change History

Changed 12 years ago by amuller

comment:1 Changed 12 years ago by amuller

Como eu não tenho permissão para commitar no branches estou anexando o diff

comment:2 Changed 12 years ago by niltonneto

Muller sempre comite no Trunk. Depois disso, eu replico no Branch 2.0

comment:3 Changed 12 years ago by amuller

É que eu fiz em cima do branches

comment:4 Changed 12 years ago by niltonneto

OK então, deixa que eu faço os commits deste ticket. Mas nos próximos, procure fazer em cima do Trunk, e por favor, informe sempre se há necessidade de aplicá-la ou não no Branch 2.0, já de trata da versão estável atualmente.

Obrigado.

comment:5 Changed 12 years ago by niltonneto

  • Version changed from trunk to branch 2.0

Corrigido no Trunk [2657]. Replicado no Branch 2.0 [2658]

comment:6 Changed 12 years ago by amuller

Para reproduzir este problema de sumir as regras basta colocar um sleep() no controller ou no class.ScriptS.inc.php. As regras somem a cada bloqueio de email pelo usuário.

Aplicado no trunk problema de concorrência

Enviando       filter.js
Transmitindo dados do arquivo .

Commit da revisão [2659]

E não aplicado no branches

Enviando       filter.js
svn: Commit falhou (detalhes a seguir):
svn: Servidor enviou valor de retorno inválido (403 Forbidden) na resposta a CHECKOUT pedido por '/!svn/ver/2658/branches/2.0/expressoMail1_2/js/filter.js'

Por favor aplicar.

comment:7 Changed 12 years ago by amuller

r2660 também dava problema pois não tinha um handler para mostar a janela, colocando um tempo fixo de 2,5 segundos, se a requisição fosse mais lenta perdia as regras. O que obviamente não é ideal.

Ficou mais rápido também a janela, pois quando é mais rápido que 2.5 segundos a janela já é mostrada.

Para aplicar no branches precisa também do sequinte patch do common_functions que esta anexado

Changed 12 years ago by amuller

comment:8 Changed 12 years ago by amuller

Deu pau no IE, segura um pouco o branches até eu corrigir

comment:9 Changed 12 years ago by amuller

r2661 corrige no IE

comment:10 Changed 12 years ago by niltonneto

Muller, apliquei todas as alterações acima no meu projeto local (branches/2.0) e não funcionou direito nem no Firefox.

comment:11 Changed 12 years ago by amuller

r2662 e r2663 também corrige outros problemas

comment:12 Changed 12 years ago by niltonneto

Muller, veja se ficou certo. Eu testei aqui e funcionou. Replicado no Branch 2.0 [2666].

comment:13 follow-up: ↓ 14 Changed 12 years ago by amuller

Acho que agora finalmente fechamos a saga dos filtros fujões

comment:14 in reply to: ↑ 13 ; follow-up: ↓ 15 Changed 12 years ago by niltonneto

Replying to amuller:

Acho que agora finalmente fechamos a saga dos filtros fujões

Agora o botão "Bloquear remetente" está dando erro. Não cria mais a regra corretamente. Aproveita e verifica também outro erro Javascript: ao tentar bloquear um remetente contendo nome acentuado, não faz nada.

comment:15 in reply to: ↑ 14 Changed 12 years ago by amuller

Replying to niltonneto: Javascript: ao tentar bloquear um remetente contendo nome acentuado, não faz nada.

Você pode anexar o fonte da mensagem? Nas msgs que testei não deu isso.

comment:16 follow-up: ↓ 17 Changed 12 years ago by amuller

r2668 corrige esse undefined e outros problemas

comment:17 in reply to: ↑ 16 Changed 12 years ago by niltonneto

Replying to amuller:

r2668 corrige esse undefined e outros problemas

Eu testei aqui e agora funcionou. Replicado no Branch 2.0 [2669]

comment:18 follow-up: ↓ 19 Changed 12 years ago by amuller

r2673 coloca mensagens amigáveis ao usuário quando o sieve está dando erro

comment:19 in reply to: ↑ 18 Changed 12 years ago by niltonneto

Replying to amuller:

r2673 coloca mensagens amigáveis ao usuário quando o sieve está dando erro

Muller, por favor, replique este commit no Branch 2.0 para fecharmos este ticket.

comment:20 Changed 12 years ago by amuller

r2788 aplica no branches

comment:21 Changed 12 years ago by niltonneto

  • Status changed from new to closed
  • Resolution set to fixed
Note: See TracTickets for help on using tickets.