Ticket #1029 (closed defeito: fixed)
Remover expressão regular desnecessária no método replace_links
Reported by: | rodsouza | Owned by: | rodsouza |
---|---|---|---|
Priority: | grave | Milestone: | Expresso 2.0.5 |
Component: | ExpressoMail | Version: | branch 2.0 |
Severity: | Keywords: | replace_links notes | |
Cc: | WorkGroup: |
Description
Com a remoção das expressões regulares que tratava de links do método replace_links, não é mais necessária a expressão regular que trata de links do notes.
Outra fator importante é que poucos são os casos que essa expressão é útil e mesmo assim ela é executada para todos os e-mails.
Change History
comment:2 Changed 14 years ago by niltonneto
Me avise quando finalizar, ou aplique diretamente nos branches 2.0 e 2.1
comment:5 follow-up: ↓ 6 Changed 14 years ago by amuller
Na verdade todas as expressões poderiam ser passadas pro JS já que, é muito pesada para o servidor e distribuindo no cliente isso se torna mais interessante. Além de que a tendência é que os navegadores sejam mais explorados visto que estão cada vez mais rápidos.
Eu vou fazer uns testes depois, mas a princípio até aquela que bloqueia as tags poderia ser migrada tbm.
comment:6 in reply to: ↑ 5 Changed 14 years ago by niltonneto
Replying to amuller:
Na verdade todas as expressões poderiam ser passadas pro JS já que, é muito pesada para o servidor e distribuindo no cliente isso se torna mais interessante. Além de que a tendência é que os navegadores sejam mais explorados visto que estão cada vez mais rápidos.
Eu vou fazer uns testes depois, mas a princípio até aquela que bloqueia as tags poderia ser migrada tbm.
Não acho seguro passar as expressões que efetuam "sanitize" no código-fonte.
Removendo expressão regular que tratava link notes.
Committed revision r2446.