ArmorySync 2.6 beta svn[234] for Roster 2.0 Beta

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

Moderators: Ulminia, poetter

ArmorySync 2.6 beta svn[221] for Roster 2.0 Beta

Postby poetter » Mon Sep 17, 2007 5:50 am

@Adric

You are right, you will need a password. Guild Level for single character syncing. Officer Level for guild syncing and Admin Level for Realm and new guild syncing.

And no, it would't be easier in util scope with admin level cause with roster 1.7.3 i was called so often to update some chars that i got botherd of that.

Now every guildmate can update his or some other single chars by himself.
Image
User avatar
poetter
Roster AddOn Dev
Roster AddOn Dev
 
Posts: 462
Joined: Sat Jun 30, 2007 9:41 pm
Location: Germany/Hamburg

ArmorySync 2.6 beta svn[221] for Roster 2.0 Beta

Postby zanix » Mon Sep 17, 2007 6:24 am

Please use $_GET or $_POST and not $_REQUEST

$_REQUEST is not populated with variables if alternative urls is turned on in RosterCP
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: 5543
Joined: Mon Jul 03, 2006 8:29 am
Location: Idaho Falls, Idaho
Realm: Doomhammer (PvE) - US

ArmorySync 2.6 beta svn[221] for Roster 2.0 Beta

Postby zanix » Mon Sep 17, 2007 6:27 am

I would also put any part of this that requires admin access into the RosterCP
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: 5543
Joined: Mon Jul 03, 2006 8:29 am
Location: Idaho Falls, Idaho
Realm: Doomhammer (PvE) - US

ArmorySync 2.6 beta svn[221] for Roster 2.0 Beta

Postby zanix » Mon Sep 17, 2007 6:34 am

Also, each page seems to automatically start the updating process

I think there should be a button that starts it, and not make it automatic
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: 5543
Joined: Mon Jul 03, 2006 8:29 am
Location: Idaho Falls, Idaho
Realm: Doomhammer (PvE) - US

ArmorySync 2.6 beta svn[221] for Roster 2.0 Beta

Postby poetter » Mon Sep 17, 2007 6:34 am

@zanix
the only part beeing disharmonic is the realm sync. I will need to call it from the realm scope. otherwise i don't get the realm as $_GET variable.

So two option:
1. Set realm sync to officer level access
2. Delete realm syncing

So what do you suggest?

BTW: $_REQUEST is on my todo list.
Image
User avatar
poetter
Roster AddOn Dev
Roster AddOn Dev
 
Posts: 462
Joined: Sat Jun 30, 2007 9:41 pm
Location: Germany/Hamburg

Re: ArmorySync 2.6 beta svn[221] for Roster 2.0 Beta

Postby poetter » Mon Sep 17, 2007 6:43 am

zanix wrote:Also, each page seems to automatically start the updating process

I think there should be a button that starts it, and not make it automatic


put on todo. your a hard critic, aren't you? ;-)
Image
User avatar
poetter
Roster AddOn Dev
Roster AddOn Dev
 
Posts: 462
Joined: Sat Jun 30, 2007 9:41 pm
Location: Germany/Hamburg

ArmorySync 2.6 beta svn[221] for Roster 2.0 Beta

Postby zanix » Mon Sep 17, 2007 7:01 am

Yes, I am
I told you I was going to give this a full workout
I'm also helping in avoiding some support issues you may have in the future


Roster does offer variables so you don't need to look in $_GET

char scope
$roster->data contains all of the character data from the players and members table for the current member
Plus a formated date ($roster->data['update_format'])

guild scope
$roster->data contains all the data for the current guild, if guild isn't specified in the url , the the default guild data is set
$roster->data['guild_name']
$roster->data['guild_id']

realm scope
$roster->data contains the realm data, if realm isn't specified in the url, the the default realm data is set
$roster->data['server']
$roster->data['region']

I know these are not documented, but hey, docs take a long time to make


I noticed you are using mb_convert_encoding

I didn't have mbstring enabled on my server so it had to be enabled
I know that not all servers have this enabled. We had this problem with Roster back around version 1.6.0
It might be good to find an alternative
Last edited by zanix on Mon Sep 17, 2007 7:02 am, edited 1 time 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: 5543
Joined: Mon Jul 03, 2006 8:29 am
Location: Idaho Falls, Idaho
Realm: Doomhammer (PvE) - US

ArmorySync 2.6 beta svn[221] for Roster 2.0 Beta

Postby zanix » Mon Sep 17, 2007 7:19 am

RaceEn, ClassEn, Power are not being set in the players table
These are needed for the char-info addon
RaceEn - English Race
ClassEn - English Class
Power - Mana, Rage, Energy, Focus, etc.. - This can be localized
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: 5543
Joined: Mon Jul 03, 2006 8:29 am
Location: Idaho Falls, Idaho
Realm: Doomhammer (PvE) - US

Re: ArmorySync 2.6 beta svn[221] for Roster 2.0 Beta

Postby poetter » Mon Sep 17, 2007 7:29 am

zanix wrote:RaceEn, ClassEn, Power are not being set in the players table
These are needed for the char-info addon
RaceEn - English Race
ClassEn - English Class
Power - Mana, Rage, Energy, Focus, etc.. - This can be localized


Are there already any delocolize function in roster2?
What iexactly is Power? Is it the secondbar value of the armory? So Mana, Rage, Energy, ...?
Image
User avatar
poetter
Roster AddOn Dev
Roster AddOn Dev
 
Posts: 462
Joined: Sat Jun 30, 2007 9:41 pm
Location: Germany/Hamburg

ArmorySync 2.6 beta svn[221] for Roster 2.0 Beta

Postby zanix » Mon Sep 17, 2007 7:33 am

I also see some links like this
Code: Select all
$link = 'index.php?p=char-armorysync&member='. $roster->data['member_id']. '&job_id='. $this->jobid;


You should use makelink() so this addon ports with no edits to other roster ports such as rosterdf

Code: Select all
$link = makelink('&member='. $roster->data['member_id']. '&job_id='. $this->jobid);
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: 5543
Joined: Mon Jul 03, 2006 8:29 am
Location: Idaho Falls, Idaho
Realm: Doomhammer (PvE) - US

Re: ArmorySync 2.6 beta svn[221] for Roster 2.0 Beta

Postby zanix » Mon Sep 17, 2007 7:39 am

poetter wrote:Are there already any delocolize function in roster2?
What iexactly is Power? Is it the secondbar value of the armory? So Mana, Rage, Energy, ...?

No, there isn't a delocalization function in Roster but there is a localization array

This will return the English value for the class
$enclass = $roster->locale->wordings['lcLC']['class_to_en']['localized_class'];
Replace lcLC with the proper character locale

There isn't one for race, but it can be added
Look at siggen, I already have the other locales' strings for this
Just ask and I can add it to the base locales

Power is the second bar, mana, energy, rage, etc..
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: 5543
Joined: Mon Jul 03, 2006 8:29 am
Location: Idaho Falls, Idaho
Realm: Doomhammer (PvE) - US

Re: ArmorySync 2.6 beta svn[221] for Roster 2.0 Beta

Postby poetter » Mon Sep 17, 2007 7:50 am

zanix wrote:There isn't one for race, but it can be added
Look at siggen, I already have the other locales' strings for this
Just ask and I can add it to the base locales


OK. I ask: Can you add them to the base locales? :D
Image
User avatar
poetter
Roster AddOn Dev
Roster AddOn Dev
 
Posts: 462
Joined: Sat Jun 30, 2007 9:41 pm
Location: Germany/Hamburg

ArmorySync 2.6 beta svn[221] for Roster 2.0 Beta

Postby zanix » Mon Sep 17, 2007 7:55 am

Yes, I can add them
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: 5543
Joined: Mon Jul 03, 2006 8:29 am
Location: Idaho Falls, Idaho
Realm: Doomhammer (PvE) - US

ArmorySync 2.6 beta svn[223] for Roster 2.0 Beta

Postby poetter » Mon Sep 17, 2007 8:03 am

k, thx and btw: new update for all non svn users (are there some?). See first post in this threat
Image
User avatar
poetter
Roster AddOn Dev
Roster AddOn Dev
 
Posts: 462
Joined: Sat Jun 30, 2007 9:41 pm
Location: Germany/Hamburg

Re: ArmorySync 2.6 beta svn[223] for Roster 2.0 Beta

Postby tuigii » Mon Sep 17, 2007 9:13 am

poetter wrote:k, thx and btw: new update for all non svn users (are there some?). See first post in this threat

Just testing the zip in your first post : no what the heck is an SVN user ?
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

PreviousNext

Return to ArmorySync - Depreciated

Who is online

Users browsing this forum: No registered users and 1 guest