[PATCH] Show Level in Broker Window

EQ2Emulator Development forum.

Moderator: Team Members

Post Reply
User avatar
Rhas
Retired
Posts: 33
Joined: Sat May 29, 2010 11:39 am
Location: Kansas City

[PATCH] Show Level in Broker Window

Post by Rhas » Sun May 30, 2010 12:44 am

This patch displays the item adventure_default_level in the broker window. In revision 437, no level is displayed. It appears that "unknown8" in the WS_BrokerItems struct is what the client uses for the level column in the broker window. I did not rename the field in WorldStructs.xml. I'll leave that up to you if you decide to commit this change.

I believe on Live it displays the required level and not recommended level. If some logic needs to be added to show the tradeskill_default_level for tradeskill items, I can change that.

Tested on my server using client 6118L.

Code: Select all

--- client.cpp (revision 437)
+++ client.cpp	(working copy)
@@ -4705,6 +4705,8 @@
 						packet->setArrayDataByName("quantity", item->stack_count, i);
 					packet->setArrayDataByName("stack_size", item->stack_count, i);
 					packet->setArrayDataByName("item_name", item->name.c_str(), i);
+					packet->setArrayDataByName("unknown8", item->generic_info.adventure_default_level, i);
+					
 					//QueuePacket(item->serialize(GetVersion(), false, GetPlayer()));
 				}
 			}
Before and after screenshots:
broker_level_before.jpg
broker_level_after.jpg
You do not have the required permissions to view the files attached to this post.

User avatar
Scatman
Retired
Posts: 1688
Joined: Wed Apr 16, 2008 5:44 am
EQ2Emu Server: Scatman's Word
Characters: Scatman
Location: New Jersey

Re: [PATCH] Show Level in Broker Window

Post by Scatman » Sun May 30, 2010 8:24 am

Cool. :) Again, later on today I'll give this a whirl. Being dragged to the beach soon :evil:

User avatar
Zcoretri
Team Member
Posts: 1642
Joined: Fri Jul 27, 2007 12:55 pm
Location: SoCal

Re: [PATCH] Show Level in Broker Window

Post by Zcoretri » Sun May 30, 2010 9:48 am

Scatman wrote:Cool. :) Again, later on today I'll give this a whirl. Being dragged to the beach soon :evil:
They have beaches on the right coast? :mrgreen:

User avatar
Scatman
Retired
Posts: 1688
Joined: Wed Apr 16, 2008 5:44 am
EQ2Emu Server: Scatman's Word
Characters: Scatman
Location: New Jersey

Re: [PATCH] Show Level in Broker Window

Post by Scatman » Sun May 30, 2010 1:14 pm

Yeah unfortunately I live right next to Atlantic City so it's becoming tourist season :(

Post Reply

Who is online

Users browsing this forum: No registered users and 0 guests