Ticket #1522 (closed defeito: fixed)
Filtros com Acentos
Reported by: | cesar.vianna | Owned by: | rodsouza |
---|---|---|---|
Priority: | normal | Milestone: | Expresso 2.2.0-pre2 |
Component: | ExpressoMail | Version: | branch 2.2 |
Severity: | grave | Keywords: | |
Cc: | WorkGroup: | Centro de Testes |
Description
Filtros contendo acentos na regra (tanto nos critérios, como nas ações) não funcionam corretamente.
Change History
comment:3 follow-up: ↓ 4 Changed 13 years ago by rodsouza
Essa questão dos filtros com acento não é nada trivial, pois palavras acentuadas são codificadas, e esta codificação pode ocorrer em alguns formatos.
Existe algum exemplo de filtro com acentuação que funcione? Até o momento nunca vi nenhum.
comment:4 in reply to: ↑ 3 Changed 13 years ago by cesar.vianna
Replying to rodsouza:
Caso seja concluído que não é possível implementar. teremos que avaliar a validação dos campos, não permitindo filtros com acentos.
comment:5 Changed 13 years ago by rodsouza
Não é impossível implementar os filtros com acentuação, todavia não é nada trivial pois exige que tenhamos exemplos nos vários formatos possíveis, ao menos dos mais significativos, e não apenas um exemplo para que seja viável avaliar a corretude.
Sugiro considerar tal modificação uma melhoria ao invés de um erro, assim deixar tal tarefa para a versão seguinte.
comment:6 Changed 13 years ago by niltonneto
Mesmo assim, antes, sugiro criticar ao salvar o filtro, caso contenha palavras acentuadas ou cedilha. Assim evita o transtorno de reclamações posteriores.
comment:7 Changed 13 years ago by niltonneto
- Owner changed from ninguem to alguem
- Component changed from API to ExpressoMail
comment:9 Changed 13 years ago by rodsouza
Será informado ao usuário que regras com caracteres especiais podem não funcionar.
A escolha de continuar ou não será do próprio usuário.
comment:10 Changed 13 years ago by rodsouza
- Status changed from new to closed
- Resolution set to fixed
Adicionado confirmação para criar filtro com acentos.
expressoMail1_2/js/filter.js expressoMail1_2/setup/phpgw_pt-br.lang
Commit da revisão r3775