ArmorySync SVN[v2.6.0.279]

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

Moderators: Ulminia, poetter

ArmorySync SVN[v2.6.0.279]

Postby tuigii » Sun Oct 28, 2007 10:28 am

Code: Select all
addons/siggen/inc/update_hook.php
  Notice line 127: Trying to get property of non-object
  Notice line 127: Trying to get property of non-object

Line 127 (addons/siggen/inc/update_hook.php ) in has this : $update->textmode and isn't defined (?).
Anyway, it seems that $update is 'out' at that moment.
Its defined global a couple of lines up.

Thinking that this is a 'siggen' issue :wink: (The author better mangles it through the AddOn SDk 'V2' (when he has some time left ... - it's a biggie) :wink: + :wink:

Also, starting with Ajax on still blocks... :cry:
Running great without Ajax activated.
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

ArmorySync SVN[v2.6.0.279]

Postby poetter » Sun Oct 28, 2007 11:16 am

Hmm, sorry. I don't really understand. Has the siggen problem something to do with AS?

And if you have a problem with ajax i am interested what kind of. Any messages in the debug infos?
Image
User avatar
poetter
Roster AddOn Dev
Roster AddOn Dev
 
Posts: 462
Joined: Sat Jun 30, 2007 9:41 pm
Location: Germany/Hamburg

ArmorySync SVN[v2.6.0.279]

Postby zanix » Sun Oct 28, 2007 2:03 pm

  • The standard update sets the variable $update as the Update class
  • It then detects if UU is uploading and sets $update->textmode to true
  • SigGen detects this so it can run the SigGen update UU compatibility mode (if enabled)


So I probably need to know what variable AS initializes the Update class as
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 SVN[v2.6.0.279]

Postby poetter » Sun Oct 28, 2007 8:29 pm

Hmm, that should be armorysync.class.php line 71 and 108

Code: Select all
            $update = new update;
            $update->fetchAddonData();


Do i have to define it globaly?
Image
User avatar
poetter
Roster AddOn Dev
Roster AddOn Dev
 
Posts: 462
Joined: Sat Jun 30, 2007 9:41 pm
Location: Germany/Hamburg

ArmorySync SVN[v2.6.0.279]

Postby zanix » Mon Oct 29, 2007 2:16 am

Yeah, I think it would have to be declared global

tuigii
Try this

Open armorysync/inc/armorysync.class.php
Find
Code: Select all
    function synchMemberByID( $server, $memberId = 0, $memberName = false, $region = false, $guildId = 0 ) {
        global $addon, $roster;

Replace with
Code: Select all
    function synchMemberByID( $server, $memberId = 0, $memberName = false, $region = false, $guildId = 0 ) {
        global $addon, $roster, $update;
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 SVN[v2.6.0.279]

Postby tuigii » Mon Oct 29, 2007 4:01 am

Yeah, that's it !

I took a fresh guild (one that I didn't synced before ever) and the eror showed up.
Putting in the $update as you said, and it was gone.

Altough, I somewhat have the impression that only an initial sync give this error - afterwards, I couldn't reproduce it.

Poetter, info about Ajax: IE7 is a no go - have to stop Ajax it before syncing, but Firefox works OK with Ajax on.
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

ArmorySync SVN[v2.6.0.279]

Postby poetter » Mon Oct 29, 2007 6:24 am

update problem is fixed in svn[283]

I did not really test with IE7, cause debugging is a mess with it. But you can configure IE7 to show script errors. Please try out and let me know what the script console says.
Image
User avatar
poetter
Roster AddOn Dev
Roster AddOn Dev
 
Posts: 462
Joined: Sat Jun 30, 2007 9:41 pm
Location: Germany/Hamburg

ArmorySync SVN[v2.6.0.279]

Postby tuigii » Mon Oct 29, 2007 9:54 am

Ok,
Activated something I never saw before : "Microsoft Scipt Editor".
It brought me to this file : /js/armorysync.js

Function : function _doUpdateDebugMessages(infoDebug)
Line 173 : rowClass = debugTableTrs[k].getElementsByTagName('td')[0].getAttribute('class').substr(10,1);
with the error
getElementsByTagName('td')[0].getAttribute('class') has the value Null or isn't an object (translated this from french to english).

Hope this says something to you.
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

Re: ArmorySync SVN[v2.6.0.279]

Postby tuigii » Mon Oct 29, 2007 10:25 am

zanix wrote:Yeah, I think it would have to be declared global

tuigii
Try this

Open armorysync/inc/armorysync.class.php
Find
Code: Select all
    function synchMemberByID( $server, $memberId = 0, $memberName = false, $region = false, $guildId = 0 ) {
        global $addon, $roster;

Replace with
Code: Select all
    function synchMemberByID( $server, $memberId = 0, $memberName = false, $region = false, $guildId = 0 ) {
        global $addon, $roster, $update;


I did this also for the next function : synchGuildByID on line 98.


Btw, poetter, doing a "ArmorySync for Guildmemberlists" (syncing the memberlist for a Guild) goes well.
but .... the transparent window that pops up when I click on Log is half sized (horizontally). This of course with IE7 - Firefox id doing well 'again' :wink:

Check out on my roster (with IE7 of course)

Should I just de-install IE7 ? - would solve some issues actually :D
Last edited by tuigii on Mon Oct 29, 2007 10:31 am, edited 1 time in total.
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

Re: ArmorySync SVN[v2.6.0.279]

Postby poetter » Mon Oct 29, 2007 4:15 pm

Yeah, this IE7 is stealing me time, causing me headaches and I DON'T LIKE IT! :twisted:

I had to try out three different methods on building overlib. The last one was working but did that half window. I already posted this problem in another topic, but got no answer from anyone. in fact i have no idea what is causing this. What should i say? A half window is better than no window :D

But i have to investigate tuigiis problem. that is the function to get the class of the last debug message line and it is doing well in FF. Lets see.
Image
User avatar
poetter
Roster AddOn Dev
Roster AddOn Dev
 
Posts: 462
Joined: Sat Jun 30, 2007 9:41 pm
Location: Germany/Hamburg

ArmorySync SVN[v2.6.0.279]

Postby tuigii » Mon Oct 29, 2007 4:41 pm

Note that for the "The half window thing" : the "Synchronise chars... " popup log is doing well in size.

What is different : we know that the list of info in the one mentioned above is not long, all info fits into it - vertically speaking.

The one cut in half has a long long list (much longer then the windows height) with updated guildmebers.
Is this what f*cks up ? Some X and Y size (line) intermix ?

poetter wrote:Yeah, this IE7 is stealing me time, causing me headaches and I DON'T LIKE IT! :twisted:

You got a point, but sins I installed this Script Editor thing yesterday, it only found 'non valid HTML' coding like example:
Code: Select all
$lang['config_reset_help'] = 'Ceci va ... ...l'autorisation... du roster';

You see the quote in quote problem? This is not an IE7 thing - and FireFox silently accepts it....

And fasten your seatbelt, poetter :D – I’ll be posting from a Mac this evening (wana try out WoW on a MAC – just for the fun) – so I’ll be using their browser…. (or what it is being called..).
Last edited by tuigii on Mon Oct 29, 2007 4:54 pm, edited 2 times in total.
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

ArmorySync SVN[v2.6.0.279]

Postby poetter » Mon Oct 29, 2007 4:45 pm

Hm maybe. i put in a div with overflow:auto to show up a scrollbar on the right side. think i have to play some more with that.
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 SVN[v2.6.0.279]

Postby tuigii » Mon Oct 29, 2007 4:54 pm

Up - you're to fast - didn't finished my post yet ..
But I already saw that were on the same wave-length :wink:
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

Re: ArmorySync SVN[v2.6.0.279]

Postby tuigii » Tue Oct 30, 2007 6:17 am

poetter wrote:Yeah, this IE7 is stealing me time, causing me headaches and I DON'T LIKE IT! :twisted:

I had to try out three different methods on building overlib. The last one was working but did that half window. I already posted this problem in another topic, but got no answer from anyone. in fact i have no idea what is causing this. What should i say? A half window is better than no window :D


Found your asperine :wink:
/armorysync/inc/armorysync.job.class.php
Line 680, change this:
Code: Select all
$array['LOG'] = "<img src="". ROSTER_PATH. "img/note.gif"". makeOverlib( "<div style="height:300px;width:500px;overflow:auto;">". $member['log']. "</div>", $roster->locale->act['update_log'] , '' ,0 , '' , ',STICKY, OFFSETX, 250, CLOSECLICK' ). " alt="" />";

for this:
Code: Select all
$array['LOG'] = "<img src="". ROSTER_PATH. "img/note.gif"". makeOverlib( "<div style="height:300px;width:500px;overflow:auto;">". $member['log']. "</div>", $roster->locale->act['update_log'] , '' ,0 , '' , ' ,STICKY, WRAP, CLOSECLICK' ). " alt="" />";


Basicly, taking out OFFSETX, 250, does the job nicely.
Looks great in Firefox.
Works ok in IE7
Even Safari is swallowing it :wink:
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

ArmorySync SVN[v2.6.0.279]

Postby poetter » Tue Oct 30, 2007 9:06 am

Great, thx tuigii. And even better i found a solution for the "Line 173 : rowClass = debugTableTrs[k].getElementsByTagName('td')[0].getAttribute('class').substr(10,1);" problem too. Will release that soon. So IE7 ajax support we're coming!
Image
User avatar
poetter
Roster AddOn Dev
Roster AddOn Dev
 
Posts: 462
Joined: Sat Jun 30, 2007 9:41 pm
Location: Germany/Hamburg


Return to ArmorySync - Depreciated

Who is online

Users browsing this forum: No registered users and 0 guests

cron