interlegis / sapl

Sistema de Apoio ao Processo Legislativo
GNU General Public License v3.0
86 stars 81 forks source link

Cadastro de matéria legislativa: Relatoria #662

Closed andreagudo closed 7 years ago

andreagudo commented 8 years ago

Na caixa de seleção Parlamentar são listados todos. Devem ser listados apenas os membros atuais da Comissão. Selecionei um parlamentar inativo e o sistema o aceitou como relator, indevidamente.

andreagudo commented 8 years ago

Resolvido.Ok.

eduardoedson commented 8 years ago

Deve ser listado somente os parlamentares que participam da comissão atual? Ou todos os ativos, independente da comissão que participa?

andreagudo commented 8 years ago

Todos os membros atuais da comissão em que a matéria está.

eduardoedson commented 8 years ago

A comissão possui várias composições, dependendo do período, deve pegar os membros de todos os períodos ou só do mais recente?

andreagudo commented 8 years ago

Deve pegar os membros do período em que está sendo indicado o relator. normalmente o atual.

LeandroJatai commented 8 years ago

@andreagudo @eduardoedson acho que foi resolvido, qdo der pra verificar...

andreagudo commented 8 years ago

Não, estão sendo listados todinhos os parlamentares, de todos os tempos.

LeandroJatai commented 8 years ago

@andreagudo um link de exemplo??

andreagudo commented 8 years ago

http://sapl31agudo.interlegis.leg.br/materia/1942/relatoria/create

LeandroJatai commented 8 years ago

Estão todos... sua base tem 108 parlamentares cadastrados... pode conferir aqui o total... http://sapl31agudo.interlegis.leg.br/admin/parlamentares/parlamentar/ vai pedir senha... acessar com admin,admin depois disso na combo de parlamentares desse link: http://sapl31agudo.interlegis.leg.br/materia/1942/relatoria/create é possível ver que estão os 108 como... ctrl-shit-i abrir a console da inspeção de elemento: rode isso

$('#id_parlamentar option').length vai mostrar 109 por causa do item 'Selecione...'

andreagudo commented 8 years ago

Devem ser listados na caixa de seleção apenas os quatro membros da composição atual da comissão.

LeandroJatai commented 8 years ago

haha... sabe a vírgula desse comentário... "Não, estão sendo listados todinhos os parlamentares, de todos os tempos."

eu não li ela, desculpe... rsrs

eduardoedson commented 8 years ago

@andreagudo Foi resolvido?

andreagudo commented 8 years ago

Não, ainda aparecem todos os parlamentares, de todos os tempos.

edwardoliveira commented 7 years ago

@andreagudo Esse erro ainda persiste???

edwardoliveira commented 7 years ago

@andreagudo O @eduardoedson Já tinha colocado um filtro para parlamentares ativo lá atrás então eu acho que os dados podem estar inconsistentes, isto é, todos os parlamentares estão como ativos no sapl31agudo. De qualquer forma coloquei mais uma restrição para filtrar somente os que fazem parte daquela comissão. Por favor veja se o bug ainda persiste. E abra a issue, caso contrário.

andreagudo commented 7 years ago

@edwardoliveira o bug persiste. Não são todos os parlamentares que estão ativos. O filtro deve ser parlamentares da comissão no período de composição atual.

eduardoedson commented 7 years ago

@andreagudo Verifique, por favor

andreagudo commented 7 years ago

Melhorou, mas deve ser adicionado mais um critério: apenas os titulares da comissão devem ser listados.

LeandroJatai commented 7 years ago

@eduardoedson seu commit 0b64341 causou erro no travis https://travis-ci.org/interlegis/sapl/builds/203938530

eduardoedson commented 7 years ago

@andreagudo verifique novamente

andreagudo commented 7 years ago

Primeiro: a inclusão de relator está sendo feita pelo operador_materias, mas não devia ser pelo operador_comissoes? Segundo: ainda estão listados todos os membros da comissão; deivam ser apenas os titulares.

eduardoedson commented 7 years ago

Verifique na comissão se não estão todo como titulares

Não. captura de tela de 2017-03-03 16 34 11

eduardoedson commented 7 years ago

A questão do operador melhor verificar com o @LeandroRoberto

eduardoedson commented 7 years ago

@andreagudo Eu tinha esquecido de enviar na sexta, poderia verificar o caso dos parlamentares? Verifiquei em: http://sapl31agudo.interlegis.leg.br/materia/1983/relatoria/create http://sapl31agudo.interlegis.leg.br/comissao/15/composicao E me parece estar correto

LeandroJatai commented 7 years ago

@eduardoedson, mudança de perfil feita em: https://github.com/interlegis/sapl/commit/bcfd859dffc260ba6f519f818115f66ad3754858

edwardoliveira commented 7 years ago

:fireworks: :fireworks: :clap: :clap:

andreagudo commented 7 years ago

Operador: ok. Listagem membros titulares: ok. Issue Resolvida.