Refactor crud layout parameter passing

comissoes/views.py

@@ -5,29 +5,29 @@ from sapl.crud import build_crud
 from .models import CargoComissao, Comissao, Periodo, TipoComissao
 
 cargo_crud = build_crud(
-    CargoComissao, 'cargo_comissao',
+    CargoComissao, 'cargo_comissao', [
 
         [_('Período de composição de Comissão'),
          [('nome', 10), ('unico', 2)]],
-)
+    ])
 
 periodo_composicao_crud = build_crud(
-    Periodo, 'periodo_composicao_comissao',
+    Periodo, 'periodo_composicao_comissao', [
 
         [_('Cargo de Comissão'),
          [('data_inicio', 6), ('data_fim', 6)]],
-)
+    ])
 
 tipo_comissao_crud = build_crud(
-    TipoComissao, 'tipo_comissao',
+    TipoComissao, 'tipo_comissao', [
 
         [_('Tipo Comissão'),
          [('nome', 9), ('sigla', 3)],
             [('dispositivo_regimental', 9), ('natureza', 3)]],
-)
+    ])
 
 comissao_crud = build_crud(
-    Comissao, 'modulo_comissoes',
+    Comissao, 'modulo_comissoes', [
 
         [_('Dados Básicos'),
          [('nome', 9), ('sigla', 3)],
@@ -52,7 +52,7 @@ comissao_crud = build_crud(
          [('data_final_prevista_temp', 4),
           ('data_prorrogada_temp', 4),
           ('data_fim_comissao', 4)]],
-)
+    ])
 
 
 class ComposicaoView(comissao_crud.CrudDetailView):

lexml/views.py

@@ -5,7 +5,7 @@ from sapl.crud import build_crud
 from .models import LexmlProvedor, LexmlPublicador
 
 lexml_provedor_crud = build_crud(
-    LexmlProvedor, 'lexml_provedor',
+    LexmlProvedor, 'lexml_provedor', [
 
         [_('Provedor Lexml'),
          [('id_provedor', 2),
@@ -14,14 +14,14 @@ lexml_provedor_crud = build_crud(
              ('email_responsavel', 5)],
             [('xml', 12)]],
 
-)
+    ])
 
 lexml_publicador_crud = build_crud(
-    LexmlPublicador, 'lexml_publicador',
+    LexmlPublicador, 'lexml_publicador', [
 
         [_('Publicador Lexml'),
          [('id_publicador', 2),
             ('nome', 5), ('sigla', 5)],
             [('id_responsavel', 2), ('nome_responsavel', 5),
              ('email_responsavel', 5)]],
-)
+    ])

materia/views.py

