Atualizar plone.app.contenttypes para 1.1.1

[hvelarde]

IDGB usa a versão 1.0 e a mais recente para o Plone 4.3 é a 1.1.1.

[idgserpro]

Pacotes que precisam ser alterados:

https://github.com/plonegovbr/brasil.gov.agenda/blob/eb3d45cf63e89407ee0c1f76827f0a5787e33686/setup.py

https://github.com/plonegovbr/brasil.gov.portlets/blob/e8c673620911b3e767748d54be7f110ac4fb9c0e/setup.py

https://github.com/plonegovbr/brasil.gov.temas/blob/8cbe4f0b443450789e803420c44db13e4dcafc2b/setup.py

284c984

Ver também collective/collective.cover#637 para ter certeza que não terá impacto. De acordo com collective/collective.cover#637 (comment), o bug foi adicionado em plone/plone.namedfile@33df023#diff-03f29bbadcdf640341c9061de5a07dccR299, mas usamos a versão 3.0.7 de namedfile e portanto não somos afetados.

[winstonf88]

Qual o impedimento do agenda em usar a versão nova do contenttypes?

[idgserpro]

@winstonf88 ver plonegovbr/brasil.gov.agenda@ec6e87f

@ericof porque isso?

[hvelarde]

tem que atualizar tanto o plone.app.contenttypes quanto o plone.app.event

[idgserpro]

Ver também o impacto em plone.app.collection, uma vez que usamos a branch 2.x e aqui se afirma que já foi incorporado em plone.app.contenttypes.

[idgserpro]

plone.app.contenttypes 1.0 incorporou a branch 2.0.x do plone.app.collection. Ver https://github.com/plone/plone.app.contenttypes/blob/1.0/CHANGES.rst e plone/plone.app.contenttypes@fa34701 onde foi feita a mescla entre os dois pacotes. Por isso que pudemos remover a pinagem de plone.app.collection 2.0b5 em plonegovbr/portal.buildout@0d4d384, pois criamos uma lógica que remove a layer do plone.app.collection para portais antigos que tinham esse pacote instalado (ver ab55f0f).

[claytonc]

@idgserpro
Atualizando o plone.app.contenttypes para a versão 1.1.1 resolve parte do #155 faltando só atualizar o collective.nitf.

[idgserpro]

Ver também se com essa atualização o relato #290 pode ser fechado. (Será analisado em outro relato, no de atualização de Plone em #331)

[claytonc]

Mais um bug relacionado a versão do plone.app.contenttypes que o provedor de conteúdos encontrou no tinymce, achei também uma issue plone/Products.TinyMCE#123

Mesmo atualizando o tinymce o erro ocorre as vezes.

Testei em um plone puro com o plone.app.contenttypes para 1.1.1 o problema não ocorreu.

[claytonc]

@idgserpro

Estou efetuando testes usando o IDG 1.1.4

1º Teste - criando um site IDG com o plone.app.contenttypes = 1.0 e depois atualizar para 1.1.1 simulando uma migração:

1. após a criação do site removi as pinagens dos produtos (agenda e temas) e no arquivo vesions.cfg; Obs.: Só não entendi porque o produto temas esta pinando o plone.app.contenttypes.
2. quando executei o upgrade ocorreu os erros:
   • AttributeError: 'NoneType' object has no attribute 'splitlines' - por causa do campo "Participantes" esta em branco;
   • AttributeError: timezone - porque a timezone gravada esta "BRT" e não é valida.
   • Possível solução- no arquivo plone.app.contenttypes/plone/app/contenttypes/migration/dxmigration.py linha 73 fiz essa alteração abaixo:

if hasattr(self.old, 'attendees'):
    if self.old.attendees:
        self.new.attendees = tuple(self.old.attendees.splitlines())
    else:
        self.new.attendees = ''
        self.new.timezone = 'UTC'

3. Após as alterações acima rodei o upgrade e o tipo "Evento" funcionou corretamente;

4. Também testei superficial adicionando os conteúdos da agenda;

5. plone.app.contenttypes 1.1.1 tem um upgrade que altera os nomes das visões "Use new view names" se rodar ira perder as customizações do IDG:

    outdated_methods = [
        'folder_listing',
        'folder_full_view',
        'folder_summary_view',
        'folder_tabular_view',
        'folder_album_view',
        'atct_album_view',
        'standard_view',
        'collection_view',
        'all_content',
        'thumbnail_view',
    ]
    new_methods = [
        'listing_view',
        'summary_view',
        'tabular_view',
        'full_view',
        'album_view',
        'event_listing',
    ]

2º Teste - criando um site IDG já com o plone.app.contenttypes = 1.1.1:

1. A maioria dos itens criados que testei funcionam corretamente até agora não encontrei outros problemas;
2. Os nomes das visões estão no novo padrão conforme o 1º teste item 5.

[idgserpro]

@clayton, devemos trabalhar nesse relato para um próximo release. Agradeço de antemão os testes efetuados, principalmente com relação a ambientes já criados - temos de ter um carinho especial nesse contexto.

[hvelarde]

eu acho que precisamos atualizar esta issues para usar plone.app.contenttypes 1.1.2 e plone.app.event 1.1.8.

@claytonc eu fiquei com algumas dúvidas:

• aparentemente o erro no campo atendees foi originado pela falta de um valor default na versão antiga do plone.app.event; acho que seria mais simples criar um upgrade step que atualize esse campo para uma tupla vazia como é definido atualmente no pacote; com essa medida evitaríamos ter que modificar o upgrade step do upstream
• no caso do campo timezone fiquei confuso pois o valor BRT é completamente válido; acho que está faltando definir uma variável de ambiente no buildout.cfg segundo descrito aqui: https://stackoverflow.com/q/23349397
• respeito das configurações da agenda, você tem informação mais precisa do que temos que fazer ai? como poderíamos solucionar esse problema?

pelo que estou entendendo atualizar fica totalmente viável após resolver esse ponto.

[claytonc]

@hvelarde

• o erro do campo atendess só ocorreu em objetos criados pelo script do brasil.gov.portal, pois quando adicionando manualmente o campo já é inserido com vazio;
• o warning BRT desapareceu quando adicionei em environment-vars no base.cfg o valor TZ America/Sao_Paulo;
• quais configurações da agenda você quer que eu teste ?

[hvelarde]

@claytonc ótimo, abre por favor uma issue diferente relatando esse problema da criação automática de conteúdo para poder solucionar.

respeito ao terceiro ponto, não ficou claro para mim também não; tu falou que ao rodar o upgrade step "ira perder as customizações do IDG"; quais customizações? tu ve algum jeito de corrigir isso diretamente upstream?

[claytonc]

@hvelarde

Mudou os nomes das visões das pastas e coleções.

Hoje por exemplo é folder_summary_view mudou para summary_view conforme o item 5 - #240 (comment)

Um exemplo é [1] tem que alterar para summary_view.

[1] https://github.com/plonegovbr/brasil.gov.portal/blob/master/src/brasil/gov/portal/browser/plone/configure.zcml#L47

[hvelarde]

@claytonc entendi; seria possível para você criar uma nova issue definindo essas mudanças que devem ser consertadas?

[idgserpro]

Acho que não precisa ser uma nova issue @hvelarde, só compilar tudo que foi discutido aqui e ou adicionar um novo comment ou editar o inicial, informando que para atualizar, "x" mudanças são necessárias.

[idgserpro]

Resolvido em

https://github.com/plonegovbr/portalpadrao.release/releases/tag/1.2

https://github.com/plonegovbr/portal.buildout/releases/tag/1.2