Emashzed wrote:Actually the generator does put ko-KR when available (out of the 2195 cards, only 814 have korean text available).
Not sure if the other cards do not exist or just mtginfo doesn't reference them
I'm on the fence about including cn/tw translations. Dotp2013 doesn't use them, but it seems handy at the time to have them for future Dotp, who knows
As you're saying, the XMLs are "stubs" and don't include the LUA scripting (because it just can't be done automatically).
The idea is just to gain time for everything else
In a future version, I'll try to auto-add scripting for the basic abilities (flying, trample, first strike, etc) + badges/helps
I think this can be done fairly effectively.
Ahh, you're right, I see it (Korean localization) on ACIDIC_SLIME_XXXXX, my mistake.
It might be handy for future versions of DotP or if you modify your generator to spit out stubs for other game engines as well, but for DotP 2013 I think it's probably just wasted space.
I agree even just stubbing things out can save people quite a bit of time when they get to coding which is why when my code can't figure out what an ability needs to do it just stubs it so I can manually add it later. I was mainly saying they were stubs because it wasn't totally clear in your previous posts and I don't think you would want regular users downloading them then reporting that they don't work. I just thought that mentioning that they were stubs might save you and some regular users some hassle.
Auto-scripting some of the basic abilities (Flying,
Lifelink, Trample, Deathtouch, Annihilator,
Battle Cry, Trample, First Strike, etc...) can most definitely be done fairly effectively (I've been experimenting with it myself) though detecting which ability to auto-add can get tricky (like when they multiple abilities on a single line
Mirri, Cat Warrior: "First strike, forestwalk, vigilance" or from abilities where they include the card name in the text like
Darksteel Citadel: "
Darksteel Citadel is indestructible.").