[PATCH] Fix ISBN AmazonFetcher::doSearch()

View: New views
2 Messages — Rating Filter:   Alert me  

[PATCH] Fix ISBN AmazonFetcher::doSearch()

by Sandor Bodo-Merle :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Without this patch current HEAD will stuck on amazon ISBN search

---
 src/fetch/amazonfetcher.cpp |    3 +--
 1 files changed, 1 insertions(+), 2 deletions(-)

diff --git a/src/fetch/amazonfetchercpp b/src/fetch/amazonfetcher.cpp
index b757b82..54f9dbd 100644
--- a/src/fetch/amazonfetcher.cpp
+++ b/src/fetch/amazonfetcher.cpp
@@ -253,11 +253,10 @@ void AmazonFetcher::doSearch() {
             isbn13 = true;
             break;
           }
-          ++it;
         }
         // if we want isbn10, then convert all
         if(!isbn13) {
-          for(QStringList::Iterator it = isbns.begin(); it != isbns.end(); ) {
+          for(QStringList::Iterator it = isbns.begin(); it != isbns.end(); ++it) {
             if((*it).length() > 12) {
               (*it) = ISBNValidator::isbn10(*it);
               (*it).remove(QLatin1Char('-'));
--


_______________________________________________
tellico-users mailing list
tellico-users@...
http://forge.novell.com/mailman/listinfo/tellico-users

Re: [PATCH] Fix ISBN AmazonFetcher::doSearch()

by robbystephenson :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

On Thursday 19 March 2009, Sandor Bodo-Merle wrote:
> Without this patch current HEAD will stuck on amazon ISBN search

Thanks, applied.

Robby
_______________________________________________
tellico-users mailing list
tellico-users@...
http://forge.novell.com/mailman/listinfo/tellico-users