@@ -9,42 +9,42 @@ from .models import (Anexada, Autor, Autoria, DocumentoAcessorio,
                      TipoProposicao, Tramitacao, UnidadeTramitacao)
 
 origem_crud = build_crud(
-    Origem, 'origem',
+    Origem, 'origem', [
 
         [_('Origem'),
          [('nome', 8), ('sigla', 4)]],
-)
+    ])
 
 tipo_materia_crud = build_crud(
-    TipoMateriaLegislativa, 'tipo_materia_legislativa',
+    TipoMateriaLegislativa, 'tipo_materia_legislativa', [
 
         [_('Tipo Matéria Legislativa'),
          [('sigla', 4), ('descricao', 8)]],
-)
+    ])
 
 regime_tramitacao_crud = build_crud(
-    RegimeTramitacao, 'regime_tramitacao',
+    RegimeTramitacao, 'regime_tramitacao', [
 
         [_('Tipo de Documento'),
          [('descricao', 12)]],
-)
+    ])
 
 tipo_documento_crud = build_crud(
-    TipoDocumento, 'tipo_documento',
+    TipoDocumento, 'tipo_documento', [
 
         [_('Regime Tramitação'),
          [('descricao', 12)]],
-)
+    ])
 
 tipo_fim_relatoria_crud = build_crud(
-    TipoFimRelatoria, 'fim_relatoria',
+    TipoFimRelatoria, 'fim_relatoria', [
 
         [_('Tipo Fim de Relatoria'),
          [('descricao', 12)]],
-)
+    ])
 
 materia_legislativa_crud = build_crud(
-    MateriaLegislativa, '',
+    MateriaLegislativa, '', [
 
         [_('Identificação Básica'),
          [('tipo', 4), ('numero', 4), ('ano', 4)],
@@ -73,10 +73,10 @@ materia_legislativa_crud = build_crud(
          [('ementa', 12)],
             [('indexacao', 12)],
             [('observacao', 12)]],
-)
+    ])
 
 Anexada_crud = build_crud(
-    Anexada, '',
+    Anexada, '', [
 
         [_('Matéria Legislativa'),
          [('tip_id_basica_FIXME', 4),
@@ -89,35 +89,34 @@ Anexada_crud = build_crud(
             ('num_ident_basica_FIXME', 4),
             ('ano_ident_basica_FIXME', 4)],
             [('data_anexacao', 6), ('data_desanexacao', 6)]],
-)
+    ])
 
 tipo_autor_crud = build_crud(
-    TipoAutor, 'tipo_autor',
+    TipoAutor, 'tipo_autor', [
 
         [_('Tipo Autor'),
          [('descricao', 12)]],
-)
-
+    ])
 
 autor_crud = build_crud(
-    Autor, 'autor',
+    Autor, 'autor', [
 
         [_('Autor'),
          [('tipo', 3), ('nome', 9)],
             [('username', 12)]],
-)
+    ])
 
 autoria_crud = build_crud(
-    Autoria, '',
+    Autoria, '', [
 
         [_('Autoria'),
          [('tip_autor_FIXME', 4),
             ('nom_autor_FIXME', 4),
             ('primeiro_autor', 4)]],
-)
+    ])
 
 documento_acessorio_crud = build_crud(
-    DocumentoAcessorio, '',
+    DocumentoAcessorio, '', [
 
         [_('Documento Acessório'),
          [('tipo', 6), ('nome', 6)],
@@ -125,18 +124,18 @@ documento_acessorio_crud = build_crud(
             [('nom_arquivo_FIXME', 12)],
             [('ementa', 12)],
             [('txt_observacao_FIXME', 12)]],
-)
+    ])
 
 numeracao_crud = build_crud(
-    Numeracao, '',
+    Numeracao, '', [
 
         [_('Numeração'),
          [('tipo_materia', 6), ('numero_materia', 6)],
             [('ano_materia', 6), ('data_materia', 6)]],
-)
+    ])
 
 orgao_crud = build_crud(
-    Orgao, 'orgao',
+    Orgao, 'orgao', [
 
         [_('Órgão'),
          [('nome', 4),
@@ -144,28 +143,28 @@ orgao_crud = build_crud(
             ('telefone', 2),
             ('endereco', 2),
             ('unidade_deliberativa', 2)]],
-)
+    ])
 
 relatoria_crud = build_crud(
-    Relatoria, '',
+    Relatoria, '', [
 
         [_('Relatoria'),
          [('data_designacao_relator', 12)],
             [('dados_FIXME', 12)],
             [('data_destituicao_relator', 6), ('tipo_fim_relatoria', 6)]],
-)
+    ])
 
 tipo_proposicao_crud = build_crud(
-    TipoProposicao, 'tipo_proposicao',
+    TipoProposicao, 'tipo_proposicao', [
 
         [_('Tipo Proposição'),
          [('descricao', 12)],
             [('materia_ou_documento', 6), ('tipo_documento', 6)],
             [('modelo', 12)]],
-)
+    ])
 
 proposicao_crud = build_crud(
-    Proposicao, '',
+    Proposicao, '', [
 
         [_('Proposição'),
          [('tipo', 4), ('dat_criacao_FIXME', 4), ('data_recebimento', 4)],
@@ -174,28 +173,28 @@ proposicao_crud = build_crud(
              ('num_ident_basica_FIXME', 4),
              ('ano_ident_basica_FIXME', 4)],
             [('nom_arquivo_FIXME', 6), ('modelo_FIXME', 6)]],
-)
+    ])
 
 status_tramitacao_crud = build_crud(
-    StatusTramitacao, 'status_tramitacao',
+    StatusTramitacao, 'status_tramitacao', [
 
         [_('Status Tramitação'),
          [('indicador', 3),
             ('sigla', 2),
             ('descricao', 7)]],
-)
+    ])
 
 unidade_tramitacao_crud = build_crud(
-    UnidadeTramitacao, 'unidade_tramitacao',
+    UnidadeTramitacao, 'unidade_tramitacao', [
 
         [_('Unidade Tramitação'),
          [('orgao', 12)],
             [('comissao', 12)],
             [('parlamentar', 12)]],
-)
+    ])
 
 tramitacao_crud = build_crud(
-    Tramitacao, '',
+    Tramitacao, '', [
 
         [_('Tramitação'),
          [('cod_ult_tram_dest_FIXME', 6), ('unidade_tramitacao_local', 6)],
@@ -204,4 +203,4 @@ tramitacao_crud = build_crud(
              ('data_encaminhamento', 4),
              ('data_fim_prazo', 4)],
             [('texto', 12)]],
-)
+    ])

