-
Notifications
You must be signed in to change notification settings - Fork 12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Ao adicionar uma notícia (collective.nitf) sem imagem num MediaCarouselTile, a capa quebra no IDG 1.0.5 e Internal Server Error no 1.1.3 #136
Comments
eu nunca entrei em detalhes de por que a capa quebrava com um erro no tile; acho que seria algo interessante de analisar. também acho que é válido abrir relatos para resolver qualquer problema que afete a experiência do usuário. não estou familiarizado com o código de esse tile e infelizmente não posso dedicar tempo agora para isso. |
No IDG 1.0.5, seguindo os passos descritos nesse issue, é possível demonstrar esse cenário. Em resumo, se algum tile lança uma exceção na renderização, a capa quebra: na minha opinião, deveria aparecer um erro igual ocorre em portlets, "ocorreu um erro na renderização desse tile" ou algo do tipo mas não sei se há alguma limitação no collective.cover ou no Plone mesmo arquitetural que impeça essa forma de tratamento. |
+1 seria uma melhoria fantástica; abre uma issue no collective.cover e vamos perguntar por ai |
@hvelarde Atualizando o plone.app.blocks para 2.1.2 resolve o problema da capa "quebrar". Peguei o mesmo ambiente 1.0.5 que está com a capa quebrada, e mudei o versions.cfg do plone.app.blocks que estava 1.1.1 para 2.1.2. Não sei se foi em 2.1.1:
ou em 2.0.0:
do plone.app.blocks que isso foi corrigido, mas isso é irrelevante porque versões mais atuais do collective.cover já pedem plone.app.blocks > 2.1.2, corrigindo o problema. Portanto, para fecharmos esse issue acho que o ideal é:
Sei que os dois últimos itens não dizem respeito a esse pacote e sim dependências, mas enquanto não for incorporado, esse erro continuará a acontecer. :( |
muito interessante, obrigado pela pesquisa! |
@hvelarde Apesar da capa "não quebrar" os tiles corretos também não são renderizados. Abrimos um outro relato na comunidade em plone/plone.app.blocks#27 |
@idgserpro obrigado! estou acompanhando |
Fix by #138 |
Como simular:
Isso quebra a capa no 1.0.5 ficando dessa forma:
Esse é o traceback do 1.0.5:
No 1.1.3 não chega a quebrar a capa pois o erro ocorre no momento da composição, ou seja, o item selecionado não é persistido. Esse é o traceback no 1.1.3:
O erro ocorre no tile mediacarousel.
Onde temos algo como:
Deveria ser (por exemplo)
Ou seja, deveria levar em consideração notícias que não possuem imagens ao invés de lançar uma exceção.
O problema é que ao fazer essa modificação no código, no carrossel não aparece sequer a notícia. Acredito que, além da correção acima, deveria ao menos aparecer o título da notícia mas sem nenhuma imagem, pois da forma como está confunde o usuário. Opiniões?
Além disso, outra coisa me intrigou: porque um erro num tile apenas quebrou toda a capa no IDG 1.0.5? No IDG 1.0.5 se quebra porque o tile não é atualizado em tempo de composição, e só após salvar que se renderizam todos os tiles: já no 1.1.3 é renderizado em tempo de composição (pois o collective.cover é o 1.0a11) e aí como não consegue renderizar e, por conseguinte, salvar, não quebra a capa: mas e outros tiles da capa? Sem contar que seria importante mostrar uma mensagem de erro pro usuário indicando que houve esse erro na atualização via ajax, mas acho que isso já é responsabilidade do collective.cover.
@hvelarde como você tem mais conhecimento do collective.cover, alguns questionamentos:
1 - É o caso de abrir um relato para tratar as requisições ajax que voltam erro 500 e indicar uma mensagem de erro ao usuário? Exemplo de url que deu erro nesse issue: http://localhost:8080/Plone/home/@@updatetilecontent?tile-type=mediacarousel&tile-id=ce5d92d477ed426f8c6903020443f2be&uid=e464e07c27504ba4aedc6f045ea3adf3
2- Esse comportamento de um tile com erro quebrar toda a capa foi corrigido em novas versões do collective.cover, ou seria o caso de abrir um issue por lá (ou é uma limitação técnica do componente e não há como consertar)? Não é possível simular isso no 1.1.3.1 já que ele dá update no tile, mas não tenho como afirmar que isso foi corrigido.
The text was updated successfully, but these errors were encountered: