Ticket #1059 (new tarefa)
Refatoração de código do javascript
Reported by: | amuller | Owned by: | ninguem |
---|---|---|---|
Priority: | média | Milestone: | |
Component: | ExpressoMail | Version: | sandbox |
Severity: | média | Keywords: | refactoring qa expressoMail |
Cc: | WorkGroup: |
Description
Este ticket vem continuar o serviço de #188, como ainda não existe boa coesão no código do ExpressoMail?, não foi possível definir testes de unidade, nem testes automatizados. Mas o presente ticket tem objetivo de:
- Diminuir as funções gigantes
- Diminuir os arquivos gigantes
- Diminuir o número de parâmetros das funções que possuem muitos
- Tentar dividir as funções que fazem muitas coisas, para várias que fazem apenas uma coisa.
- Procurar pedaços de códigos semelhantes que podem ser procedimentalizados.
Change History
comment:4 Changed 13 years ago by rodsouza
Removendo palavra reservada causadora de problemas no IE e no Webkit.
phpgwapi/js/expressoAjax/dom.js
Committed revision r2733.
comment:5 Changed 13 years ago by amuller
Também vou fazer como em um commit anterior, vou efetuar um commit mesmo não estando pronto ainda só para não ficar dessincronizado com o trunk. Ele não estragará nada que está funcionando. r2734
comment:6 Changed 13 years ago by amuller
r2747 torna o draw_folder mais otimizado e de quebra economiza uma requisição na troca de pasta
comment:7 Changed 13 years ago by rodsouza
Organizando a função draw_tree_folders para viabilizar a legibilidade.
expressoMail1_2/js/jscode/draw_api_folders.js
Committed revision r2899.
comment:8 Changed 13 years ago by rodsouza
Exibindo layout do ExpressoMail? independente do carregamento dos e-mails.
expressoMail1_2/js/jscode/draw_api_folders.js
Committed revision r2904.
comment:9 Changed 13 years ago by amuller
r2929 diminui o tamanho das variávies com mais de 25 caracteres e poem underline na frente para o compactador utilizar o nome delas.
comment:10 Changed 13 years ago by amuller
r2930 faz o mesmo
comment:12 Changed 13 years ago by niltonneto
- Owner changed from amuller to ninguem
- Version changed from trunk to sandbox
- Severity set to média
- Milestone changed from Backlog do Produto to Sandbox - Expresso 3.0
Como a tarefa ficou um pouco mais árdua que o normal, vou enviar para o svn para evitar de ficar dessincronizado
r2694 começa uma parte e a outra vou fazer em seguida