Sinalizador de Acompanhamento e Marcadores de mensagens

[douglas]

Ticket que centraliza commits relacionados a esta melhoria.

Mudanças realizadas nesta melhoria:

• Permitir ao usuário sinalizar mensagens do Expresso Mail para acompanhar a mesma em outro momento definido;
• Permitir ao usuário configurar alarmes para mensagens sinalizadas e ser notificado pelos alarmes definidos para as respectivas mensagens inalizadas, na data e hora configuradas em cada alarme;
• Permitir ao usuário classificar suas mensagens com o uso de marcadores.

Ticket da Prognus:  #1045.

[douglas]

[5497]

• Commit inicial: Alteração dos menus de contexto e de ferramentas para a funcionalidade.

[adriano]

Replying to douglas:

[5497]

• Commit inicial: Alteração dos menus de contexto e de ferramentas para a funcionalidade.

Foi atualizada também a versão da biblioteca jQuery para v1.7.1.

[adriano]

Foi feito na r5533 o commit inicial do codigo das funcionalidades de marcadores e sinalizadores (ainda com algumas correções a serem feitas)

[adriano]

Commit na r5534 para retirar codigos acidentalmente incluidos na r5533.

[thiago]

[5535]

• Suporte a exibição de marcadores das mensagens entre o assunto e a lista de anexos de cada mensagem aberta.

[thiago]

[5536]

• Ajuste na exibição de marcadores de cada mensagem aberta.

[thiago]

[5538]

• Outro reajuste na exibição de marcadores de cada mensagem aberta.

[douglasz]

[5537]

• Link para abrir configuração do marcador no módulo principal do ExpressoMail.
• Ajustes de interface.
• Definição de cores padrões para os marcadores.

[adriano]

Commit nas r5540 e r5541, contendo:

• modificação para salvamento dos sinalizadores no imap (com informações extra no postgresql);
• correção de erros de salvamento e recuperação de sinalizadores;
• correção de erro que ocorrina no primeiro clique em um marcador da lista de marcadores para listar mensagens marcadas com ele;

[thiago]

[5544]

*Traducao *ajustes na exibição dos labels nas mensagens.

[thiago]

[5545]

• Reajuste na traducao que estava faltando nas flags.

[douglasz]

[5547]

• Foi atualizado o arquivo responsável por criar as tabelas novas no setup e os includes padrões.
• Correção de duplicação no salvamento dos sinalizadores.
• Correção de erro que não removia um sinalizador concluído quando salvo novamente sem que esteja a combox marcada.
• Ajeitado um pequeno detalhe de que criava scroll no modulo principal do ExpressoMail.

[marcieli]

[5548]

Resolvida lógica para:

• Sinalizar/marcar mensagens listadas no resultado da pesquisa.
• Exibir sinalizadores e marcadores nessas mensagens.
• E ao marcar/sinalizar mensagens na busca as mesmas devem ser marcadas/sinalizadas na sua pasta atual, e vice-versa.

[adriano]

Replying to thiago:

[5544]

*Traducao *ajustes na exibição dos labels nas mensagens.

Thiago, nos locais onde aparece a palavra em inglês "Marker", referem-se, na verdade à palavra "Label" que é a tradução adotada para marcadores em inglês.

[thiago]

[5551]

• Correção no arquivo de tradução, 'Marker' para 'Label'.

[thiago]

[5552]

• Correção no arquivo de tradução, 'Marker' para 'Label'.

• -- Não tinha sido enviado o arquivo label_configure.ejs.

[5553]

• envio do arquivo draw_api.js

[niltonneto]

Fui rodar o setup do Expressomail e apareceu o seguinte erro:

Database error: Invalid SQL: INSERT INTO expressomail_followupflag(name) VALUES ('Follow up'), ('Read'), ('Forward'), ('Answer'), ('Don''t forward'), ('Don''t answer');
pgsql Error: -1 (ERRO: erro de sintaxe em ou próximo a "," at character 65)

[marcieli]

[5554]

Resolvidos:

• Desmarcar mensagens selecionadas após sinalizar ou marcar, criar marcador ou sinalizador.
• Salvar marcador a partir do ícone editar dos marcadores listados na tela de configuração do marcador.

