Ticket #1669 (closed defeito: fixed)

Opened 13 years ago

Last modified 13 years ago

Horário incorreto de envio e recebimento de e-mails

Reported by: yasmin.kamal Owned by: kolling
Priority: normal Milestone: Expresso 2.2.0-beta
Component: ExpressoMail Version: branch 2.2
Severity: média Keywords:
Cc: WorkGroup: Centro de Testes

Description

Plataforma de Teste: FF3 - Win

O sistema está enviando e recebendo os e-mails com o horário incorreto.

Caso de teste:  http://testlink.expressolivre.org/lib/execute/execSetResults.php?version_id=1925&level=testcase&id=1924&tplan_id=2664&setting_build=20&setting_platform=4

Attachments

compara_datas_20_22.png Download (36.1 KB) - added by niltonneto 13 years ago.
Comparação entre as datas da 2.0 (certo) e 2.2 (errado)

Change History

comment:1 Changed 13 years ago by kolling

  • Owner changed from ninguem to kolling
  • Status changed from new to assigned

Erro tinha sido corrigido no Ticket #1322, voltou a ocorrer devido à alteração para corrigir Ticket #1485. Ao que parece foi um código que retornou de uma versão antiga, provavelmente através de um merge ou resolução de conflitos. Não parece fazer parte da solução para o Ticket #1485.

comment:2 Changed 13 years ago by kolling

As linhas que retornaram foram as linhas 3823, 3824, 3825 do arquivo class.imap_functions.inc.php. Que transforma o timestamp (que deveria ser GMT) em um timestamp do timezone local do servidor.

{{{ Prepare udate from mailDate (DateTime? arrived with TZ) for fixing summertime problem. $pdate = date_parse($header->MailDate?); $header->udate += $pdatezone?*(-60);}}}

comment:3 Changed 13 years ago by kolling

reeditando...

// Prepare udate from mailDate (DateTime arrived with TZ) for fixing summertime problem.
$pdate = date_parse($header->MailDate);
$header->udate +=  $pdate['zone']*(-60);

comment:4 Changed 13 years ago by kolling

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

Linhas removidas na revisão [3906].

comment:5 Changed 13 years ago by niltonneto

Kolling, o horário ficou correto mas o dia da semana não. Agora as mensagens de ontem aparecem como se fosse hoje, ou seja, apenas hora:minuto. E as mensagens de anteontem aparecem escrito "ontem".

Changed 13 years ago by niltonneto

Comparação entre as datas da 2.0 (certo) e 2.2 (errado)

comment:6 Changed 13 years ago by niltonneto

Veja a imagem em anexo.

comment:7 Changed 13 years ago by kolling

Sim Nilton, eu tinha notado isso, já da outra vez que eu implementei isso.

Mas isso aconteceu por causa da revisão [3113], que resolve o ticket #974. Que define a hora corrente e o minuto corrente como zero. Comparando a data do e-mail com o horário 00:00 do dia corrente.

Até foi por isso que eu não corrigi na época (Ticket #1322). Achando que era proposital.

comment:8 Changed 13 years ago by kolling

Do modo que se está comparando, a data do e-mail está sendo comparada com a meia-noite do dia corrente, ou seja, se o e-mail foi enviado às 10h da manhã não terá se passado as 24h e o expresso colocará a hora em que o e-mail foi enviado. Anteriormente a hora somente era mostrada após 24h terem se passado do envio do e-mail. A outra solução é definir a hora corrente como 23:59:59:59, nesse caso qualquer mudança de dia, fará com que a diferença seja maior que 24h.

Se quiser eu faço e comito.

comment:9 Changed 13 years ago by niltonneto

Kolling, sugiro manter como está hoje na versão 2.0, conforme imagem em anexo. Mantenha a comparação com o dia corrente, ok?

comment:10 Changed 13 years ago by kolling

  • Status changed from closed to reopened
  • Resolution fixed deleted

comment:11 Changed 13 years ago by kolling

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

Commit na revisão [3927].

Note: See TracTickets for help on using tickets.