norma/views.py

@@ -6,23 +6,23 @@ from .models import (AssuntoNorma, LegislacaoCitada, NormaJuridica,
                      TipoNormaJuridica)
 
 assunto_norma_crud = build_crud(
-    AssuntoNorma, 'assunto_norma_juridica',
+    AssuntoNorma, 'assunto_norma_juridica', [
 
         [_('Assunto Norma Jurídica'),
          [('assunto', 6), ('descricao', 6)]],
-)
+    ])
 
 tipo_norma_crud = build_crud(
-    TipoNormaJuridica, 'tipo_norma_juridica',
+    TipoNormaJuridica, 'tipo_norma_juridica', [
 
         [_('Tipo Norma Jurídica'),
          [('descricao', 4),
             ('sigla', 4),
             ('equivalente_lexml', 4)]],
-)
+    ])
 
 norma_crud = build_crud(
-    NormaJuridica, '',
+    NormaJuridica, '', [
 
         [_('Identificação Básica'),
          [('tipo', 4), ('numero', 4), ('ano', 4)],
@@ -43,10 +43,10 @@ norma_crud = build_crud(
          [('assunto_norma_FIXME', 12)],
             [('assunto_norma_FIXME', 12)],
             [('assunto_norma_FIXME', 12)]],
-)
+    ])
 
 legislacao_citada_crud = build_crud(
-    LegislacaoCitada, '',
+    LegislacaoCitada, '', [
 
         [_('Legislação Citada'),
          [('tip_norma_FIXME', 4),
@@ -55,4 +55,4 @@ legislacao_citada_crud = build_crud(
             [('disposicoes', 3), ('parte', 3), ('livro', 3), ('titulo', 3)],
             [('capitulo', 3), ('secao', 3), ('subsecao', 3), ('artigo', 3)],
             [('paragrafo', 3), ('inciso', 3), ('alinea', 3), ('item', 3)]],
-)
+    ])

parlamentares/views.py

@@ -8,15 +8,15 @@ from .models import (CargoMesa, Coligacao, Dependente, Filiacao, Legislatura,
                      TipoDependente)
 
 cargo_mesa_crud = build_crud(
-    CargoMesa, 'cargo_mesa',
+    CargoMesa, 'cargo_mesa', [
 
         [_('Cargo na Mesa'),
          [('descricao', 10),
             ('unico', 2)]],
-)
+    ])
 
 legislatura_crud = build_crud(
-    Legislatura, 'tabelas_auxiliares#legislatura',
+    Legislatura, 'tabelas_auxiliares#legislatura', [
 
         [_('Legislatura'),
          [('id', 3),
@@ -24,38 +24,38 @@ legislatura_crud = build_crud(
             ('data_fim', 2),
             ('data_eleicao', 2)],
             [('data_inicio', 4), ('data_fim', 4), ('data_eleicao', 4)]],
-)
+    ])
 
 coligacao_crud = build_crud(
-    Coligacao, 'coligacao',
+    Coligacao, 'coligacao', [
 
         [_('Coligação'),
          [('nome', 5),
             ('legislatura', 5),
             ('numero_votos', 2)]],
-)
+    ])
 
 partido_crud = build_crud(
-    Partido, 'partidos',
+    Partido, 'partidos', [
 
         [_('Partido Político'),
          [('nome', 6),
             ('sigla', 2),
             ('data_criacao', 2),
             ('data_extincao', 2)]],
-)
+    ])
 
 dependente_crud = build_crud(
-    Dependente, '',
+    Dependente, '', [
 
         [_('Dependentes'),
          [('nome', 12)],
             [('tipo', 4), ('sexo', 4), ('data_nascimento', 4)],
             [('cpf', 4), ('rg', 4), ('titulo_eleitor', 4)]],
-)
+    ])
 
 sessao_legislativa_crud = build_crud(
-    SessaoLegislativa, 'sessao_legislativa',
+    SessaoLegislativa, 'sessao_legislativa', [
 
         [_('Sessão Legislativa'),
          [('numero', 2),
@@ -64,10 +64,10 @@ sessao_legislativa_crud = build_crud(
             ('data_fim', 2),
             ('data_inicio_intervalo', 2),
             ('data_fim_intervalo', 2)]],
-)
+    ])
 
 dependente_crud = build_crud(
-    Parlamentar, '',
+    Parlamentar, '', [
 
         [_('Cadastro do Parlamentar'),
          [('nome_parlamentar', 4), ('login_FIXME', 4), ('ativo', 4)],
@@ -86,17 +86,17 @@ dependente_crud = build_crud(
             [('biografia', 12)],
             [('observacao_FIXME', 12)],
             [('parlamentar_salvar_FIXME', 12)]],
-)
+    ])
 
 filiacao_crud = build_crud(
-    Filiacao, '',
+    Filiacao, '', [
 
         [_('Filiações Partidárias '),
          [('partido', 4), ('data', 4), ('data_desfiliacao', 4)]],
-)
+    ])
 
 mandato_crud = build_crud(
-    Mandato, '',
+    Mandato, '', [
 
         [_('Mandato'),
          [('legislatura', 4), ('coligacao', 4), ('votos_recebidos', 4)],
@@ -105,32 +105,32 @@ mandato_crud = build_crud(
              ('data_fim_mandato', 3),
              ('data_expedicao_diploma', 3)],
             [('observacao', 12)]],
-)
+    ])
 
 tipo_dependente_crud = build_crud(
-    TipoDependente, 'tipo_dependente',
+    TipoDependente, 'tipo_dependente', [
 
         [_('Tipo de Dependente'),
          [('descricao', 12)]],
-)
+    ])
 
 nivel_instrucao_crud = build_crud(
-    NivelInstrucao, 'nivel_instrucao',
+    NivelInstrucao, 'nivel_instrucao', [
 
         [_('Nível Instrução'),
          [('descricao', 12)]],
-)
+    ])
 
 tipo_afastamento_crud = build_crud(
-    TipoAfastamento, 'tipo_afastamento',
+    TipoAfastamento, 'tipo_afastamento', [
 
         [_('Tipo de Afastamento'),
          [('descricao', 5), ('dispositivo', 5), ('afastamento', 2)]],
-)
+    ])
 
 tipo_militar_crud = build_crud(
-    SituacaoMilitar, 'tipo_situa_militar',
+    SituacaoMilitar, 'tipo_situa_militar', [
 
         [_('Tipo Situação Militar'),
          [('descricao', 12)]],
-)
+    ])

