Refactor _checkAuthorized to include operator parameter

Author: Amxx

contracts/token/ERC1155/ERC1155.sol

@@ -97,7 +97,7 @@ abstract contract ERC1155 is Context, ERC165, IERC1155, IERC1155MetadataURI, IER
 
     /// @inheritdoc IERC1155
     function safeTransferFrom(address from, address to, uint256 id, uint256 value, bytes memory data) public virtual {
-        _checkAuthorized(from);
+        _checkAuthorized(_msgSender(), from);
         _safeTransferFrom(from, to, id, value, data);
     }
 
@@ -109,15 +109,14 @@ abstract contract ERC1155 is Context, ERC165, IERC1155, IERC1155MetadataURI, IER
         uint256[] memory values,
         bytes memory data
     ) public virtual {
-        _checkAuthorized(from);
+        _checkAuthorized(_msgSender(), from);
         _safeBatchTransferFrom(from, to, ids, values, data);
     }
 
-    /// @dev Checks if the caller is authorized to transfer tokens from the given address. Reverts with {ERC1155MissingApprovalForAll} if not.
-    function _checkAuthorized(address from) internal view virtual {
-        address sender = _msgSender();
-        if (from != sender && !isApprovedForAll(from, sender)) {
-            revert ERC1155MissingApprovalForAll(sender, from);
+    /// @dev Checks if `operator` is authorized to transfer tokens owned by `owner`. Reverts with {ERC1155MissingApprovalForAll} if not.
+    function _checkAuthorized(address operator, address owner) internal view virtual {
+        if (owner != operator && !isApprovedForAll(owner, operator)) {
+            revert ERC1155MissingApprovalForAll(operator, owner);
         }
     }