Ticket #912 (closed melhoria: fixed)
Adição de Teclas de Atalho M e O no expressoMail.
| Reported by: | valmir.sena | Owned by: | valmir.sena |
|---|---|---|---|
| Priority: | normal | Milestone: | Expresso 2.2 |
| Component: | ExpressoMail | Version: | branch 2.2 |
| Severity: | média | Keywords: | tecla atalho |
| Cc: | WorkGroup: |
Description
Quando aberta mensagem a tecla "O" expande as opções e a tecla "M" expande os emails (destinatários). Um novo pressionamento de "O" recolhe as opções e de "M" recolhe a lista de emails.
Change History
comment:1 Changed 14 years ago by valmir.sena
- Status changed from new to closed
- Resolution set to fixed
comment:4 Changed 14 years ago by rodsouza
- Status changed from closed to reopened
- Resolution fixed deleted
Por favor, antes de realizar uma modificação atualize sua revisão. A alteração feita voltou a ocasionar problemas.
E o PROBLEMA novamente é a CHAVE VOADORA que tinha sido removida em #919 na revisão r2046 e voltou a ocorrer na revisão r2051.
Caso essa a chave voadora funcione em seu ambiente por favor me instrua como se faz mágica.
comment:6 Changed 14 years ago by valmir.sena
- Status changed from new to closed
- Resolution set to fixed
comment:7 Changed 14 years ago by valmir.sena
Caro Rodrigo,
O que aconteceu foi o seguinte:
/Situação 1 /
if (Element('div_toaddress_'+ID) != null)
esqueci esta chave {
show_div_address_full(ID,'to');
}
if (Element('div_ccaddress_'+ID) != null) {
show_div_address_full(ID,'cc');
}
// O código ficou assim:
if (Element('div_toaddress_'+ID) != null)
show_div_address_full(ID,'to');
}
if (Element('div_ccaddress_'+ID) != null) {
show_div_address_full(ID,'cc');
}
//
Vc percebeu o erro, e ao invês de manter o padrão do if embaixo e colocar a chave faltante em cima vc retirou a chave de baixo deixando o código assim:
/*/
if (Element('div_toaddress_'+ID) != null)
show_div_address_full(ID,'to');
if (Element('div_ccaddress_'+ID) != null) {
show_div_address_full(ID,'cc');
}
//
Como você disse isto é tão básico que qualquer idiota pode perceber que não funciona.
rodsouza escreveu: Conceito básico de qualquer linguagem de programação, um bloco de código precisa respeitar >>seus delimitadores.
Como isso pode funcionar em algum lugar???
/ Assim, eu percebi o erro que havia cometido, executei o update, mas não percebi que vc havia retirado a chave debaixo na revisão r2046, e a recoloquei na r2051, por isso ocasionou de novo o problema, só que agora com a chave em cima.
if (Element('div_toaddress_'+ID) != null)
Recoloquei a chave {
show_div_address_full(ID,'to');
if (Element('div_ccaddress_'+ID) != null) {
show_div_address_full(ID,'cc');
}
O que ocasionou o problema descrito.
Rodrigo,
Infelizmente houve um problema com o servidor IMAP que eu uso nos testes com tunelamento e como eu havia testado o código no expresso que fica nele antes de dar problema, coloquei os ifs no código e resolvi comitar sem testar por se tratar de uma mudança simples, e assim, acabei esquecendo os parênteses.
Sou formado em Ciência da Computação, estou terminando mestrado, programo em C++ a mais de 7 anos em em PHP e javascript à 4 anos, não sou nenhum principiante. Programai muito em java no eclipse e infelizmente para javascript o eclipse não avisa como faz com java para avisar de parentese faltando. Sinceramente não vejo motivo para haver este tipo de tratamento grosseiro da sua parte menosprezando os outros. Esta não é primeira vez que isto acontece.
Resolvido na revisão [2017].