Melhorias de performance no código do Expresso.

[douglasz]

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.

[douglasz]

Correção na r7655.

[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

[douglasz]

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

[douglasz]

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

[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]

[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.

[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].

[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].

[wmerlotto]

Replying to douglasz:

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

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

[marcieli]

Commit na revisão r7730 e r7731.

Melhorias de performance sobre o segundo relatório emitido.