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

Encoding sugerido #37

Closed
likid opened this issue May 22, 2019 · 7 comments
Closed

Encoding sugerido #37

likid opened this issue May 22, 2019 · 7 comments
Assignees
Labels
editor Editor related issues enhancement New feature or request

Comments

@likid
Copy link

likid commented May 22, 2019

Pessoal, na documentação está sendo informado que o encoding sugerido deve ser o ISO-8859-16, até então eu estava utilizando o Windows-1252 que é o mesmo utilizado no TDS no Eclipse. Ao alterar para o encoding sugerido, os arquivos ficaram com problema nos caracteres especiais e com isso acabei voltando para o Windows-1252.
Dessa forma, o correto não seria sugerir o uso do encoding Windows-1252?

@ricardomansano
Copy link
Contributor

@likid , conversei isso hj a tarde com o time de Frame e eles confirmaram que usam o Windows-1252.
em alinhamento com o @matheus-sales resolvemos fazer o seguinte:
-Criar uma Tag em nosso Json com Windows 1252
-Quando alguem alterar (ex: pra 1251 (cirilico), agente joga essa informacao no nosso Json, respeitando a configuracao do usuario

Vc acha que assim funciona?

@lwtnb-wrk lwtnb-wrk added the editor Editor related issues label May 23, 2019
@likid
Copy link
Author

likid commented May 23, 2019

@ricardomansano usar um encoding diferente do Windows-1252 não poderia gerar erros de apresentação dos textos no ERP? Até onde eu conheço o sistema, ele trabalha com este encoding, o ideal seria sugerir que o desenvolvedor sempre use Windows-1252 no meu ponto de vista.

@ricardomansano
Copy link
Contributor

@likid , concordo, eu posso sugerir, mas não posso obrigar.
Se eu forçar a mudança a cada abertura do VS e o analista voltar para o que ele deseja, o plugin acaba sendo um problema, e não uma solução.
Vamos documentar "com destaque" no Git que o padrão adotado pela TOTVS é o W1252, na esperança que apenas o time da Russia altere, já que precisará do W1251. (@matheus-sales PSC)
Creio que na próxima semana ja tenhamos essa feature na liberação, amanhã faremos nosso primeiro update com correções/melhorias, fique atento.
Grato pelo apoio,

@matheus-sales matheus-sales self-assigned this May 23, 2019
@matheus-sales matheus-sales added the enhancement New feature or request label May 23, 2019
@matheus-sales
Copy link
Contributor

Na abertura do workspace criamos uma opção que o usuário pode definir se deseja alterar o encoding dos fontes advpl para o padrão (Windows1252). Caso aceite, alteramos o encoding apenas para os fontes advpl e o padrão do vscode continua como está. Caso não aceite, na próxima abertura é apresentado a pergunta novamente. Temos também uma opção de não perguntar novamente.
encoding

@fabiofurlanc
Copy link

@matheus-sales, no vscode usando o W1252 OK, no GITHUB para ver as alterações nos fontes só mostra corretamente se estiver em UTF-8, não achei tem como mudar isso no GITHUB para apresentar o arquivo em W1252 e mostrar os acentos corretamente ?

@lwtnb-wrk
Copy link
Contributor

@fabiofurlanc Esta é uma issue muito antiga, se estiver com problemas nas versoes mais atuais do tds-vscode, favor abrir um novo chamdo.

@totvs totvs locked and limited conversation to collaborators Sep 26, 2024
@lwtnb-wrk
Copy link
Contributor

@fabiofurlanc Só um ponto, se o problema for na visualização do github isso não tem relação com o tds-vscode e o encoding utilizado nos fontes AdvPL/4GL.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
editor Editor related issues enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

5 participants