Changeset 222 for cleverbox/trunk/cleverbox

Timestamp:

02/13/08 20:33:03 (4 years ago)

Author:

trivoallan

Message:

cleverbox :

• documentation cleanup
• started implementation of parameter checking method

Location:

cleverbox/trunk/cleverbox/model

Files:

• __init__.py (modified) (1 diff)
• client.py (modified) (1 diff)
• project.py (modified) (2 diffs)

cleverbox/trunk/cleverbox/model/__init__.py

@@ -1 +1 @@
 # -*- coding: utf-8 -*-
 version = '0.5dev'
+
+def check_required_parameters(names, parameters):
+    
+    for required_param in names:
+        # This will raise a KeyError if a parameter is missing
+        # TODO : it could be better to raise a more specific exception
+        parameters[required_param]

cleverbox/trunk/cleverbox/model/client.py

@@ -137 +137 @@
     Deletes client from supplied environment.
     
-    TODO : tunr that into an "archive" command to avoid complete loss of data
+    TODO : turn that into an "archive" command to avoid complete loss of data
     """
     

cleverbox/trunk/cleverbox/model/project.py

@@ -5 +5 @@
 import logging, os, re, ConfigParser, traceback
 from cleverbox.utils import filesystem, styles
+from cleverbox import model
 
 def exists(environment, client_name, project_name, project_status = None):
@@ -87 +88 @@
     """
     
-    # Parameters required for project creation
-    required = ('client', 'short_name', 'full_name', 'profile', 'enable')
-
     # Merging for consistency
     parameters['client'] = client_name
     parameters['short_name'] = project_name
 
-    # Sanity checks before project creation
-    for required_param in required:
-        # This will raise a KeyError if a parameter is missing
-        # TODO : it could be better to raise a more specific exception
-        parameters[required_param]
-    
+    # Check that all required parameters are here
+    required = ('client', 'short_name', 'full_name', 'profile', 'enable')
+    model.check_required_parameters(required, parameters)
+
     # Make sure client exists
     from cleverbox.model import client