Ticket #1669 (closed defeito: fixed)
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.
Attachments
Change History
comment:1 Changed 12 years ago by kolling
- Owner changed from ninguem to kolling
- Status changed from new to assigned
comment:2 Changed 12 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 12 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 12 years ago by kolling
- Status changed from assigned to closed
- Resolution set to fixed
Linhas removidas na revisão [3906].
comment:5 Changed 12 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 12 years ago by niltonneto
-
attachment
compara_datas_20_22.png
added
Comparação entre as datas da 2.0 (certo) e 2.2 (errado)
comment:7 Changed 12 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 12 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 12 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 12 years ago by kolling
- Status changed from closed to reopened
- Resolution fixed deleted
comment:11 Changed 12 years ago by kolling
- Status changed from reopened to closed
- Resolution set to fixed
Commit na revisão [3927].
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.