Fix linter review

bitmark-inc · Dec 13, 2024 · a8851ad · a8851ad
1 parent 2eb6555
commit a8851ad
Showing 1 changed file with 11 additions and 11 deletions.
diff --git a/contracts/SeriesIndexer.sol b/contracts/SeriesIndexer.sol
@@ -1,9 +1,9 @@
 // SPDX-License-Identifier: MIT
 pragma solidity ^0.8.0;
 
-import "@openzeppelin/contracts/access/Ownable.sol";
+import "@openzeppelin/contracts/access/Ownable.sol" as Ownable;
 
-contract SeriesIndexer is Ownable {
+contract SeriesIndexer is Ownable.Ownable {
     // Counter
     uint256 private nextSeriesID = 1;
     uint256 private nextArtistID = 1;
@@ -30,7 +30,7 @@ contract SeriesIndexer is Ownable {
     mapping(uint256 => mapping(uint256 => uint256)) private artistPendingCoArtistRequestIndex;// artistID => SeriesID => Index
 
     // Events
-    event ArtistAddressUpdated(uint256 indexed artistID, address oldAddress, address newAddress);
+    event ArtistAddressUpdated(uint256 indexed artistID, address indexed oldAddress, address indexed newAddress);
     event SeriesIndexed(
         uint256 indexed seriesID,
         uint256[] artistIDs,
@@ -178,8 +178,8 @@ contract SeriesIndexer is Ownable {
     // ------------------------
 
     function addSeries(
-        string memory metadata,
-        string memory tokenIDsMapCID
+        string calldata metadata,
+        string calldata tokenIDsMapCID
     ) external returns (uint256) {
         address[] memory artistAddrs = new address[](1);
         artistAddrs[0] = msg.sender;
@@ -188,8 +188,8 @@ contract SeriesIndexer is Ownable {
 
     function ownerAddSeries(
         address[] memory artistAddrs,
-        string memory metadata,
-        string memory tokenIDsMapCID
+        string calldata metadata,
+        string calldata tokenIDsMapCID
     ) external onlyOwner returns (uint256) {
         require(artistAddrs.length > 0, "No artists");
         _checkArtistsNotRevoked(artistAddrs);
@@ -198,8 +198,8 @@ contract SeriesIndexer is Ownable {
 
     function ownerBatchAddSeries(
         address[][] memory artistsArray,
-        string[] memory metadatas,
-        string[] memory tokenIDsMapCIDs
+        string[] calldata metadatas,
+        string[] calldata tokenIDsMapCIDs
     ) external onlyOwner returns (uint256[] memory) {
         uint256 length = artistsArray.length;
         require(
@@ -221,8 +221,8 @@ contract SeriesIndexer is Ownable {
 
     function updateSeries(
         uint256 seriesID,
-        string memory metadata,
-        string memory tokenIDsMapCID
+        string calldata metadata,
+        string calldata tokenIDsMapCID
     ) external seriesExists(seriesID) onlyOwnerOrArtist(seriesID) {
         _checkMetadataAndTokenCID(metadata, tokenIDsMapCID);