Ticket #843 (closed defeito: fixed)

Opened 11 years ago

Last modified 10 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 10 years ago.
Patch0 de correção de anexos em rascunhos
prognus_r382.diff Download (7.8 KB) - added by wmerlotto 10 years ago.
Patch1 de correção de anexos em rascunhos

Change History

comment:1 Changed 11 years ago by amuller

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

comment:2 Changed 11 years ago by zapa

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

comment:3 Changed 10 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 10 years ago by wmerlotto

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

Changed 10 years ago by wmerlotto

Patch0 de correção de anexos em rascunhos

Changed 10 years ago by wmerlotto

Patch1 de correção de anexos em rascunhos

comment:5 in reply to: ↑ 4 Changed 10 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 10 years ago by kolling

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

comment:7 Changed 10 years ago by kolling

  • Status changed from new to assigned

comment:8 Changed 10 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 10 years ago by cesar.vianna

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

comment:10 Changed 10 years ago by niltonneto

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

comment:11 in reply to: ↑ 9 Changed 10 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 10 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 10 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.