From 08008fddeca37a2884026352bf48d7fe8e5f0fd5 Mon Sep 17 00:00:00 2001
From: Sebastien Luttringer <sebastien.luttringer@smartjog.com>
Date: Tue, 20 Dec 2011 20:48:16 +0100
Subject: [PATCH] repository name checking moved inside reposiory class
---
installsystems/repository.py | 24 ++++++++++++++----------
1 file changed, 14 insertions(+), 10 deletions(-)
diff --git a/installsystems/repository.py b/installsystems/repository.py
index 0c5fb7a..261c72d 100644
--- a/installsystems/repository.py
+++ b/installsystems/repository.py
@@ -28,6 +28,19 @@ class Repository(object):
Repository class
'''
+ @staticmethod
+ def is_repository_name(name):
+ return re.match("^[-_\w]+$", name) is not None
+
+ @staticmethod
+ def check_repository_name(name):
+ '''
+ Raise exception is repository name is invalid
+ '''
+ if not Repository.is_repository_name(name):
+ raise Exception("Invalid repository name %s" % name)
+ return name
+
@classmethod
def diff(cls, repo1, repo2):
'''
@@ -694,18 +707,9 @@ class RepositoryConfig(object):
Repository configuration container
'''
- @staticmethod
- def check_repository_name(name):
- '''
- Raise exception is repository name is invalid
- '''
- if re.match("^[-_\w]+$", name) is None:
- raise Exception("Invalid repository name %s" % name)
- return name
-
def __init__(self, name, **kwargs):
# set default value for arguments
- self.name = self.check_repository_name(name)
+ self.name = Repository.check_repository_name(name)
self.path = ""
self._offline = False
self._dbpath = None
--
GitLab