Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Autofrenzy ON does not work #185

Open
shrx opened this issue May 1, 2016 · 1 comment
Open

Autofrenzy ON does not work #185

shrx opened this issue May 1, 2016 · 1 comment

Comments

@shrx
Copy link

shrx commented May 1, 2016

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
Copy link

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants