@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.

Please use $_GET or $_POST and not $_REQUEST

$_REQUEST is not populated with variables if alternative urls is turned on in RosterCP

I would also put any part of this that requires admin access into the RosterCP

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

@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.

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? ;-)

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

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

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, ...?

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);

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..

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?

Yes, I can add them

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

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 ?