Weird use of approver id

[lightclient]

When calling acceptFulfillment an _approverId is passed in as a parameter. That is then passed to the modifier to determine if the user is authorized to accept fulfillments. The modifier looks like this:

modifier canApprove(uint _approverId) {
     require(msg.sender == controller || msg.sender == approvers[_approverId]);
     _;
}

I think it is weird that if msg.sender == controller it doesn't matter what the approverId is. I believe a better approach would be to overload the function with two different signatures where one could be called with the approveId and one could be called without.

[lightclient]

i.e.

function acceptFulfillment(uint _fulfillmentId, address[] _payoutTokens, uint[] _tokenVersions, uint[][] _tokenAmounts)
       public
       onlyController
{
       _acceptFulfillment(_fulfillmentId, _payoutTokens,  _tokenVersions, _tokenAmounts)
}

function acceptFulfillment(uint _approverId, uint _fulfillmentId, address[] _payoutTokens, uint[] _tokenVersions, uint[][] _tokenAmounts) 
       public
       canApprove(_approverId)
{
      _acceptFulfillment(_fulfillmentId, _payoutTokens,  _tokenVersions, _tokenAmounts)
}

function _acceptFufillment(uint _fulfillmentId, address[] _payoutTokens, uint[] _tokenVersions, uint[][] _tokenAmounts) 
      private
      validateFulfillmentArrayIndex(_fulfillmentId)
      validateApproverArrayIndex(_approverId)
      sameLength(_payoutTokens.length, _tokenVersions.length)
{
      // do accepting logic
}

[lightclient]

same issue with fulfillAndAccept