[GRASS-SVN] r72941 - grass/trunk/lib/db/dbmi_base

[svn_grass at osgeo.org]

Author: mmetz
Date: 2018-07-02 09:40:02 -0700 (Mon, 02 Jul 2018)
New Revision: 72941

Modified:
   grass/trunk/lib/db/dbmi_base/connect.c
   grass/trunk/lib/db/dbmi_base/default_name.c
Log:
lib/dbmibase: fix get/set connection parameters, clarify default and current parameters

Modified: grass/trunk/lib/db/dbmi_base/connect.c
===================================================================
--- grass/trunk/lib/db/dbmi_base/connect.c	2018-07-01 21:03:44 UTC (rev 72940)
+++ grass/trunk/lib/db/dbmi_base/connect.c	2018-07-02 16:40:02 UTC (rev 72941)
@@ -29,15 +29,19 @@
 {
     /* TODO: add checks and return DB_* error code if needed */
 
+    G_unsetenv2("DB_DRIVER", G_VAR_MAPSET);
     if (connection->driverName)
 	G_setenv2("DB_DRIVER", connection->driverName, G_VAR_MAPSET);
 
+    G_unsetenv2("DB_DATABASE", G_VAR_MAPSET);
     if (connection->databaseName)
 	G_setenv2("DB_DATABASE", connection->databaseName, G_VAR_MAPSET);
 
+    G_unsetenv2("DB_SCHEMA", G_VAR_MAPSET);
     if (connection->schemaName)
 	G_setenv2("DB_SCHEMA", connection->schemaName, G_VAR_MAPSET);
 
+    G_unsetenv2("DB_GROUP", G_VAR_MAPSET);
     if (connection->group)
 	G_setenv2("DB_GROUP", connection->group, G_VAR_MAPSET);
 
@@ -59,7 +63,7 @@
 }
 
 /*!
-  \brief Get default DB connection settings
+  \brief Get default DB connection settings for the current mapset
   
   \param[out] connection pointer to dbConnection to be modified
 

Modified: grass/trunk/lib/db/dbmi_base/default_name.c
===================================================================
--- grass/trunk/lib/db/dbmi_base/default_name.c	2018-07-01 21:03:44 UTC (rev 72940)
+++ grass/trunk/lib/db/dbmi_base/default_name.c	2018-07-02 16:40:02 UTC (rev 72941)
@@ -19,9 +19,9 @@
 #include <grass/glocale.h>
 
 /*!
-  \brief Get default driver name
+  \brief Get driver name from current DB connection settings
 
-  \return pointer to default driver name
+  \return pointer to driver name
   \return NULL if not set
 */
 const char *db_get_default_driver_name(void)
@@ -35,9 +35,9 @@
 }
 
 /*!
-  \brief Get default database name
+  \brief Get database name from current DB connection settings
 
-  \return pointer to default database name
+  \return pointer to database name
   \return NULL if not set
 */
 const char *db_get_default_database_name(void)
@@ -51,9 +51,9 @@
 }
 
 /*!
-  \brief Get default schema name
+  \brief Get schema name from current DB connection settings
   
-  \return pointer to default schema name
+  \return pointer to schema name
   \return NULL if not set
 */
 const char *db_get_default_schema_name(void)
@@ -67,9 +67,9 @@
 }
 
 /*!
-  \brief Get default group name
+  \brief Get group name from current DB connection settings
   
-  \return pointer to default group name
+  \return pointer to group name
   \return NULL if not set
 */
 const char *db_get_default_group_name(void)
@@ -85,6 +85,9 @@
 /*!
   \brief Sets up database connection settings using GRASS default from dbmi.h
 
+  This function ignores current DB connection settings and uses GRASS 
+  default settings instead.
+
   \todo DB_OK on success, DB_* error code on fail
 
   \return returns DB_OK 
@@ -97,8 +100,8 @@
     G_debug(1,
 	    "Creating new default DB params with db_set_default_connection()");
 
-    /* is this really needed ? */
-    db_get_connection(&connection);
+    /* do not use default DB connection settings for the current mapset */
+    G_zero(&connection, sizeof(dbConnection));
 
     if (strcmp(DB_DEFAULT_DRIVER, "dbf") == 0) {
 	/* Set default values and create dbf db dir */