summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorskotlex <skotlex@54d463be-8e91-2dee-dedb-b68131a5f0ec>2006-11-28 19:07:38 +0000
committerskotlex <skotlex@54d463be-8e91-2dee-dedb-b68131a5f0ec>2006-11-28 19:07:38 +0000
commit0b0a39e39623321141c51115214a1b5233fd7b1d (patch)
treee7492cbf23cbf32b0c4fbefda84a71814441a9c7
parentde9ba8dc89aaa1ba7d0d7bab06001e3e284a2d4d (diff)
downloadhercules-0b0a39e39623321141c51115214a1b5233fd7b1d.tar.gz
hercules-0b0a39e39623321141c51115214a1b5233fd7b1d.tar.bz2
hercules-0b0a39e39623321141c51115214a1b5233fd7b1d.tar.xz
hercules-0b0a39e39623321141c51115214a1b5233fd7b1d.zip
- Moved the blocking of casting supportive skills on homunculus from battle_check_target to status_check_skilluse as bct is not invoked unless the skill is offensive (or party/guild only)
- Cleaned up status_calc_bl_sub_hom, fixed recasting dex/str/vit/int bonuses skills making their atk/def/mdef stack with itself over and over. git-svn-id: https://rathena.svn.sourceforge.net/svnroot/rathena/trunk@9350 54d463be-8e91-2dee-dedb-b68131a5f0ec
-rw-r--r--Changelog-Trunk.txt6
-rw-r--r--src/map/battle.c4
-rw-r--r--src/map/status.c42
3 files changed, 30 insertions, 22 deletions
diff --git a/Changelog-Trunk.txt b/Changelog-Trunk.txt
index 132e2fe6e..35a257a31 100644
--- a/Changelog-Trunk.txt
+++ b/Changelog-Trunk.txt
@@ -4,6 +4,12 @@ AS OF SVN REV. 5091, WE ARE NOW USING TRUNK. ALL UNTESTED BUGFIXES/FEATURES GO
IF YOU HAVE A WORKING AND TESTED BUGFIX PUT IT INTO STABLE AS WELL AS TRUNK.
2006/11/28
+ * Moved the blocking of casting supportive skills on homunculus from
+ battle_check_target to status_check_skilluse as bct is not invoked unless
+ the skill is offensive (or party/guild only) [Skotlex]
+ * Cleaned up status_calc_bl_sub_hom, fixed recasting dex/str/vit/int
+ bonuses skills making their atk/def/mdef stack with itself over and over.
+ [Skotlex]
* Updated sql files [Playtester]
* Changing setting the Guild Master from pc_authok to clif_parse_LoadEndAck
so that it will not fail to set the gmaster flag when the first person of a
diff --git a/src/map/battle.c b/src/map/battle.c
index c168f8bdb..efe9e161e 100644
--- a/src/map/battle.c
+++ b/src/map/battle.c
@@ -3226,10 +3226,8 @@ int battle_check_target( struct block_list *src, struct block_list *target,int f
return 0;
}
break;
+ //Valid targets with nothing else specified.
case BL_HOM:
- //Except for the master/itself, hom can't be targetted with non-offensive skills. [Skotlex]
- if (t_bl != s_bl && !(flag&BCT_ENEMY))
- return 0;
break;
//All else not specified is an invalid target.
default:
diff --git a/src/map/status.c b/src/map/status.c
index 306273087..0abff57a9 100644
--- a/src/map/status.c
+++ b/src/map/status.c
@@ -1100,7 +1100,12 @@ int status_check_skilluse(struct block_list *src, struct block_list *target, int
//TODO: Would be nice if this could be used to judge whether the player can or not pick up the item it targets. [Skotlex]
if (status->mode&MD_LOOTER)
return 1;
- else
+ return 0;
+ case BL_HOM:
+ //Can't use support skills on homun
+ //Placed here instead of battle_check_target because support skill
+ //invocations don't call that function.
+ if (skill_num && skill_get_inf(skill_num)&INF_SUPPORT_SKILL)
return 0;
default:
//Check for chase-walk/hiding/cloaking opponents.
@@ -2792,6 +2797,7 @@ void status_calc_bl_sub_pc(struct map_session_data *sd, unsigned long flag)
void status_calc_bl_sub_hom(struct homun_data *hd, unsigned long flag) //[orn]
{
struct status_data *status = &hd->battle_status, *b_status = &hd->base_status;
+ struct status_change *sc = &hd->sc;
int skill = 0;
@@ -2806,11 +2812,6 @@ void status_calc_bl_sub_hom(struct homun_data *hd, unsigned long flag) //[orn]
if(status->hp > status->max_hp)
status->hp = status->max_hp;
}
- if(flag&SCB_VIT)
- {
- flag|=SCB_DEF;
- status->def += (status->vit/5 - b_status->vit/5);
- }
if(flag&(SCB_MAXSP|SCB_INT))
{
flag|=SCB_MAXSP;
@@ -2821,24 +2822,27 @@ void status_calc_bl_sub_hom(struct homun_data *hd, unsigned long flag) //[orn]
if(status->sp > status->max_sp)
status->sp = status->max_sp;
}
- if(flag&SCB_SPEED)
- {
- if (battle_config.slaves_inherit_speed && hd->master)
- status->speed = status_get_speed(&hd->master->bl);
- else
- status->speed = DEFAULT_WALK_SPEED;
+ if(flag&SCB_VIT)
+ { //Since vit affects def, recalculate def.
+ flag|=SCB_DEF;
+ status->def = status_calc_def(&hd->bl, sc, b_status->def);
+ status->def+= (status->vit/5 - b_status->vit/5);
}
- if(flag&SCB_INT) {
+ if(flag&SCB_INT)
+ {
flag|=SCB_MDEF;
- status->mdef += (status->int_/5 - b_status->int_/5);
+ status->mdef = status_calc_mdef(&hd->bl, sc, b_status->mdef);
+ status->mdef+= (status->int_/5 - b_status->int_/5);
}
if(flag&SCB_DEX) {
flag |=SCB_WATK;
- status->rhw.atk += (status->dex - b_status->dex);
+ status->rhw.atk = status_calc_watk(&hd->bl, sc, b_status->rhw.atk);
+ status->rhw.atk+= (status->dex - b_status->dex);
}
if(flag&SCB_STR) {
flag |=SCB_WATK;
- status->rhw.atk += (status->str - b_status->str);
+ status->rhw.atk2 = status_calc_watk(&hd->bl, sc, b_status->rhw.atk2);
+ status->rhw.atk2+= (status->str - b_status->str);
}
if(flag|SCB_WATK && status->rhw.atk2 < status->rhw.atk)
status->rhw.atk2 = status->rhw.atk;
@@ -2852,7 +2856,7 @@ void status_calc_bl_sub_hom(struct homun_data *hd, unsigned long flag) //[orn]
skill = (1000 -4*status->agi -status->dex)
*hd->homunculusDB->baseASPD/1000;
- status->aspd_rate = status_calc_aspd_rate(&hd->bl, &hd->sc , b_status->aspd_rate);
+ status->aspd_rate = status_calc_aspd_rate(&hd->bl, sc , b_status->aspd_rate);
if(status->aspd_rate != 1000)
skill = skill*status->aspd_rate/1000;
@@ -2863,14 +2867,14 @@ void status_calc_bl_sub_hom(struct homun_data *hd, unsigned long flag) //[orn]
if(flag&(SCB_AGI|SCB_DSPD)) {
skill = 800-status->agi*4;
status->dmotion = cap_value(skill, 400, 800);
- status->dmotion = status_calc_dmotion(&hd->bl, &hd->sc, b_status->dmotion);
+ status->dmotion = status_calc_dmotion(&hd->bl, sc, b_status->dmotion);
}
if(flag&(SCB_INT|SCB_MAXSP|SCB_VIT|SCB_MAXHP) && flag != SCB_ALL)
status_calc_regen(&hd->bl, status, &hd->regen);
if(flag&SCB_REGEN)
- status_calc_regen_rate(&hd->bl, &hd->regen, &hd->sc);
+ status_calc_regen_rate(&hd->bl, &hd->regen, sc);
if (flag == SCB_ALL)
return; //Refresh is done on invoking function (status_calc_hom)