ActiveCampaignAction._get_request() - Code Metrics - Inspection of "Remove print statement" - StackStorm/st2contrib - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#546)

by Lakshmi

created 2016-08-17 19:49 UTC

ActiveCampaignAction._get_request() A

↳ Parent: Project

Complexity

Conditions

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	2
dl	0
loc	19
rs	9.4285
c	0
b	0
f	0

import requests
import urllib
import urlparse

from st2actions.runners.pythonrunner import Action


class ActiveCampaignAction(Action):

    def run(self, **kwargs):
        if kwargs['api_key'] is None:
            kwargs['api_key'] = self.config['api_key']

        return self._get_request(kwargs)

    def _get_request(self, params):
        url = urlparse.urljoin(self.config['url'], 'admin/api.php')
        headers = {}
        headers['Content-Type'] = 'application/x-www-form-urlencoded'

        params = self._format_params(params)
        data = urllib.urlencode(params)
        response = requests.get(url=url,
                                headers=headers, params=data)

        results = response.json()
        if results['result_code'] is not 1:
            failure_reason = ('Failed to perform action: %s \
                              (status code: %s)' % (response.text,
                              response.status_code))
            self.logger.exception(failure_reason)
            raise Exception(failure_reason)

        return results

    def _format_params(self, params):
        output = {}
        for k, v in params.iteritems():
            if isinstance(v, dict):
                for pk, pv in v.iteritems():
                    param_name = "%s[%s]" % (k, pk)
                    output[param_name] = pv
            else:
                output[k] = v
        return output


1			import requests
2			import urllib
3			import urlparse
4
5			from st2actions.runners.pythonrunner import Action
6
7
8			class ActiveCampaignAction(Action):
9
10			def run(self, **kwargs):
11			if kwargs['api_key'] is None:
12			kwargs['api_key'] = self.config['api_key']
13
14			return self._get_request(kwargs)
15
16			def _get_request(self, params):
17			url = urlparse.urljoin(self.config['url'], 'admin/api.php')
18			headers = {}
19			headers['Content-Type'] = 'application/x-www-form-urlencoded'
20
21			params = self._format_params(params)
22			data = urllib.urlencode(params)
23			response = requests.get(url=url,
24			headers=headers, params=data)
25
26			results = response.json()
27			if results['result_code'] is not 1:
28			failure_reason = ('Failed to perform action: %s \
29			(status code: %s)' % (response.text,
30			response.status_code))
31			self.logger.exception(failure_reason)
32			raise Exception(failure_reason)
33
34			return results
35
36			def _format_params(self, params):
37			output = {}
38			for k, v in params.iteritems():
39			if isinstance(v, dict):
40			for pk, pv in v.iteritems():
41			param_name = "%s[%s]" % (k, pk)
42			output[param_name] = pv
43			else:
44			output[k] = v
45			return output
46

StackStorm / st2contrib

Pull Request — master (#546)

ActiveCampaignAction._get_request() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like