added option add_email_prevent_duplicates

[pkg/abook.git] / abook.c

diff --git a/abook.c b/abook.c
index 4808dac37abf7e10ea3c05569cc025f53e10f2f6..d4de3ec5679d71620fa311c790998618ec03072e 100644 (file)
--- a/abook.c
+++ b/abook.c
@@ -45,8 +45,8 @@ static void           add_email(int);
 char *datafile = NULL;
 char *rcfile = NULL;
 
-int alternative_datafile = FALSE;
-int alternative_rcfile = FALSE;
+bool alternative_datafile = FALSE;
+bool alternative_rcfile = FALSE;
 
 static int
 datafile_writeable()
@@ -103,7 +103,8 @@ init_abook()
 {
        set_filenames();
        check_abook_directory();
-       init_options();
+       init_opts();
+       load_opts(rcfile);
 
        signal(SIGKILL, quit_abook_sig);
        signal(SIGTERM, quit_abook_sig);
@@ -121,7 +122,7 @@ init_abook()
                if(load_database(datafile) || !statusline_ask_boolean(
                                        "If you continue all changes will "
                                "be lost. Do you want to continue?", FALSE)) {
-                       close_config();
+                       free_opts();
                        /*close_database();*/
                        close_ui();
                        exit(1);
@@ -135,12 +136,12 @@ init_abook()
 void
 quit_abook()
 {
-       if( options_get_int("autosave") )
+       if( opt_get_bool(BOOL_AUTOSAVE) )
                save_database();
        else if( statusline_ask_boolean("Save database", TRUE) )
                save_database();
 
-       close_config();
+       free_opts();
        close_database();
 
        close_ui();
@@ -242,7 +243,7 @@ set_filename(char **var, char *path)
 
        cwd = my_getcwd();
 
-       *var = strconcat(cwd, path, NULL);
+       *var = strconcat(cwd, "/", path, NULL);
 
        free(cwd);
 }
@@ -404,7 +405,7 @@ static void
 quit_mutt_query(int status)
 {
        close_database();
-       close_config();
+       free_opts();
 
        exit(status);
 }
@@ -417,7 +418,7 @@ muttq_print_item(FILE *file, int item)
 
        split_emailstr(item, emails);
 
-       for(i = 0; i < (options_get_int("mutt_return_all_emails") ?
+       for(i = 0; i < (opt_get_bool(BOOL_MUTT_RETURN_ALL_EMAILS) ?
                        MAX_EMAILS : 1) ; i++)
                if( *emails[i] )
                        fprintf(file, "%s\t%s\t%s\n", emails[i],
@@ -458,7 +459,8 @@ static void
 init_mutt_query()
 {
        set_filenames();
-       init_options();
+       init_opts();
+       load_opts(rcfile);
 
        if( load_database(datafile) ) {
                printf("Cannot open database\n");
@@ -506,7 +508,7 @@ void
 launch_mutt(int item)
 {
        char *cmd = NULL, *mailstr = NULL;
-       char *mutt_command = options_get_str("mutt_command");
+       char *mutt_command = opt_get_str(STR_MUTT_COMMAND);
 
        if(mutt_command == NULL || !*mutt_command)
                return;
@@ -550,7 +552,7 @@ launch_wwwbrowser(int item)
 
        if( database[item][URL] )
                cmd = mkstr("%s '%s'",
-                               options_get_str("www_command"),
+                               opt_get_str(STR_WWW_COMMAND),
                                safe_str(database[item][URL]));
        else
                return;
@@ -603,17 +605,18 @@ FILE *
 abook_fopen (const char *path, const char *mode)
 {      
        struct stat s;
+       bool stat_ok;
 
-       if( ! strchr(mode, 'r') )
-               return fopen(path, mode);
-
-       if ( (stat(path, &s)) == -1 )
-               return NULL;
-
-       return S_ISREG(s.st_mode) ? fopen(path, mode) : NULL;
+       stat_ok = (stat(path, &s) != -1);
+       
+       if(strchr(mode, 'r'))
+               return (stat_ok && S_ISREG(s.st_mode)) ?
+                       fopen(path, mode) : NULL;
+       else
+               return (stat_ok && S_ISDIR(s.st_mode)) ?
+                       NULL : fopen(path, mode);
 }
 
-
 static void
 convert(char *srcformat, char *srcfile, char *dstformat, char *dstfile)
 {
@@ -633,7 +636,8 @@ convert(char *srcformat, char *srcfile, char *dstformat, char *dstfile)
 #endif
 
        set_filenames();
-       init_options();
+       init_opts();
+       load_opts(rcfile);
 
        switch( import_file(srcformat, srcfile) ) {
                case -1:
@@ -663,7 +667,7 @@ convert(char *srcformat, char *srcfile, char *dstformat, char *dstfile)
                }
 
        close_database();
-       close_config();
+       free_opts();
        exit(ret);
 }
 
@@ -700,8 +704,9 @@ init_add_email()
 {
        set_filenames();
        atexit(free_filenames);
-       init_options();
-       atexit(close_config);
+       init_opts();
+       load_opts(rcfile);
+       atexit(free_opts);
 
        /*
         * we don't actually care if loading fails or not
@@ -718,6 +723,16 @@ add_email_add_item(int quiet, char *name, char *email)
 {
        list_item item;
 
+       if(opt_get_bool(BOOL_ADD_EMAIL_PREVENT_DUPLICATES)) {
+               int search_fields[] = { EMAIL, -1 };
+               if(find_item(email, 0, search_fields) >= 0) {
+                       if(!quiet)
+                               printf("Address %s already in addressbook\n",
+                                               email);
+                       return 0;
+               }
+       }
+                                       
        if(!quiet) {
                FILE *in = fopen("/dev/tty", "r");
                char c;
@@ -754,6 +769,12 @@ add_email(int quiet)
 {
        char *line;
        char *name = NULL, *email = NULL;
+       struct stat s;
+
+       if( (fstat(fileno(stdin), &s)) == -1 || S_ISDIR(s.st_mode)) {
+               fprintf(stderr, "stdin is a directory or cannot stat stdin\n");
+               exit(1);
+       }
 
        init_add_email();