Ticket #715 (closed defeito: fixed)

Opened 14 years ago

Last modified 14 years ago

Evento criado como restrito é visualizado como privado e vice-versa

Reported by: david.ianakiara Owned by: david.ianakiara
Priority: grave Milestone: Expresso 2.0.3
Component: Calendar Version: branch 2.0
Severity: Keywords:
Cc: WorkGroup:

Description

Ao criar um evento marcando-o como restrito, ao visualiza-lo o expresso mostra o evento como "privado", e quando marcamos o evento como privado ele vêm como restrito.

Attachments

Visualizar Evento.jpg Download (46.6 KB) - added by david.ianakiara 14 years ago.
restrito.png Download (63.0 KB) - added by rafaelraymundo 14 years ago.
Agendamento criado como restrito

Change History

comment:1 Changed 14 years ago by david.ianakiara

Percebi que no combo onde voce seleciona o tipo de evento, os values estao sendo montados assim: Restrito => value="private" Privado => value="privateHiddenFields"

é pra ser assim mesmo ou isto está invertido?

comment:2 Changed 14 years ago by niltonneto

Temos que tomar muito cuidado com os tipos "privado" e "restrito".
"Privado" é utilizado para quem deseja marcar um compromisso particular, onde muitos campos desnecessários são escondidos.
Já o "Restrito", o usuário cria um compromisso normal no Expresso, convidando participantes, etc, e restringe suas informações automaticamente, para que não apareça na hora na tela de conflitos quando outro usuário tentar marcar junto com ele. Fui claro ou está confuso? Aqui na Celepar usamos muito compromisso restritos...

comment:3 follow-up: ↓ 4 Changed 14 years ago by david.ianakiara

E na hora de visualizar o evento, se um evento for privado ele deverá ser visualizado como restrito? e se for restrito deverá ser visualizado como privado?

Changed 14 years ago by david.ianakiara

comment:4 in reply to: ↑ 3 Changed 14 years ago by niltonneto

Replying to david.ianakiara:

E na hora de visualizar o evento, se um evento for privado ele deverá ser visualizado como restrito? e se for restrito deverá ser visualizado como privado?

Ah sim, então tá invertido.

comment:5 Changed 14 years ago by david.ianakiara

O conceito 'tipo do compromisso' está atrelado ao 'acesso do compromisso'?

na tela de edição de eventos, o combo tipo está sendo carregado desta forma: <option value="private" '.(!$eventpublic??'SELECTED':).' >'. lang('Restrict').'</option> <option value="privateHiddenFields" '.(!$eventpublic??'SELECTED':).' >'. lang('Private').'</option>

quando um evento é de acesso publico e quando é de acesso privado? quando o evento do tipo é "privado" ou "restrito", o combo está sendo montado com dois elementos marcados como 'selected' é pra ser assim mesmo? quando visualizo um evento que foi criado como restrito, o acesso dele é mostrado como "privado", e ao clicar em editar o evento é carregado como se fosse do tipo "privado".

comment:6 Changed 14 years ago by niltonneto

Na verdade, em implementações anteriores, o "acesso do compromisso" foi unificado junto ao "tipo de compromisso", pois era um checkbox que confundia os usuários. Então foi criado uma combobox para definidir o tipo: normal (publico), restrito e apontamento de horas. Depois disso ainda, foi inserido o quarto tipo "privado", para compromissos particulares. Talvez tenha faltado tratar o tipo do evento na visualização e edição do evento.

comment:7 follow-up: ↓ 8 Changed 14 years ago by david.ianakiara

eu vou alterar então a forma como o combo está sendo montado, e deixar selecionado o evento do mesmo tipo que foi criado, para isso eu tenho que ter salvo no banco alguma informação sobre o tipo do evento.

Na tabela phpgw_cal existe o campo cal_type, mas que só está sendo preenchido com os valores "H"(Apontamento de Horas) e "E" para os outros tipos de eventos, teria algum problema se quando o evento ser salvo eu cadastrar no cal_type outros valores?

ex: "P" private "R" restrict "N" normal

assim eu poderia recuperar o tipo de evento salvo para montar o combo corretamente.

