Skip to content

Commit

Permalink
Merge branch 'license-118n' into release-v1.7.2
Browse files Browse the repository at this point in the history
  • Loading branch information
rossjones committed Jul 19, 2012
2 parents f0adba7 + 239ec45 commit a351034
Showing 1 changed file with 12 additions and 10 deletions.
22 changes: 12 additions & 10 deletions ckan/model/license.py
Original file line number Diff line number Diff line change
Expand Up @@ -41,15 +41,8 @@ def as_dict(self):
class LicenseRegister(object):
"""Dictionary-like interface to a group of licenses."""

def __init__(self):
group_url = config.get('licenses_group_url', None)
if group_url:
self.load_licenses(group_url)
else:
self.licenses = self.get_license_info

@property
def get_license_info(self):
def licenses(self):
# this allows the license info to be localised see #2575
default_license_list = [
{
Expand Down Expand Up @@ -256,6 +249,13 @@ def get_license_info(self):
]
return [License(entity) for entity in default_license_list]

def __init__(self):
group_url = config.get('licenses_group_url', None)
if group_url:
# this will override licenses function
self.load_licenses(group_url)


def load_licenses(self, license_url):
try:
response = urllib2.urlopen(license_url)
Expand All @@ -272,9 +272,11 @@ def load_licenses(self, license_url):

def _create_license_list(self, license_data):
if isinstance(license_data, dict):
self.licenses = [License(entity) for entity in license_data.values()]
# override the class attribute
self.__class__.licenses = [License(entity) for entity in license_data.values()]
elif isinstance(license_data, list):
self.licenses = [License(entity) for entity in license_data]
# override the class attribute
self.__class__.licenses = [License(entity) for entity in license_data]
else:
msg = "Licenses at %s must be dictionary or list" % license_url
raise ValueError(msg)
Expand Down

0 comments on commit a351034

Please sign in to comment.