Ticket #3236 (closed melhoria: fixed)

Opened 7 years ago

Last modified 7 years ago

Melhorias de performance no código do Expresso.

Reported by: douglasz Owned by: douglasz
Priority: normal Milestone: Expresso 2.5.0
Component: ExpressoMail Version:
Severity: média Keywords: Prognus
Cc: WorkGroup:

Description

Realizar as correções no código do Expresso afim de melhorar a performance e otimização com base em um relatório gerado através do Yasca mostrando os pontos que podem ser alterados.

Change History

comment:1 Changed 7 years ago by douglasz

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

Correção na r7655.

comment:2 Changed 7 years ago by douglasz

  • Status changed from closed to reopened
  • Resolution fixed deleted

comment:3 Changed 7 years ago by douglasz

Deletados os arquivos da pasta DB no qual são desnecessárias na r7656.

Também foram comitados as melhorias do código do expresso na r7656 na parte de include das tags scripts para os arquivos: contactcenter/templates/classic/head.inc.php contactcenter/templates/default/head.inc.php expressoMail1_2/programed_archiving.php phpgwapi/templates/default/head.inc.php

comment:4 Changed 7 years ago by douglasz

Correções para Performance: Function Within Loop Declaration na r7673.

comment:5 Changed 7 years ago by douglasz

Correções para para Best Practice: Short Open Tag e Best Practice: Always Quote Array Keys na revisão r7681.

comment:6 Changed 7 years ago by douglasz

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

comment:7 Changed 7 years ago by asaikawa

Desfeita uma alteração no arquivo Smarty_Compiler.class.php pois causou uma inconsistência na compilação de templates.

Na revisão r7655, havia sido removida uma chamada da função "count", do array $tokens, de dentro de uma condição da repetição "for", atribuindo este valor a uma variável e passou-se a utilizar esta variável na condição, porém, dentro do loop este array é manipulado, podendo ter seu tamanho alterado, o que implica em ter que fazer esta contagem a cada iteração, por isso, foi corrigido em [7684]

comment:8 follow-up: ↓ 10 Changed 7 years ago by rodsouza

A revisão r7655 causará diversos problemas como o apontado no ticket #3236.

As alterações realizadas sem a devida atenção afeta todas as iterações que modifica o contexto de execução, e que no Expresso não são poucas.

Uma revisão cuidadosa de tudo o que foi realizado sob o título "Melhorias de performance no código do Expresso." e correlatos deveria ser considerada como uma ação imprescindível e iminente, que não pode ser diferida.

comment:9 Changed 7 years ago by douglas

  • Status changed from closed to reopened
  • Resolution fixed deleted

comment:10 in reply to: ↑ 8 Changed 7 years ago by douglas

Replying to rodsouza:

A revisão r7655 causará diversos problemas como o apontado no ticket #3236.

As alterações realizadas sem a devida atenção afeta todas as iterações que modifica o contexto de execução, e que no Expresso não são poucas.

Uma revisão cuidadosa de tudo o que foi realizado sob o título "Melhorias de performance no código do Expresso." e correlatos deveria ser considerada como uma ação imprescindível e iminente, que não pode ser diferida.

Rodrigo, reabri o ticket. Foi precipitado comitarmos as alterações sem antes a devida análise para cada caso, como ocorreu com a correção em [7684]. Vamos revisar o contexto de cada correção aplicada, em busca de possíveis falhas introduzidas após [7655].

comment:11 follow-up: ↓ 12 Changed 7 years ago by douglasz

Devido ao fato de que muitos arquivos foram modificados se fez necessário a revisão de alguns arquivos novamente no qual foi levado em consideração a inconsistência encontrada pelo Asaikawa e por esse motivo deu-se mais atenção para a verificação sob a revisão [7673] para tratar de problemas conforme a descrição também do Asaikawa na revisão [7684].

Correção na [7712][7713].

comment:12 in reply to: ↑ 11 Changed 7 years ago by wmerlotto

Replying to douglasz:

Correção na [7712][7713].

Ainda bem que foi revisado... Pelo visto haviam vários loops inconsistentes.

comment:13 Changed 7 years ago by marcieli

Commit na revisão r7730 e r7731.

Melhorias de performance sobre o segundo relatório emitido.

comment:14 Changed 7 years ago by marcieli

  • Status changed from reopened to closed
  • Resolution set to fixed
Note: See TracTickets for help on using tickets.