Fortimanager lock_adom() does not work

[robertpenz]

Hi!

fortimanager.lock_adom() calls return self._request(data), but _requests is defined as

 def _request(self, method, url, option=None, data=None, request_id=1,
                 verbose=False):

which leads to TypeError: _request() takes at least 3 arguments (2 given). Should the data json dict split up and provided as parameters for the method?

[robertpenz]

if I change the methods to following it works

    @login_required
    def lock_adom(self, adom):
        '''
        Lock an ADOM
        '''
        return self._exec(url="pm/config/adom/{}/_workspace/lock".format(adom), request_id = 5612)

    @login_required
    def unlock_adom(self, adom):
        '''
        Unclock an ADOM
        '''
        return self._exec(url="pm/config/adom/{}/_workspace/unlock".format(adom), request_id = 5613)

    @login_required
    def commit(self, adom):
        '''
        Commit changes made to ADOM
        '''
        return self._exec(url="pm/config/adom/{}/_workspace/commit".format(adom), request_id = 5614)

[pschmitt]

Thanks for letting me know. What FortiManager version are you using? Can you issue a PR with these changes?

[robertpenz]

Did a PR.