got a link to the roster page showing the problem?

-trell

trell wrote:got a link to the roster page showing the problem?

-trell

the link is here.

http://www.crematoria-gilde.de/pages/ht ... =guildbank

i think you can simulate the problem when you remove the "item=..."
line from one or two items in your characterprofiler.lua.

after upload you will have items with length(itemid) = 0

Using latest CP and GP.

Wowroster fresh downloaded today, and patched via the file updates mentioned in previous links ... Roster Diag shows no files to update.

Using Catergorized Guild Bank , and receive the already mentioned :
Notice: Undefined offset: 1 in ....


Have checked the already mentioned SQL queries and see nothing that was mentioned ...

Catorized Guild Bank thread , says to check here , and the last page on that thread says to Check these forums ...

So Anyone actually have a valid working fix for this ?

Temporarily fixed by deleting:

.':'.$itemid_array[1].':'.$itemid_array[2]

from gbank.php .

However, I found this fix in the http://www.wowroster.net/Forums/viewtopic/t=2289.html thread from someone stating the exact same thing that I'm about to, in that I am not sure that this is a valid fix for this issue.

It removes the symptom and everything appears to be working , but I don't code so I would still like to have this looked into if possible for a more proper fix if there is one .

Hi,
I've only done some quick testing of this, but the fix above leaves some broken links within the roster page itself, but does supress the error.

I think a better solution would be to skip that row in the database if it doesn't have a valid item id. I quickly tried this and it seems to work cleanly for me (I have even tested it and it seems ok).

On lines 246-247 of gbank.php you'll see the following:

Code: Select all


$itemid_array = explode(':', $itemArrayRow['item_id']);
$real_itemid = $itemid_array[0].':'.$itemid_array[1].':'.$itemid_array[2];
 

In between those lines add the following:

Code: Select all

if (! isset($itemid_array[2])) { continue; }
 

Thus, the code will now look like:

Code: Select all


$itemid_array = explode(':', $itemArrayRow['item_id']);
if (! isset($itemid_array[2])) { continue; }
$real_itemid = $itemid_array[0].':'.$itemid_array[1].':'.$itemid_array[2];
 

All this does is check to see if the 2nd array offset is valid (which it will be in a normal item id, and won't be if some corrupt data is in the database) and then skip the rest of the code that puts it into the various categories.

I've done rudimentary testing and it works fine here. G'luck.

-trell