Changeset 18926 for trunk/psModules/src/config/pmConfig.c

Timestamp:

Aug 5, 2008, 3:13:14 PM (18 years ago)

Author:

Paul Price

Message:

Adding function pmConfigUserSite to get values from the user/site configurations. Using this for database details.

File:

• trunk/psModules/src/config/pmConfig.c (modified) (2 diffs)

trunk/psModules/src/config/pmConfig.c

@@ -1264 +1264 @@
 }
 
-// Look up a value from the user or site configuration
-#define CONFIG_USER_SITE(TYPE, TARGET, TYPENAME, NAME, DEFAULT) \
-    TYPE TARGET; \
-    { \
-        bool status = false;                /* Status of MD lookup */ \
-        TARGET = psMetadataLookup##TYPENAME(&status, config->user, NAME); \
-        if (!status) { \
-            TARGET = psMetadataLookup##TYPENAME(&status, config->site, NAME); \
-            if (!status) { \
-                TARGET = DEFAULT; \
-            } \
-        } \
-    }
+psMetadataItem *pmConfigUserSite(const pmConfig *config, const char *name, psDataType type)
+{
+    PS_ASSERT_PTR_NON_NULL(config, NULL);
+    PS_ASSERT_STRING_NON_EMPTY(name, NULL);
+
+    psMetadataItem *item = psMetadataLookup(config->user, name);
+    if (!item) {
+        item = psMetadataLookup(config->site, name);
+        if (!item) {
+            psError(PS_ERR_BAD_PARAMETER_VALUE, true,
+                    "Unable to find %s in user or site configuration", name);
+            return NULL;
+        }
+    }
+    if (item->type != type) {
+        psError(PS_ERR_BAD_PARAMETER_TYPE, true,
+                "Type of %s (%x) in user/site configuration does not match expected (%x)",
+                name, item->type, type);
+        return NULL;
+    }
+
+    return item;
+}
 
 
@@ -1296 +1306 @@
     }
 
-    CONFIG_USER_SITE(const char*, dbServer, Str, "DBSERVER",   NULL);
-    CONFIG_USER_SITE(const char*, dbUser,   Str, "DBUSER",     NULL);
-    CONFIG_USER_SITE(const char*, dbPass,   Str, "DBPASSWORD", NULL);
-    CONFIG_USER_SITE(const char*, dbName,   Str, "DBNAME",     NULL);
-    CONFIG_USER_SITE(int,         dbPort,   S32, "DBPORT",     0);
-
-    if (!dbServer || !dbUser || !dbPass || !dbName) {
+    // Connection details
+    psMetadataItem *server = pmConfigUserSite(config, "DBSERVER",   PS_DATA_STRING);
+    psMetadataItem *user   = pmConfigUserSite(config, "DBUSER",     PS_DATA_STRING);
+    psMetadataItem *pass   = pmConfigUserSite(config, "DBPASSWORD", PS_DATA_STRING);
+    psMetadataItem *name   = pmConfigUserSite(config, "DBNAME",     PS_DATA_STRING);
+    psMetadataItem *port   = pmConfigUserSite(config, "DBPORT",     PS_TYPE_S32);
+
+    if (!server || !user || !pass || !name) {
         psWarning("Cannot find DBSERVER/DBUSER/DBPASSWORD/DBNAME in user or site configuration: "
                   "unable to connect to database.");
+        psErrorClear();
         return NULL;
     }
-
-    if (strcasecmp(dbName, "XXX") == 0 || strcasecmp(dbName, "NONE") == 0) {
-        psTrace("psModules.config", 1, "Database initialisation skipped: database is %s.", dbName);
+    if (!port) {
+        psTrace("psModules.config", 1, "Database port defaulting to 0");
+        psErrorClear();
+    }
+
+    if (strcasecmp(name->data.str, "XXX") == 0 || strcasecmp(name->data.str, "NONE") == 0) {
+        psTrace("psModules.config", 1, "Database initialisation skipped: database is %s.", name->data.str);
         return NULL;
     }
 
-    config->database = psDBInit(dbServer, dbUser, dbPass, dbName, dbPort);
+    config->database = psDBInit(server->data.str, user->data.str, pass->data.str, name->data.str,
+                                port ? port->data.S32 : 0);
     return config->database;