Ticket #815 (closed defeito: fixed)
Problema ao exportar mensagens com aspas simples no assunto
Reported by: | niltonneto | Owned by: | rafaelraymundo |
---|---|---|---|
Priority: | média | Milestone: | Expresso 2.0.7 |
Component: | ExpressoMail | Version: | branch 2.0 |
Severity: | Keywords: | exportar mensagem mensagens problema | |
Cc: | WorkGroup: |
Description
Não é possível exportar mensagens do expresso que contenham no campo assunto o caracter aspas simples ('). O arquivo .zip baixado tem 0kb. O caracter (') não está sendo substituído no nome do arquivo, gerando um nome de arquivo inválido.
Vide tópico no Fórum
Attachments
Change History
comment:3 Changed 14 years ago by rafaelraymundo
- Status changed from new to closed
- Resolution set to fixed
Implementado em [1802] - Foi modificado o método remove_accents sendo adicionada as aspas simples.
comment:4 Changed 14 years ago by niltonneto
- Status changed from closed to reopened
- Resolution fixed deleted
Eu ia te reportar o problema, mas o pessoal da comunidade foi mais rápido e reportou no Fórum. Favor corrigir. Rafael, cuidado com a desatenção. É interessante testar sempre antes de comitar.
comment:5 follow-up: ↓ 6 Changed 14 years ago by rafaelraymundo
Funcionalidade testada nos navegadores IE6, IE7, Firefox3.0 e Firefox3.5 Já estamos utilizando a funcionalidade.
Qual problema a comunidade encontrou?
comment:6 in reply to: ↑ 5 Changed 14 years ago by niltonneto
Replying to rafaelraymundo:
Funcionalidade testada nos navegadores IE6, IE7, Firefox3.0 e Firefox3.5 Já estamos utilizando a funcionalidade.
Qual problema a comunidade encontrou?
Funcionou por sorte. Veja melhor como funciona a função strtr(). Além disso, já estão explicando no Fórum ....
comment:7 follow-up: ↓ 9 Changed 14 years ago by rafaelraymundo
- Status changed from reopened to closed
- Resolution set to fixed
comment:9 in reply to: ↑ 7 Changed 14 years ago by niltonneto
- Status changed from closed to reopened
- Resolution fixed deleted
Replying to rafaelraymundo:
Corrigido em [1936] - Correção referente ao ticket [851] e adicionado caracter em remove_accents
Rafael, e então? Replicou a [1936] no Branch 2.1?
comment:11 Changed 14 years ago by rafaelraymundo
A alteração está no Branch2.1 na revisão [1936]
comment:12 follow-up: ↓ 13 Changed 14 years ago by niltonneto
Rafael, no Branch 2.1 não funcionou direito. Se você exporta pela lista de emails somente a mensagem com aspas simples, dá pau ao gerar o zip. Se você seleciona outras juntas, não.
comment:13 in reply to: ↑ 12 Changed 14 years ago by rafaelraymundo
Olá Nilton,
Não consegui reproduzir o problema no branch 21 Criei mensagens com aspas simples no assunto e exportei/importei uma delas na lista de mensagens. Fiz o mesmo com uma série de mensagens. Em ambos os casos o zip (mensagens.zip) foi gerado com sucesso e a importação ocorreu normalmente...
O teste seria de outra forma?
Caso contrário acredito que esse Ticket pode ser fechado.
Replying to niltonneto:
Rafael, no Branch 2.1 não funcionou direito. Se você exporta pela lista de emails somente a mensagem com aspas simples, dá pau ao gerar o zip. Se você seleciona outras juntas, não.
comment:14 follow-up: ↓ 15 Changed 14 years ago by rodsouza
Não querendo ser novamente o criador de controvérsias entretanto o ExpressoLivre? Branch 2.1 não está descontinuado?
comment:15 in reply to: ↑ 14 Changed 14 years ago by niltonneto
- Milestone changed from Expresso 2.2 to Expresso 2.0.6
Replying to rodsouza:
Não querendo ser novamente o criador de controvérsias entretanto o ExpressoLivre? Branch 2.1 não está descontinuado?
Olha, também não querendo ser chato, mas mandei um email para todos os desenvolvedores avisando sobre a descontinuação da 2.1. Bem, voltando ao ticket, esse problema acontece no Branch 2.0, e acho interessante que seja corrigido lá também. Rafael, poderia anexar o patch de correção para aplicarmos na 2.0?
Changed 14 years ago by rafaelraymundo
- attachment expressoMail1_2_07052010.patch added
Branch2.0 - expressoMail1_2/inc/class.exporteml.inc.php
comment:16 follow-up: ↓ 17 Changed 14 years ago by rafaelraymundo
- Status changed from new to closed
- Resolution set to fixed
Patch para o branch 2.0 adicionado ao ticket.
comment:17 in reply to: ↑ 16 Changed 14 years ago by niltonneto
- Status changed from closed to reopened
- Version changed from trunk to branch 2.0
- Resolution fixed deleted
- Milestone changed from Expresso 2.0.6 to Expresso 2.0.7
Replying to rafaelraymundo:
Patch para o branch 2.0 adicionado ao ticket.
Rafael, você não deveria ter fechado este ticket, pois ainda faltava aplicar o patch no Branch 2.0. Agora a versão 2.0.6 já foi publicada sem aplicar esse patch. Vou ter que reabrir este ticket e realocá-lo para "Expresso 2.0.7", pois não adianta ele ficar na 2.0.6.
comment:18 Changed 14 years ago by niltonneto
Corrigido no Branch 2.0 [2831]
comment:19 Changed 14 years ago by rafaelraymundo
Ok Nilton,
Acabo de testar no branches/2.0 e a exportação/importação de mensagens com aspas(') no assunto funcionou corretamente após o update da cópia de trabalho. Ticket ok
comment:20 Changed 14 years ago by niltonneto
- Status changed from reopened to closed
- Resolution set to fixed
Neste caso, podemos entar finalizar.
Um problema parecido foi resolvido no Ticket #93