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

Refactor world elements cost calculation code #61

Closed
wants to merge 164 commits into from

Conversation

mdziekon
Copy link
Owner

@mdziekon mdziekon commented Aug 31, 2019

Closes #60

Changelog:

  • Improvements
    • Add new abstractions & helpers:
      • Purchase / upgrade / downgrade costs calculation
      • Structures' queue abstraction
    • Initial work on moving towards pages modularisation / componentisation
    • "Development" pages refactor
      • Structures ("compact")
      • Structures ("legacy")
      • Research ("compact")
      • Research ("legacy")
      • Shipyard ("compact")
      • Shipyard ("legacy")
    • "Development" commands improvements (cleaner code, better validation)
      • Structures (both "compact" & "legacy")
      • Research (both "compact" & "legacy")
      • Shipyard (both "compact" & "legacy")
    • "Development" queues manipulation improvements (cleaner, less repetitive code)
      • Structures' queue
      • Research queue
      • Shipyard's queue
  • Bugfixes
    • Incorrect deuterium income calculation when income is negative & storage is full

@mdziekon mdziekon added the pr:enhancement_request All pull requests related to enhancements proposals label Aug 31, 2019
@mdziekon mdziekon added this to In progress in Basic modernisation via automation Aug 31, 2019
(to better emphasise that this is about calculating resources which will be taken from the planetary storages)
mdziekon added a commit that referenced this pull request Feb 6, 2020
…ions-part7

Port minor fixes to the codebase from PR #61 | Part 7
mdziekon added a commit that referenced this pull request Feb 9, 2020
…ions-part8

Port minor fixes to the codebase from PR #61 | Part 8
mdziekon added a commit that referenced this pull request Feb 17, 2020
…ions-part9

Port minor fixes to the codebase from PR #61 | Part 9
@mdziekon mdziekon force-pushed the mdziekon/60/worldelem-cost-functions branch from d52747e to 69a47fa Compare February 17, 2020 21:09
mdziekon added a commit that referenced this pull request Feb 19, 2020
…ions-part10

Port minor fixes to the codebase from PR #61 | Part 10
mdziekon added a commit that referenced this pull request Feb 24, 2020
…ions-part11

Port minor fixes to the codebase from PR #61 | Part 11
mdziekon added a commit that referenced this pull request Feb 26, 2020
…ions-part12

Port minor fixes to the codebase from PR #61 | Part 12
mdziekon added a commit that referenced this pull request Feb 28, 2020
…ions-part13

Port minor fixes to the codebase from PR #61 | Part 13
mdziekon added a commit that referenced this pull request Mar 2, 2020
…ions-part14

Port minor fixes to the codebase from PR #61 | Part 14
mdziekon added a commit that referenced this pull request Mar 5, 2020
…ions-part15

Port minor fixes to the codebase from PR #61 | Part 15
mdziekon added a commit that referenced this pull request Mar 5, 2020
…ions-part16

Port minor fixes to the codebase from PR #61 | Part 16
mdziekon added a commit that referenced this pull request Mar 8, 2020
…ions-part17

Port minor fixes to the codebase from PR #61 | Part 16
mdziekon added a commit that referenced this pull request Mar 11, 2020
…ions-part18

Port minor fixes to the codebase from PR #61 | Part 18
mdziekon added a commit that referenced this pull request Mar 20, 2020
…ions-part19

Port minor fixes to the codebase from PR #61 | Part 19
@@ -579,6 +579,11 @@ function calculateRealResourceIncome($resourceKey, &$planet, $params) {
$resourceCurrentAmount = $planet[$resourceKey];
$resourceMaxStorage = ($planet["{$resourceKey}_max"] * MAX_OVERFLOW);

// FIXME: this should be removed as it prevents "negative income" from being
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Issue reported as #88

mdziekon added a commit that referenced this pull request Mar 20, 2020
…ions-part20

Port minor fixes to the codebase from PR #61 | Part 20
@mdziekon
Copy link
Owner Author

Superseded by #63 and all of its related PRs.

@mdziekon mdziekon closed this Mar 20, 2020
@mdziekon mdziekon deleted the mdziekon/60/worldelem-cost-functions branch March 20, 2020 01:27
@mdziekon mdziekon moved this from In progress to Canceled in Basic modernisation Mar 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr:enhancement_request All pull requests related to enhancements proposals
Projects
Basic modernisation
  
Canceled
Development

Successfully merging this pull request may close these issues.

Refactor world elements cost calculation code
1 participant