Changeset 222 for cleverbox/trunk/cleverbox/model/project.py

Timestamp:

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

Author:

trivoallan

Message:

cleverbox :

• documentation cleanup
• started implementation of parameter checking method

Files:

• cleverbox/trunk/cleverbox/model/project.py (modified) (2 diffs)

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