Ticket #1498 (closed defeito: fixed)

Opened 13 years ago

Last modified 13 years ago

Marcar mensagem offline como Importante

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

Description (last modified by diogenesduarte) (diff)

EL-1047: Marcar mensagem offline como Importante

FF3-Ubuntu8.04 Unificação 2.2

O sistema mostra a mensagem do passo 4, porém não classifica a mesma como importante. Retornando a visualização das mensagens da pasta Caixa de entrada.

Change History

comment:1 Changed 13 years ago by diogenesduarte

  • Owner changed from alguem to diogenesduarte
  • Description modified (diff)

comment:2 Changed 13 years ago by diogenesduarte

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

O problema ocorria pelos seguintes fatores:

  • Ao clicar no link, não estava sendo chamado o proxy_mensagens, e sim direto set_messages_flag, ou seja, mensagens locais(offlines) estavam sendo desconsideradas.
  • No set_messages_flag de local_messages faltava adicionar a informação no array serializado da mensagem, pois antes só era adicionado no cabeçalho. Como essa informação agora estava sendo necessária ao clicar no link dentro da mensagem, precisamos adicionar esse valor no array do e-mail também.
  • A lógica do método changeLinkState em draw_api apresentava problemas, inclusive em mensagens não locais, fazendo com que se mudássemos na mesma tela a mensagem de normal para importante e depois de importante para normal, o expresso mostrava mensagens atrapalhadas e não apresentava o retorno esperado.

Todas essas problemas acima citados foram resolvidos na revisão [3716].

Além disso, a chamada para mudar o flag era feita a set_messages_flag, e o proxy chama set_message_flag para o caso de mudança de flag de apenas uma mensagem. O set_message_flag não atualizava a mensagem na sua pasta, caso a pasta estivesse aberta. Corrigimos também em [3716] a lógica de set_message_flag, afim de que a correção não gerasse um outro defeito em mensagens não locais.

Note: See TracTickets for help on using tickets.