Ticket #2100 (closed defeito: fixed)
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:2 follow-up: ↓ 3 Changed 13 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 13 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 13 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?