[thiago]

[5558]

• Adicionando outras traduções que estavam faltando.

[douglasz]

[5557]

Nilton, foi alterado a forma de montar o insert para os sinalizadores padrões, favor testar novamente para ver se ocorre o mesmo erro. Caso isso ocorra pode ser a versão do banco, pois tivemos problemas com os escapes na palavra "Don't".

Replying to niltonneto:

Fui rodar o setup do Expressomail e apareceu o seguinte erro:

Database error: Invalid SQL: INSERT INTO expressomail_followupflag(name) VALUES ('Follow up'), ('Read'), ('Forward'), ('Answer'), ('Don''t forward'), ('Don''t answer');
pgsql Error: -1 (ERRO: erro de sintaxe em ou próximo a "," at character 65)

[douglasz]

[5560]

• Foi comentado o código onde mostrava as antigas listagem de pastas.

[thiago]

[5561]

• Estava com inconsistência na abertura da edição de algum Label, o problema era que ele não estava setando corretamente o Label que o usuário clicou.

[wmerlotto]

Replying to douglasz:

[5557]

Nilton, foi alterado a forma de montar o insert para os sinalizadores padrões, favor testar novamente para ver se ocorre o mesmo erro. Caso isso ocorra pode ser a versão do banco, pois tivemos problemas com os escapes na palavra "Don't".

Replying to niltonneto:

Fui rodar o setup do Expressomail e apareceu o seguinte erro:

Database error: Invalid SQL: INSERT INTO expressomail_followupflag(name) VALUES ('Follow up'), ('Read'), ('Forward'), ('Answer'), ('Don''t forward'), ('Don''t answer');
pgsql Error: -1 (ERRO: erro de sintaxe em ou próximo a "," at character 65)

Nilton, se vc estiver utilizando o postgres 9.0 ou superior, coloque este parâmetro na configuração do mesmo:

standard_conforming_strings = off

[thiago]

[5563]

• Corrigido um bug na janela dos marcadores.

[marcieli]

[5564]

• Removidos comentários no código - search.js
• Atualizar a variável onceOpenedHeadersMessages após excluir um sinalizador.
• Corrigido salvar label, não estava salvando a partir do botão Salvar na tela de configuração do marcador.

[douglasz]

[5566]

• E foi trocado o da link que faz abrir as mensagens que estao selecionado com aquele marcador.
• Foi corrigido o problema do botão Criar da tela de marcador, pois quando criando um novo clicando pelo botão, o mesmo não estava salvando.
• E ajeitado as formas de chamar a listagem dos marcadores.

[thiago]

[5567]

• Recuperar o valor do tipo da flag, e enviar para a janela dos sinalizadores

[douglas]

Thiago, sugiro que na tela de criar/editar marcadores, você não use a expressão "Nenhum marcador cadastrado." para indicar que não existe nenhum marcador, pois, marcadores não são "cadastrados" tal qual um cadastro de clientes, ou de produto, tal qual a palavra sugere.

Sugiro a expressão "Nenhum marcador existente.", ou "Nenhum marcador criado.".

[adriano]

Commit na r5570 - alterações no interceptor para suportar o filtro utilizado para exibir os alarmes dos sinalizadores e ajustes na implementação da operação de create.

[adriano]

Replying to douglas:

Thiago, sugiro que na tela de criar/editar marcadores, você não use a expressão "Nenhum marcador cadastrado." para indicar que não existe nenhum marcador, pois, marcadores não são "cadastrados" tal qual um cadastro de clientes, ou de produto, tal qual a palavra sugere.

Sugiro a expressão "Nenhum marcador existente.", ou "Nenhum marcador criado.".

Eu sugiro "Nenhum marcador encontrado," pois trata-se apenas de uma ordem de listagem dada pelo usuário. Sugiro também uma mensagem mais descritiva para a correspondente em inglês. No caso de ser aceita minha sugestão, sugiro a correspondente "No labels found." em inglês.

[adriano]

Commit na r5571: Correções de bugs nos alarmes; refactory de parte do código dos alarmes.