comment:8 in reply to: ↑ 7 Changed 14 years ago by niltonneto

  • Priority changed from média to grave
  • Milestone changed from Expresso 2.1 to Expresso 2.0.2

Replying to david.ianakiara:

eu vou alterar então a forma como o combo está sendo montado, e deixar selecionado o evento do mesmo tipo que foi criado, para isso eu tenho que ter salvo no banco alguma informação sobre o tipo do evento.

Na tabela phpgw_cal existe o campo cal_type, mas que só está sendo preenchido com os valores "H"(Apontamento de Horas) e "E" para os outros tipos de eventos, teria algum problema se quando o evento ser salvo eu cadastrar no cal_type outros valores?

ex: "P" private "R" restrict "N" normal

assim eu poderia recuperar o tipo de evento salvo para montar o combo corretamente.

Na versão 1.2, que não tinha o tipo "privado", funcionava perfeitamente o tipo "restrito", tanto na criação, quanto na visualização e edição. Depois disso, deu problema. Mais uma deterioração causada por melhoria. Precisamos corrigir isso urgente e tomar muito cuidado nas novas implementações!!![[BR]] Olha só como funciona no 1.2:
Normal => cal_type = E cal_public = 1
Restrito => cal_ype = E cal_public = 0
Apontamento => cal_type = H cal_public = 1

comment:9 Changed 14 years ago by niltonneto

Vi no banco que o tipo "privado" no Expresso 2.0 é igual ao normal (cal_type=E cal_public=1). Só esconde os campos na criação, por isso dá confusão..... Quem usa o tipo "privado" já em produção deve se manifestar para sabermos do impacto caso mude o tipo dele de "E" para "P". O restrito devemos deixar como está.

comment:10 Changed 14 years ago by niltonneto

Davi, você está corrigindo isso? Fico no aguardo então. Obrigado.

comment:11 Changed 14 years ago by david.ianakiara

Eu estava esperando para ver se alguem se manifestava a respeito da alteração do tipo "E" para "P" do evento privado, posso fazer essa alteração?

comment:12 Changed 14 years ago by niltonneto

Melhor mandar um email para lista desenvolvedores arroba serpro. Se ninguém se manifestar, então pode alterar.

comment:13 follow-up: ↓ 14 Changed 14 years ago by niltonneto

  • Version changed from trunk to branch 2.0

comment:14 in reply to: ↑ 13 ; follow-up: ↓ 15 Changed 14 years ago by niltonneto

Replying to niltonneto:

David, como está o andamento desse ticket?

comment:15 in reply to: ↑ 14 Changed 14 years ago by wmerlotto

Replying to niltonneto:

Replying to niltonneto:

David, como está o andamento desse ticket?

David, conseguiu solucionar este problema? Precisamos lançar a próxima versão (2.0.3) e para isso, é necessário encerrar este ticket...

comment:16 Changed 14 years ago by david.ianakiara

Já estou resolvendo isso!

comment:17 Changed 14 years ago by niltonneto

Espero que sim, pois eu já tinha perguntado isso há 7 semanas atrás....

comment:18 Changed 14 years ago by david.ianakiara

A demora foi referente ao meu periodo de férias e ao aguardo de um retorno da equipe do serpro sobre a alteração. Como eles nao se manifestaram,acabei fazendo a alteração na revisao [1949]

comment:19 Changed 14 years ago by david.ianakiara

  • Status changed from new to closed
  • Resolution set to fixed

comment:20 Changed 14 years ago by niltonneto

Replicado em [1955] no Branch 2.0, já que se trata de um erro crítico.

Changed 14 years ago by rafaelraymundo

Agendamento criado como restrito

comment:21 Changed 14 years ago by rafaelraymundo

Estou na revisão 2228 do trunk, mas os agendamentos criados como "Restrito" estão sendo mostrados como "Privado" - Segue a figura  http://trac.expressolivre.org/attachment/ticket/715/restrito.png

Esse comportamento está correto?

comment:22 Changed 14 years ago by niltonneto

Se quiser corrigir algo em relação a esse assunto, abra um novo ticket. Esse aqui foi fechado e publicado no marco Expresso 2.0.3.

Note: See TracTickets for help on using tickets.