Changeset 12042 for trunk

Timestamp:

09/08/11 21:49:58 (13 years ago)

Author:

tbretz

Message:

Added program name to call of parse_database; fixed database query; removed setting 'unregistered' option - this is done in a later step by boost itself

File:

• trunk/FACT++/src/Configuration.cc (modified) (7 diffs)

trunk/FACT++/src/Configuration.cc

@@ -391 +391 @@
 #ifdef HAS_SQL
 po::basic_parsed_options<char>
-    Configuration::parse_database(const string &database, const po::options_description& desc, bool allow_unregistered)
+    Configuration::parse_database(const string &prgname, const string &database, const po::options_description& desc, bool allow_unregistered)
 {
     //static const boost::regex expr("(([[:word:].-]+)(:(.+))?@)?([[:word:].-]+)(:([[:digit:]]+))?(/([[:word:].-]+))?");
@@ -422 +422 @@
     if (!db.empty())
         cout << "/" << db;
-    cout << "'" << endl;
+    cout << "' for " << prgname << endl;
 
     mysqlpp::Connection conn(db.c_str(), server.c_str(), user.c_str(), passwd.c_str(), port);
@@ -433 +433 @@
 
     const mysqlpp::StoreQueryResult res =
-        conn.query("SELECT fProgram, CONCAT(fKey1,fKey2), fValue "
-                   "FROM ProgramOption WHERE fCounter="
-                   "(SELECT MAX(fCounter) FROM History)").store();
+        conn.query("SELECT CONCAT(fKey1,fKey2), fValue "
+                   "FROM ProgramOption "
+                   "WHERE fCounter=(SELECT MAX(fCounter) FROM History) "
+                   "AND (fProgram='"+prgname+"' OR fProgram='*')").store();
     /* throws exceptions
     if (!res)
@@ -477 +478 @@
         n.value.clear();          // Fixme: composing?
         n.value.push_back((*v)[1].c_str());
-        n.unregistered = unregistered;
+        //n.unregistered = unregistered;
 
         // If any parsing will be done in the future...
@@ -487 +488 @@
     }
 
-    cout << endl;
-
     return result;
 }
 #else
 po::basic_parsed_options<char>
-    Configuration::parse_database(const string &, const po::options_description &desc, bool)
+    Configuration::parse_database(const string &, const string &, const po::options_description &desc, bool)
 {
     return po::parsed_options(&desc);
@@ -1146 +1145 @@
     {
         fDatabase = getdatabase["database"].as<string>();
-        cerr << "Requesting options from '" << fDatabase << "'." << endl;
+        cerr << "Requesting options from database for '" << fName << "'" << endl;
     }
 
@@ -1153 +1152 @@
     const po::parsed_options parsed_database =
         fDatabase.empty() ? po::parsed_options(&opt_database) :
-        parse_database(fDatabase, opt_database, !checkdb);
+        parse_database(path.filename(), fDatabase, opt_database, !checkdb);
 
     // ------------------------ (11) -------------------------