protocoloadm/views.py

@@ -7,14 +7,14 @@ from .models import (DocumentoAcessorioAdministrativo, DocumentoAdministrativo,
                      TipoDocumentoAdministrativo, TramitacaoAdministrativo)
 
 tipo_documento_administrativo_crud = build_crud(
-    TipoDocumentoAdministrativo, '',
+    TipoDocumentoAdministrativo, '', [
 
         [_('Tipo Documento Administrativo'),
          [('sigla', 4), ('sigla', 4), ('descricao', 4)]],
-)
+    ])
 
 documento_administrativo_crud = build_crud(
-    DocumentoAdministrativo, '',
+    DocumentoAdministrativo, '', [
 
         [_('Formulário de Cadastro'),
          [('tipo', 4), ('numero', 4), ('ano', 4)],
@@ -35,10 +35,10 @@ documento_administrativo_crud = build_crud(
         [_('Outras Informações'),
          [('dias_prazo', 4), ('data_fim_prazo', 4), ('observacao', 4)],
             [('observacao', 12)]],
-)
+    ])
 
 documento_acessorio_administrativo_crud = build_crud(
-    DocumentoAcessorioAdministrativo, '',
+    DocumentoAcessorioAdministrativo, '', [
 
         [_('Documento Administrativo'),
          [('tipo', 4), ('nome', 4), ('data', 4)],
@@ -51,10 +51,10 @@ documento_acessorio_administrativo_crud = build_crud(
             [('autor', 12)],
             [('arquivo', 12)],
             [('assunto', 12)]],
-)
+    ])
 
 protocolo_crud = build_crud(
-    Protocolo, '',
+    Protocolo, '', [
 
         [_('Formulário de Cadastro'),
          [('tipo_documento', 4),
@@ -83,10 +83,10 @@ protocolo_crud = build_crud(
             ('dat_fim_prazo_FIXME', 4),
             ('observacao', 4)],
             [('observacao', 12)]],
-)
+    ])
 
 status_tramitacao_administrativo_crud = build_crud(
-    StatusTramitacaoAdministrativo, '',
+    StatusTramitacaoAdministrativo, '', [
 
         [_('Status Tramitação Administrativo'),
          [('sigla', 3),
@@ -95,10 +95,10 @@ status_tramitacao_administrativo_crud = build_crud(
             ('descricao', 3)],
             [('sigla', 6), ('ind_tramitacao_FIXME', 6)],
             [('descricao', 12)]],
-)
+    ])
 
 tramitacao_administrativo_crud = build_crud(
-    TramitacaoAdministrativo, '',
+    TramitacaoAdministrativo, '', [
 
         [_('Documento Administrativo'),
          [('cod_ult_tram_dest_FIXME', 6), ('unidade_tramitacao_local', 6)],
@@ -111,4 +111,4 @@ tramitacao_administrativo_crud = build_crud(
             [('status', 6), ('unidade_tramitacao_destino', 6)],
             [('data_encaminhamento', 6), ('data_fim_prazo', 6)],
             [('texto', 12)]],
-)
+    ])

sapl/crud.py

@@ -59,7 +59,7 @@ class Crud(object):
     pass
 
 
-def build_crud(model, help_path, *layout):
+def build_crud(model, help_path, layout):
     crud = Crud()
     crud.model = model
     crud.help_path = help_path

sapl/test_crud.py

@@ -24,7 +24,7 @@ pytestmark = pytest.mark.django_db
      ['nome', 'sigla', 'tipo', 'data_criacao', 'unidade_deliberativa']),
 ])
 def test_listview_get_fieldnames(layout, result):
