Ticket #843 (closed defeito: fixed)

Opened 10 years ago

Last modified 8 years ago

Salvar rascunho não grava imagem inserida

Reported by: zapa Owned by: kolling
Priority: grave Milestone: Expresso 2.2.0.1
Component: ExpressoMail Version: branch 2.2
Severity: média Keywords: TAG2.2.0.1.5
Cc: WorkGroup: Centro de Testes

Description

Ao gravarmos uma mensagem nos rascunhos que contenha imagem inserida no corpo, a imagem não é salva.


1.Crie uma nova mensagem e insira uma imagem; 2.Salve nos rascunhos e feche a mensagem; 3.Va na pasta rascunhos e edite a mensagem; 4.Veirfique se a imagem inserida aparece;

Attachments

prognus_r381.diff Download (8.1 KB) - added by wmerlotto 8 years ago.
Patch0 de correção de anexos em rascunhos
prognus_r382.diff Download (7.8 KB) - added by wmerlotto 8 years ago.
Patch1 de correção de anexos em rascunhos

Change History

comment:1 Changed 10 years ago by amuller

Acho que este ticket não está claro o suficiente.

comment:2 Changed 10 years ago by zapa

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

comment:3 Changed 8 years ago by cesar.vianna

  • Status changed from closed to reopened
  • Severity set to média
  • Summary changed from Salvar rascunho não grava imagem inserida(ticket#????) to Salvar rascunho não grava imagem inserida
  • Version changed from trunk to branch 2.2
  • Milestone changed from Expresso 2.1 to Expresso 2.2.0.1
  • Keywords TAG2.2.0.1.5 added
  • Resolution invalid deleted
  • WorkGroup set to Centro de Testes

Erro ocorre conforme descrição do Zapa.

comment:4 follow-up: ↓ 5 Changed 8 years ago by wmerlotto

Cesar e Zapa, tivemos este mesmo problema e já fizemos uma correção. Anexei os patches para vcs avaliarem.

Changed 8 years ago by wmerlotto

Patch0 de correção de anexos em rascunhos

Changed 8 years ago by wmerlotto

Patch1 de correção de anexos em rascunhos

comment:5 in reply to: ↑ 4 Changed 8 years ago by wmerlotto

Replying to wmerlotto:

Cesar e Zapa, tivemos este mesmo problema e já fizemos uma correção. Anexei os patches para vcs avaliarem.

Em nosso branch (2.2.0.2) já enviamos esta correção, conforme o ticket #1888.

comment:6 Changed 8 years ago by kolling

  • Owner changed from brunocosta to kolling
  • Status changed from reopened to new

comment:7 Changed 8 years ago by kolling

  • Status changed from new to assigned

comment:8 Changed 8 years ago by kolling

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

Commit da correção aplicada na 2.2.0.2. Com uma pequena correção no arquivo draw_api.js. [4683]

comment:9 follow-up: ↓ 11 Changed 8 years ago by cesar.vianna

Mário, foi aplicada no Branch 2.2.0.2 ou 2.2.0.1?

comment:10 Changed 8 years ago by niltonneto

Pessoal, o ticket #1578 não corrige isto?

comment:11 in reply to: ↑ 9 Changed 8 years ago by kolling

Replying to cesar.vianna:

Mário, foi aplicada no Branch 2.2.0.2 ou 2.2.0.1?

Desculpem, foi aplicado no branch 2.2.0.1, não no 2.2.0.2.

comment:12 Changed 8 years ago by zapa

  • Status changed from closed to reopened
  • Resolution fixed deleted

Estou reabrindo-o em função do problema ainda persistir, e confirmado.

comment:13 Changed 8 years ago by kolling

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

Commit na revisão [4848]. Erro na inserção da imagem acontecia devido à parâmetro incorreto (msg_part), que não apontava corretamente à parte do rascunho que correspondia à imagem inserida. Também foi revertido parte da implementação da prognus (acima) no arquivo main.js devido à erro de javascript.

Note: See TracTickets for help on using tickets.