godotengine / godot-git-plugin

Git implementation of the VCS interface in Godot
MIT License
656 stars 64 forks source link

Incorrect display of Russian characters #218

Closed Purpursarkans closed 3 months ago

Purpursarkans commented 5 months ago

изображение

OS: archlinux OS lang: ru godot installed from steam

bruvzg commented 5 months ago

Nothing in the plugin seems to be Unicode aware.

https://github.com/godotengine/godot-git-plugin/blob/2f160a2b7926efb728025ab07a29858ff71981a1/godot-git-plugin/src/git_plugin.cpp#L385

Probably should be godot::String msg = godot::String::utf8(git_commit_message(commit.get()));

https://github.com/godotengine/godot-git-plugin/blob/2f160a2b7926efb728025ab07a29858ff71981a1/godot-git-plugin/src/git_plugin.cpp#L278

Same here for branch names and probably every single other use of godot::String.

Purpursarkans commented 5 months ago

the previous comment has already suggested why this is so, but I want to add 1 more place without Russian characters, because of which it is impossible to index and commit changes (file names in Russian) изображение

Calinou commented 5 months ago

Would https://github.com/godotengine/godot-git-plugin/pull/190 help with this?

bruvzg commented 5 months ago

Would https://github.com/godotengine/godot-git-plugin/pull/190 help with this?

It seems to be fixing at least some of the strings, not sure if it's all.

Purpursarkans commented 5 months ago

change

https://github.com/godotengine/godot-git-plugin/blob/259ae989f48486b9c9f6fedbe09af3da99fc3a31/godot-git-plugin/src/git_plugin.cpp#L385

on

godot::String msg = godot::String::utf8(git_commit_message(commit.get()));

it really helped to fix the Russian language in the commit list, but the Russian language still needs fixing in the index and in the version control tab (I highlighted in green the part that started working correctly) изображение

Calinou commented 3 months ago

Thanks for the report! Consolidating in https://github.com/godotengine/godot-git-plugin/issues/150.