plonegovbr / brasil.gov.portal / 1466

90%

push

travis-ci

<a href="https://github.com/plonegovbr/brasil.gov.portal/commit/<a class=hub.com/plonegovbr/brasil.gov.portal/commit/ae2054a2d84026378819ea8490dd024481fe1480">ae2054a2d<a href="https://github.com/plonegovbr/brasil.gov.portal/commit/ae2054a2d84026378819ea8490dd024481fe1480">&quot;&gt;Issue 359 corrige erros visoes colecao summary e listing view (#367)

* Atualiza custom templates p.a.contenttypes 1.1.1.

Com a atualização do plone.app.contenttypes para 1.1.1, as templates
nesse commit foram totalmente modificadas. Achamos melhor copiar do
jeito que estão em

https://github.com/plone/plone.app.contenttypes/tree/1.1.1/plone/app/contenttypes/browser/templates

Para, num outro commit, aplicar as customizações relativas a nome de autor e
data adicionados em

    https://github.com/plonegovbr/brasil.gov.portal/pull/270/

Dessa forma, o diff entre a customização e a template original ficará
mais simples.

* Customizações de templates revisadas (fecha #359)

Com a atualização de plone.app.contenttypes para 1.1.1, várias templates
foram alteradas. Em commits anteriores, tínhamos mantido a customização
ainda presente em brasil.gov.portal (que havia customizado as presentes
em plone.app.contenttypes 1.1b3) mas achamos melhor pegar as templates
de plone.app.contenttypes da versão 1.1.1 e reaplicar as principais
motivações da customização, presente em

    https://github.com/plonegovbr/brasil.gov.portal/pull/270

Principais alterações:

src/brasil/gov/portal/browser/plone/templates/folder_listing.pt
(alteração com base na listing_view de plone.app.contenttypes 1.1.1)

src/brasil/gov/portal/browser/plone/templates/folder_summary_view.pt
(alteração com base na summary_view de plone.app.contenttypes 1.1.1 e
nas customizações originais de brasil.gov.portal)

src/brasil/gov/portal/browser/plone/templates/folder_tabular_view.pt
(alteração com base na tabular_view de plone.app.contenttypes 1.1.1)

Segue uma pequena tabela para entender o de/para:

https://github.com/plone/plone.app.contenttypes/blob/1.1.1/plone/app/contenttypes/upgrades.py#L11
https://github.com/plone/plone.app.contenttypes/blob/1.1.1/plone/app/contenttypes/browser/configure.zcml#L16
LISTING_VIEW_MAPPING = {  # OLD (AT and old DX) : NEW (NEW Template Name)
    &amp;#39;folder_listing&amp;#39;: &amp;#39;listing_view&amp;#39;, (templates/listing.pt)
    &amp;#39;folder_listing_view&amp;#39;: &amp;#39;listing_view&amp;#39;, (templates/listing.pt)
    &amp;#39;folder_summary_view&amp;#39;: &amp;#39;summary_view&amp;#39;, (templates/listing_summary.pt)
    &amp;#39;folder_tabular_view&amp;#39;: &amp;#39;tabular_view&amp;#39;, (templates/listing_tabular.pt)
    &amp;#39;standard_view&amp;#39;: &amp;#39;listing_view&amp;#39;, (templates/listing.pt)
}

As customizações mais substanciais foram as presentes em folder_summary_view.pt,
uma vez que o layout dos itens nessa view é bem diferente do que o
apresentado oficialmente em plone.app.contenttypes e, portanto, além de
focar no que PR 270 implementava, também manteve a identidade visual
presente no brasil.gov.portal.

Esse commit também fecha o relato https://github.com/plonegovbr/brasil.gov.portal/issues/359.

* Renomeia templates.

Segue o padrão das novas viewnames em plone.app.contenttypes.

folder_listing.pt -&amp;gt; listing.pt
folder_summary_view.pt -&amp;gt; listing_summary.pt
folder_tabular_view.pt -&amp;gt; listing_tabular.pt

custom_summary_view foi removido pois a partir da atualização para
plone.app.contenttypes 1.1.1, a visão tanto para coleção quando para folder
foi unificada. Ver:

https://github.com/plone/plone.app.contenttypes/blob/1.0/plone/app/contenttypes/browser/configure.zcml#L33,L38
https://github.com/plone/plone.app.contenttypes/blob/1.1.1/plone/app/contenttypes/browser/configure.zcml#L94,L143

e principalmente:

https://github.com/plone/plone.app.contenttypes/commit/</a><a class="double-link" href="https://github.com/plonegovbr/brasil.gov.portal/commit/<a class="double-link" href="https://github.com/plonegovbr/brasil.gov.portal/commit/d16749fbfa0e9ba8d4d75474afeefc88463cd80b">d16749fbf</a>">d16749fbf</a><a href="https://github.com/plonegovbr/brasil.gov.portal/commit/ae2054a2d84026378819ea8490dd024481fe1480">#diff-b3255a8bbf7a04cbd9a0d2fe786918ba

* Code review.

- Adicionar rel=&quot;nofollow&quot; para evitar que motores de busca acessem o
formulário de @@busca;
- Inicializar as variáveis que são usadas em comum por todos os itens
fora do loop de itens.

3480 of 3955 relevant lines covered (87.99%)

0.88 hits per line