Issue information

Issue ID
#4466
Status
Duplicate
Severity
None
Started
Hercules Elf Bot
Oct 5, 2010 20:08
Last Post
Hercules Elf Bot
May 4, 2012 6:15
Confirmation
Yes (1)
No (0)

Hercules Elf Bot - Oct 5, 2010 20:08

Originally posted by [b]Inquisetor90[/b]
http://www.eathena.ws/board/index.php?autocom=bugtracker&showbug=4466

When you cast frenzy, it'll cancel 2H quicken. This is the new renewal effect making it harder to get 190 aspd.

How to implement (my suggestion):
In battle.c, change
CODE
    { "berserk_cancels_buffs",              &battle_config.berserk_cancels_buffs,           0,      0,      1,              },

which is default at 0, minimum at 0, maximum at 1.

to
CODE
    { "berserk_cancels_buffs",              &battle_config.berserk_cancels_buffs,           2,      0,      2,              },

meaning default at 2 (for RE), minimum at 0, maximum at 2.

Now go to status.c, change

CODE
    case SC_BERSERK:
        if(battle_config.berserk_cancels_buffs == 1)
        {
            status_change_end(bl,SC_ONEHAND,-1);
            status_change_end(bl,SC_TWOHANDQUICKEN,-1);
            status_change_end(bl,SC_CONCENTRATION,-1);
            status_change_end(bl,SC_PARRYING,-1);
            status_change_end(bl,SC_AURABLADE,-1);
            status_change_end(bl,SC_MERC_QUICKEN,-1);
//I think it should be included here since all knight buffs are included. [The Quality Maker]
            status_change_end(bl,SC_ENCHANTBLADE,-1);
        }
        break;


to

CODE
    case SC_BERSERK:
        if(battle_config.berserk_cancels_buffs == 1)
        {
            status_change_end(bl,SC_ONEHAND,-1);
            status_change_end(bl,SC_TWOHANDQUICKEN,-1);
            status_change_end(bl,SC_CONCENTRATION,-1);
            status_change_end(bl,SC_PARRYING,-1);
            status_change_end(bl,SC_AURABLADE,-1);
            status_change_end(bl,SC_MERC_QUICKEN,-1);
//I think it should be included here since all knight buffs are included. [The Quality Maker]
            status_change_end(bl,SC_ENCHANTBLADE,-1);
        }
        else if(battle_config.berserk_cancels_buffs == 2)
        { // RE: Not sure whether SC_MERC_QUICKEN is also cancelled. [Inquisetor90]
            status_change_end(bl,SC_ONEHAND,-1);
            status_change_end(bl,SC_TWOHANDQUICKEN,-1);
        }
        break;

Hercules Elf Bot - Dec 17, 2011 4:40

Originally posted by [b]Ind[/b]
would like confirmation / source. thanks

Hercules Elf Bot - Apr 19, 2012 7:59

Originally posted by [b]Angezerus[/b]
Confirmed, also related to [url="http://rathena.org/board/tracker/issue-5492-frenzy-lord-knight/"]http://rathena.org/board/tracker/issue-5492-frenzy-lord-knight/[/url]

The way I see it was fixed in r15848