Age | Commit message (Collapse) | Author | Files | Lines |
|
- Parameters passed to bindatcmd-invoked labels are now properly
space-delimited.
- This is in order to support strings containing multiple spaces or
containing a trailing space. Previously it was impossible to create a
bindatcmd command that could accept a player name such as
'This name has two spaces' or 'Sir Trailingspace '.
- Added documentation and usage examples, especially wrt space handling.
See doc/script_commands.txt for details.
- NOTE: Your custom atcommand labels may need edits, as we're no longer
trimming multiple sequantial spaces or trailing spaces, in order to
gain more flexibility. It is your care to do that.
- Special thanks to Dastgir, jaBote.
Signed-off-by: Haru <haru@dotalux.com>
|
|
- This reverts commit 4ac673941714032ada6d26fb60936ec510bbe496.
- The commit breaks some legacy code. It'll be pushed again once some
things are fixed.
Signed-off-by: Haru <haru@dotalux.com>
|
|
checkquest deprecated; Use questprogress instead for a more logical
quest log checking command.
getbrokencount command added to get amount of broken equipment.
setdragon and setmadogear deprecated; use setriding instead.
setriding now handles all combat mounts.
|
|
- Please use the values from enum scstart_flag when calling
status_change_start or related functions.
- This also applies to the sc_start script commands.
Signed-off-by: Haru <haru@dotalux.com>
|
|
its so frustrating that rathena already having this for month, but we don't have =/
https://github.com/rathena/rathena/commit/eba1539
|
|
MAX_REFINE has different value for renewal(20) and non-renewal(10)
and some server even adjust this value
so its better to add this constant so there's no need to add a configuration when making custom refiner scripts
|
|
https://github.com/HerculesWS/Hercules/commit/a06e53479131c25ce2343c431c688c57b3bfa2c0
to mimic the *sc_start documentation to rathena
and fixed return 0; into return true;
thx to Litro for bringing up
http://hercules.ws/board/topic/6997-debug-whats-wrong/
|
|
Updated script command 'sc_start' to allow for more variables and options rAthena 17326 (bug:8217)
Fix Cart Tornado damage to official
elemental crashing/memory error (thanks to Ind)
|
|
- When under Mind Break Matk Effect and Mdef reduction should not show on ALT Q(bug:7915)
- Plagiarism should not copy New 3rd class skills. Thanks to OmegaRed (bug:8118)
- Update Runstone Lux Anima to official formula (bug:8285)
- Add official formula for Demonic Fire
- Update damage formula for Unlimit, apply Def and Mdef to 1 during the skill and adjust the cooldown to official. (bug:8227)
- Flash Combo should not consume hp-sp and the skill not ignore the distance.
- Fix a bug with Full Throttle that not consuming sp correctly and adjust the cooldown to official.
- All WoE items are inactive outside WoE or PVP
- Add bonus3 bSPVanishRate for Vellum Items.
- Script Vellum Items.
- Add 2 new bonus bSetDefRace and bSetMDefRace for the Vellum Items. (rAthena c046668034)
- Fixed Bowling Bash, Brandish Spear, Grand Darkness and Meteor Assault reducing def during cast; the only skill that does this officially is Grand Cross (rAthena 694719048e)
- Implemented official Bowling Bash with all its special behaviors including the gutter line
* As many servers probably want to remove the gutter line problem, it is configurable; just adjust the bowling_bash_area setting in skill.conf (Bug:4209)
- Sonic Blow now has a fixed range of 1, even for monsters; a monster can't use this skill if you tank it from farther away (rAthena a15cee063d)
|
|
http://hercules.ws/board/index.php?app=tracker&showissue=8247
Corrected *getexp documentation
Documented and changed pc_gainexp type
|
|
Updated *strcharinfo documentation in order to follow 1a477c4a628decb330b8083bbd4625ffae26c709
|
|
Just reflecting correct behavior of addtimercount script command, which expects the parameters in the reverse order the documentation formerly said.
|
|
Fixed documentation errors:
-> Strings are always delimited by double quotes ("), not single qotes(');
-> Errors on optional parameters of monster and areamonster script commands;
-> Also fixed curlies for optional parameters in sprintf and sscanf.
|
|
- Updated item_db.conf in preparation for the source edits.
Signed-off-by: Haru <haru@dotalux.com>
|
|
|
|
http://hercules.ws/board/tracker/issue-8182-cloaking-on-ice-wall/
Fixed minor typo on script_commands
|
|
- Updated examples to a more modern syntax
- Corrected examples using the wrong case for script commands (fixes
bugreport:8192 thanks to jaBote)
http://hercules.ws/board/tracker/issue-8192-camelcase-in-scripting-documentation/
- Reworded or rewritten some misleading/no longer correct notes to match
the current script engine behavior.
- Marked some commands as deprecated and suggested alternatives, to
discourage their use in new scripts (set, goto, jump_zero, menu)
- Changed use of temporary character variables into scope variables in
examples, where appropriate.
Signed-off-by: Haru <haru@dotalux.com>
|
|
Update atcommands.txt
|
|
Compartmentalized npc file processing
Corrected event initalisation order in npc_reload
Changed loading order, now item db constants are loaded before npc processing in order to avoid
labels overwriting constants values, like Aegis Names
Added information regarding reloading scripts in /doc/script_commands.txt
|
|
Added enumered returns to npc_cashshop_buy and to npc_cashshop_buylist
|
|
@newmount => @mount2
|
|
*http://hercules.ws/board/tracker/issue-8175-missing-delequip/
*Documentation wrote by @emistry
#Added return values for *delequip
|
|
#Fixed issue where a corrupted map cache would lead to a crash
*Moved Big-endian compatibility functions to common/utils.h
#Fixed issue 8162
*http://hercules.ws/board/tracker/issue-8162-loadnpc-doesnt-trigger-oninit-of-duplicate-npcs/
*Added options to npc_parse_duplicate
#Fixed issue 8169
*http://hercules.ws/board/tracker/issue-8169-script-command-guildskill-skill-idlevel-not-working-as-intended/
*Changed *guildskill behavior, now it behaves exactly as depicted in the documentation
*Updated *guildskill documentation
#Added missing GBI types to mapif_parse_GuildBasicInfoChange now it's possible to change guild exp, lv, skill point and skill information
#GeoIP revamp
*GeoIP module was partially rewritten
*Added several data checks to prevent corruption and crashes
*Updated GeoIP database
*See https://github.com/maxmind/geoip-api-c/blob/master/libGeoIP/GeoIP.c for more information
#Added packetver checks regarding NST_MARKET
*Now *tradertype warns if user is trying to use this feature with older clients
|
|
|
|
http://hercules.ws/board/tracker/issue-8140-toucan-npc-error-message/
That error usually happens when the player clicked on a NPC that has the view id
of a mob, to activate this kind of npc it's needed to be in a 2,2 range
from it. If the OnTouch area of a npc, coincides with the 2,2 range of
another it's expected that the OnTouch event be put first in stack, because
unit_walktoxy_timer is executed before any other function in this case.
So it's best practice to put an 'end;' before OnTouch events in npcs that
have view ids of mobs to avoid this kind of error.
Also updated script_commands.txt to add this recomendation.
|
|
http://hercules.ws/board/tracker/issue-8131-sc-chat/
-Documented SC_NOCHAT
-Corrected the way that pc_setparam was handling SP_MANNER
-Added documentation on *sc_status regarding SC_NOCHAT, to mute a user via script you should set Manner
|
|
(fixed issue: 7916 http://hercules.ws/board/tracker/issue-7916-wrong-comment-in-pc-statusup2/)
Now pc_statusup2 returns stat increase amount as stated in previous documentation
Updated *statusup documentation it was wrong
Added last update in upgrade index
@console.c/.h
Documented partially
Now two different parsing categories can have functions with same name
e.g.
- server exit
- sql exit
|
|
Fix hunting mission not able to take mission after a character has deleted
http://hercules.ws/board/tracker/issue-8104-hunting-missiontxt-delete-character/
Clarify Weight and MaxWeight unit as in 0.1
http://hercules.ws/board/tracker/issue-7836-weight-and-maxweight/
|
|
Fix certain type of bonus can actually trigger with weapon skills
http://hercules.ws/board/tracker/issue-7897-bug-or-bad-description/
Fix duplicate sample typo
http://hercules.ws/board/tracker/issue-4984-wrong-test-duplicate-npc-sample/
skill script command doesn't has type 4
http://hercules.ws/board/tracker/issue-7565-skill-script-command-invalid-flag/
|
|
- The script command pcre_match and the operator ~= will return the
number of regular expression matches in a given string (roughly
equivalent to the php function preg_match or the perl operator =~)
- The operator ~! is the opposite of ~= (roughly equivalent to the perl
operator !~)
- See script_commands and npc/custom/test.txt for more information.
Signed-off-by: Haru <haru@dotalux.com>
|
|
A bonus. Fixed unitwalk's non-x/y possibility, NPCs can now be set to chase players around for as long as they keep walking, which opens a number of possibilities e.g. help npc y get thru z.
Signed-off-by: shennetsind <ind@henn.et>
|
|
Added simple overflow check in pc_heal to fix issue: 8082
http://hercules.ws/board/tracker/issue-8082-itemheal-kills-with-high-rand-value/
|
|
- - Fixed some structuring of the file
- - Cleaned up some sentences
|
|
|
|
Added a new optional parameter to specify upgrade quantity, so its consistent with downrefitem which provides a optional parameter to specify downgrade quantity.
As proposed by kyeme in
http://hercules.ws/board/topic/2403-successrefitem/
Signed-off-by: shennetsind <ind@henn.et>
|
|
Thanks to KeyWorld.
Signed-off-by: shennetsind <ind@henn.et>
|
|
- Added option to disable hash check by GM group_id (specify 'disabled'
as hash for a certain group_id to let them log in with any client,
even if client_hash_check is enabled
- Updated and reworded related documentation for the feature, following
user feedback about certain parts being incomplete or confusing.
|
|
- Added script command strtol (conforming to the ISO C90 function)
- Modified script command axtoi to internally use strtol instead of an
unnecessary own implementation.
- Fixed sscanf behavior to conform to the C specifications in case the
input string is empty. It now correctly returns -1, or 0 if the format
string is also empty. Fixes bugreport:8009, thanks to AnnieRuru -
http://hercules.ws/board/tracker/issue-8009-sscanf-should-return-1-if-the-string-field-is-an-empty-string/
Signed-off-by: Haru <haru@dotalux.com>
|
|
Fixed scripting deficiency of being unable to tell whether a user is assigned to a specific instance or not ( 20bdc01fa687b174a732be4483ddea4982d67ce9 removed that ability ), because users can be assigned to more than a single instance on hercules, the previous instance_id(<va>) isn't viable, so I'm introducing has_instance2("<map_name>"), returns the instance id of that map for the user (as long as the user is assigned to a instead with that map), -1 upon failure -- valid instance ids are >= 0.
Signed-off-by: shennetsind <ind@henn.et>
|
|
- Updated array size info from 127 to the new +4billion
- Improved trader type documentation, thanks to emistry for pointing out some deficiencies!
Signed-off-by: shennetsind <ind@henn.et>
|
|
Signed-off-by: Haru <haru@dotalux.com>
|
|
Info:
http://hercules.ws/board/topic/3614-christmas-patch-gifto/
Signed-off-by: shennetsind <ind@henn.et>
|
|
- [ This commit is part of a larger script engine related update ]
- Operator precedence rules now closely follow those of languages such
as C and derivates/related (C++, Java, PHP, etc.)
- Please note that if you had custom scripts with non parenthesized
expressions containing bitwise |, &, ^ operators, they may behave
incorrectly now (or perhaps they were already behaving incorrectly,
since the previous behavior was undocumented).
- Added an up to date operator precedence/associativity table in the
script documentation.
- Added an operator/keyword self-test script in the npc/custom folder,
in case if may be of some use for future regression-testing.
|
|
- Special thanks to ossi0110, kyeme
- Fixes bugreport:7821
http://hercules.ws/board/tracker/issue-7821-missing-item-dbconf-columne/
Signed-off-by: Haru <haru@dotalux.com>
|
|
http://hercules.ws/board/topic/2954-item-db-file-structure-overhaul
- Item db was changed to libconfig format.
- This new format is larger than the original format, but it's less
subject to conflicts when some items are edited and the file gets
updated.
- It is no longer necessary to specify fields with no value, and only
the actually used fields are specified.
- Items scripts (especially the long ones) are made more readable by
splitting them into multiple lines, with proper indentation.
- A converter perl script is provided in the tools directory (and a link
to the web-based version of the same script canbe found in the above
forum topic)
- All of this was made possible thanks to Ind, Yommy.
Signed-off-by: Haru <haru@dotalux.com>
|
|
- Added a note about variables and command scripts case sensitivity.
Even though the engine still accepts variable, function and command
names in a case insensitive way, it is discouraged to rely on such
behavior. Please update your custom scripts as soon as possible.
Signed-off-by: Haru <haru@dotalux.com>
|
|
Special Thanks to Haruna
Signed-off-by: shennetsind <ind@henn.et>
|
|
Signed-off-by: shennetsind <ind@henn.et>
Conflicts:
conf/messages.conf
src/common/mmo.h
src/map/pc_groups.c
src/map/pc_groups.h
|
|
- Warps don't require a facing, so their duplicates shouldn't either.
- Credits to jaBote.
Signed-off-by: Haru <haru@dotalux.com>
|
|
- Small tweaks to the command, fixed merge conflicts
Signed-off-by: Haru <haru@dotalux.com>
|