Config outdated?

[DrKN]

The following self condition seems to be replaced by whenEquipped and whenNotEquipped

Code: Select all

	equip_topHead
	equip_midHead
	equip_lowHead
	equip_leftHand
	equip_rightHand
	equip_leftAccessory
	equip_rightAccessory
	equip_robe
	equip_armor
	equip_shoes
	equip_arrow

Should we delete them and modify it?

I cant search the checking of these lines am i correct or not?

[sofax222]

In the Msic.pm, there is still the code lines:

Code: Select all

	# check skill use SP if this is a 'use skill' condition
	if ($prefix =~ /skill|attackComboSlot/i) {
		my $skill = Skill->new(auto => $config{$prefix});
		return 0 unless ($char->getSkillLevel($skill)
						|| $config{$prefix."_equip_leftAccessory"}
						|| $config{$prefix."_equip_rightAccessory"}
						|| $config{$prefix."_equip_leftHand"}
						|| $config{$prefix."_equip_rightHand"}
						|| $config{$prefix."_equip_robe"}
						);
		return 0 unless ($char->{sp} >= $skill->getSP($config{$prefix . "_lvl"} || $char->getSkillLevel($skill)));
	}

for "skill" and "attackComboSlot" blocks !

[DrKN]

but read it carefully,
the checking condition is nonsense.
why attackComboSlot fail when part of the equips are filled?

[DrKN]

Sorry i have to correct what i have said:
These fields are entered so the attackCombo or skill use is always success.
Why? this is nonsense at all

[EternalHarvest]

That's a check if you have specified skill at all (it allows to ignore blocks with skills you don't currently have). Some skills are available via equipping something, so if a block has instructions to equip something in order to cast a skill, this check would be skipped.