# HG changeset patch
# User jbe
# Date 1281885047 -7200
# Node ID bdccc56fb70506ac198348dd8e1404651adf9146
# Parent  6dba5c0eb0ec5da2a7fa2bdb091bb4314b406839
lf_update continues on error (but still reports errors)
and version number changed to v1.2.4 (without any other schema changes)

diff -r 6dba5c0eb0ec -r bdccc56fb705 core.sql
--- a/core.sql	Sat Jul 24 16:52:51 2010 +0200
+++ b/core.sql	Sun Aug 15 17:10:47 2010 +0200
@@ -6,7 +6,7 @@
 BEGIN;
 
 CREATE VIEW "liquid_feedback_version" AS
-  SELECT * FROM (VALUES ('1.2.3', 1, 2, 3))
+  SELECT * FROM (VALUES ('1.2.4', 1, 2, 4))
   AS "subquery"("string", "major", "minor", "revision");
 
 
diff -r 6dba5c0eb0ec -r bdccc56fb705 lf_update.c
--- a/lf_update.c	Sat Jul 24 16:52:51 2010 +0200
+++ b/lf_update.c	Sun Aug 15 17:10:47 2010 +0200
@@ -6,6 +6,7 @@
 int main(int argc, char **argv) {
 
   // variable declarations:
+  int err = 0;
   int i, count;
   char *conninfo;
   PGconn *db;
@@ -57,97 +58,105 @@
   status = PQexec(db, "DELETE FROM \"expired_session\"");
   if (!status) {
     fprintf(stderr, "Error in pqlib while sending SQL command deleting expired sessions\n");
-    return 1;
-  }
-  if (
+    err = 1;
+  } else if (
     PQresultStatus(status) != PGRES_COMMAND_OK &&
     PQresultStatus(status) != PGRES_TUPLES_OK
   ) {
     fprintf(stderr, "Error while executing SQL command deleting expired sessions:\n%s", PQresultErrorMessage(status));
-    return 1;
+    err = 1;
+    PQclear(status);
+  } else {
+    PQclear(status);
   }
 
   // calculate member counts:
   status = PQexec(db, "SELECT \"calculate_member_counts\"()");
   if (!status) {
     fprintf(stderr, "Error in pqlib while sending SQL command calculating member counts\n");
-    return 1;
-  }
-  if (
+    err = 1;
+  } else if (
     PQresultStatus(status) != PGRES_COMMAND_OK &&
     PQresultStatus(status) != PGRES_TUPLES_OK
   ) {
     fprintf(stderr, "Error while executing SQL command calculating member counts:\n%s", PQresultErrorMessage(status));
-    return 1;
+    err = 1;
+    PQclear(status);
+  } else {
+    PQclear(status);
   }
 
   // update open issues:
   list = PQexec(db, "SELECT \"id\" FROM \"open_issue\"");
   if (!list) {
     fprintf(stderr, "Error in pqlib while sending SQL command selecting open issues\n");
-    return 1;
-  }
-  if (PQresultStatus(list) != PGRES_TUPLES_OK) {
+    err = 1;
+  } else if (PQresultStatus(list) != PGRES_TUPLES_OK) {
     fprintf(stderr, "Error while executing SQL command selecting open issues:\n%s", PQresultErrorMessage(list));
-    return 1;
+    err = 1;
+    PQclear(list);
+  } else {
+    count = PQntuples(list);
+    for (i=0; i<count; i++) {
+      const char *params[1];
+      params[0] = PQgetvalue(list, i, 0);
+      status = PQexecParams(
+        db, "SELECT \"check_issue\"($1)", 1, NULL, params, NULL, NULL, 0
+      );
+      if (!status) {
+        fprintf(stderr, "Error in pqlib while sending SQL command to call function \"check_issue\"(...):\n");
+        err = 1;
+      } else if (
+        PQresultStatus(status) != PGRES_COMMAND_OK &&
+        PQresultStatus(status) != PGRES_TUPLES_OK
+      ) {
+        fprintf(stderr, "Error while calling SQL function \"check_issue\"(...):\n%s", PQresultErrorMessage(status));
+        err = 1;
+        PQclear(status);
+      } else {
+        PQclear(status);
+      }
+    }
+    PQclear(list);
   }
-  count = PQntuples(list);
-  for (i=0; i<count; i++) {
-    const char *params[1];
-    params[0] = PQgetvalue(list, i, 0);
-    status = PQexecParams(
-      db, "SELECT \"check_issue\"($1)", 1, NULL, params, NULL, NULL, 0
-    );
-    if (!status) {
-      fprintf(stderr, "Error in pqlib while sending SQL command to call function \"check_issue\"(...):\n");
-      return 1;
-    }
-    if (
-      PQresultStatus(status) != PGRES_COMMAND_OK &&
-      PQresultStatus(status) != PGRES_TUPLES_OK
-    ) {
-      fprintf(stderr, "Error while calling SQL function \"check_issue\"(...):\n%s", PQresultErrorMessage(status));
-      return 1;
-    }
-    PQclear(status);
-  }
-  PQclear(list);
 
   // calculate ranks after voting is finished:
   // (NOTE: This is a seperate process to avoid long transactions with locking)
   list = PQexec(db, "SELECT \"id\" FROM \"issue_with_ranks_missing\"");
   if (!list) {
     fprintf(stderr, "Error in pqlib while sending SQL command selecting issues where ranks are missing\n");
-    return 1;
-  }
-  if (PQresultStatus(list) != PGRES_TUPLES_OK) {
+    err = 1;
+  } else if (PQresultStatus(list) != PGRES_TUPLES_OK) {
     fprintf(stderr, "Error while executing SQL command selecting issues where ranks are missing:\n%s", PQresultErrorMessage(list));
-    return 1;
+    err = 1;
+    PQclear(list);
+  } else {
+    count = PQntuples(list);
+    for (i=0; i<count; i++) {
+      const char *params[1];
+      params[0] = PQgetvalue(list, i, 0);
+      status = PQexecParams(
+        db, "SELECT \"calculate_ranks\"($1)", 1, NULL, params, NULL, NULL, 0
+      );
+      if (!status) {
+        fprintf(stderr, "Error in pqlib while sending SQL command to call function \"calculate_ranks\"(...):\n");
+        err = 1;
+      } else if (
+        PQresultStatus(status) != PGRES_COMMAND_OK &&
+        PQresultStatus(status) != PGRES_TUPLES_OK
+      ) {
+        fprintf(stderr, "Error while calling SQL function \"calculate_ranks\"(...):\n%s", PQresultErrorMessage(status));
+        err = 1;
+        PQclear(status);
+      } else {
+        PQclear(status);
+      }
+    }
+    PQclear(list);
   }
-  count = PQntuples(list);
-  for (i=0; i<count; i++) {
-    const char *params[1];
-    params[0] = PQgetvalue(list, i, 0);
-    status = PQexecParams(
-      db, "SELECT \"calculate_ranks\"($1)", 1, NULL, params, NULL, NULL, 0
-    );
-    if (!status) {
-      fprintf(stderr, "Error in pqlib while sending SQL command to call function \"calculate_ranks\"(...):\n");
-      return 1;
-    }
-    if (
-      PQresultStatus(status) != PGRES_COMMAND_OK &&
-      PQresultStatus(status) != PGRES_TUPLES_OK
-    ) {
-      fprintf(stderr, "Error while calling SQL function \"calculate_ranks\"(...):\n%s", PQresultErrorMessage(status));
-      return 1;
-    }
-    PQclear(status);
-  }
-  PQclear(list);
 
   // cleanup and exit
   PQfinish(db);
-  return 0;
+  return err;
 
 }
diff -r 6dba5c0eb0ec -r bdccc56fb705 update/core-update.v1.2.3-v1.2.4.sql
--- /dev/null	Thu Jan 01 00:00:00 1970 +0000
+++ b/update/core-update.v1.2.3-v1.2.4.sql	Sun Aug 15 17:10:47 2010 +0200
@@ -0,0 +1,9 @@
+BEGIN;
+
+CREATE OR REPLACE VIEW "liquid_feedback_version" AS
+  SELECT * FROM (VALUES ('1.2.4', 1, 2, 4))
+  AS "subquery"("string", "major", "minor", "revision");
+
+-- no changes in database scheme except version number
+
+COMMIT;