[adriano]

Commit na r5572: retirado código commitado por engano na r5571.

[niltonneto]

Colunas desemparelhadas em relação ao seus headers.

[niltonneto]

Replying to wmerlotto:

Replying to douglasz:

[5557]

Nilton, foi alterado a forma de montar o insert para os sinalizadores padrões, favor testar novamente para ver se ocorre o mesmo erro. Caso isso ocorra pode ser a versão do banco, pois tivemos problemas com os escapes na palavra "Don't".

Replying to niltonneto:

Fui rodar o setup do Expressomail e apareceu o seguinte erro:

Database error: Invalid SQL: INSERT INTO expressomail_followupflag(name) VALUES ('Follow up'), ('Read'), ('Forward'), ('Answer'), ('Don''t forward'), ('Don''t answer');
pgsql Error: -1 (ERRO: erro de sintaxe em ou próximo a "," at character 65)

Nilton, se vc estiver utilizando o postgres 9.0 ou superior, coloque este parâmetro na configuração do mesmo:

standard_conforming_strings = off

Agora funcionou o SETUP! Estou usando postgreSQL 8 no ambiente de homologação. O problema agora está na visualização dos emails. As colunas ficaram todas desemparelhadas em relação aos seus headers, conforme a imagem em anexo.

[douglas]

Nilton, somente na Caixa de Entrada está assim?

Qual versão de browser é?

Limpe o cookies do browser e veja o que acontece.

[thiago]

[5573]

• novas traduções atualizadas.

[thiago]

[5574]

• Correção do bug presente na listagem de e-mails na atualização do menu dos marcadores.

[marcieli]

[5575]

Foi identificado problema já existente no Expresso Mail, quando realizada a busca rápida onde ids de elementos estão sendo duplicados. Foi avaliado o impacto de uma manutenção corretiva, e optou-se por não corrigir no momento, mas sim tentar contornar essa situação quando se tratando de aplicar marcadores e sinalizadores, onde os mesmos são aplicados apenas no primeiro elemento encontrado com o id específico.

[adriano]

Commit na r5576 - parte do código para salvamento de novos tipos de sinalizadores na interface de sinalização de mensagens.

[niltonneto]

Replying to douglas:

Nilton, somente na Caixa de Entrada está assim?

Qual versão de browser é?

Limpe o cookies do browser e veja o que acontece.

O Firefox é o 9.0.1 - Ubuntu. Havia limpado somente o cache. Depois que limpei os Cookies funcionou!

[adriano]

Commit na r5577 - implementado o salvamento de novos tipos de sinalizadores na interface de sinalização de mensagens

[thiago]

[5579]

• Correção do bug presente na tela dos sinalizadores, quando selecionava varios emails e abria a tela de edição dos sinalizadores, ele não estava recuperando corretamente o tipo, data de inicio, conclusão, cor da flag...

[adriano]

Commit na r5583 - correcao de erros na janela de aplicacao de sinalizadores

[marcieli]

[5584]

• Corrigido: quando sinalizar msg do retorno da busca rápida, havia inconsistencia em marcar a mesma msg na sua pasta de origem aberta.
• Aplicado novo atributo identificador para tr das msgs do retorno da busca rápida, contornando ids repetidos.

[thiago]

[5585]

• Correção na abertura da configuração apenas de 1 flag.

[douglasz]

[5586]

• Foi corrigido o problema de quando editado ou excluído um marcador não estava sendo aplicados nas listas de mensagens.
• Foi refeito algumas correções nos códigos que já foram comitados, pois por algum motivo quando efetuado um commit esses códigos foram alterados, refletindo o problema na tela de F9 na aba de nova mensagem.
• Correção em algumas palavras nas traduções.

[marcieli]

[5588]

• Corrigidos mais alguns bugs encontrados ao marcar/sinalizar e selecionar mensagens listadas no retorno da busca rápida.

[thiago]

[5597]

• Correção da inconsistência no tamanho dos menus.

[thiago]

[5599]

• Voltando na versão anterior desse arquivo.

[douglasz]

[5603]

