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

Ore Purifier should double all income sources. #37

Open
ghost opened this issue Oct 30, 2015 · 6 comments · Fixed by OpenRA/ra2#618
Open

Ore Purifier should double all income sources. #37

ghost opened this issue Oct 30, 2015 · 6 comments · Fixed by OpenRA/ra2#618

Comments

@ghost
Copy link

ghost commented Oct 30, 2015

The effect of the building is (supposed to be) to double income from all sources. Currently it appears to do nothing.

Revision 41d9a20.

@phrohdoh phrohdoh changed the title Ore Purifier should purify ore Ore Purifier should double all income sources. Oct 30, 2015
@phrohdoh
Copy link
Owner

Renamed to better reflect what the logic should actually do.

@GraionDilach
Copy link

It was giving 125% actually, not doubling. See [General]PurifierBonus= in rules https://github.com/Phrohdoh/oramod-ra2/blob/master/extra-files/rules.ini#L251.

@ghost
Copy link
Author

ghost commented Oct 30, 2015

Oops, so it was. Why'd I think doubling...

Suppose you get $100 from a source. How should the CashTick be displayed with the Purifier?:

  1. Display +125 instead.
  2. Display +100, and also +25 at the purifier.
  3. Display +100 (+25).

@MustaphaTR
Copy link

I think second is better.

@GraionDilach
Copy link

2 sounds the most scalable even though it might be confusing to players. Multiple Purifiers worked in the engine (the BuildLimit was there due to balance) and 3 doesn't sound nice with multiple purifiers to me.

@ghost
Copy link
Author

ghost commented Nov 2, 2015

Some thoughts/notes on this:

  • Selling buildings does not grant bonuses. At least one source says that crates and grinders do, however.
  • There are a couple of implementation options:
    • An interface which is notified of cash changes
    • Modifying existing cash/resource-granting traits
    • Probably at least one other decent design

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

Successfully merging a pull request may close this issue.

3 participants