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

Corrige incompatibilidade com spree1.3.1 e atualiza alguns .erb para .haml #1

Closed
wants to merge 1 commit into from

Conversation

michelboaventura
Copy link

No description provided.

@andreteves
Copy link

Oi Michel, agora que percebi que você corrigiu o problema que relatei aqui #2

Eu tentei usar o seu fork (que realmente corrige este problema), mas devido a utilização do haml eu estou tendo problema com codificação das palavras com acento do partial "_order_details.html.haml" (através, por exemplo). Veja o erro abaixo:

ActionView::Template::Error ("\xC3" from ASCII-8BIT to UTF-8):

Quando eu simplesmente troco o seu partial para erb o problema é automaticamente resolvido.

Você entende por quê isso está acontecendo?

Obrigado,
André

@michelboaventura
Copy link
Author

André,

O arquivo está em UTF-8, então não sei porque ele está tentando converter de ASCII-8BIT para UTF-8. Se você colocar um '#coding: utf-8' no controller resolve? Você está rodando em qual sistema com qual server?

Abraço

@andreteves
Copy link

Michel, tentei colocar "#coding: utf-8" no PagSeguroController e o problema continua. Realmente a única solução que encontrei até agora foi utilizar o erb ao invés do haml. Mas não consigo entender o motivo disso resolver.

Eu estou usando ruby 1.9.2, rails 3.2.11, spree 1.3.2, server WEBrick 1.3.1 em um Mac OS X (10.8.2).

Alguma outra sugestão?

Obrigado!

@michelboaventura
Copy link
Author

A única coisa que pensei foi em você atualizar o ruby para 1.9.3. Eu uso
Linux, e aqui tudo funcionou perfeitamente. Se não funcionar eu posso
atualizar o PR para usar erb ao invés de haml.

@andreteves
Copy link

Oi Michel! Testei agora com o ruby 1.9.3 e o problema é o mesmo...deve ser relacionado ao meu OS!

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.

2 participants