• Correção de quando excluído um marcador ele não estava removendo o tooltip do marcador na lista de mensagens.
• Correção de validação de quantidade de caracteres para criação dos marcadores.
• Correção de no setup.

[marcieli]

[5605]

• Corrigidos bugs referentes a atualizar sinalizador, sinalizar/marcar msgs.

[thiago]

[5606] [5607]

• Considerando o espaço no input do nome do marcador, ele estava considerando os espaço como caracteres, sendo assim, possibilitava o usuario criar um label "sem nome".

[thiago]

[5610]

• Ao abrir a janela de configuração do acompanhamento ele não estava selecionando corretamente o enable e o disable do campo alarme(date) e alarme(time).

[thiago]

[5612]

• Corrigido a parte de abrir a lista dos emails marcados.a parte de abrir a lista dos emails marcados.

[adriano]

Commits:

• r5609 - impedimento de criação de mais de um sinalizador para cada mensagem
• r5611 - corrige interceptor de deepness para o conceito followupflagged que antes retornava todos os followupflag iguais

[thiago]

[5623]

• Melhoria no codigo da revisão: [5612].

Com essa melhoria foi gerado um bug na tela dos marcadores e que foi corrigido nessa revisão: [5624]

[marcieli]

[5625] Implementei a funcionalidade de:

• Ao marcar uma mensagem (botão direito sobre a mensagem -> Marcadores), seja fornecida a opção de criar um Novo Marcador.
• A(s) mensagem(s) selecionada(s) devem ser automaticamente marcadas com o(s) novo(s) marcador(es) criados nesse processo.

[marcieli]

[5640]

• Estilização dos botões Nova Mensagem, Atualizar e Ferramentas, além do menu exibido por Ferramentas.
• Resolvido bug referente a sinalizar mensagens com sinalizador customizado, isto é, criar novo sinalizador ao sinalizar uma mensagem, o sinalizador estava sendo salvo duplicado no banco.

[thiago]

[5643]

• Correção na edição do label clicando no botão do lado direito do label.=> ele não estava salvando corretamente, e ao retirar o bug, surgiu outro, mas agora ambos estão corrigidos.
• Ao abrir a tela de edição das flags, independente se abrir 1 ou varias, ele abrirá com a configuração correta daquela flag.

[marcieli]

[5649]

• Implementada funcionalidade na busca rápida: ao clicar sobre a bandeirinha (sinalizador) de uma mensagem onde esse sinalizador não é padrão, o mesmo deve ser exibido na tela de acompanhamento.
• Corrigida inconsistencia em sinalizar mensagem clicando na bandeirinha na busca rápida.

[douglasz]

[5658]

• Correções de sintaxe.
• Correções de interface na tela de sinalizadores.

[marcieli]

[5659]

Bugs Corrigidos:

• inconsistência na exibição de mensagens "concluindo hoje", na dialog.
• quando retirado o sinalizador de uma mensagem, o nome do sinalizador continuava visível.

[thiago]

[5660]

• Correção na atualização das tuplas na tabela de flags.

[thiago]

[5664]

• Correção na atualização das tuplas na tabela de flags. Melhoria.

[thiago]

[5665]

• Correção na atualização das tuplas na tabela de flags. Melhoria, Correção da melhoria.

[thiago]

[5666] [5667]

• O svn não deu merge no codigo, então atribui novamente o codigo da marcieli.

[thiago]

[5673]

• Quando adiciona um marcador, o salvar não fica desabilitado.

[thiago]

[5674]

• Na tela do followupflag, não habilitava o botão salvar quando editava a cor pelo color Picker, e nem quando clicava no select do "Sinalizar como"

[marcieli]

[5675]

Corrigidos bugs:

• Botão Criar permanece habilitado quando todos os labels são deletados na tela de configuração de marcador.

• Não deve ser permitido que o usuário digite dentro dos campos referentes a cor, tanto na tela de acompanhamento do sinalizador quanto na tela de configuração do marcador.

• Ajustado tamanho do campo referente as datas na tela de acompanhamento do sinalizador.

[thiago]

[5676]

