[Gnucash-changes] Improve debugging messages in HBCI.

Christian Stimming cstim at cvs.gnucash.org
Thu May 5 07:40:26 EDT 2005


Log Message:
-----------
Improve debugging messages in HBCI.

2005-05-05  Christian Stimming  <stimming at tuhh.de>

	* src/import-export/hbci/gnc-hbci-utils.c: Improve debugging
	messages.

Tags:
----
gnucash-1-8-branch

Modified Files:
--------------
    gnucash:
        ChangeLog
    gnucash/src/import-export/hbci:
        druid-hbci-initial.c
        gnc-hbci-utils.c

Revision Data
-------------
Index: ChangeLog
===================================================================
RCS file: /home/cvs/cvsroot/gnucash/ChangeLog,v
retrieving revision 1.1461.2.410
retrieving revision 1.1461.2.411
diff -LChangeLog -LChangeLog -u -r1.1461.2.410 -r1.1461.2.411
--- ChangeLog
+++ ChangeLog
@@ -1,3 +1,8 @@
+2005-05-05  Christian Stimming  <stimming at tuhh.de>
+
+	* src/import-export/hbci/gnc-hbci-utils.c: Improve debugging
+	messages.
+
 2005-04-21  Christian Stimming  <stimming at tuhh.de>
 
 	* src/import-export/hbci/gnc-hbci-getbalance.c: Fix forgotten NULL
Index: druid-hbci-initial.c
===================================================================
RCS file: /home/cvs/cvsroot/gnucash/src/import-export/hbci/druid-hbci-initial.c,v
retrieving revision 1.26.2.18
retrieving revision 1.26.2.19
diff -Lsrc/import-export/hbci/druid-hbci-initial.c -Lsrc/import-export/hbci/druid-hbci-initial.c -u -r1.26.2.18 -r1.26.2.19
--- src/import-export/hbci/druid-hbci-initial.c
+++ src/import-export/hbci/druid-hbci-initial.c
@@ -450,7 +450,8 @@
   HBCIInitialInfo *info = user_data;
   GWEN_BUFFER *buf;
   int res;
-  const char *backend_name = "aqhbci";
+  const GWEN_STRINGLIST *active_backends;
+  const char *backend_name;
 
   /* This is the point where we look for and start an external
      application shipped with aqhbci that contains the setup druid for
@@ -466,6 +467,15 @@
   AB_BANKING *banking = info->api;
   g_assert(info->druid);
 
+  active_backends = AB_Banking_GetActiveProviders (banking);
+  if (GWEN_StringList_Count(active_backends) <= 1)
+    backend_name = "aqhbci";
+  else {
+    /* FIXME: Present a selection dialog to select a particular
+       backend */
+    backend_name = "aqhbci";
+  }
+
   buf = GWEN_Buffer_new(NULL, 300, 0, 0);
   AB_Banking_FindWizard(banking, backend_name, NULL, buf);
   wizard_exists = (strlen(GWEN_Buffer_GetStart(buf)) > 0);
Index: gnc-hbci-utils.c
===================================================================
RCS file: /home/cvs/cvsroot/gnucash/src/import-export/hbci/gnc-hbci-utils.c,v
retrieving revision 1.23.2.17
retrieving revision 1.23.2.18
diff -Lsrc/import-export/hbci/gnc-hbci-utils.c -Lsrc/import-export/hbci/gnc-hbci-utils.c -u -r1.23.2.17 -r1.23.2.18
--- src/import-export/hbci/gnc-hbci-utils.c
+++ src/import-export/hbci/gnc-hbci-utils.c
@@ -28,6 +28,7 @@
 #include <iconv.h>
 #include <langinfo.h>
 #include <gwenhywfar/directory.h>
+#include <gwenhywfar/logger.h>
 
 #include "gnc-ui.h"
 #include "gnc-hbci-kvp.h"
@@ -189,6 +190,7 @@
 /*   list_int *list; */
 /*   const char *msg; */
   int cause = 0;
+  AB_JOB_STATUS jobstatus;
   
   g_assert (job);
 /*   if (AB_JOB_status (job) != HBCI_JOB_STATUS_DONE) */
@@ -197,12 +199,23 @@
 /*     return; */
 
   if (verbose) {
-    printf("OutboxJob status: %s", AB_Job_Status2Char(AB_Job_GetStatus(job)));
+    printf("gnc_hbci_debug_outboxjob: Job status: %s", AB_Job_Status2Char(AB_Job_GetStatus(job)));
 
     printf(", result: %s", AB_Job_GetResultText(job));
     printf("\n");
   }
 
+  jobstatus = AB_Job_GetStatus (job);
+  if (jobstatus == AB_Job_StatusError) {
+    if (!verbose)
+      printf("gnc_hbci_debug_outboxjob: Job %s had an error: %s\n",
+	     AB_Job_Type2Char(AB_Job_GetType(job)),
+	     AB_Job_GetResultText(job));
+    cause = 9000;
+  } else {
+    cause = 0;
+  }
+
 #if 0  
   list = AB_JOB_resultCodes (job);
   if (list_int_size (list) > 0) {
@@ -420,17 +433,23 @@
 {
   int err;
   int resultcode;
+  int be_verbose = FALSE;
 	  
   if (inter)
     GNCInteractor_show (inter);
 
   if (gnc_lookup_boolean_option("_+Advanced", 
 				"HBCI Verbose Debug Messages", FALSE)) {
-/*     GWEN_Logger_SetLevel(0, GWEN_LoggerLevelDebug); */
-/*     HBCI_Hbci_setDebugLevel (4); */
+    GWEN_Logger_SetLevel(GWEN_LOGDOMAIN, GWEN_LoggerLevelNotice);
+    GWEN_Logger_SetLevel(AQBANKING_LOGDOMAIN, GWEN_LoggerLevelInfo);
+    GWEN_Logger_SetLevel("aqhbci", GWEN_LoggerLevelInfo);
+    be_verbose = TRUE;
+  }
+  else {
+    GWEN_Logger_SetLevel(GWEN_LOGDOMAIN, GWEN_LoggerLevelError);
+    GWEN_Logger_SetLevel(AQBANKING_LOGDOMAIN, GWEN_LoggerLevelError);
+    GWEN_Logger_SetLevel("aqhbci", GWEN_LoggerLevelError);
   }
-/*   else */
-/*     HBCI_Hbci_setDebugLevel (0); */
 
   do {
     if (inter) {
@@ -450,7 +469,7 @@
     
   } while (gnc_hbci_Error_retry (parent, err, inter));
   
-  resultcode = gnc_hbci_debug_outboxjob (job, FALSE);
+  resultcode = gnc_hbci_debug_outboxjob (job, be_verbose);
   if (!hbci_Error_isOk(err)) {
 /*     char *errstr =  */
 /*       g_strdup_printf("gnc_AB_BANKING_execute: Error at executeQueue: %s", */


More information about the gnucash-changes mailing list