culturagovbr / mapasculturais_descontinuado

Mapas Culturais
16 stars 2 forks source link

Botão excluir - SaaS #72

Closed thaisrigolon closed 7 years ago

thaisrigolon commented 7 years ago

Após excluir um SaaS que estava na aba ativo, o SaaS vai automaticamente para a aba lixeira. Nesta aba deveria ter o botão excluir para dar a possibilidade de excluir o SaaS definitivamente. No entanto, esse botão aparece somente para algumas instalações e deveria aparecer para todas:

1

marylly commented 7 years ago

@thaisrigolon @robertkapa @GustavoVS essa situação é porque o usuário que estava acessando neste exemplo, não é proprietário da instalação SaaS, então ele não tem permissões para excluir a instalação. Tem que ser o dono (owner) da instalação ou um usuário saasSuperAdmin.

thaisrigolon commented 7 years ago

Ok.

thaisrigolon commented 7 years ago

@marylly eu e o @marcus2703 estamos acessando o ambiente de produção com usuários SaaSAdmin e não conseguimos excluir definitivamente algumas instalações:

erro

marylly commented 7 years ago

Reforçando que só o usuário com role superSaasAdmin tem acesso de excluir qualquer instalação definitivamente, se a opção não está surgindo, pode ser que vocês não sejam os Publicadores/Responsáveis pelas instalações que não exibem o botão de exclusão definitiva. @thaisrigolon @marcus2703

marcus2703 commented 7 years ago

Sim @marylly. Meu usuário do SaaS é "saasSuperAdmin". E mesmo assim não conseguimos excluir os subsites. Acho que talvez seja uma confusão na exibição. Pois no exemplo acima você pode ver que a instalação do Museus aparece na lixeira, sendo uma instalação ativa. Ocorre isso também na aba rascunho e arquivados. Temos instalações ativas nessas abas também.

thaisrigolon commented 7 years ago

@marylly mais um erro ao tentar excluir definitivamente o SaaS em produção:

Ao clicar no botão excluir definitivamente ocorreu o erro:

1

thaisrigolon commented 7 years ago

@marylly

E não consigo criar outro subsite com a mesma URL enquanto não excluo definitivamente esse:

1

marylly commented 7 years ago

Ao clicar no botão excluir definitivamente ocorreu o erro:

@thaisrigolon @GustavoVS está dando erro porque está tentando excluir os registros relacionados. Será que é prudente criar a deleção para apagar o subsite? porque os relacionamentos do banco de dados no exige que apaguemos todo o conteúdo do subsite o que incluí: Agentes/Espaços/Projeto/Eventos/Selos/Permissões de Usuários/, isso dá poder para algum usuário apagar uma instalação inteira, o que é bem preocupante. Será que podemos deixar a instalação apenas na lixeira ou arquivada? Essa questão de criar uma instalação com a mesma url de outro subsite já existente eu posso ignorar a validação para verificar apenas as instalações ativas. O que acham?

marcus2703 commented 7 years ago

@marylly é possivel que as informações das entidades permaneçam na instalação "master" que é o MapasBr, ao invés de serem excluídas? Atualmente todas as informações aparecem no MapasBr, e este é um dos objetivos do SaaS. Entendo que se em um determinado momento um subsite vir a ser excluído, suas informações deveriam permanecer.

marylly commented 7 years ago

@thaisrigolon @marcus2703 poderiam verificar, por gentileza? Está na branch rc.

GustavoVS commented 7 years ago

@marylly está dando erro ao atualizar para o branch RC:

selecao_019

Antes do erro na página, também aparecem esses erros no deploy.sh (PHP Fatal error: Call to a member function getRevisionData() on a non-object in /srv/mapas/mapasculturais/src/protected/application/lib/MapasCulturais/Traits/EntityRevision.php on line 137):

selecao_020

marylly commented 7 years ago

@GustavoVS foram reiniciados os serviços e gerados novamente os caches?

thaisrigolon commented 7 years ago

Está disponível para teste. @marcus2703 poderia verificar se está ok?

thaisrigolon commented 7 years ago

@marylly é possível que as informações das entidades permaneçam na instalação "master" que é o MapasBr, ao invés de serem excluídas? Atualmente todas as informações aparecem no MapasBr, e este é um dos objetivos do SaaS. Entendo que se em um determinado momento um subsite vir a ser excluído, suas informações deveriam permanecer.

É possível, @marylly?

marcus2703 commented 7 years ago

Ainda estão aparecendo em todas as abas as listas de subsites ativos. O que torna muito complexo o uso deste painel no sentido de excluir subsites. Acredito que os substites ativos deveriam aparecer somente nesta aba, e assim respectivamente. Facilitando a manutenção e organização. Ainda não ficou claro se o subsite irá "desaparecer" de alguma lista. Eu criei um subsite de teste, e após excluir ele, ele aparece na aba Lixeira, e só tem as opções "editar" e "recuperar" image

avaloni commented 7 years ago

Ainda estão aparecendo em todas as abas as listas de subsites ativos. O que torna muito complexo o uso deste painel no sentido de excluir subsites.

Alteração realizada na branch RC deste repositório.

Eu criei um subsite de teste, e após excluir ele, ele aparece na aba Lixeira, e só tem as opções "editar" e "recuperar"

Na branch RC está normal, mas só usuários "superSaasAdmin" conseguem excluir definitivamente.

marcus2703 commented 7 years ago

Testes efetuados. Exclusão realizada. Em relação a situação do subsite id ser atualizado com valor null, seria possivel que ao excluir o sistema solicite para qual instalação os objetos criados devem ser repassados? Ou ainda, que a instalação definida como Master fosse a MapasBR no caso do MinC, colocar um campo com essa informação. Tipo, nas abas dos subsites, ter uma checkbox com uma flag de instalação principal. Preocupa-me este campo ficar vazio

intrepidocabral commented 7 years ago

Em relação a situação do subsite id ser atualizado com valor null, seria possivel que ao excluir o sistema solicite para qual instalação os objetos criados devem ser repassados?

Isso é uma nova funcionalidade não prevista nesse escopo. Vamos deixar ela em standby para ser realizada postumamente numa renovação de parcerias. =)

diegodorgam commented 7 years ago

sim é possível substitutir o id pelo id de outro subsite ao invés de manter ele como null, mas isso é uma nova funcionalidade como disse o @photonauta35, vai exigir um esforço que está além do escopo fechado, podemos negociar ela dentro de um novo escopo. Sugiro que essa issue seja fechada e que seja aberta em uma nova issue com essa demanda, caracterizada como uma evolução da funcionalidade.

thaisrigolon commented 7 years ago

@marcus2703 vou fechar esta issue porque foi corrigido, ok? Por favor, abra uma nova issue com essa demanda pra incluirmos no novo escopo.