X-Git-Url: https://git.deb.at/?a=blobdiff_plain;f=abook.c;h=bb0a34792a0f8b313a652102a61472a08272b11c;hb=4f7637954591ecfb575642709bd12a069ee90aab;hp=6147aeeadcd28fabb98d341222e614e397e0cf4a;hpb=a99180ae57f15c923ac78be34df3acb432d8fce7;p=pkg%2Fabook.git

diff --git a/abook.c b/abook.c
index 6147aee..bb0a347 100644
--- a/abook.c
+++ b/abook.c
@@ -1,7 +1,7 @@
 /*
  * $Id$
  *
- * by JH <jheinonen@bigfoot.com>
+ * by JH <jheinonen@users.sourceforge.net>
  *
  * Copyright (C) Jaakko Heinonen
  */
@@ -273,44 +273,60 @@ quit_mutt_query(int status)
 }
 
 
+static char *
+make_mailstr(int item)
+{
+	char email[MAX_EMAIL_LEN];
+	char *ret;
+	char *name = mkstr("\"%s\"", database[item][NAME]);
+
+	get_first_email(email, item);
+
+	ret = *database[item][EMAIL] ?
+		mkstr("%s <%s>", name, email) :
+		strdup(name);
+
+	free(name);
+	
+	return ret;
+}
+
 void
 launch_mutt(int item)
 {
-	/*
-	 * This is _broken_
-	 */
-#if 0
-	struct db_enumerator e = init_db_enumerator(ENUM_ALL);
-	char email[MAX_EMAIL_LEN];
-	char *cmd;
-	char *tmp = options_get_str("mutt_command");
-
-	cmd = strconcat(tmp, " '", NULL );
-
-	db_enumerate_items(e) {
-		if( e.item != item );
-		get_first_email(email, e.item);
-		tmp = mkstr("%s, \"%s\"", cmd, database[e.item][NAME]);
-		my_free(cmd);
-		if( *database[e.item][EMAIL] ) {
-			cmd = mkstr("%s <%s>", tmp, email);
+	char *cmd = NULL, *mailstr = NULL;
+	char *mutt_command = options_get_str("mutt_command");
+
+	if(mutt_command == NULL || !*mutt_command)
+		return;
+
+	if( is_valid_item(item) )
+		mailstr = make_mailstr(item);
+	else {
+		struct db_enumerator e = init_db_enumerator(ENUM_SELECTED);
+		char *tmp = NULL;
+		db_enumerate_items(e) {
+			tmp = mailstr;
+			mailstr = tmp ?
+				strconcat(tmp, ",", make_mailstr(e.item), NULL):
+				strconcat(make_mailstr(e.item), NULL);
 			free(tmp);
-			tmp = cmd;
 		}
-		cmd = strconcat(tmp, " ", NULL);
-		free(tmp);
 	}
 
-	tmp = strconcat(cmd, "\'", NULL);
-	free(cmd);
-	cmd = tmp;
+	cmd = strconcat(mutt_command, " \'", mailstr,
+				"\'", NULL);
+	free(mailstr);
 #ifdef DEBUG
 	fprintf(stderr, "cmd: %s\n", cmd);
 #endif
 	system(cmd);	
-
 	free(cmd);
-#endif
+	
+	/*
+	 * we need to make sure that curses settings are correct
+	 */
+	ui_init_curses();
 }
 
 void
@@ -332,6 +348,11 @@ launch_wwwbrowser(int item)
 		system(cmd);
 
 	free(cmd);
+
+	/*
+	 * we need to make sure that curses settings are correct
+	 */
+	ui_init_curses();
 }
 
 void *
@@ -396,11 +417,13 @@ convert(char *srcformat, char *srcfile, char *dstformat, char *dstfile)
 	strlower(srcformat);
 	strlower(dstformat);
 
+#ifndef DEBUG
 	if( !strcmp(srcformat, dstformat) ) {
 		printf(	"input and output formats are the same\n"
 			"exiting...\n");
 		exit(1);
 	}
+#endif
 
 	set_filenames();
 	init_options();