-    crud = build_crud(Comissao, *layout)
+    crud = build_crud(Comissao, 'stub_help_path', layout)
     view = crud.CrudListView()
     assert view.field_names == result
 
@@ -89,7 +89,7 @@ def test_get_field_display():
 def test_crud_detail_view_fieldsets(monkeypatch):
 
     crud = build_crud(
-        Comissao, '',
+        Comissao, 'stub_help_path', [
 
             ['Dados Básicos',
              [('nome', 9), ('sigla', 3)],
@@ -99,7 +99,7 @@ def test_crud_detail_view_fieldsets(monkeypatch):
             ['Dados Complementares',
              [('finalidade', 12)]
              ],
-    )
+        ])
 
     view = crud.CrudDetailView()
     stub = mommy.make(Comissao,

sapl/teststubs/urls_for_list_test.py

@@ -1,11 +1,11 @@
 from django.conf.urls import include, url
 
 from comissoes.models import Comissao
-from comissoes.urls import comissoes_urls
+from comissoes.urls import comissao_url_patterns
 from sapl.crud import build_crud
 
 crud = build_crud(
-    Comissao, '',
+    Comissao, '', [
 
         ['Dados Básicos',
          [('nome', 9), ('sigla', 3)],
@@ -15,12 +15,10 @@ crud = build_crud(
         ['Dados Complementares',
          [('finalidade', 12)]
          ],
-)
-
-patterns, namespace, app_name = comissoes_urls
+    ])
 
 urlpatterns = [
     url(r'^comissoes/', include((
-        crud.urlpatterns + patterns[len(crud.urlpatterns):],
-        namespace, app_name))),
+        crud.urlpatterns + comissao_url_patterns[len(crud.urlpatterns):],
+        crud.namespace, crud.namespace))),
 ]

