Hi Ramza,
I've just detected a bug (I'm on MV) :
With the Barehanded Attack option set to OFF, if my ambidextrous actor has only one hand equipped as a mainhand, he attacks with a punch using the offhand.
I tested this on a blank project, with only YEP_BattleEngineCore and Ramza_dualWield, and didn't touch any parameters other than setting Barehanded Attack to OFF.
This is the last version : v2.75
Good catch.
The problem was with a function that returned whether or not an actor was currently dual wielding. For some reason, when I rewrote that function on a previous update, I completely neglected to take that plugin parameter into account, so it was always considered dual wielding, whether or not the parameter was set.
I have uploaded the fix for that just a few minutes ago.
I've just tested your modification but there's another bug:
- when you equip a Hand alone, everything's fine, just one sword strike.
- But if you equip an Offhand weapon only, only one blow is triggered, but it's the MainHand punch, and the Offhand weapon is useless.
EDIT: To be more precise, it's the Offhand weapon's animation that's played, but the Mainhand's damage is applied
(I tested it on a blank project to be sure)...
Alas, that one is intended.
The intent was that you could turn off an extra barehanded attack if your offhand was empty, mostly for two handed weapon use, honestly, but if you're using no weapons at all, in the normal engine, the actor would still make a barehanded attack, so his main hand should always be considered armed, even if he has a weapon in his offhand.
Changing that in particular would be difficult because the plugin was not designed with that in mind. Attacks always come from the main hand first, the only exception being if the offhand is locked during a skill. To change it permanently would be much more involved than the change I did to make the offhand empty attack not trigger.
~Ramza
