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 (The author better mangles it through the AddOn SDk 'V2' (when he has some time left ... - it's a biggie) +

Also, starting with Ajax on still blocks...
Running great without Ajax activated.

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?

• 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

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?

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;

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.

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.

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.

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'

Check out on my roster (with IE7 of course)

Should I just de-install IE7 ? - would solve some issues actually

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

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

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.

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!

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

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.

Up - you're to fast - didn't finished my post yet ..
But I already saw that were on the same wave-length

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

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

Found your asperine
/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

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!