Skip to content
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

Adiciona diretiva do plone4.csrffixes no dependencies.zcml para que seja carregado este produto quando os testes são executados #297

Merged
merged 1 commit into from
Aug 10, 2016

Conversation

idgserpro
Copy link
Member

@idgserpro idgserpro commented Jul 4, 2016

Adiciona diretiva do plone4.csrffixes no dependencies.zcml para que seja carregado este produto quando os testes são executados.

fixes #279

@idgserpro idgserpro force-pushed the issue_279 branch 4 times, most recently from 85c4e4b to 54134c7 Compare July 5, 2016 14:58
@idgserpro idgserpro force-pushed the issue_279 branch 2 times, most recently from d0e618c to 120e21a Compare July 20, 2016 13:10
@idgserpro
Copy link
Member Author

Favor revisar

@hvelarde
Copy link
Member

vocês estão misturando coisas aqui; por favor separar em dois commits; não entendi porque precisa de essa keyword nos testes da capa.

@idgserpro
Copy link
Member Author

não entendi porque precisa de essa keyword nos testes da capa.

@hvelarde ver: plone/buildout.coredev#225 (comment)

@hvelarde
Copy link
Member

então, isso deveria ser consertado no collective.cover e não aqui.

@idgserpro
Copy link
Member Author

@hvelarde esse problema não ocorre no collective.cover porque ele não utiliza o plone4.csrffixes nos seus testes. Até tinhamos incluido o plone4.csrffixes no cover mas vimos que não é necessário.

@hvelarde
Copy link
Member

o certo seria substituir essa linha aqui pelo seguinte:

Wait Until Page Contains  Add Cover

@idgserpro
Copy link
Member Author

@hvelarde pensando melhor, seria bom corrigir no cover e no futuro remover essa customização no brasil.gov.portal.

hvelarde added a commit to collective/collective.cover that referenced this pull request Jul 21, 2016
@idgserpro idgserpro force-pushed the issue_279 branch 2 times, most recently from b870881 to 0d72d89 Compare July 21, 2016 18:37
@idgserpro
Copy link
Member Author

idgserpro commented Jul 21, 2016

@hvelarde o

Wait Until Page Contains Add Cover

não funcionou:

https://travis-ci.org/plonegovbr/brasil.gov.portal/builds/146457659

hvelarde added a commit to collective/collective.cover that referenced this pull request Jul 21, 2016
@hvelarde
Copy link
Member

hvelarde commented Jul 21, 2016

eu vi, ele está dando problemas num teste; acho que da para consertar... acabei de abrir um branch com isso se por acaso querem trabalhar ai: collective/collective.cover#636

@idgserpro
Copy link
Member Author

Acho que temos que fazer funcionar aqui primeiro, uma vez que no cover não tem plone4.csrffixes. Lá vai funcionar.

Aqui, eu acho que tem que esperar algum evento js assíncrono. Por isso acho que vai ter que ser Sleep e não Wait Until.

hvelarde added a commit to collective/collective.cover that referenced this pull request Jul 22, 2016
@idgserpro idgserpro force-pushed the issue_279 branch 2 times, most recently from 359efa8 to 6be82ab Compare July 26, 2016 18:06
@idgserpro
Copy link
Member Author

@hvelarde só funcionou com Sleep. Agora precisamos fazer essa alteração no cover e colocar um fixme aqui para remover esse customização quando sair uma nova versão do cover.

idgserpro pushed a commit to collective/collective.cover that referenced this pull request Jul 29, 2016
hvelarde added a commit to collective/collective.cover that referenced this pull request Aug 2, 2016
idgserpro pushed a commit to collective/collective.cover that referenced this pull request Aug 2, 2016
idgserpro pushed a commit to collective/collective.cover that referenced this pull request Aug 2, 2016
hvelarde added a commit to collective/collective.cover that referenced this pull request Aug 3, 2016
hvelarde added a commit to collective/collective.cover that referenced this pull request Aug 4, 2016
…eja carregado este produto quando os testes são executados
@idgserpro
Copy link
Member Author

idgserpro commented Aug 9, 2016

@hvelarde atualizei o PR com o Input Text For Sure e adicionei um fixme para remover a customização quando estivermos utilizando uma versão do collective.cover que contemple a correção. Pode revisar e fazer o merge?

@hvelarde hvelarde merged commit 8e51786 into master Aug 10, 2016
@hvelarde hvelarde deleted the issue_279 branch August 10, 2016 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implementar proteção CSRF
2 participants