From 083285a9a14d10288970992c5edc1c4fcd23ed8c Mon Sep 17 00:00:00 2001
From: zuies <zurabichkhikvadze66@gmail.com>
Date: Fri, 14 Jun 2024 11:24:23 +0300
Subject: [PATCH] fix issues based on scanner reports

---
 contracts/Engagement.sol  | 17 ++++++++++-------
 contracts/IEngagement.sol |  2 ++
 2 files changed, 12 insertions(+), 7 deletions(-)

diff --git a/contracts/Engagement.sol b/contracts/Engagement.sol
index 310afda..f2d4461 100644
--- a/contracts/Engagement.sol
+++ b/contracts/Engagement.sol
@@ -7,7 +7,7 @@ import "@openzeppelin/contracts/utils/Strings.sol";
 import "./IEngagement.sol";
 
 contract Engagement is IEngagement, ERC1155, AccessControl {
-    uint private _counter = 0;
+    uint private _counter;
     bytes32 public constant PROVIDER_ROLE = keccak256("PROVIDER_ROLE");
 
     mapping(uint => string) private _tokenMetadata;
@@ -33,10 +33,11 @@ contract Engagement is IEngagement, ERC1155, AccessControl {
     }
 
     function issue(string memory hash_) external {
-        _tokenMetadata[_counter] = hash_;
-        _mint(msg.sender, _counter, 1, "");
-        emit Issue(msg.sender, _counter);
-        _counter++;
+        uint counterCache = _counter;
+        _tokenMetadata[counterCache] = hash_;
+        _mint(msg.sender, counterCache, 1, "");
+        emit Issue(msg.sender, counterCache);
+        _counter = counterCache + 1;
     }
 
     function mint(
@@ -45,7 +46,7 @@ contract Engagement is IEngagement, ERC1155, AccessControl {
         uint amount,
         bytes memory data
     ) external override validTokenId(tokenId) {
-        if (balanceOf(account, tokenId) > 0) {
+        if (balanceOf(account, tokenId) >= 1) {
             revert MintLimit(account, tokenId);
         }
         _mint(account, tokenId, 1, data);
@@ -93,7 +94,9 @@ contract Engagement is IEngagement, ERC1155, AccessControl {
 
     function supportsInterface(
         bytes4 interfaceId
-    ) public view override(AccessControl, ERC1155) returns (bool) {}
+    ) public view override(AccessControl, ERC1155) returns (bool) {
+        return super.supportsInterface(interfaceId);
+    }
 
     function uri(
         uint tokenId
diff --git a/contracts/IEngagement.sol b/contracts/IEngagement.sol
index eaabdcc..c159b21 100644
--- a/contracts/IEngagement.sol
+++ b/contracts/IEngagement.sol
@@ -1,3 +1,5 @@
+pragma solidity ^0.8.24;
+
 interface IEngagement {
     event Issue(address indexed account, uint indexed tokenId);
     event Mint(address indexed account, uint indexed tokenId, uint amount);