plonegovbr/brasil.gov.tiles

Remover patch to tile banner rotativo que foi adicionado após atualizar o cover e corrigir de forma definitiva

Closed this issue · 8 comments

Motivado por #145. No PR #173 isso começou a ser tratado. Explicação completa do erro em #166 (comment).

Lembrar de ver a questão do csrf (#145 (comment)).

Na versão do IDG 1.3 indo para 1.4 o tile Carrossel ocorre o seguinte erro ao tentar excluir um item

http://localhost:8080/Plone/home/@@removeitemfromlisttile
Traceback (innermost last):
Module ZPublisher.Publish, line 138, in publish
Module ZPublisher.mapply, line 77, in mapply
Module ZPublisher.Publish, line 48, in call_object
Module collective.cover.browser.compose, line 174, in call
Module brasil.gov.tiles.patches_banner_rotativo, line 60, in render
TypeError: render() takes exactly 1 argument (2 given)

@claytonc por favor abre uma issue no brasil.gov.portal visualizar todo no milestone 1.4.

@idgserpro podem esclarecer se isso aqui é um enhancement ou um bug mesmo?

@claytonc não entendi. Ocorre o erro na 1.3 ou na 1.4?

É um bug mesmo, foi feito um patch mas o correto é mexer em #173

Poderia dar um print em

tile_type = self.request.form.get('tile-type')
? Era pra cair no if do "banner_rotativo" mas está entrando no else.

Ah, me desculpe, tile Carrossel e não "banner_rotativo".

@idgserpro

Isso no Carrossel.
Ocorreu nas duas versões, pensei que um erro exclusivo da 1.4 mas testei também na 1.3 e ocorreu o mesmo problemas.

@idgserpro

Eu sugiro se for possível e fácil a remoção do item "excluir" do tile Carrossel e deixar a exclusão ser feita link Editar como hoje, pois ocorre outro problema que é "Could not extract a stage height from the CSS. Traced height: 0px." e só editando e removendo o item problemático que ele volta a funcionar.

no IDG v2 o banner rotativo está descontinuado e esse patch foi removido (refs. #239).