summaryrefslogtreecommitdiff
path: root/src/map/pc.c
AgeCommit message (Collapse)AuthorFilesLines
2017-09-17Split 'View' in two different fields in the item databaseHaru1-52/+53
- The field was used both for sprite view IDs and for weapon or ammunition subtypes. Now the fields 'ViewSprite' and 'Subtype' are used respectively. Signed-off-by: Haru <haru@dotalux.com>
2017-07-30Implementation of RoDEXKirieZ1-0/+3
2017-07-11Removed 'NO MSG' message in clients < 20101123. Follow up ↵Jedzkie1-2/+1
https://github.com/HerculesWS/Hercules/commit/60f9a68f90d495323e9e4533c7a1573ca5d86e71
2017-06-25Add packet 0xabd for send job and base levels to party members.Andrei Karas1-0/+20
Also improve packets send on base level change. Based on commit from rathena: commit 816d8ab2d0934f823a1d26f7fcac244f8245f14b Author: Lemongrass3110 <lemongrass@kstp.at> Date: Mon Jun 5 23:07:09 2017 +0200 Fixed party window for 2017 clients Thanks to @mrdiablo for his help again!
2017-06-25Fix vending list packet for 2016 clients. Based on rathena commit:Andrei Karas1-0/+29
commit 9716233c842f731df3fed5281370e324b5f5f024 Author: Lemongrass3110 <lemongrass@kstp.at> Date: Mon May 8 23:40:05 2017 +0200 Introducing the equip preview window Fixes some bugs for 2016-09-21 onward, where you can preview how a item would like if you put it on. This works on other people's vending and on your own inventory. Thanks to @Rytech2 and @hazimjauhari90
2017-05-27Merge pull request #1722 from Smokexyz/mmo_charstatus-fixHaru1-53/+66
Dismemberment of `storage_data` from `mmo_charstatus`.
2017-05-22Add storage_data reception, parsing and sending to/from the map-server.Smokexyz1-53/+66
Remove loading and saving of storage_data through char.c Re-declaration of structure storage_data as a vector. Re-code of portions in the map-server using storage_data. A new approach is taken by saving the loaded storage data from sql into memory for the duration of the session, thereby removing the need of querying the database to re-load all items everytime a storage save routine is issued from the map-server. Saving of storage items is done through a new function that significantly reduces the number of queries compared to char_memitemdata_tosql(), and therefore run-time speed. This method potentially reduces the number of update and delete queries from MAX_STORAGE (which could be >= 600) times to literally 1. Storage items are stored in a dynamically allocated array and handled accordingly. struct mmo_charstatus size reduces by 34,800 bytes. Update pc_checkitem() with masks for item checks. `sd->state.itemcheck` has been changed to `sd->itemcheck` of type `enum pc_checkitem_types` `battle/items.conf` has been updated to reflect configuration changes. Further updates to assert a successful reception of storage data in related functions.
2017-05-04Merge pull request #1695 from Jedzkie/7-CashFoodFixHaru1-26/+3
Cash Foods Update
2017-05-04Merge pull request #1717 from mekolat/vault2Haru1-0/+15
expose the bank vault to the script engine
2017-04-29Fixes #1721Jedzkie1-1/+1
Signed-off-by: Jedzkie <jedzkie13@rocketmail.com>
2017-04-29Merge pull request #1713 from Jedzkie/17-DropAnnounceHaru1-7/+0
Fixes #994
2017-04-29Removal of rare_drop_announce config.Jedzkie1-7/+0
2017-04-26Merge pull request #1673 from Smokexyz/IROHaru1-55/+65
Fixes an issue where the unequipped items with options do not re-calculate status.
2017-04-25expose bank vault to the script enginegumi1-0/+15
2017-04-25Removal of cashfood_use_interval setting.Jedzkie1-26/+3
2017-04-25Merge pull request #1689 from Jedzkie/2-GiantFlyWingHaru1-3/+33
Implemented official Giant Fly Wing Effect
2017-04-25Merge pull request #1694 from Jedzkie/6-NPCWorkInProgressFixHaru1-4/+5
Added the correct PACKETVER Date for MSG_NPC_WORK_IN_PROGRESS message.
2017-04-24Merge pull request #1704 from Jedzkie/13-AddHealRateHaru1-9/+29
Added AB_HIGHNESSHEAL in skill_add_heal_rate configuration
2017-04-23Implemented official Giant Fly Wing EffectJedzkie1-3/+33
Update the *warpparty script commmand, credits to Dastgir Coding-style fixes & whitespace adjustments in warpparty part in script.c
2017-04-23Added the correct PACKETVER Date for MSG_NPC_WORK_IN_PROGRESS message.Jedzkie1-4/+5
2017-04-23Merge pull request #1697 from Jedzkie/9-PreventLogoutHaru1-2/+4
Made prevent_logout effect on log-in optional
2017-04-21Added AB_HIGHNESSHEAL in skill_add_heal_rate configurationJedzkie1-9/+29
2017-04-20Fix showing NO_MSG messages when using below 20101123 clientsJedzkie1-9/+16
2017-04-21Update GX Poison itemsJedzkie1-6/+0
2017-04-21Update Rune Knight Runes & OresJedzkie1-5/+0
2017-04-21Removed hard coded checks for no use while sitting in Aloevera and Anodyne item.Jedzkie1-5/+1
2017-04-20Made prevent_logout effect on log-in optionalJedzkie1-2/+4
kRO does not prevent players from logging out after connecting to its zone servers. Credits to @secretdataz of rAthena
2017-04-16Code-styling fixes to pc_unequipitem() in pc.cSmokexyz1-50/+49
2017-04-16Fixes an issue where the unequipped items with options do not re-calculate ↵Smokexyz1-8/+19
status.
2017-04-08Merge pull request #1663 from Smokexyz/bow-unequip-fixHaru1-0/+5
Unequip arrows when a bow is unequipped. (RE only)
2017-04-04Unequip arrows when a bow is unequipped. (RE only)Smokexyz1-0/+5
Forces the unequipment of arrows when a bow is unequipped. Based on original PR by @Jedzkie in #1079
2017-03-11Remove a duplicate comment lineHaru1-1/+0
- follow-up to 0f5fdca8945ec3afd5ba2e67a9d414f1ef5565c3 Signed-off-by: Haru <haru@dotalux.com>
2016-12-03Remove the platform-dependent variables from struct status_dataHaru1-6/+6
`struct status_data::class_` was platform dependent since c30bb75ec50624429bff7b4106db4be0fda366d6 in order to silence some compile warnings (uint32 on 64 bit builds and uint16 on 32 bit builds). It's now been changed to the correct type, int32, on all platforms. Since the change has potential to silently break third party code, the variable was renamed to `class`. Signed-off-by: Haru <haru@dotalux.com>
2016-12-03Add function to retrieve the appropriate fame list type for a job mapidHaru1-7/+27
This commit adds the function `pc->famelist_type()` to retrieve the appropriate fame list for a given job (common operation). When the given job ID doesn't have an appropriate fame list, the newly introduced value RANKTYPE_UNKNOWN is returned. Signed-off-by: Haru <haru@dotalux.com>
2016-12-03Change the argument to pc->famerank() to a rank type (instead of job mapid)Haru1-15/+33
For consistency with `pc->addfame()`, the argument to pc->famerank() is now an enum fame_list_type. The function was renamed to `pc->fame_rank()` to avoid silently compiling old non-compliant code. Signed-off-by: Haru <haru@dotalux.com>
2016-12-03Ensure that pc->addfame() increments the correct fame pointsHaru1-13/+30
The function now takes the rank type as argument, rather than guessing it from the character's class. If the wrong fame point type for the current character is requested, the request is ignored. This fixes some (unofficial) edge cases where a Taekwon or an Alchemist refined a signed item, they could obtain rank points. Signed-off-by: Haru <haru@dotalux.com>
2016-12-03Clarify some Job Class vs MapID confusion (2/2)Haru1-123/+156
This commit ensures that `sd->job` and related variables only contain MapIDs (i.e. the serverside optimized values, where High Novice is MAPID_NOVICE | JOBL_UPPER) and are never checked against the client-based Job Class values. As a rule of thumb, from now on, when a variable is named `job` or `jobid`, it is intended to contain a MapID and not a Job Class ID. The type of such variable shall be an unsigned `uint16` or `uint32`. To ensure that related third party code is also verified when this commit is merged, the variable `struct map_session_data::class_ (i.e. `sd->class_`) is renamed to `job`. Some issues in related lines are also fixed, including: - The atcommand `@mount` would not check properly that the requesting character is a 2nd class Swordsman type. - `pc->addfame()` would silently accept invalid MapIDs, sending unknown values to the client (more on this in a subsequent commit). - `pc->famerank()` would not use the passed job as a bitmask, causing the caller to have to mask it beforehand (more on this in a subsequent commit). - The Soul Linker check in TK_JUMPKICK wasn't future-proof (no harm caused currently). - Gunslingers would be able to be targeted by Spiritual Bestowment (`MO_KITRANSLATION`) and Spiritual Sphere Absorption (`MO_ABSORBSPIRITS`) due to a faulty check introduced when the Rebellion class was implemented (causing unintended interaction with Coins). Signed-off-by: Haru <haru@dotalux.com>
2016-12-03Clarify some Job Class vs MapID confusion (1/2)Haru1-103/+106
This commit ensures that `sd->status.class` and related variables only contain Job Classes (i.e. the client-compatible values, where High Novice 4001) and are never checked against the bitmask-based MapID values. As a rule of thumb, from now on, when a variable is named `class`, it is intended to contain a Job Class ID and not a MapID. The type of such variable shall be a signed `int16` or `int`. To ensure that related third party code is also verified when this commit is merged, the variable `struct mmo_charstatus::class_ (i.e. `sd->status.class_`) is renamed to `class`. Some issues in related lines are also fixed, including: - A wrong check in the char server would prevent the correct detection of babies in code related to the family exp sharing. - Baby Arch Bishops would not be affected by Eucharistica. - A wrong check would cause the `questinfo()` script command not to display its information for most classes (except 1-1 classes). - Map IDs and Job Classes were mixed up in `itemdb_jobid2mapid()` and `itemdb_jobmask2mapid()` for 1-1 classes (causing currently no harm, since they just happen to coincide). - The Baby Sura class would not cause parties to be marked as containing a monk type character (for SLS Team Up purposes). - Baby Geneticists would bypass the cart check when trying to equip ammunitions. - Baby Mechanics would bypass the Mado Gear check when trying to equip ammunitions. - Transcendent Shadow Chasers would lose the Stalkers' ability to clone transcendent skills. Signed-off-by: Haru <haru@dotalux.com>
2016-11-20Synchronized the AegisNames to the in-source ITEMID_ constantsHaru1-5/+5
Some missing items were copied to the pre-renewal database. In general, when an item ID is hardcoded in source, it's preferable that the item exists in both databases. Signed-off-by: Haru <haru@dotalux.com>
2016-10-22Implemented SC_BITESCAR:Dastgir Pojee1-0/+3
Cannot be reset by dispell. Cannot be healed by item/NPC once in BITESCAR. Heal Skill would end the BiteScar Effect.
2016-10-22Implemented SU_SPRITEMABLE Skill.Dastgir Pojee1-0/+6
MaxHP + 1000, MaxSP + 100. Show's Spirit of Sea, Land and Life around the sprite when skill is learned.
2016-10-22Implemented SU_HIDE Skill.Dastgir Pojee1-0/+1
Transforms into Bush.
2016-10-22Implemented SU_BASIC_SKILL Skill of SummonerDastgir Pojee1-1/+17
Added Function for Basic Skills check.
2016-10-22Added Summoner Class.Dastgir Pojee1-2/+9
(Only Placeholder, other things related to summoner will follow-up soon) Added SQL-Upgrade: Added `class` column in charlog
2016-09-30Add different fixes for gcc 7 warnings.Andrei Karas1-1/+1
Some possible buffer overflows. Add attribute for mark fallthrough cases. Skipped libconfig warnings.
2016-08-14Update pc_steal_coin parameterEmistry1-6/+7
Added skill_lv as parameter to *pc_steal_coin() Closes #1395 as merged Signed-off-by: Haru <haru@dotalux.com>
2016-07-25Added a missing entry into messages.conf and added some comments where ↵epuncker1-4/+4
missing, ref #1282
2016-07-14Merge pull request #1361 from HerculesWS/coverity-fixesAndrei Karas1-60/+59
Fixed various issues reported by coverity
2016-07-14Fixes warp facing direction.KirieZ1-0/+5
Fixes #1240 Closes #1353 as merged
2016-07-14Changed mmo_charstatus::status_point and mmo_charstatus::skill_point to intHaru1-6/+7
Fixes several -Wsign-compare issues Signed-off-by: Haru <haru@dotalux.com>