SteemNova: some bugfixes in Research and Buildings

Words
265
Reading
2 min
Listen
Play
8y

SteemNova is an open source space strategy browsergame for Steemians.

Repository

https://github.com/steemnova/steemnova

Bug Fixes

  • What was the issue(s)?
  1. During the research of the first level of a Technology, there's a bug in the link text to research the next level of that technology. The text should show 'Research to the next level 2' instead of 'Research'. The bug is is described in this bug report.
  2. During construction of the first level of a building, there's a bug in the link text to upgrade to the next level of that building. The text should show 'Upgrade 2' instead of 'Build'. It's a similar bug as 1. but there was no extra bug report.
  • What was the solution?
  1. To fix the research issue, I had to fix the Smarty code in the following template: styles/templates/game/page.research.default.tpl
    In this file were two wrong if conditions, which choose the text to display (one for the green text when research is possible and one for the red text when research is not possible). Instead of just check the actual level of the research, a check of the level to research is needed.
    Screenshot of the above changes:
    bugfix_research.png
    https://github.com/steemnova/steemnova/pull/154/files#diff-6239085e8c839a693a2dd83b6b9e2e87
  2. To fix the building issue, I had to fix the Smarty code in the following template:styles/templates/game/page.buildings.default.tpl
    In this file were two wrong if conditions, which choose the text to display (one for the green text when building is possible and one for the red text when building is not possible). Instead of just check the actual level of the building the check of the level to build is needed.
    Screenshot of the above changes:
    bugfix_buildings.png
    https://github.com/steemnova/steemnova/pull/154/files#diff-5f9b47849684313cee8c35b6f1eb21e9

GitHub Account

https://github.com/cryptoriddlerUIO

SteemNova: some bugfixes in Research and Buildings | Ecency