• Ao colocar uma flag em um email X, depois selecionar esse email X que esta com flag e selecionar o email de cima que esta sem flag, ao tentar abrir a janela de edição dos sinalizadores gerava um erro.

[douglasz]

[5677]

• Correção no disable do botão Criar, quando for criar um novo marcador pelo campo "Nome do marcador:"

[marcieli]

[5683]

Corrigido problema com setup:

• Não estavam sendo inseridos os registros default para marcadores e sinalizadores.
• Não estava sendo criada chave estrangeira para a tabela associativa de mensagens e sinalizadores.
• Não estava sendo criada a tabela referente aos anexos das mensagens.

[marcieli]

[5684]

Padronizado nome da tabela referente a anexos.

[marcieli]

[5685]

Padronizado nome da tabela referente a anexos.(faltou alterar em um arquivo).

[thiago]

[5689]

• Correção de quando abrir uma mensagem pela telinha do alarme das flags.

Ele estava deletando a aba de algum email que ja estava selecionada , e adicionava por cima.

[douglasz]

[5690]

• Correções de includes de css, pois no IE não estava abrindo as funcionalidades dos marcadores.
• Corrigido alguns detalhes de interface.

[adriano]

Commit na r5697: implementação da pasta "para acompanhamento" e correção de problemas no exclusão.