sessao/views.py

@@ -7,14 +7,14 @@ from .models import (ExpedienteMateria, OrdemDia, RegistroVotacao,
                      TipoSessaoPlenaria)
 
 tipo_sessao_crud = build_crud(
-    TipoSessaoPlenaria, 'tipo_sessao_plenaria',
+    TipoSessaoPlenaria, 'tipo_sessao_plenaria', [
 
         [_('Tipo de Sessão Plenária'),
          [('nome', 6), ('quorum_minimo', 6)]],
-)
+    ])
 
 sessao_crud = build_crud(
-    SessaoPlenaria, '',
+    SessaoPlenaria, '', [
 
         [_('Dados Básicos'),
          [('numero', 3),
@@ -28,10 +28,10 @@ sessao_crud = build_crud(
              ('hora_fim', 2),
              ('tipo_expediente', 6)],
             [('url_audio', 6), ('url_video', 6)]],
-)
+    ])
 
 expediente_materia_crud = build_crud(
-    ExpedienteMateria, '',
+    ExpedienteMateria, '', [
 
         [_('Cadastro de Matérias do Expediente'),
          [('data_ordem', 4), ('tip_sessao_FIXME', 4), ('numero_ordem', 4)],
@@ -40,10 +40,10 @@ expediente_materia_crud = build_crud(
              ('ano_ident_basica_FIXME', 4)],
             [('tipo_votacao', 12)],
             [('observacao', 12)]],
-)
+    ])
 
 ordem_dia_crud = build_crud(
-    OrdemDia, '',
+    OrdemDia, '', [
 
         [_('Cadastro de Matérias da Ordem do Dia'),
          [('data_ordem', 4), ('tip_sessao_FIXME', 4), ('numero_ordem', 4)],
@@ -52,24 +52,24 @@ ordem_dia_crud = build_crud(
              ('ano_ident_basica_FIXME', 4)],
             [('tipo_votacao', 12)],
             [('observacao', 12)]],
-)
+    ])
 
 tipo_resultado_votacao_crud = build_crud(
-    TipoResultadoVotacao, 'tipo_resultado_votacao',
+    TipoResultadoVotacao, 'tipo_resultado_votacao', [
 
         [_('Tipo de Resultado da Votação'),
          [('nome', 12)]],
-)
+    ])
 
 tipo_expediente_crud = build_crud(
-    TipoExpediente, 'tipo_expediente',
+    TipoExpediente, 'tipo_expediente', [
 
         [_('Tipo de Expediente'),
          [('nome', 12)]],
-)
+    ])
 
 registro_votacao_crud = build_crud(
-    RegistroVotacao, '',
+    RegistroVotacao, '', [
 
         [_('Votação Simbólica'),
          [('numero_votos_sim', 3),
@@ -80,7 +80,7 @@ registro_votacao_crud = build_crud(
              ('ind_votacao_presidente_FIXME', 6)],
             [('tipo_resultado_votacao', 12)],
             [('observacao', 12)]],
-)
+    ])
 
 
 class ExpedienteView(sessao_crud.CrudDetailView):