Minor ArmorySync 2.6 beta svn[x] for R. V Gender issue

Sync Blizzards' Armory data with WoWRoster (addon depreciated no longer works see ApiSync)

Moderators: Ulminia, poetter

Minor ArmorySync 2.6 beta svn[x] for R. V Gender issue

Postby tuigii » Tue Dec 25, 2007 2:51 am

Hi Poetter

When twitling a little bit with SigGen, I discovered that sexes form chars, imported by the Armory, weren't "as should be".
I switched SigGen to the start char images. Meaning : sexes are involved here.
File names are like bloodelf-female-hunter.png
Opening the table {rosterprefix}_players showed me that all char imported from the Armory (indicated in the CPversion column with a 2.6.0) avec sex field which set as '(H)' or '(F)'.
Btw : sexId = ok.

This should be the localized value of 'Homme' or 'Femme' (these values are brought back to localized values), and will it be recognized by SigGen (and others) as a valid sex indication.

I think some works has to be done around line 747 - armory/inc/armorysync.class.php, there, were you marked :
// This is an ugly workaround for an encoding error in the armory :wink: )

On line 747, you just assign : $this->data["Sex"] = $char->gender;
I thinks it's time to introduce yet another lookup array here.

Is this new ?
Must be again another way of Bliz making things different between IG -> CP and The Armory ?

Anyway : Happy Christmas to you all, folks ! Joyeux Noël ^^
User avatar
tuigii
WR.net Master
WR.net Master
 
Posts: 891
Joined: Wed Dec 27, 2006 12:57 pm
Location: Somewhere in the South Ouest of France

Minor ArmorySync 2.6 beta svn[x] for R. V Gender issue

Postby poetter » Tue Dec 25, 2007 6:14 pm

Hi tuigii, Happy Christmas!

Yes you are right. What would be the exact translation of male and female that siggen is expecting? Are "Homme" and "Femme" correct?

BTW: The french locales of the armory seem to be the only one setting (h) and (f) instead of whole words.
Image
User avatar
poetter
Roster AddOn Dev
Roster AddOn Dev
 
Posts: 462
Joined: Sat Jun 30, 2007 9:41 pm
Location: Germany/Hamburg

Re: Minor ArmorySync 2.6 beta svn[x] for R. V Gender issue

Postby tuigii » Wed Dec 26, 2007 1:40 am

To make SigGen happy, I used this addons/siggen/locale/frFR.php
// Race, Gender Translation
$lang['translate']['Elfe de sang'] = 'Blood Elf';
.....
$lang['translate']['Homme'] = 'Male';
$lang['translate']['Femme'] = 'Female';
$lang['translate']['(M)']= 'Male';
$lang['translate']['(F)']= 'Female';

SigGen already translated some words. I just added the last two lines.

Normally, SigGen would except the two lines before, as the entire roster does (does it ?!) (roster is using sexId ?!).
But it would be better if AS translated '(H)' to 'Homme' and '(F)' to 'Femme' - as CP also extract theses values.
User avatar
tuigii
WR.net Master
WR.net Master
 
Posts: 891
Joined: Wed Dec 27, 2006 12:57 pm
Location: Somewhere in the South Ouest of France


Return to ArmorySync - Depreciated

Who is online

Users browsing this forum: No registered users and 1 guest

cron