• falta padronizar a utilização de ids do conceito followuoflagged no cliente, trocando o id do banco pelo id composto (!pasta/subpasta/<messageNumber>#<idDoBanco>)

Commit na r5697: inclusão da biblioteca de geração de ids para viabilizar a criação de Message-IDs para mensagens que não passam pelo postifix (já que é o postifix o responsável por este campo do cabeçalho) como os rascunhos e cópias das enviadas. Este campo do cabeçalho servirá para futuras mudanças na forma de armazenamento dos sinalizadores, por conta da limitação de número de flags dada pelo cyrus e outras implementações do IMAP.

[adriano]

Replying to adriano:

Commit na r5697: implementação da pasta "para acompanhamento" e correção de problemas no exclusão.

• falta padronizar a utilização de ids do conceito followuoflagged no cliente, trocando o id do banco pelo id composto (!pasta/subpasta/<messageNumber>#<idDoBanco>)

Commit na r5697: inclusão da biblioteca de geração de ids para viabilizar a criação de Message-IDs para mensagens que não passam pelo postifix (já que é o postifix o responsável por este campo do cabeçalho) como os rascunhos e cópias das enviadas. Este campo do cabeçalho servirá para futuras mudanças na forma de armazenamento dos sinalizadores, por conta da limitação de número de flags dada pelo cyrus e outras implementações do IMAP.

Correção: Commit na r5698: inclusão da biblioteca...

[thiago]

[5700]

• Na modal de alarme, ao clicar no check box "Não mostrar essa janela por hoje." logo em seguida em "Ok", a modal concluído e Alarme desapareciam, sendo que o correto é desaparecer apenas a modal que você desejou ocultar durante durante o período especificado.

Retirando o espaço: [5701]

[douglasz]

[5704]

• Foi compatibilizado a tela dos marcadores para abrir no IE.

[thiago]

[5705]

• Corrigido o bug presente nas janelas de alarmes, o problema era que não estava sendo tratado a parte converter as horas que a mensagem esta sinalizada, para hh:mm.

[marcieli]

[5708]

• Correção de problemas em salvar followupflagged e adequação de novos ids.

[thiago]

[5709]

• Melhoria no codigo da correção anterior das janelas dos alarmes.

[5710]

• Foi aumentado o tamanho dos marcadores dentro dos emails.

[marcieli]

[5711]

• Correção na adequação dos ids compostos dos followupflagged.

[douglasz]

[5713]

• Foi compatibilizado as telas dos marcadores e sinalizadores para abrir no IE.
• Foram retirado os addClass removeClass hidden para hide e show para funcionar no IE.
• Foi aumentado o z-index do zebra_dialog quando excluir um marcador.

[marcieli]

[5716]

• Adequei as bandeiras dos sinalizadores, corrigindo falhas de preenchimento da img, substituindo a imagem do corpo da bandeira pelo background-color da div.

[marcieli]

[5717]

• Diminui o comprimento da bandeira de sinalizadores.

[gustavo]

[5718]

• Corrigido problema do contextmenu no linux;

[marcieli]

[5719]

• Adequei as bandeiras dos sinalizadore na tela de acompanhamento.

[thiago]

[5730]

• Problema do timepicker corrigido.

[marcieli]

[5738]

• Internacinalizados marcadores e sinalizadores.

[thiago]

[5740][5743]

• Implementação de recuperação dos labels e acompanhamentos pelo php.

[thiago]

[5748]

• Correção do bug na hora de listar os emails do anexar mensagem.

[douglasz]

[5749]

• Adicionado os ícones que aparecem na caixa de entrada na tela de anexar mensagens. (Obs: As funcionalidades para os sinalizadores e marcadores, ainda serão definidas para a implementação).

[douglasz]

[5757]

Foram adicionado os ícones faltantes do commit na r[5749].

[douglasz]

[5760]

• Correção nos títulos dos ícones da tela de anexar mensagens.

[thiago]

[5814]

• Implementação da recuperação das flags e dos labels pelo imap_functions.

[thiago]

[5816]

• Atribuindo a função count em uma variavel.

[marcieli]

[5817]

• Corrigido problema com ids dos followupflaggeds ao atualizá-los.

[thiago]

[5820]

• Atribuir nome dos labels na opção excluir.

[thiago]

[5824]

• Deixar em negrito o nome do label na model de excluir.

[thiago]

[5825]

• Problema de AM e PM nos alarmes, Fixed.

[douglasz]

[5826]

• Correções dos sinalizadores e marcadores na tela de anexar mensagens.
• Correções das listagem de marcadores.
• Correção do ícone importante nas mensagens.

[thiago]

[5827][5828]

• problema no contador e abrir abas fixed.

[marcieli]

[5829]

Corrigido na tela de acompanhamento dos sinalizadores:

• Trocar 'ás' para 'às';
• 'Concluido' para 'Concluído';
• Quando aplicado o sinalizador a tela não fecha.

[marcieli]

[5830]

Resolvido conflito no arquivo expressoMail1_2/setup/phpgw_pt-br.lang

[douglasz]

[5831][5832]

• Correção para os sinalizadores que estão concluídos para aparecer na tela de anexar mensagens.

[niltonneto]

Ao responsável pelo ticket ou pela correção da revisão [5748]:

Encontramos alguns problemas referente ao commit [5748], e gostaríamos de saber se podemos corrigi-lo, alterando o caminho relativo do seguinte "include":

De: include_once '../prototype/api/controller.php';
Para: include_once dirname(__FILE__).'/../../prototype/api/controller.php';

O novo caminho relativo segue o modelo dos demais "includes" da classe imap_functions, e acredito que não gere nenhum efeito colateral. Podemos comitar essa correção ou vocês fazem?

[niltonneto]

Outra correção necessária de caminho relativo na classe imap_functions:

De: require_once('../library/CssToInlineStyles/css_to_inline_styles.php');

Para: require_once(dirname(__FILE__).'/../../library/CssToInlineStyles/css_to_inline_styles.php');

[marcieli]

Olá Nilton,

desculpa a demora, esta solução sugerida foi commitada na revisão [5993] (Corrigido caminho relativo em includes da classe imap_functions).

Replying to niltonneto:

Outra correção necessária de caminho relativo na classe imap_functions:

De: require_once('../library/CssToInlineStyles/css_to_inline_styles.php');

Para: require_once(dirname(__FILE__).'/../../library/CssToInlineStyles/css_to_inline_styles.php');

[niltonneto]

Ótimo, muito obrigado!!!

Replying to marcieli:

Olá Nilton,

desculpa a demora, esta solução sugerida foi commitada na revisão [5993] (Corrigido caminho relativo em includes da classe imap_functions).

Replying to niltonneto:

Outra correção necessária de caminho relativo na classe imap_functions:

De: require_once('../library/CssToInlineStyles/css_to_inline_styles.php');

Para: require_once(dirname(__FILE__).'/../../library/CssToInlineStyles/css_to_inline_styles.php');