8488 Commits

Author SHA1 Message Date
Euphy
2acf9a401a Fixed item script for Glorious Holy Avenger. (bugreport:9244)
http://rathena.org/board/tracker/issue-9244-9-glorious-holy-avenger/

Script command 'preg_match' will throw a debug instead of an error if PCRE is not enabled.

Signed-off-by: Euphy <euphy.raliel@rathena.org>
2014-08-22 23:56:08 -04:00
aleos89
4ffe723d8c Merge branch 'master' of https://github.com/rathena/rathena 2014-08-22 17:52:28 -04:00
aleos89
c0ed836e87 * Changed how the Unique ID System saves and creates IDs for items. (Hercules dd49dbc)
- System is now enabled by default.
- All non-stackable items that previously existed will get unique IDs if items::item_check is enabled.
- Added 'getequipuniqueid' script command to get the unique ID of an equipment on a player.
- Don't forget to apply the SQL update!
* Renamed SQL update file that applied to logs.
2014-08-22 17:52:15 -04:00
Cydh Ramdh
416f6d7ba8 - Added error messages for some script commands
- Halt script if no item equipped for 'delequip' just like 'delitem' does

Signed-off-by: Cydh Ramdh <house.bad@gmail.com>
2014-08-23 01:47:30 +07:00
Cydh Ramdh
4fd7e46345 Follow up 70b8b8b, an error for PRE-RE
Signed-off-by: Cydh Ramdh <house.bad@gmail.com>
2014-08-22 23:42:13 +07:00
Cydh Ramdh
70b8b8bc38 Bug Fixes:
- Fixed map-server crashed caused by Wall of Thorn when atatcked by Fire element (with wide range skill unit, bugreport9102). Also reverted some changes in c046668
- Added config conf/battle/skill.conf 'arrow_shower_knockback', as follow up 453b6d0. Direction of Arrow Shower knockback depends on Arrow Shower location

Signed-off-by: Cydh Ramdh <house.bad@gmail.com>
2014-08-22 23:28:48 +07:00
lighta
402170c018 -Update doc/souce_doc.txt for refactoring change
-Add doc/packet_interserv.txt and doc/packet_client.txt, to refer packets
-Upd src inner documentation, map/intif mostly
-Add  common::utils::check_filepath(), fixing bugreport:9131 @loadnpc crash
-Fix pc_level_penalty_mod not using mob_class correctly
-Fix db/import/ for produce, arrow, abra, impro db...
-Upd install script (config.pl) to use mariadb for fedora/centos distrib
2014-08-20 20:58:48 -04:00
Cydh Ramdh
3efe5375f3 - Added check to clear skill unit group that created by NPC/script before the NPC/script unloaded.
- Changed enum for NPC subtype, added prefix NPCTYPE_. See map.h::npc_subtype.
- Changed Channel_Config struct usage to -> extern struct Channel_Config channel_config.
- Added restriction for atcommand that cannot be executed when Player is in autotrade, to prevent map crash. (bugreport:9196) TODO: Check other atcommand or change this behavior.
- Added enum for atcommand restriction with prefix ATCMD_NO. See atcommand.c::e_atcmd_restict.
- Updated atcommand documentations.
- Changed dummy_sd's attributes to group_id & group_level 99, and gives all permissions (PC_PERM_ALLPERMISSION). dummy_sd is being used by 'atcommand' or 'useatcmd' script when no rid/player attached.

