Ticket #815 (closed defeito: fixed)

Opened 14 years ago

Last modified 14 years ago

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

expressoMail1_2_07052010.patch Download (416 bytes) - added by rafaelraymundo 14 years ago.
Branch2.0 - expressoMail1_2/inc/class.exporteml.inc.php

Change History

comment:1 Changed 14 years ago by niltonneto

Um problema parecido foi resolvido no Ticket #93

comment:2 Changed 14 years ago by rafaelraymundo

  • Owner changed from alguem to rafaelraymundo

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

Corrigido em [1936] - Correção referente ao ticket [851] e adicionado caracter em remove_accents

comment:8 Changed 14 years ago by niltonneto

Favor replicar as correções no Branch 2.1

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:10 Changed 14 years ago by niltonneto

  • Status changed from reopened to new

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

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.

Note: See TracTickets for help on using tickets.