zackdreaver / ROenglishRE

An unofficial english translation project for Korea Ragnarok Online (kRO).
144 stars 174 forks source link

iteminfo have errors: #13

Closed fmendezy closed 7 years ago

fmendezy commented 7 years ago

have very much error in the file.

Example line 19556 image

zackdreaver commented 7 years ago

Does this affect your client, like crashing or the description isn't read properly?

hurtsky commented 7 years ago

i also have this problem, after merging the last commit that you merge with JRO item i think.. but i fix this manually by adding all < " >, @zackdreaver yup, it does give error at client

hurtsky commented 7 years ago

just a suggestion @zackdreaver , i think you should try use sever database editor, it will make your live easier :+1: https://rathena.org/board/files/file/3269-server-database-editor/

zackdreaver commented 7 years ago

Well I have no problem so far like what you guys have here. I only set my text editor, which ever it is, to ANSI encoding. I see from your screen shot that you're not using this encoding.

Please set your text editor to this encoding first, then open the itemInfo

zackdreaver commented 7 years ago

Here is mine from notepad++ with ANSI encoding new picture This item using two languages Korean and Japan.

new picture 1 And this item using two languages Korean and Chinese.

hurtsky commented 7 years ago

Here mine, using ANSI also, i also keep having conflict during cherry-pick even my repo is same as you (not modified one)

capture

zackdreaver commented 7 years ago

1.What's your OS main language? 2.What's your git client? 3.Can you check your default git client encoding?

hurtsky commented 7 years ago

im pretty sure all git default at UTF-8, and my OS main language is english, not adding any others language here. from what i can see right now, from git diff, File is considered as binary: invalid character 0x00 in line 14778, at position 7 there some Japanese words is using invalid character, which make those error. maybe due to copy paste thing?, because before you're merged with jro, no issues in cherry-picking.

zackdreaver commented 7 years ago

The git encoding for this project is cp1252.

hurtsky commented 7 years ago

@zackdreaver okay thanks, will try set encoding to cp1252

zackdreaver commented 7 years ago

Closing this, because it was GIT client's issue.

hurtsky commented 7 years ago

@zackdreaver sorry for reply at this issue again, but i still having problem after changing the encoding to cp1252, its like the git change the encoding after git checkout.. if possible, can you not adding non translated (japanese) into git? i means, to avoid errors.. will appreciate any of your respond, just a request.

zackdreaver commented 7 years ago

Going backwards is not my style when working on something.

You can always use the pre-merge one here 39c248b37ade9c818e666c7394bf955050273b67

hurtsky commented 7 years ago

it's okay, it just a request, i know i can always use the pre-merge. thank you for your hard work :D

zackdreaver commented 7 years ago

@hurtsky try to update, I have pushed a workaround though don't know if it works or not. https://github.com/zackdreaver/ROenglishRE/commit/e849392840bc600ed135fcf94cc90a48f79f3cf3

hurtsky commented 7 years ago

@zackdreaver thanks, i will test it let you know.. i think i'm wrongly explain the last comment, its not , not adding the japanese word, but replace the japanese word with "????" something like that, because from what i saw, the git change the encoding after -git checkout because they detect invalid character.. but let me test first the one you recommend. and will let you know again. thanks again

hurtsky commented 7 years ago

@zackdreaver looks like it same sir, have error at the same line, actually i'm not really bothered with this, i can just fix it with a few commands but the others of your users might not know.. At yours local repo its already okay but after you do git checkout, the git change its encoding because invalid character i get this when using git diff (invalid character 0x00), thus make your users error.. my idea is create another branch (-pretranslate) that have non translated japanese word, and the (master) branch replace the japanese word with "????". and only merge the japanese word to master after it been translated. im pretty sure your work/repo have many user/follower, just like myself, i always keep my repo update with yours :+1:

thanks again

zackdreaver commented 7 years ago

Try download this one. itemInfo.zip

hurtsky commented 7 years ago

no errors in that itemInfo @zackdreaver , looks good

zackdreaver commented 7 years ago

Ok then, that explain it's GIT client's fault. will fix related issue, maybe in 1 or 2 weeks.

hurtsky commented 7 years ago

thanks :+1: