Chucksta

I'll look at this right away, seeing as it most definitely makes the game unplayable, or at least playable only for as long as the toon has any life (3 or 4 bouts of combat), lol

I tested the output in the game window when solo and grouped, and it got the correct messages when looting money:

"You received 2 copper"

"Your shares is 1 copper"

Therefore the opcode is being received by the client.

Perhaps there should be another opcode sent that tells the client to update the total money displayed?

Duplicate - closing and archiving

Original issue: 

This has been fixed. Issue was that the loading of Warden's default settings was resulting in all previously read in settings for everything else, being set to 0.

I just checked the characters table, and the total amount of money is correct under "money" for the characters.

So, just need to figure out how the client is told the amount of money the character has.

Fix:

https://github.com/mangosthree/server/pull/106

When buying equipment and looting, the items go into the bag slots, but you cannot move the items around; they are stuck in the slot they first go into.

.modify money <amount> gives you money, as I was able to then buy armour, but...

the total in the interface (bottom right of opened up backpack) was not going up; it just showed as 0

Edit: looting of money also results in the correct amount showing up in the characters table (after logoff, ofc)

It should appear on the site as soon as you hit Publish / Synch - top right of the github client.

Are you on Skype?

Windows 10!!! 

use branch Develop21

https://github.com/mangostwo/database/tree/Develop21

You'll need to fork the branch to your own profile, just click on Fork button on the mangostwo database page

https://github.com/mangostwo/database

If your profile is called necrovoice and you don't have any other project with the name of database, you'll then have mangosteo database at:

https://github.com/necrovoive/database

You can commit changes via the github client, or directly from a git terminal using git commands

client: https://desktop.github.com/

git for windows: https://git-for-windows.github.io/

with git for windows, you can right click on a folder and run git bash - a git terminal. Very useful.

Say you have a folder on your PC called Mangos, to clone (download) the mangostwo repo, you'll need to work from your copy firth, then PR it. To do that:

right click on the Mangos folder

select git bash

the terminal will open up

then in the terminal you type:

git clone --recursive git://github.com/necrovoice/database -b Develop21 

You'll have under your Mangos folder, a folder called database

Make the changes - adding a new update under World/Updates/Dev21 in your case

Then if using the client, drag and drop the database folder to it - enter details - hit commit - hit Publish/Synch

The go to your github account and your fork of the mangostwo database, and hit Create Pull Request

Someone will then check it over and merge it for you

necrovoice, can you commit that fix to the Two repo ?

Leave Three; I'll add it to Three's base :-)

Yay, nice one, necrovoice

Everything looks correct in the database on both the Horde and the Alliance side. The quest givers 

Alliance: 23729 Baron Ulrik von Stromheart

Horde: 26471 Image of Archmage Aethas Sunreaver

correctly correctly set up to teleport the player.

I am unable to test in-game, because Two keeps on crashing on me

Further details on the quests:

Alliance quest to enter Dalaran

http://www.wowhead.com/quest=12794/the-magical-kingdom-of-dalaran

Horde quest to enter Dalaran

http://www.wowhead.com/quest=12791/the-magical-kingdom-of-dalaran

More information, including the NPCs who can teleport you once you have the quest:

http://wowwiki.wikia.com/wiki/Quest:The_Magical_Kingdom_of_Dalaran

Alliance Quest giver:

http://www.wowhead.com/npc=23729/baron-ulrik-von-stromhearth

Horde quest giver:

http://www.wowhead.com/npc=26471/image-of-archmage-aethas-sunreaver

Spell cast by the NPC to teleport the player:

http://www.wowhead.com/spell=53360/teleport-to-dalaran

Thanks for the link, gabrielesani. I'll have read of it now.
:-)

I just tested this in-game, with a few clients up, and it successfully broadcast the message to all of them.

Has this been fixed, or am I not understanding the issue ?

[url]https://github.com/mangoszero/database/pull/31[/url]

Lines removed.

They were supposed to replace 'tabel' with 'table', but that spelling error no longer exists. I have no idea how both fields in that script ended up as the same.

[quote=Corsol]yes! This is what i mean as second step to fix the problem.
;)[/quote]

Coolio, Corsol :)

Merry xmas, and 'appy New Year :D

[quote=Corsol].. Each vehicle has his own spell to active the dismount action... This spell can be discovered from search on wowhead or other portal that describe the vehicle in object.[/quote]

This might be what Corsol is talking about:

[url]http://www.wowhead.com/spells?filter=na=dismount[/url]

Lots of dismount spells :)

Without looking at the code, I presume something like:

[QUOTE]Deathknight exits mount
cast Dismount Deathcharger [COLOR="#FF0000"]

[quote=Nagilum]Ok, but because i'm a noop at git, how to implement your update ?

Do i just copy and past your addition in to the Opcode.ccp and do a build?

Thats where i get lost...

Nagilum[/quote]

If that is the fix for it, then yes.

Look for those lines in the file ([B]src/game/Server/Opcodes.cpp[/B]), and if the ones highlighted in red are still there, then replace them with the lines highlighted in green. Then rebuild :)


[quote=Nagilum]Noop, Noob i just dont get it :D[/quote]

Welcome to my world :D

-------------------------------------------------------------------------------
I just checked that file, and that code has already been applied.
(I hadn't read the whole of the thread, lol)

Oops, my bad :o

Thanks, shenron :)

I'll check One as well, just-in-case

------------------------------------

Fix PR'd and merged for One
[url]https://github.com/mangosone/database/pull/23#[/url]

Fix PR'd and merged for Two
[url]https://github.com/mangostwo/database/pull/36[/url]

[quote=H0zen]The fix for this issue has been [URL="https://github.com/mangoszero/server/pull/60"]PR`ed[/URL]. The problem was the destruction of various static ACE_TSS objects. A great hint was quite the [URL="http://www.dre.vanderbilt.edu/Doxygen/5.7.9/html/ace/a00737.html#_details"]documentation[/URL] (the note)[/quote]

Nice one, H0zen :D

I am going to create an effigy to your awesomeness and show my devotion by sacrificing MadMax before your altar :)
Or would you prefer the standard virgin maid?

(moments of madness, priceless) :D

Keep up the awesome work, matey :D

Lots of nice work there, necrovoice :-)

When performing an INSERT, you must DELETE the record first, otherwise if it exists, the INSERT will fail.

e.g.
[QUOTE]delete from creature where guid = 151032;
insert into creature (guid, id, map, spawnMask, phaseMask, modelid, equipment_id, position_x, position_y, position_z, orientation, spawntimesecs, spawndist, currentwaypoint, curhealth, curmana, DeathState, MovementType) values(151032,16315,530,1,1,0,0,7051.7,-5811.42,46.5964,2.50734,472,5,0,270,295,0,1);[/QUOTE]



I took the scripts and made a version for One and a version for Two (database record structures are not the same), applied them and they work as intended.

I'll PR them now on One and Two.

[B]ONE:[/B]
[url]https://github.com/mangosone/database/pull/20[/url]

[B]TWO:[/B]
[url]https://github.com/mangostwo/database/pull/33[/url]