0.8.10claws76
[claws.git] / src / news.c
index f1d33847bf2c8c0b6d70a00c728925741b7aa68a..54b519d051954fa728376a88563335591f7641a2 100644 (file)
 #include "prefs_account.h"
 #include "inputdialog.h"
 #include "alertpanel.h"
-#if USE_SSL
+#include "log.h"
+#include "progressindicator.h"
+#if USE_OPENSSL
 #  include "ssl.h"
 #endif
 
 #define NNTP_PORT      119
-#if USE_SSL
+#if USE_OPENSSL
 #define NNTPS_PORT     563
 #endif
 
@@ -60,7 +62,7 @@ static void news_folder_init           (Folder        *folder,
                                          const gchar   *name,
                                          const gchar   *path);
 
-#if USE_SSL
+#if USE_OPENSSL
 static Session *news_session_new        (const gchar   *server,
                                          gushort        port,
                                          const gchar   *userid,
@@ -86,31 +88,25 @@ static gint news_select_group                (NNTPSession   *session,
                                          gint          *num,
                                          gint          *first,
                                          gint          *last);
-static GSList *news_get_uncached_articles(NNTPSession  *session,
-                                         FolderItem    *item,
-                                         gint           cache_last,
-                                         gint          *rfirst,
-                                         gint          *rlast);
 static MsgInfo *news_parse_xover        (const gchar   *xover_str);
 static gchar *news_parse_xhdr           (const gchar   *xhdr_str,
                                          MsgInfo       *msginfo);
-static GSList *news_delete_old_articles         (GSList        *alist,
-                                         FolderItem    *item,
-                                         gint           first);
-static void news_delete_all_articles    (FolderItem    *item);
-
 static gint news_remove_msg             (Folder        *folder, 
                                          FolderItem    *item, 
                                          gint           num);
-GSList *news_get_num_list               (Folder        *folder, 
-                                         FolderItem    *item);
-MsgInfo *news_fetch_msginfo             (Folder        *folder, 
+gint news_get_num_list                  (Folder        *folder, 
+                                         FolderItem    *item,
+                                         GSList       **list);
+MsgInfo *news_get_msginfo               (Folder        *folder, 
                                          FolderItem    *item,
                                          gint           num);
-GSList *news_fetch_msginfos             (Folder        *folder,
+GSList *news_get_msginfos               (Folder        *folder,
                                          FolderItem    *item,
                                          GSList        *msgnum_list);
 
+gint news_post_stream                   (Folder        *folder, 
+                                         FILE          *fp);
+
 Folder *news_folder_new(const gchar *name, const gchar *path)
 {
        Folder *folder;
@@ -121,18 +117,25 @@ Folder *news_folder_new(const gchar *name, const gchar *path)
        return folder;
 }
 
-void news_folder_destroy(NewsFolder *folder)
+void news_folder_destroy(Folder *folder)
 {
+       gchar *dir;
+
+       dir = folder_get_path(folder);
+       if (is_dir_exist(dir))
+               remove_dir_recursive(dir);
+       g_free(dir);
+
        folder_remote_folder_destroy(REMOTE_FOLDER(folder));
 }
 
 static void news_folder_init(Folder *folder, const gchar *name,
                             const gchar *path)
 {
-       folder_remote_folder_init(folder, name, path);
-
        folder->type = F_NEWS;
 
+       folder_remote_folder_init(folder, name, path);
+
 /*
        folder->get_msg_list = news_get_article_list;
 */
@@ -140,13 +143,14 @@ static void news_folder_init(Folder *folder, const gchar *name,
 /*
        folder->scan         = news_scan_group;
 */
+       folder->destroy      = news_folder_destroy;
        folder->remove_msg   = news_remove_msg;
        folder->get_num_list = news_get_num_list;
-       folder->fetch_msginfo = news_fetch_msginfo;
-       folder->fetch_msginfos = news_fetch_msginfos;
+       folder->get_msginfo  = news_get_msginfo;
+       folder->get_msginfos = news_get_msginfos;
 }
 
-#if USE_SSL
+#if USE_OPENSSL
 static Session *news_session_new(const gchar *server, gushort port,
                                 const gchar *userid, const gchar *passwd,
                                 SSLType ssl_type)
@@ -163,7 +167,7 @@ static Session *news_session_new(const gchar *server, gushort port,
 
        log_message(_("creating NNTP connection to %s:%d ...\n"), server, port);
 
-#if USE_SSL
+#if USE_OPENSSL
        if (userid && passwd)
                nntp_sock = nntp_open_auth(server, port, buf, userid, passwd,
                                           ssl_type);
@@ -188,18 +192,21 @@ static Session *news_session_new(const gchar *server, gushort port,
        SESSION(session)->phase            = SESSION_READY;
        SESSION(session)->last_access_time = time(NULL);
        SESSION(session)->data             = NULL;
+
+       SESSION(session)->destroy          = news_session_destroy;
+
        session->group = NULL;
 
        return SESSION(session);
 }
 
-void news_session_destroy(NNTPSession *session)
+void news_session_destroy(Session *session)
 {
-       nntp_close(session->nntp_sock);
-       session->nntp_sock = NULL;
-       SESSION(session)->sock = NULL;
+       nntp_close(NNTP_SESSION(session)->nntp_sock);
+       NNTP_SESSION(session)->nntp_sock = NULL;
+       session->sock = NULL;
 
-       g_free(session->group);
+       g_free(NNTP_SESSION(session)->group);
 }
 
 static Session *news_session_new_for_folder(Folder *folder)
@@ -209,6 +216,7 @@ static Session *news_session_new_for_folder(Folder *folder)
        const gchar *userid = NULL;
        gchar *passwd = NULL;
        gushort port;
+       gchar buf[NNTPBUFSIZE];
 
        g_return_val_if_fail(folder != NULL, NULL);
        g_return_val_if_fail(folder->account != NULL, NULL);
@@ -223,7 +231,7 @@ static Session *news_session_new_for_folder(Folder *folder)
                                                             userid);
        }
 
-#if USE_SSL
+#if USE_OPENSSL
        port = ac->set_nntpport ? ac->nntpport
                : ac->ssl_nntp ? NNTPS_PORT : NNTP_PORT;
        session = news_session_new(ac->nntp_server, port, userid, passwd,
@@ -232,6 +240,8 @@ static Session *news_session_new_for_folder(Folder *folder)
        port = ac->set_nntpport ? ac->nntpport : NNTP_PORT;
        session = news_session_new(ac->nntp_server, port, userid, passwd);
 #endif
+       if ((session != NULL) && ac->use_nntp_auth && ac->use_nntp_auth_onconnect)
+               nntp_forceauth(NNTP_SESSION(session)->nntp_sock, buf, userid, passwd);
 
        g_free(passwd);
 
@@ -248,20 +258,18 @@ NNTPSession *news_session_get(Folder *folder)
 
        if (!rfolder->session) {
                rfolder->session = news_session_new_for_folder(folder);
-               statusbar_pop_all();
                return NNTP_SESSION(rfolder->session);
        }
 
        if (time(NULL) - rfolder->session->last_access_time < SESSION_TIMEOUT) {
                rfolder->session->last_access_time = time(NULL);
-               statusbar_pop_all();
                return NNTP_SESSION(rfolder->session);
        }
 
        if (nntp_mode(NNTP_SESSION(rfolder->session)->nntp_sock, FALSE)
            != NN_SUCCESS) {
-               log_warning(_("NNTP connection to %s:%d has been"
-                             " disconnected. Reconnecting...\n"),
+               log_warning("NNTP connection to %s:%d has been"
+                             " disconnected. Reconnecting...\n",
                            folder->account->nntp_server,
                            folder->account->set_nntpport ?
                            folder->account->nntpport : NNTP_PORT);
@@ -271,55 +279,9 @@ NNTPSession *news_session_get(Folder *folder)
 
        if (rfolder->session)
                rfolder->session->last_access_time = time(NULL);
-       statusbar_pop_all();
        return NNTP_SESSION(rfolder->session);
 }
 
-GSList *news_get_article_list(Folder *folder, FolderItem *item,
-                             gboolean use_cache)
-{
-       GSList *alist;
-       NNTPSession *session;
-
-       g_return_val_if_fail(folder != NULL, NULL);
-       g_return_val_if_fail(item != NULL, NULL);
-       g_return_val_if_fail(folder->type == F_NEWS, NULL);
-
-       session = news_session_get(folder);
-
-       if (!session) {
-               alist = procmsg_read_cache(item, FALSE);
-               item->last_num = procmsg_get_last_num_in_cache(alist);
-       } else if (use_cache) {
-               GSList *newlist;
-               gint cache_last;
-               gint first, last;
-
-               alist = procmsg_read_cache(item, FALSE);
-
-               cache_last = procmsg_get_last_num_in_cache(alist);
-               newlist = news_get_uncached_articles
-                       (session, item, cache_last, &first, &last);
-               alist = news_delete_old_articles(alist, item, first);
-
-               alist = g_slist_concat(alist, newlist);
-               item->last_num = last;
-       } else {
-               gint last;
-
-               alist = news_get_uncached_articles
-                       (session, item, 0, NULL, &last);
-               news_delete_all_articles(item);
-               item->last_num = last;
-       }
-
-       procmsg_set_flags(alist, item);
-
-       statusbar_pop_all();
-
-       return alist;
-}
-
 gchar *news_fetch_msg(Folder *folder, FolderItem *item, gint num)
 {
        gchar *path, *filename;
@@ -336,7 +298,7 @@ gchar *news_fetch_msg(Folder *folder, FolderItem *item, gint num)
        g_free(path);
 
        if (is_file_exist(filename)) {
-               debug_print(_("article %d has been already cached.\n"), num);
+               debug_print("article %d has been already cached.\n", num);
                return filename;
        }
 
@@ -347,19 +309,19 @@ gchar *news_fetch_msg(Folder *folder, FolderItem *item, gint num)
        }
 
        ok = news_select_group(session, item->path, NULL, NULL, NULL);
-       statusbar_pop_all();
        if (ok != NN_SUCCESS) {
-               g_warning(_("can't select group %s\n"), item->path);
+               g_warning("can't select group %s\n", item->path);
                g_free(filename);
                return NULL;
        }
 
-       debug_print(_("getting article %d...\n"), num);
+       debug_print("getting article %d...\n", num);
        ok = news_get_article(NNTP_SESSION(REMOTE_FOLDER(folder)->session),
                              num, filename);
-       statusbar_pop_all();
        if (ok < 0) {
-               g_warning(_("can't read article %d\n"), num);
+               g_warning("can't read article %d\n", num);
+               session_destroy(SESSION(session));
+               REMOTE_FOLDER(folder)->session = NULL;
                g_free(filename);
                return NULL;
        }
@@ -381,7 +343,7 @@ gint news_scan_group(Folder *folder, FolderItem *item)
 
        ok = news_select_group(session, item->path, &num, &first, &last);
        if (ok != NN_SUCCESS) {
-               log_warning(_("can't set group: %s\n"), item->path);
+               log_warning("can't set group: %s\n", item->path);
                return -1;
        }
 
@@ -403,10 +365,6 @@ gint news_scan_group(Folder *folder, FolderItem *item)
        else {
                if (min < first)
                        min = first;
-               else if (first < min) {
-                       num -= min - first;
-                       first = min;
-               }
 
                if (last < max)
                        max = last;
@@ -481,12 +439,10 @@ GSList *news_get_group_list(Folder *folder)
 
                if (nntp_list(session->nntp_sock) != NN_SUCCESS) {
                        g_free(filename);
-                       statusbar_pop_all();
                        return NULL;
                }
-               statusbar_pop_all();
                if (recv_write_to_file(SESSION(session)->sock, filename) < 0) {
-                       log_warning(_("can't retrieve newsgroup list\n"));
+                       log_warning("can't retrieve newsgroup list\n");
                        session_destroy(SESSION(session));
                        REMOTE_FOLDER(folder)->session = NULL;
                        g_free(filename);
@@ -532,8 +488,6 @@ GSList *news_get_group_list(Folder *folder)
 
        list = g_slist_sort(list, (GCompareFunc)news_group_info_compare);
 
-       statusbar_pop_all();
-
        return list;
 }
 
@@ -568,7 +522,6 @@ void news_remove_group_list_cache(Folder *folder)
 
 gint news_post(Folder *folder, const gchar *file)
 {
-       NNTPSession *session;
        FILE *fp;
        gint ok;
 
@@ -576,24 +529,36 @@ gint news_post(Folder *folder, const gchar *file)
        g_return_val_if_fail(folder->type == F_NEWS, -1);
        g_return_val_if_fail(file != NULL, -1);
 
-       session = news_session_get(folder);
-       if (!session) return -1;
-
        if ((fp = fopen(file, "rb")) == NULL) {
                FILE_OP_ERROR(file, "fopen");
                return -1;
        }
 
+       ok = news_post_stream(folder, fp);
+
+       fclose(fp);
+
+       return ok;
+}
+
+gint news_post_stream(Folder *folder, FILE *fp)
+{
+       NNTPSession *session;
+       gint ok;
+
+       g_return_val_if_fail(folder != NULL, -1);
+       g_return_val_if_fail(folder->type == F_NEWS, -1);
+       g_return_val_if_fail(fp != NULL, -1);
+
+       session = news_session_get(folder);
+       if (!session) return -1;
+
        ok = nntp_post(session->nntp_sock, fp);
        if (ok != NN_SUCCESS) {
-               log_warning(_("can't post article.\n"));
+               log_warning("can't post article.\n");
                return -1;
        }
 
-       fclose(fp);
-
-       statusbar_pop_all();
-
        return 0;
 }
 
@@ -610,7 +575,7 @@ static gint news_get_article_cmd(NNTPSession *session, const gchar *cmd,
        g_free(msgid);
 
        if (recv_write_to_file(session->nntp_sock->sock, filename) < 0) {
-               log_warning(_("can't retrieve article %d\n"), num);
+               log_warning("can't retrieve article %d\n", num);
                return -1;
        }
 
@@ -619,22 +584,13 @@ static gint news_get_article_cmd(NNTPSession *session, const gchar *cmd,
 
 static gint news_remove_msg(Folder *folder, FolderItem *item, gint num)
 {
-       MsgInfo * msginfo;
-       gchar * filename;
-       MsgFlags msgflags = { 0, 0 };
+       gchar * dir;
        gint r;
 
-       filename = folder_item_fetch_msg(item, num);
-       if (filename == NULL)
-               return -1;
-
-       msginfo = procheader_parse_file(filename, msgflags, FALSE, FALSE);
-       if (msginfo == NULL)
-               return -1;
-
-       r = news_cancel_article(folder, msginfo);
-
-       procmsg_msginfo_free(msginfo);
+       dir = folder_item_get_path(item);
+       debug_print("news_remove_msg: removing msg %d in %s\n",num,dir);
+       r = remove_numbered_files(dir, num, num);
+       g_free(dir);
 
        return r;
 }
@@ -681,142 +637,6 @@ static gint news_select_group(NNTPSession *session, const gchar *group,
        return ok;
 }
 
-static GSList *news_get_uncached_articles(NNTPSession *session,
-                                         FolderItem *item, gint cache_last,
-                                         gint *rfirst, gint *rlast)
-{
-       gint ok;
-       gint num = 0, first = 0, last = 0, begin = 0, end = 0;
-       gchar buf[NNTPBUFSIZE];
-       GSList *newlist = NULL;
-       GSList *llast = NULL;
-       MsgInfo *msginfo;
-
-       if (rfirst) *rfirst = 0;
-       if (rlast)  *rlast  = 0;
-
-       g_return_val_if_fail(session != NULL, NULL);
-       g_return_val_if_fail(item != NULL, NULL);
-       g_return_val_if_fail(item->folder != NULL, NULL);
-       g_return_val_if_fail(item->folder->type == F_NEWS, NULL);
-
-       ok = news_select_group(session, item->path, &num, &first, &last);
-       if (ok != NN_SUCCESS) {
-               log_warning(_("can't set group: %s\n"), item->path);
-               return NULL;
-       }
-
-       /* calculate getting overview range */
-       if (first > last) {
-               log_warning(_("invalid article range: %d - %d\n"),
-                           first, last);
-               return NULL;
-       }
-       if (cache_last < first)
-               begin = first;
-       else if (last < cache_last)
-               begin = first;
-       else if (last == cache_last) {
-               debug_print(_("no new articles.\n"));
-               return NULL;
-       } else
-               begin = cache_last + 1;
-       end = last;
-
-       if (rfirst) *rfirst = first;
-       if (rlast)  *rlast  = last;
-
-       if (prefs_common.max_articles > 0 &&
-           end - begin + 1 > prefs_common.max_articles)
-               begin = end - prefs_common.max_articles + 1;
-
-       log_message(_("getting xover %d - %d in %s...\n"),
-                   begin, end, item->path);
-       if (nntp_xover(session->nntp_sock, begin, end) != NN_SUCCESS) {
-               log_warning(_("can't get xover\n"));
-               return NULL;
-       }
-
-       for (;;) {
-               if (sock_gets(SESSION(session)->sock, buf, sizeof(buf)) < 0) {
-                       log_warning(_("error occurred while getting xover.\n"));
-                       return newlist;
-               }
-
-               if (buf[0] == '.' && buf[1] == '\r') break;
-
-               msginfo = news_parse_xover(buf);
-               if (!msginfo) {
-                       log_warning(_("invalid xover line: %s\n"), buf);
-                       continue;
-               }
-
-               msginfo->folder = item;
-               msginfo->flags.perm_flags = MSG_NEW|MSG_UNREAD;
-               msginfo->flags.tmp_flags = MSG_NEWS;
-               msginfo->newsgroups = g_strdup(item->path);
-
-               if (!newlist)
-                       llast = newlist = g_slist_append(newlist, msginfo);
-               else {
-                       llast = g_slist_append(llast, msginfo);
-                       llast = llast->next;
-               }
-       }
-
-       if (nntp_xhdr(session->nntp_sock, "to", begin, end) != NN_SUCCESS) {
-               log_warning(_("can't get xhdr\n"));
-               return newlist;
-       }
-
-       llast = newlist;
-
-       for (;;) {
-               if (sock_gets(SESSION(session)->sock, buf, sizeof(buf)) < 0) {
-                       log_warning(_("error occurred while getting xhdr.\n"));
-                       return newlist;
-               }
-
-               if (buf[0] == '.' && buf[1] == '\r') break;
-               if (!llast) {
-                       g_warning("llast == NULL\n");
-                       continue;
-               }
-
-               msginfo = (MsgInfo *)llast->data;
-               msginfo->to = news_parse_xhdr(buf, msginfo);
-
-               llast = llast->next;
-       }
-
-       if (nntp_xhdr(session->nntp_sock, "cc", begin, end) != NN_SUCCESS) {
-               log_warning(_("can't get xhdr\n"));
-               return newlist;
-       }
-
-       llast = newlist;
-
-       for (;;) {
-               if (sock_gets(SESSION(session)->sock, buf, sizeof(buf)) < 0) {
-                       log_warning(_("error occurred while getting xhdr.\n"));
-                       return newlist;
-               }
-
-               if (buf[0] == '.' && buf[1] == '\r') break;
-               if (!llast) {
-                       g_warning("llast == NULL\n");
-                       continue;
-               }
-
-               msginfo = (MsgInfo *)llast->data;
-               msginfo->cc = news_parse_xhdr(buf, msginfo);
-
-               llast = llast->next;
-       }
-
-       return newlist;
-}
-
 #define PARSE_ONE_PARAM(p, srcp) \
 { \
        p = strchr(srcp, '\t'); \
@@ -917,58 +737,6 @@ static gchar *news_parse_xhdr(const gchar *xhdr_str, MsgInfo *msginfo)
                return g_strdup(p);
 }
 
-static GSList *news_delete_old_articles(GSList *alist, FolderItem *item,
-                                       gint first)
-{
-       GSList *cur, *next;
-       MsgInfo *msginfo;
-       gchar *dir;
-
-       g_return_val_if_fail(item != NULL, alist);
-       g_return_val_if_fail(item->folder != NULL, alist);
-       g_return_val_if_fail(item->folder->type == F_NEWS, alist);
-
-       if (first < 2) return alist;
-
-       debug_print(_("Deleting cached articles 1 - %d ... "), first - 1);
-
-       dir = folder_item_get_path(item);
-       remove_numbered_files(dir, 1, first - 1);
-       g_free(dir);
-
-       for (cur = alist; cur != NULL; ) {
-               next = cur->next;
-
-               msginfo = (MsgInfo *)cur->data;
-               if (msginfo && msginfo->msgnum < first) {
-                       procmsg_msginfo_free(msginfo);
-                       alist = g_slist_remove(alist, msginfo);
-               }
-
-               cur = next;
-       }
-       debug_print(_("done.\n"));
-
-       return alist;
-}
-
-static void news_delete_all_articles(FolderItem *item)
-{
-       gchar *dir;
-
-       g_return_if_fail(item != NULL);
-       g_return_if_fail(item->folder != NULL);
-       g_return_if_fail(item->folder->type == F_NEWS);
-
-       debug_print(_("\tDeleting all cached articles... "));
-
-       dir = folder_item_get_path(item);
-       remove_all_numbered_files(dir);
-       g_free(dir);
-
-       debug_print(_("done.\n"));
-}
-
 gint news_cancel_article(Folder * folder, MsgInfo * msginfo)
 {
        gchar * tmp;
@@ -986,7 +754,7 @@ gint news_cancel_article(Folder * folder, MsgInfo * msginfo)
        }
        if (change_file_mode_rw(tmpfp, tmp) < 0) {
                FILE_OP_ERROR(tmp, "chmod");
-               g_warning(_("can't change file mode\n"));
+               g_warning("can't change file mode\n");
        }
        
        fprintf(tmpfp, "From: %s\r\n", msginfo->from);
@@ -1010,35 +778,42 @@ gint news_cancel_article(Folder * folder, MsgInfo * msginfo)
        return 0;
 }
 
-GSList *news_get_num_list(Folder *folder, FolderItem *item)
+gint news_get_num_list(Folder *folder, FolderItem *item, GSList **msgnum_list)
 {
        NNTPSession *session;
-       gint i, ok, num, first, last;
-       GSList *msgnum_list = NULL;
+       gint i, ok, num, first, last, nummsgs = 0;
+       gchar *dir;
+
+       g_return_val_if_fail(item != NULL, -1);
+       g_return_val_if_fail(item->folder != NULL, -1);
+       g_return_val_if_fail(item->folder->type == F_NEWS, -1);
 
        session = news_session_get(folder);
-       g_return_val_if_fail(session != NULL, NULL);
-       g_return_val_if_fail(item != NULL, NULL);
-       g_return_val_if_fail(item->folder != NULL, NULL);
-       g_return_val_if_fail(item->folder->type == F_NEWS, NULL);
+       g_return_val_if_fail(session != NULL, -1);
 
        ok = news_select_group(session, item->path, &num, &first, &last);
        if (ok != NN_SUCCESS) {
                log_warning(_("can't set group: %s\n"), item->path);
-               return NULL;
+               return -1;
        }
 
        if(last < first) {
                log_warning(_("invalid article range: %d - %d\n"),
                            first, last);
-               return NULL;
+               return 0;
        }
 
        for(i = first; i <= last; i++) {
-               msgnum_list = g_slist_prepend(msgnum_list, GINT_TO_POINTER(i));
+               *msgnum_list = g_slist_prepend(*msgnum_list, GINT_TO_POINTER(i));
+               nummsgs++;
        }
 
-       return msgnum_list;
+       dir = folder_item_get_path(item);
+       debug_print("removing old messages from %d to %d in %s\n", first, last, dir);
+       remove_numbered_files(dir, 1, first - 1);
+       g_free(dir);
+
+       return nummsgs;
 }
 
 #define READ_TO_LISTEND(hdr) \
@@ -1049,7 +824,7 @@ GSList *news_get_num_list(Folder *folder, FolderItem *item)
                } \
        }
 
-MsgInfo *news_fetch_msginfo(Folder *folder, FolderItem *item, gint num)
+MsgInfo *news_get_msginfo(Folder *folder, FolderItem *item, gint num)
 {
        NNTPSession *session;
        MsgInfo *msginfo = NULL;
@@ -1119,12 +894,13 @@ MsgInfo *news_fetch_msginfo(Folder *folder, FolderItem *item, gint num)
        return msginfo;
 }
 
-static GSList *news_fetch_msginfo_from_to(NNTPSession *session, FolderItem *item, guint begin, guint end)
+static GSList *news_get_msginfos_for_range(NNTPSession *session, FolderItem *item, guint begin, guint end)
 {
        gchar buf[NNTPBUFSIZE];
        GSList *newlist = NULL;
        GSList *llast = NULL;
        MsgInfo *msginfo;
+       guint count = 0, lines = (end - begin + 1) * 3;
 
        g_return_val_if_fail(session != NULL, NULL);
        g_return_val_if_fail(item != NULL, NULL);
@@ -1141,6 +917,9 @@ static GSList *news_fetch_msginfo_from_to(NNTPSession *session, FolderItem *item
                        log_warning(_("error occurred while getting xover.\n"));
                        return newlist;
                }
+               count++;
+               progressindicator_set_percentage
+                       (PROGRESS_TYPE_NETWORK, session->fetch_base_percentage + ((gfloat) count / (gfloat) lines));
 
                if (buf[0] == '.' && buf[1] == '\r') break;
 
@@ -1175,6 +954,9 @@ static GSList *news_fetch_msginfo_from_to(NNTPSession *session, FolderItem *item
                        log_warning(_("error occurred while getting xhdr.\n"));
                        return newlist;
                }
+               count++;
+               progressindicator_set_percentage
+                       (PROGRESS_TYPE_NETWORK, session->fetch_base_percentage + ((gfloat) count / (gfloat) lines));
 
                if (buf[0] == '.' && buf[1] == '\r') break;
                if (!llast) {
@@ -1200,6 +982,9 @@ static GSList *news_fetch_msginfo_from_to(NNTPSession *session, FolderItem *item
                        log_warning(_("error occurred while getting xhdr.\n"));
                        return newlist;
                }
+               count++;
+               progressindicator_set_percentage
+                       (PROGRESS_TYPE_NETWORK, session->fetch_base_percentage + ((gfloat) count / (gfloat) lines));
 
                if (buf[0] == '.' && buf[1] == '\r') break;
                if (!llast) {
@@ -1221,11 +1006,12 @@ gint news_fetch_msgnum_sort(gconstpointer a, gconstpointer b)
        return (GPOINTER_TO_INT(a) - GPOINTER_TO_INT(b));
 }
 
-GSList *news_fetch_msginfos(Folder *folder, FolderItem *item, GSList *msgnum_list)
+GSList *news_get_msginfos(Folder *folder, FolderItem *item, GSList *msgnum_list)
 {
        NNTPSession *session;
        GSList *elem, *msginfo_list = NULL, *tmp_msgnum_list, *tmp_msginfo_list;
        guint first, last, next;
+       guint tofetch, fetched;
        
        g_return_val_if_fail(folder != NULL, NULL);
        g_return_val_if_fail(folder->type == F_NEWS, NULL);
@@ -1238,21 +1024,29 @@ GSList *news_fetch_msginfos(Folder *folder, FolderItem *item, GSList *msgnum_lis
        tmp_msgnum_list = g_slist_copy(msgnum_list);
        tmp_msgnum_list = g_slist_sort(tmp_msgnum_list, news_fetch_msgnum_sort);
 
+       progressindicator_start(PROGRESS_TYPE_NETWORK);
+       tofetch = g_slist_length(tmp_msgnum_list);
+       fetched = 0;
+
        first = GPOINTER_TO_INT(tmp_msgnum_list->data);
        last = first;
        for(elem = g_slist_next(tmp_msgnum_list); elem != NULL; elem = g_slist_next(elem)) {
                next = GPOINTER_TO_INT(elem->data);
                if(next != (last + 1)) {
-                       tmp_msginfo_list = news_fetch_msginfo_from_to(session, item, first, last);
+                       session->fetch_base_percentage = (gfloat) fetched / (gfloat) tofetch;
+                       tmp_msginfo_list = news_get_msginfos_for_range(session, item, first, last);
                        msginfo_list = g_slist_concat(msginfo_list, tmp_msginfo_list);
+                       fetched = last - first + 1;
                        first = next;
                }
                last = next;
        }
-       tmp_msginfo_list = news_fetch_msginfo_from_to(session, item, first, last);
+       tmp_msginfo_list = news_get_msginfos_for_range(session, item, first, last);
        msginfo_list = g_slist_concat(msginfo_list, tmp_msginfo_list);
 
        g_slist_free(tmp_msgnum_list);
        
+       progressindicator_stop(PROGRESS_TYPE_NETWORK);
+
        return msginfo_list;
 }