Ticket #2100 (closed defeito: fixed)

Opened 8 years ago

Last modified 8 years ago

Problema na exibição de thumbs no arquivamento local

Reported by: diogenesduarte Owned by: diogenesduarte
Priority: normal Milestone: Expresso 2.2.8
Component: ExpressoMail Version: branch 2.2
Severity: média Keywords:
Cc: WorkGroup:

Description

Na exibição dos thumbs de mensagens arquivadas localmente, são adicionados os caracteres aspas e ponto-e-virgula para cada vez que o e-mail é aberto. Para reproduzir, arquive um e-mail que possua uma imagem como anexo, e abra no arquivamento local. Note que ao lado do thumb aparecerá os caracteres citados, e cada vez que for aberto o e-mail novamente, eles são duplicados.

Change History

comment:1 Changed 8 years ago by diogenesduarte

  • Owner changed from ninguem to diogenesduarte

comment:2 follow-up: ↓ 3 Changed 8 years ago by diogenesduarte

  • Status changed from new to closed
  • Resolution set to fixed

Esse problema é gerado ao abrir um e-mail, quando é chamada a função set_messages_flag, e essa captura o e-mail na base, altera o flag e insere o e-mail novamente. O erro está na hora de inserir novamente o e-mail, pois retira-se todas as aspas simples para duas aspas simples(escape de aspas no sqlite) para inserir o conteúdo na base e logo após isso, o expresso serializa array. Como a serialização conta as aspas a mais como posições, esse valor fica inconsistente, pois essas aspas se perdem quando inseridas no banco.

Para resolver, mudamos a forma de insert de set_messages_flag para a mesma utilizada em mail_sync na hora de inserir o e-mail, utilizando prepared statements... Assim a própria biblioteca cuida dos escapes e o serialize não fica inconsistente. Isso também serve como correção para o ticket 2099.

Corrigido na revisão [4857].

Só uma dúvida. Como ficaria a replicação dessa correção para os ramos 2.3 e 2.4?

comment:3 in reply to: ↑ 2 Changed 8 years ago by wmerlotto

Replying to diogenesduarte:

Só uma dúvida. Como ficaria a replicação dessa correção para os ramos 2.3 e 2.4?

O Serpro é responsável por sincronizar as correções/melhorias do branch 2.2 para o 2.3.

Nós da Prognus somos responsáveis por sincronizar do branch 2.3 para o 2.4. Mas como conversamos, a Prodeb está realizando esta atividade.

comment:4 Changed 8 years ago by diogenesduarte

É essa minha dúvida.

Nós aqui na PRODEB já ajudamos na sincronização de todas as revisões da 2.3 até a ultima sexta para a 2.4, devido a ter uma fila muito grande de tickets a serem replicados. A partir daí a idéia seria continuarmos replicando novas alterações ou a Prognus assumiria?

E esse caso por exemplo, só será corrigido na 2.4 após a sincronização do serpro na 2.3, certo?

Note: See TracTickets for help on using tickets.