Ticket #1059 (new tarefa)

Opened 10 years ago

Last modified 9 years ago

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:1 Changed 10 years ago by amuller

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

comment:2 Changed 10 years ago by amuller

r2699 continua o trabalho de refatoração.

comment:3 Changed 10 years ago by amuller

r2701 remodela draw_message

comment:4 Changed 10 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 10 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 10 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 10 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 10 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 9 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 9 years ago by amuller

r2930 faz o mesmo

comment:11 Changed 9 years ago by niltonneto

  • Milestone changed from Expresso 2.2 to Expresso 3.0

comment:12 Changed 9 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
Note: See TracTickets for help on using tickets.