PolkaWar is a blockchain based NFT gaming platform and marketplace.
Inspired by the recent crypto NFT and gaming narratives, PolkaWar will combine and synergize them both to build an attractive platform.
The game allows every player to build a character and participate in battle against other players.
Further, PolkaWar will also include an NFT marketplace, where players can sell their items, weapons and upgraded equipment for crypto. Some of the items can even be redeemed for real world replicas.
Unable to find manual contract audit (e.g. Certik, PeckShield, Solidity...)
Redundant expression "this (@openzeppelin/contracts/GSN/Context.sol#13)" inContext (@openzeppelin/contracts/GSN/Context.sol#4-16)
Remove redundant statements if they congest code but offer no value.
Additional information: link
ERC20Detailed.constructor(string,string,uint8).name (@openzeppelin/contracts/token/ERC20/ERC20Detailed.sol#12) shadows:
- ERC20Detailed.name() (@openzeppelin/contracts/token/ERC20/ERC20Detailed.sol#19-21) (function)
ERC20Detailed.constructor(string,string,uint8).symbol (@openzeppelin/contracts/token/ERC20/ERC20Detailed.sol#12) shadows:
- ERC20Detailed.symbol() (@openzeppelin/contracts/token/ERC20/ERC20Detailed.sol#24-26) (function)
ERC20Detailed.constructor(string,string,uint8).decimals (@openzeppelin/contracts/token/ERC20/ERC20Detailed.sol#12) shadows:
- ERC20Detailed.decimals() (@openzeppelin/contracts/token/ERC20/ERC20Detailed.sol#29-31) (function)
Rename the local variables that shadow another component.
Additional information: link
Context._msgData() (@openzeppelin/contracts/GSN/Context.sol#12-15) is never used and should be removed
SafeMath.div(uint256,uint256) (@openzeppelin/contracts/math/SafeMath.sol#41-43) is never used and should be removed
SafeMath.div(uint256,uint256,string) (@openzeppelin/contracts/math/SafeMath.sol#46-53) is never used and should be removed
SafeMath.mod(uint256,uint256) (@openzeppelin/contracts/math/SafeMath.sol#56-58) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (@openzeppelin/contracts/math/SafeMath.sol#61-64) is never used and should be removed
SafeMath.mul(uint256,uint256) (@openzeppelin/contracts/math/SafeMath.sol#26-38) is never used and should be removed
Remove unused functions.
Additional information: link
Pragma version^0.6.0 (@openzeppelin/contracts/GSN/Context.sol#1) allows old versions
Pragma version^0.6.0 (@openzeppelin/contracts/math/SafeMath.sol#1) allows old versions
Pragma version^0.6.0 (@openzeppelin/contracts/token/ERC20/ERC20.sol#1) allows old versions
Pragma version^0.6.0 (@openzeppelin/contracts/token/ERC20/ERC20Burnable.sol#1) allows old versions
Pragma version^0.6.0 (@openzeppelin/contracts/token/ERC20/ERC20Detailed.sol#1) allows old versions
Pragma version^0.6.0 (@openzeppelin/contracts/token/ERC20/IERC20.sol#1) allows old versions
Pragma version>=0.6.0 (contracts/PolkaWar.sol#1) allows old versions
solc-0.6.2 is not recommended for deployment
Deploy with any of the following Solidity versions: 0.5.16 - 0.5.17, 0.6.11 - 0.6.12, 0.7.5 - 0.7.6 Use a simple pragma version that allows any of these versions. Consider using the latest version of Solidity for testing.
Additional information: link
Different versions of Solidity is used:
- Version used: ['>=0.6.0', '^0.6.0']
- ^0.6.0 (@openzeppelin/contracts/GSN/Context.sol#1)
- ^0.6.0 (@openzeppelin/contracts/math/SafeMath.sol#1)
- ^0.6.0 (@openzeppelin/contracts/token/ERC20/ERC20.sol#1)
- ^0.6.0 (@openzeppelin/contracts/token/ERC20/ERC20Burnable.sol#1)
- ^0.6.0 (@openzeppelin/contracts/token/ERC20/ERC20Detailed.sol#1)
- ^0.6.0 (@openzeppelin/contracts/token/ERC20/IERC20.sol#1)
- >=0.6.0 (contracts/PolkaWar.sol#1)
Use one Solidity version.
Additional information: link
totalSupply() should be declared external:
- ERC20.totalSupply() (@openzeppelin/contracts/token/ERC20/ERC20.sol#17-19)
balanceOf(address) should be declared external:
- ERC20.balanceOf(address) (@openzeppelin/contracts/token/ERC20/ERC20.sol#22-24)
transfer(address,uint256) should be declared external:
- ERC20.transfer(address,uint256) (@openzeppelin/contracts/token/ERC20/ERC20.sol#27-30)
allowance(address,address) should be declared external:
- ERC20.allowance(address,address) (@openzeppelin/contracts/token/ERC20/ERC20.sol#33-35)
approve(address,uint256) should be declared external:
- ERC20.approve(address,uint256) (@openzeppelin/contracts/token/ERC20/ERC20.sol#38-41)
transferFrom(address,address,uint256) should be declared external:
- ERC20.transferFrom(address,address,uint256) (@openzeppelin/contracts/token/ERC20/ERC20.sol#44-48)
increaseAllowance(address,uint256) should be declared external:
- ERC20.increaseAllowance(address,uint256) (@openzeppelin/contracts/token/ERC20/ERC20.sol#50-53)
decreaseAllowance(address,uint256) should be declared external:
- ERC20.decreaseAllowance(address,uint256) (@openzeppelin/contracts/token/ERC20/ERC20.sol#56-59)
burn(uint256) should be declared external:
- ERC20Burnable.burn(uint256) (@openzeppelin/contracts/token/ERC20/ERC20Burnable.sol#9-11)
burnFrom(address,uint256) should be declared external:
- ERC20Burnable.burnFrom(address,uint256) (@openzeppelin/contracts/token/ERC20/ERC20Burnable.sol#14-16)
name() should be declared external:
- ERC20Detailed.name() (@openzeppelin/contracts/token/ERC20/ERC20Detailed.sol#19-21)
symbol() should be declared external:
- ERC20Detailed.symbol() (@openzeppelin/contracts/token/ERC20/ERC20Detailed.sol#24-26)
decimals() should be declared external:
- ERC20Detailed.decimals() (@openzeppelin/contracts/token/ERC20/ERC20Detailed.sol#29-31)
Use the external attribute for functions never called from the contract.
Additional information: link
Unable to find Youtube account