'raceEN' and 'Night Elf' issues [1337]

Posts from previous Beta sessions

'raceEN' and 'Night Elf' issues [1337]

Postby tuigii » Thu Sep 20, 2007 11:08 pm

Found something about Night Elf’s: just have a look at their main char. page : yup, no class icon for them (male or female).
Example : http://www.papy-team.fr/roster_20b1/ind ... &member=15

(In database ‘players’ the field is named RaceEN and contains “Night Elf”.

This value is used over here like this:
[file addons/info/inc/header.php – line 74]
Code: Select all
$char->data['char_icon'] = $addon['image_path'] . 'portrait/' . strtolower($char->data['raceEn']) . '-' . ($char->data['sexid'] == '0' ? 'male' : 'female');

So, the resulting image file will be:
…<img src="/roster_20b1/addons/info/images/portrait/night elf-male.gif"…. (did you saw the space in it ?)

Or, the concerning files are named:
nightelf-male.gif
nightelf-female.gif
in dir. addons/info/images/portrait, without the space between night & elf-…..

Proposed solution:
Insert the space in the filename (if this doesn’t introduce server-OS complications)
Or
Change line 74 in header.php so it removes spaces in 'raceEn' before usage.
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

'raceEN' and 'Night Elf' issues [1337]

Postby Diska » Thu Sep 20, 2007 11:17 pm

Looks like a bug in ArmorySync and not in Roster Tuigi.

Char Diska has been updated with a cp.lua: http://roster.unbridled.eu/index.php?p= ... &member=81
Char Noldor has been updated through Armory: http://roster.unbridled.eu/index.php?p= ... &member=55

Looks like ArmorySync is not filling the field properly.
Last edited by Diska on Thu Sep 20, 2007 11:17 pm, edited 1 time in total.
User avatar
Diska
Roster AddOn Dev
Roster AddOn Dev
 
Posts: 179
Joined: Tue Jul 04, 2006 2:05 pm

'raceEN' and 'Night Elf' issues [1337]

Postby zanix » Thu Sep 20, 2007 11:28 pm

CP data provides the EN name without the space
Armory Sync is using translation in the locales to convert the names to EN since armory doesn't provide the English name in other locales

Armory sync needs to parse out the space
Code: Select all
str_replace(' ','',{RACEEN}) 
Last edited by zanix on Thu Sep 20, 2007 11:33 pm, edited 2 times in total.
Read the Forum Rules, the WiKi, and Search before posting!
WoWRoster v2.1 - SigGen v0.3.3.523 - WoWRosterDF
User avatar
zanix
Admin
Admin
WoWRoster.net Dev Team
WoWRoster.net Dev Team
UA/UU Developer
UA/UU Developer
 
Posts: 5546
Joined: Mon Jul 03, 2006 8:29 am
Location: Idaho Falls, Idaho
Realm: Doomhammer (PvE) - US


Return to Archived

Who is online

Users browsing this forum: No registered users and 1 guest

cron