Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fixes in gameStore #186

Merged
merged 1 commit into from Feb 6, 2017
Merged

fixes in gameStore #186

merged 1 commit into from Feb 6, 2017

Conversation

jlcvp
Copy link
Collaborator

@jlcvp jlcvp commented Feb 6, 2017

verifying cap before add items and avoid removing coins if the items were not delivered.

fixes #176

verifying cap before add items and avoid removing coins if the items were not delivered
@brewsterl brewsterl merged commit 639725b into mattyx14:otxserv3 Feb 6, 2017
@jlcvp
Copy link
Collaborator Author

jlcvp commented Feb 6, 2017

dude, i didn`t understand what you said. sorry.

My code checks for cap with stackable and non-stackable items.

I`ve also fixed an amateurism in the code where stackable items were added 1 by 1 inside a for loop.

@Eduardo-Dantas
Copy link

Eduardo-Dantas commented Feb 6, 2017

Ooooh sorry mate, I had not looked at the updated script code. Perfect -
Much better than the seams I had made. hahahaha

Can you verify the error in soul for news vocations and tibia coin in market?

mattyx14 added a commit that referenced this pull request Mar 12, 2017
Added function to find mount by name (TFC)
Drop commands support in favor of talkactions (TFT)
Fix wrong return value from reload (TFT)
Fix CMake command for source directory (TFT)
Fix memory leak in talkactions (TFC)
Fix setMaster behaviour (TFC)
Fix Player:getClosestFreePosition behavior (TFC)
Fix removeMoney unintended behavior (TFC)
Fix #175, #183, #186 #205 to ##208 (OTXS-C)
Lazily load monsters from XML (TFT)
Move !sellhouse to talkactions (TFT)
Move WaitingList attributes to private implementation (TFT)
Move levitate to Lua (TFT)
Storing TalkAction instances instead of pointers (TFC)

Ty all :D
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FIX] Error in store in-game
3 participants