Icehawk78 / FrozenCookies

An automated Cookie Clicker tool.
120 stars 171 forks source link

Autofrenzy ON does not work #185

Open shrx opened 8 years ago

shrx commented 8 years ago

The "Autofrenzy ON" option does not enable autoclicking the big cookie during frenzies. I checked with both x7 and x777 frenzies, it didn't trigger in either case.

llamasoft commented 2 years ago

I worked out the root cause of this issue: the clicking upgrades aren't taken into consideration.

To determine if a frenzy is active, the clickBuffBonus() function looks at the multClick attribute of the currently active buffs. However, most regular frenzies only have a multCpS attribute, buffing the overall CpS. If you have no Clicking Upgrades, these multCpS frenzies do absolutely nothing to boost your cookies per click. Conversely, if you do have Clicking Upgrades then the buff to your overall CpS increases your cookies per click.

My rough suggestion would be making hasClickBuff into something like this:

function hasMouseUpgrades() {
  // TODO
}

function hasClickBuff() {
  return Game.hasBuff("Cursed finger") || clickBuffBonus() > 1 || (cpsBonus() > 1 && hasMouseUpgrades());
}

I'll open a PR for this in a few days unless the author has a more robust solution in mind.