Signed-off-by: Cydh Ramdh <house.bad@gmail.com>
2014-08-18 19:59:16 +07:00
aleos89
191f65bdcd Merge pull request #94 from anubisK/else_correction
Follow up to adabedb. Preprocessor cleanup.
2014-08-16 18:35:05 -04:00
aleos89
25969358e4 Merge branch 'master' of https://github.com/rathena/rathena 2014-08-16 16:16:40 -04:00
aleos89
60b0ed9c8b Bug Fixes
* Fixed mouse cursor displaying incorrectly for PvP scenarios. Thanks to Napster. (bugreport:8605)
* Fixed summons getting stuck when master gets too far off screen or changes maps. (bugreport:9152)
* Added a battle_config 'spawn_direction' for keeping a  character's face direction when teleporting/changing maps/logging in. Default is always North (official). (bugreport:8754)
* Cast sensor mobs should always target the caster whenever any skill is used. (bugreport:8555)
* Added a packet version check to char_del_option based on client date. (bugreport:8521)
* Script command query_sql will now return -1 on an empty result or failed result. (bugreport:5937)
2014-08-16 16:15:50 -04:00
Cydh Ramdh
d270047312 * Fixed bugreport:9211. Basilica causes map-server crashed when caster changing map
* Some source docs for skill.c/h

Signed-off-by: Cydh Ramdh <house.bad@gmail.com>
2014-08-17 00:00:19 +07:00
Cristian Gonzalez
5936a67cb3 Enhancement to a if else... sounds better I guess 2014-08-12 20:55:36 +02:00
aleos89
adabedba23 Bug Fixes
* Fixed Shield Spell not being able to be casted. Can now be casted even with skill_unconditional permission while wearing no shield. (bugreport:9198)
* Fixed divest chance of Earth Strain to official. (bugreport:8999)
* Cleaned up ATK and WATK calculation for some skills to only work in pre-renewal.
* Fixed Magnum Break not giving 20% Fire property damage. (bugreport:9199)
2014-08-12 12:33:34 -04:00
Cydh Ramdh
a0cefe9479 Follow up fea3489, just 'char' to 'uint8'
Signed-off-by: Cydh Ramdh <house.bad@gmail.com>
2014-08-11 21:52:41 +07:00
Cydh Ramdh
98c83eeb7f * EXP updates for Renewal
-- Base Level 150 ~ 175 for 3rd classes & Expanded classes (256 ~ 1000 use custom gap 10m)
-- Job Level 50 ~ 60 for 3rd classes (61 ~ 255 use custom gap 5m)
Reference: http://www.divine-pride.net/database/experience

* Follow up fea3489
- Got extra ';' there fea348968c (commitcomment-7332456)
- Reduce w1's length

* Added some newline at end of file

Signed-off-by: Cydh Ramdh <house.bad@gmail.com>
2014-08-11 01:06:19 +07:00
Cydh Ramdh
fea348968c Some CPPCheck clean ups
Signed-off-by: Cydh Ramdh <house.bad@gmail.com>
2014-08-10 23:49:24 +07:00
Cydh Ramdh
453b6d0023 * Bug Fixes:
- Added base level modifier for Great Echo. (bugreport:9179).
- Fixed Max HP/SP rate reduction, should be limited to -100% if the value less than it. (bugreport:9195).
- Fixed Knuckle Arrow (SR_KNUCKLEARROW, Sura) knockback damage bonus behavior. (bugreport:9096).
- Fixed Knuckle Arrow (SR_KNUCKLEARROW, Sura) knockback direction by saving last direction before attacker slides to target first.
- Fixed Overbrand (LG_OVERBRAND_BRANDISH, Royal Guard) knockback damage bonus behavior. (bugreport:9096).
- Fixed knockback direction for Arrow Shower (AC_SHOWER, Archer), now using target to attacker direction. (bugreport:1709).

* Misc:
- Added new flags for Skill Unit to remove hardcoded checks:
--- UF_NOKNOCKBACK: Cannot be knocked back (only unit that can be damaged).
--- UF_REM_CRAZYWEED: Removed if be overlapped by GN_CRAZYWEED.
--- UF_REM_FIRERAIN: Removed if be overlapped by RL_FIRE_RAIN.
- Implemented knockback damage bonus for Round Trip (RL_R_TRIP, Rebellion).

