Looty

External page that displays guild bank data from the Roster database

Moderator: trell

Re: Looty

Postby derangedtaco » Fri Dec 15, 2006 12:16 am

Nôrrec wrote:I have a porblem with looty and the new WoW Patch.

I've updated my Wowroster 1.7.1 to 1.7.2 and uploaded the guildbank data
now looty is not showing the items any longer. It's just telling an error:

Warning: Invalid argument supplied for foreach() in /usr/export/www/vhosts/funnetwork/hosting/calistoo/looty/index.php on line 476


check: http://calistoo.ca.funpic.de/e107_plugi ... wrap.php?3
and http://calistoo.ca.funpic.de/yroster/ for the roster itself

happens to me as well!! someone help pls!
derangedtaco
WR.net Apprentice
WR.net Apprentice
 
Posts: 39
Joined: Fri Nov 24, 2006 12:24 am

Looty

Postby Anaxent » Fri Dec 15, 2006 2:36 am

Yeah just like wow addons many things have broken due to the 2.0 patch blizzard released. We devs are relying on all the addon developers of roster to rebuild there addons to be complaint with roster 1.7.2, but since there was a major change in character and guild profiler it is taking some time to get this version fully fixed. As soon as roster is a release version im sure you all will see more and more addons being fixed and created.
Last edited by Anaxent on Fri Dec 15, 2006 2:43 am, edited 1 time in total.
User avatar
Anaxent
WoWRoster.net Dev Team
WoWRoster.net Dev Team
 
Posts: 642
Joined: Tue Jul 04, 2006 6:27 am
Location: Phoenix, Az

Re: Looty

Postby derangedtaco » Thu Dec 21, 2006 1:36 am

when will it be ready.. i know devs hate that question..but.....when

or has a fix already been made? i havent checked around much

thanx
derangedtaco
WR.net Apprentice
WR.net Apprentice
 
Posts: 39
Joined: Fri Nov 24, 2006 12:24 am

Re: Looty

Postby derangedtaco » Mon Jan 01, 2007 3:26 pm

plz someone fix it
derangedtaco
WR.net Apprentice
WR.net Apprentice
 
Posts: 39
Joined: Fri Nov 24, 2006 12:24 am

Looty

Postby Nôrrec » Tue Jan 02, 2007 8:51 am

1.7.2 released.
I'm still looking for a working Looty version but don't have enough skills in php to fix it myself :(
Nôrrec
WR.net Apprentice
WR.net Apprentice
 
Posts: 2
Joined: Thu Dec 14, 2006 3:02 pm

Re: Looty

Postby derangedtaco » Mon Jan 08, 2007 10:05 pm

Nôrrec wrote:1.7.2 released.
I'm still looking for a working Looty version but don't have enough skills in php to fix it myself :(

lol same.... H E L P
derangedtaco
WR.net Apprentice
WR.net Apprentice
 
Posts: 39
Joined: Fri Nov 24, 2006 12:24 am

Looty

Postby trell » Tue Jan 09, 2007 10:20 am

It took a little bit of hunting down, but the fix seems to be pretty simple.

On line 226 of index.php you'll see the following piece of code:

Code: Select all

$item
['color']          = preg_replace('/^ff/''#'$item['item_color']);
 


Changing it to the following fixes this problem for me:

Code: Select all
if (strlen($item['color'] == 8)) {
        
$item['color'] = preg_replace('/^ff/''#'$item['item_color']);
} else {
        
$item['color'] = preg_replace('/^/''#'$item['item_color']);
}

 


Hope that helps.

I've noticed that it doesn't seem to work when sorting by item class. I'll see if I can work that out too.

-trell
Last edited by trell on Tue Jan 09, 2007 12:11 pm, edited 3 times in total.
trell
Roster AddOn Dev
Roster AddOn Dev
 
Posts: 19
Joined: Wed Dec 06, 2006 4:11 pm
Location: UK

Looty

Postby trell » Tue Jan 09, 2007 12:43 pm

Ok, and a fix to get selection by item class working.

In index.php at line 164 you should see the following code:

Code: Select all
if ($itemclass) {
        
$tmpcolor 'ff' preg_replace('/^#/'''$itemclass);
        
$wheres[] = "item_color = '$tmpcolor'";
        }
 



Replace that with the following:

Code: Select all
if ($itemclass) {
        
$tmpcolor preg_replace('/^#/'''$itemclass);
        
$wheres[] = "item_color = '$tmpcolor' or item_color = 'ff$tmpcolor'";
        }
 


And you should be all ok.

-trell
trell
Roster AddOn Dev
Roster AddOn Dev
 
Posts: 19
Joined: Wed Dec 06, 2006 4:11 pm
Location: UK

Re: Looty

Postby derangedtaco » Tue Jan 09, 2007 1:12 pm

Thanks champ
derangedtaco
WR.net Apprentice
WR.net Apprentice
 
Posts: 39
Joined: Fri Nov 24, 2006 12:24 am

Looty

Postby trell » Tue Jan 09, 2007 1:25 pm

And if you are still having problems with slashes appearing in the tooltips:

At about line 256 of index.php (it might be different because I've now made a few customisations in the version of looty I'm working on) you'll see a line that looks like:

Code: Select all

$item
['tooltip']        = htmlspecialchars(trim($item['item_tooltip']));
 


If you change that to the following, you should lose the extra slashes:

Code: Select all

$item
['tooltip']        = htmlspecialchars(stripslashes(trim($item['item_tooltip'])));
 


Hope that helps.

-trell
trell
Roster AddOn Dev
Roster AddOn Dev
 
Posts: 19
Joined: Wed Dec 06, 2006 4:11 pm
Location: UK

Re: Looty

Postby Finja » Wed Jan 10, 2007 11:57 pm

Hi.

Apologies for the lack of updates: I left my old guild (Citizens of Azeroth) way back in Christmas 2005 (sniff... a sad day, after 7 months as guild master, brings a tear to my eye even now), and didn't really have a chance to use Looty myself to test it and update it etc.

Thanks very much to trell for posting fixes - and honestly credit to you for being able to figure out what was causing the problem in that uncommented mess of code.

Hopefully his fixes should be merged into Looty now properly, along with a couple of other minor things. I am still not in a position to do much work on Looty (read: not playing WoW; not in an active guild with a bank when I am playing; don't have Roster installed anymore), but feel free to get in touch anyway if there's something broken / you'd like added / you're lonely - see my profile for contact details. (Shitloads of them! Whatever happened to the good old days of just email and IRC?) Note: please don't expect any quick replies.

A (slightly less impressive than previous) demo version of Looty can be seen here:

- http://pgl.yoyo.org/wow/looty/

and the latest version is stored here:

- http://pgl.yoyo.org/wow/looty/files/looty-latest.zip (just a symlink to the most recent zip file)
- http://pgl.yoyo.org/wow/looty/files/looty-1.1.zip (... which is this file right here, at the time of writing)

Please see the readme.txt for the changelog and general information:

- http://pgl.yoyo.org/wow/looty/files/readme.txt

cheers,

- Fin
Attachments
looty-1.1.zip
Looty 2007-01-11 v1.1
(9.94 KiB) Downloaded 71 times
Last edited by Finja on Thu Jan 11, 2007 12:08 am, edited 1 time in total.
User avatar
Finja
WR.net Apprentice
WR.net Apprentice
 
Posts: 4
Joined: Wed Jan 10, 2007 11:35 pm
Location: Prague, the Czech Republic

Looty

Postby Finja » Thu Jan 11, 2007 12:01 am

Oh, yeah, one thing: as I don't have Roster installed (and no test characters with inventories I can use for testing publically), would someone be so kind as to send me a dump of their players and items tables from Roster? While I've tested the changes made with the data I had, it would be nice to check with something a bit more recent than 6 months ago.

Thanks in advance for any help with this.

cheers,

- Fin
User avatar
Finja
WR.net Apprentice
WR.net Apprentice
 
Posts: 4
Joined: Wed Jan 10, 2007 11:35 pm
Location: Prague, the Czech Republic

Re: Looty

Postby robojerk » Sat Jan 13, 2007 3:47 am

Thanks for the updates.. I'm on a break from WoW myself and havent logged on for a few months.

trell wrote:Sorry for the delay in getting back to you all. The expansion has been distracting me.

If you are using looty 1.1 and are suffering from the 'Invalid argument supplied for foreach()' on line 485 error then this is the fix for you.

It would appear that the logic is inverted in a statement earlier in the code. So to fix this edit line 230.

It should currently look like:

Code: Select all

$item
['color']        = preg_replace((strlen($item['color']) == '/^/' '/^ff/'), '#'$item['item_color']);
 


Change that line to look like:

Code: Select all

$item
['color']      = preg_replace((strlen($item['item_color']) == '/^ff/' '/^/'), '#'$item['item_color']);
 


I hope that helps.

-trell
Last edited by robojerk on Tue Jan 23, 2007 2:06 pm, edited 2 times in total.
Image
For the Horde!
Image
User avatar
robojerk
WR.net Master
WR.net Master
 
Posts: 484
Joined: Wed Jul 05, 2006 12:17 am
Location: -The OmniMatrix- Web 3.0

Looty

Postby peterplys » Sun Jan 14, 2007 3:49 pm

i cant get any items to show up, it says i have 51 items. But none shows. my rosterbank works just fine, showing everything. Any ideas (and yes i aint the pc handyman ^^)
peterplys
WR.net Apprentice
WR.net Apprentice
 
Posts: 2
Joined: Sun Jan 14, 2007 2:49 am

Re: Looty

Postby Artillery » Sun Jan 14, 2007 7:48 pm

[url=http://emberguild.com/looty/]Ember's Looty
[/url]

Please help... I can't seem to get it to work. The configuration file is configured correctly, as far as I can tell.
Artillery
WR.net Apprentice
WR.net Apprentice
 
Posts: 9
Joined: Sun Jan 14, 2007 7:46 pm

PreviousNext

Return to Looty

Who is online

Users browsing this forum: No registered users and 0 guests

cron