Signed-off-by: Cydh Ramdh <house.bad@gmail.com>
2014-08-10 15:16:20 +07:00
Cydh Ramdh
05d3c6bd43 Fixed skill unit effect issue, wrong used packet for newer client. Causing some skills shows wrong animation. (Partial merge from 3CeAM 2.0 r16, r17, & r18)
Signed-off-by: Cydh Ramdh <house.bad@gmail.com>
2014-08-09 05:16:50 +07:00
Cydh Ramdh
45a8e8fec0 Bug fixes:
- Wrong message when attempt to open boxes while inventory is full. (bugreport:9170)
- Revert a change on d670952, wrong commented line makes misc damage won't be relected. (bugreport:9185)
- Fixed 'devotion_rdamage' doesn't work for Magic Skill. (bugreport:8906)
- Follow up 8a81940, use vending/buyingstore db accessor on searchstore_query() for db_iterator()

Note for 'devotion_rdamage' for magic reflect.
- When this active to reflect magic damage, it still displays dummy damage value on devoted player, and this is custom config at first place. :P

Signed-off-by: Cydh Ramdh <house.bad@gmail.com>
2014-08-04 22:03:43 +07:00
Cydh Ramdh
8a819403c1 Bug fixes & misc:
- Follow up 7fd6776, correcting how to make player sit by autotrade persistance.
- Follow up 7fd6776, pc_setstand() does a check if player can stand or not.
- bugreport:9174, buyingstore_close() doesn't clear buyingstores data on table.
- Fixed "missing" buyingstore search (purchase type search) in searchstore, by adding DBMap *buyingstore_db.
- Added nullpo checks on some buyingstore.c functions.
- Added do_init_buyingstore() and do_final_buyingstore().
- Failing autotrader from pc_setpos(). (Since autotrade doesn't have 'fd'? FIXME later)

Signed-off-by: Cydh Ramdh <house.bad@gmail.com>
2014-08-02 11:07:26 +07:00
aleos89
4eeb10042c Added some options for idletime. (Herules e08c05a) 2014-07-31 09:54:22 -04:00
aleos89
c8758478b4 Bug Fixes
* Fixed the stun effect from Berkana Rune. (bugreport:9057)
* Adjusted Thorn Wall to be a placeable skill and to ignore caster's % damage cards.
* Field Manuals (SC_EXPBOOST and SC_JEXPBOOST) now increase EXP gained from NPC quests (through the getexp script command). (bugreport:9091)
* Lif's Mental Change no longer fully restores her HP and SP upon activation.
* Fixed fakename atcommand not working while disguised. (bugreport:8896) (Hercules 1e492e3)
* Fixed speed atcommand resetting when swapping equipment/changing maps. (bugreport:8915) (Hercules 37fdcba)
* Cleaned up some compile warnings.
2014-07-24 11:45:57 -04:00
Cydh Ramdh
7fd6776194 - SC_BANANA_BOMB_SITDOWN & SC_SITDOWN_FORCE force player to wait unti duration ended before standing. (bugreport:9074) Confirmed by @raynra & reigneil
- Corrected check to finalizing vending/buyingstore autotrader datas, follow up 171e2f2. Thank Stingor
- Autotrade persistance updates: (suggestion from Napster)
--- Kicked player will be removed from vending or buyingstore table. This also avoid 'confusion' lookup when server restarted after the kicked player logged back and does autotrade.
--- feature.autotrade_direction, added -1 value to use last player's direction
--- feature.autotrade_head_direction, added for player's head direction
--- feature.autotrade_sit, added -1 value to use last player's condition (standing/sitting)
--- Please import/execute upgrade_20140723.sql

Signed-off-by: Cydh Ramdh <house.bad@gmail.com>
2014-07-23 13:26:04 +07:00
Cydh Ramdh
2bfe1d6f6e - Fixed bugreport:9146, wrong message for msg 380. Thank Napster
- Changed Monster, Mercenary, & Elemental defense element check, from %10 to %20, to support 10 custom elements. Thank @Atemo
- Updated some arrays in ACMD_FUNC(mobinfo)
- Removed element number limit for each level in db/[pre-]re/attr_fix.txt
- Allowing 'getitembound' and 'getitembound2' to put bound type 0 (no bound) for scripting easier
- Updated doc/mob_db.txt, follow up ca4a0ea
- Updated map_msg_idn.conf

Signed-off-by: Cydh Ramdh <house.bad@gmail.com>
2014-07-19 13:50:46 +07:00
Cydh Ramdh
9e0d78a3c0 Follow up a24da6d, bugreport:9143
Signed-off-by: Cydh Ramdh <house.bad@gmail.com>
2014-07-17 18:46:33 +07:00
Cydh Ramdh
401a7161ce * Follow up 462b2be, wrong check for itemdb_isstackable(). Thank Napster
* Follow up 171e2f2, added battle config 'at_monsterignore' at conf/battle/misc.conf to makes autotrader cannot be attacked. Thank @Lemongrass3110

Signed-off-by: Cydh Ramdh <house.bad@gmail.com>
2014-07-16 22:03:36 +07:00
Cydh Ramdh
171e2f2be9 Bug Fixes:
- Failure when initializing autotrade persitance for PACKETVER < 20120201 or NEW_CARTS, ENABLE_SC_SAVING aren't defined
- Added 'feature.autotrade_open_delay' config at conf/battle/feature.conf to give delay to open vending/buyingstore by autotrade persistency (bugreport:9077)
- Add immediate save after vending/buyingstore opened to avoid missing sc_data (for SC_PUSH_CART) when server terminated before saving routine (bugreport:9077)
- Disable to trigger NPCE_LOGIN (and NPCE_LOADMAP?) for autotraders, avoid vending_reopen & buyingstore_reopen failure (bugreport:9077)
- Set player's state, 'monster_ignore' when autotrade
- EXP calculation overload (bugreport:9127)
- Added macro apply_rate(val,rate) & apply_rate2(val,rate,per)

Thank you all for reports & helps

Signed-off-by: Cydh Ramdh <house.bad@gmail.com>
2014-07-16 17:21:58 +07:00
Cydh Ramdh
171de747e3 Follow up a24da6d, (bugreport:9135)
- Fixed compile error
- Added filters for MSVS
Follow up 237579c

Signed-off-by: Cydh Ramdh <house.bad@gmail.com>
2014-07-16 12:02:45 +07:00
lighta
a24da6d302 -Apply login_refac (from old refac branch) in same way as char_refac, each interface now have his own file.
-Remove _sql from some file.
Thx to Euphy for helping with documentation.
2014-07-15 19:49:32 -04:00
lighta
7ccd13e904 -Fix bugreport:9070-ranged-forced-attack-movement (now when OFFICIAL_PATH is off and clicking outside target, target is being erased, and no longer attempt to walk back to him)
-Cleanup / speedup few stuff, and disabled some slave_timer part for review.
2014-07-15 15:37:42 -04:00
aleos89
237579c957 Bug Fixes
* Corrected some battle calculations for status changes to be applied during skill calculations. (bugreport:9109)
* Fixed Fire Expansion hitting players in all maps. (bugreport:9056)
* Fixed skill duration for Elemental Shield. (bugreport:9078)
* Fixed bSetDefRace and bSetMDefRace not zeroing out values. (bugreport:9108)
* Corrected Inspiration checking for the wrong damage formula. (bugreport:9098)
* Fixed combo type skills causing issues with usable items. (bugreport:9090) (Hercules 0100bae)
* Updated usable skills while riding a Warg. (Hercules cc4c87c)
* Updated the display message for using items that have a delay. (bugreport:9107)
* Fixed an issue when players couldn't attack they also couldn't talk to NPC with mob view IDs. (Hercules 9dd6ee6)
* Reverted 7a9aa7f so that the prompt script command will return 255 on cancel. (bugreport:9104)
2014-07-15 13:47:43 -04:00
Cydh Ramdh
a8f239fc88 Bug Fixes:
- Client stuck while deleting char (bugreport:9101), thank leke
- Wrong check at pc_is_same_equip_index() triggers double damage for two-handed weapon (bugreport:9124), thank Myskelz & Anacondaqq
- Follow up 3d27b7e, Item avail check for npc_cashshop_buy() (bugreport:9122), thank Napster
- Missing enum 'K' for LOG_TYPE_BANK at `zenylog` (bugreport:9122), thank Napster

Signed-off-by: Cydh Ramdh <house.bad@gmail.com>
2014-07-13 12:46:41 +07:00
Cydh Ramdh
0105bc7191 Follow up 4a4bced
Signed-off-by: Cydh Ramdh <house.bad@gmail.com>
2014-07-12 12:50:05 +07:00
Cydh Ramdh
4a4bced1b1 Follow up 3d27b7e
Signed-off-by: Cydh Ramdh <house.bad@gmail.com>
2014-07-12 12:37:27 +07:00
Cydh Ramdh
3d27b7e109 Bug Fixes:
- Kunai Splash (KO_HAPPOKUNAI) ammo check failure crashing map-server (bugreport:9112)
- Fear Breeze status (SC_FEARBREEZE) makes impossible to critical for single hit (bugreport:8197)
- Missing item avail check at NPC Cashshop causing item with item_avail cannot be bought (bugreport:9106)
- Swapped value between 'nk' and 'inf' for MH_LIGHT_OF_REGENE & MH_OVERED_BOOST
- Multi-hit by SC_FEARBREEZE that possibly causing map-server crashed if no ammo equipped
- Thank to all bug reporters
Misc:
- Add enum for clif_damage type, e_damage_type
- Add check for Autotrade Persistency, make sure to kick player who fail open vending/buyingstore

Signed-off-by: Cydh Ramdh <house.bad@gmail.com>
2014-07-12 07:38:32 +07:00
Cydh Ramdh
be9ffaf7e6 - Fixed 'atcommand' script failure for group with 'command_enable: false' (bugreport:9050)
- Replaced 'atcommand_max_stat_bypass' config (conf/gm.conf) to group permission 'bypass_max_stat'

Signed-off-by: Cydh Ramdh <house.bad@gmail.com>
2014-07-07 12:26:07 +07:00
Cydh Ramdh
fcd79d63aa Bug Fixes:
- Fixed wrong check added at e7b654a (bugreport:9095)
- Spirit sphere reduces damage for skill hit num < 0, eg. MO_TRIPLEATTACK has hit num -3. (bugreport:9094)
Misc:
- Removed unused parameters in battle_calc_bg_damage() and battle_calc_gvg_damage()
- Some random src docs.

Signed-off-by: Cydh Ramdh <house.bad@gmail.com>
2014-07-06 20:53:25 +07:00
lighta
79f4600dcd -Fix some CppCheck
-Fix bugreport:9088 (constant not always recognize if stating by number) thx to Hercule is_number
-Add infos on read_constant for check/debug
2014-07-04 11:22:40 -04:00
Cydh Ramdh
cd41c6d028 Macro clean up for Element, Race, Race2, Class, & Size validation check
Signed-off-by: Cydh Ramdh <house.bad@gmail.com>
2014-07-04 19:53:30 +07:00
Cydh Ramdh
a051af4cf1 Bugfixed, bugreport:9085. Wrong var in SP_ADD_ITEMGROUP_HEAL_RATE statement.
Signed-off-by: Cydh Ramdh <house.bad@gmail.com>
2014-07-03 19:58:14 +07:00
lighta
0544f8fde8 -Fix bugreport:9084 getitemX unable to found item by id
-Upd itemdb, itemdb_combo, itemdb_group change key for unsigned int, since they no longer signed
2014-07-03 01:52:15 -04:00
lighta
a2433494cb -Fix bugreport:9079 'skill_chk’ differ in signedness [-Wpointer-sign] 2014-07-02 02:25:56 -04:00
lighta
f90fea2f86 -Fix bugreport:8637 itemcombo bug with card
-Fix weird check reported by cppcheck (Redundant condition at status_readdb_attrfix)
2014-07-02 02:18:37 -04:00
Cydh Ramdh
462b2bebdb * Bug Fixes:
-- Added itemstack check when init autotrader (bugreport:9077)
-- 'getitem' issue, gave unidentified item (bugreport:9075), follow up 62a2813
-- Invalid Def & Mdef reduction of SC_ARMORCHANGE & duration (NPC_STONESKIN & NPC_ANTIMAGIC), should be percentage (bugreport:9076)
-- Invalid Effect duration & Reflect chance of SC_MAGICMIRROR (NPC_MAGICMIRROR)
-- Wrong usage of script_isstring() for script commands: addspiritball, delspiritball, & countspiritball
* Misc:
-- Moved mail send progress to mail.c mail_send() from clif.c clif_parse_Mail_send()
-- Moved hardcoded flood protection when send a mail to 'mail_delay' in misc.conf

Signed-off-by: Cydh Ramdh <house.bad@gmail.com>
2014-07-01 16:29:19 +07:00
Cydh Ramdh
c0326df96b * Added setting for resetting reuse skill delay homunculus, 0x80 (Disabled by default).
* Follow up 22dea1f

PS:
Hmm, as "suggested" by Darkpurple at bugreport:8989, idk it's bug or it should be like that. And, I added the setting option for a whole Homunculus (not for spesific skill/homun id yet), just like "reset reuse skill delay when vaporized" does.

Signed-off-by: Cydh Ramdh <house.bad@gmail.com>
2014-06-28 15:38:49 +07:00
Cydh Ramdh
22dea1f600 Follow up e7b654a. Fixed changes for bugreport:9062 wasn't staged there
Signed-off-by: Cydh Ramdh <house.bad@gmail.com>
2014-06-28 13:04:48 +07:00
aleos89
62a2813adc Bug Fixes
* Fixed Concentration ending when Frenzy is used. (bugreport:8942)
* Fixed Hells Plant hitting Ghost element enemies. (bugreport:8799)
* Forced guild notice changes to save immediately. (bugreport:8961)
* Fixed Cart Tornado damage formula. (bugreport:9065)
* Adjust Amistr's Castling to now switch to any target type. Still needs skill switching support. (bugreport:1784)
* Removed hard coded 1 second skill duration reduction for Spore Explosion.
* Fixed the party window not displaying the correct map name when in an instance. (bugreport:7949)
* Fixed hatching a pet causing client disconnections. (bugreport:9059) (Hercules 533ff4f)
* Updated Kagerou/Oboro skill cast times. (bugreport:9035)
* Removed deprecated negative nameid value for getitem/getitem2.
* Renamed incuvate to incubate throughout source and SQL.
* Added skill ID to the error display for battle_calc_attack to help resolve issues. (bugreport:8876)
* Cleaned and corrected some Sorcerer Elemental skills. (bugreport:9058)
* Cleaned up some more compile warnings.
2014-06-27 15:45:35 -04:00
lighta
b124498263 Fix -WShadow definition warning
Cleanup some leftover
2014-06-27 06:07:50 -04:00
Cydh Ramdh
e7b654aaab - Follow up a8a4425 (bugreport:9062)
- Added check for item group ID usage on item bonus
- Moved this '(sd->class_&MAPID_UPPERMASK) == MAPID_TAEKWON && sd->status.base_level >= battle_config.taekwon_ranker_min_lv && pc_famerank(sd->status.char_id, MAPID_TAEKWON)' to a macro 'pc_is_taekwon_ranker(sd)'

Signed-off-by: Cydh Ramdh <house.bad@gmail.com>
2014-06-26 22:12:19 +07:00