OSDN Git Service

- Correct issue with importing when not a premium member.
[neighbornote/NeighborNote.git] / src / cx / fbn / nevernote / threads / SyncRunner.java
index 8ad6119..9933f25 100644 (file)
@@ -1,5 +1,5 @@
 /*\r
- * This file is part of NeverNote \r
+ * This file is part of NixNote \r
  * Copyright 2009 Randy Baumgarte\r
  * \r
  * This file may be licensed under the terms of of the\r
@@ -29,6 +29,7 @@ import java.util.ArrayList;
 import java.util.Calendar;\r
 import java.util.Date;\r
 import java.util.GregorianCalendar;\r
+import java.util.HashMap;\r
 import java.util.List;\r
 import java.util.TreeSet;\r
 import java.util.Vector;\r
@@ -53,6 +54,7 @@ import com.evernote.edam.error.EDAMNotFoundException;
 import com.evernote.edam.error.EDAMSystemException;\r
 import com.evernote.edam.error.EDAMUserException;\r
 import com.evernote.edam.notestore.NoteStore;\r
+import com.evernote.edam.notestore.NoteStore.Client;\r
 import com.evernote.edam.notestore.SyncChunk;\r
 import com.evernote.edam.notestore.SyncState;\r
 import com.evernote.edam.type.Data;\r
@@ -70,6 +72,7 @@ import com.trolltech.qt.core.QByteArray;
 import com.trolltech.qt.core.QFile;\r
 import com.trolltech.qt.core.QIODevice.OpenModeFlag;\r
 import com.trolltech.qt.core.QObject;\r
+import com.trolltech.qt.core.QTextCodec;\r
 import com.trolltech.qt.gui.QMessageBox;\r
 \r
 import cx.fbn.nevernote.signals.NoteIndexSignal;\r
@@ -90,12 +93,17 @@ public class SyncRunner extends QObject implements Runnable {
                private DatabaseConnection              conn;\r
                private boolean                                 idle;\r
                public boolean                                  error;\r
+               public volatile Vector<String>  errorSharedNotebooks;\r
+               public volatile HashMap<String,String>  errorSharedNotebooksIgnored;\r
                public volatile boolean                 isConnected;\r
                public volatile boolean                 keepRunning;\r
                public volatile String                  authToken;\r
                private long                                    evernoteUpdateCount;\r
+               private final String userAgent = "NixNote/" + System.getProperty("os.name")\r
+                                                               +"/"+System.getProperty("java.vendor") + "/"\r
+                                                               + System.getProperty("java.version") +";";\r
                \r
-               public volatile NoteStore.Client                noteStore;\r
+               public volatile NoteStore.Client                localNoteStore;\r
                private UserStore.Client                                userStore;\r
                \r
                public volatile StatusSignal                    status;\r
@@ -115,15 +123,15 @@ public class SyncRunner extends QObject implements Runnable {
            public volatile String username = ""; \r
            public volatile String password = ""; \r
                public volatile String userStoreUrl;\r
-           private final static String consumerKey = "baumgarte"; \r
-           private final static String consumerSecret = "eb8b5740e17cb55f";\r
+//         private final static String consumerKey = "baumgarte"; \r
+//         private final static String consumerSecret = "eb8b5740e17cb55f";\r
            public String noteStoreUrlBase;\r
            private THttpClient userStoreTrans;\r
            private TBinaryProtocol userStoreProt;\r
-           private AuthenticationResult authResult;\r
+           //private AuthenticationResult authResult;\r
            private AuthenticationResult linkedAuthResult;\r
            private User user; \r
-           private long authTimeRemaining;\r
+//         private long authTimeRemaining;\r
            public long authRefreshTime;\r
            public long failedRefreshes = 0;\r
            public  THttpClient noteStoreTrans;\r
@@ -136,15 +144,18 @@ public class SyncRunner extends QObject implements Runnable {
                private static int MAX_QUEUED_WAITING = 1000;\r
                String dbuid;\r
                String dburl;\r
+               String indexUrl;\r
+               String resourceUrl;\r
                String dbpswd;\r
                String dbcpswd;\r
                private final TreeSet<String> ignoreTags;\r
                private final TreeSet<String> ignoreNotebooks;\r
                private final TreeSet<String> ignoreLinkedNotebooks;\r
+               private HashMap<String,String> badTagSync;\r
        \r
                \r
                \r
-       public SyncRunner(String logname, String u, String uid, String pswd, String cpswd) {\r
+       public SyncRunner(String logname, String u, String i, String r, String uid, String pswd, String cpswd) {\r
                logger = new ApplicationLogger(logname);\r
                \r
                noteSignal = new NoteSignal();\r
@@ -156,6 +167,8 @@ public class SyncRunner extends QObject implements Runnable {
                searchSignal = new SavedSearchSignal();\r
                syncSignal = new SyncSignal();\r
                resourceSignal = new NoteResourceSignal();\r
+               resourceUrl = r;\r
+               indexUrl = i;\r
                dbuid = uid;\r
                dburl = u;\r
                dbpswd = pswd;\r
@@ -167,9 +180,6 @@ public class SyncRunner extends QObject implements Runnable {
                authRefreshNeeded = false;\r
                keepRunning = true;\r
                idle = true;\r
-               noteStore = null;\r
-               userStore = null;\r
-               authToken = null;\r
                disableUploads = false;\r
                ignoreTags = new TreeSet<String>();\r
                ignoreNotebooks = new TreeSet<String>();\r
@@ -180,20 +190,22 @@ public class SyncRunner extends QObject implements Runnable {
        }\r
        @Override\r
        public void run() {\r
+               errorSharedNotebooks = new Vector<String>();\r
+               errorSharedNotebooksIgnored = new HashMap<String,String>();\r
                try {\r
                        logger.log(logger.EXTREME, "Starting thread");\r
-                       conn = new DatabaseConnection(logger, dburl, dbuid, dbpswd, dbcpswd, 200);\r
+                       conn = new DatabaseConnection(logger, dburl, indexUrl, resourceUrl, dbuid, dbpswd, dbcpswd, 200);\r
                        while(keepRunning) {\r
+                               logger.log(logger.EXTREME, "Blocking until work is found");\r
                                String work = workQueue.take();\r
+                               logger.log(logger.LOW, "Dirty Notes Before Sync: " +new Integer(conn.getNoteTable().getDirtyCount()).toString());\r
                                logger.log(logger.EXTREME, "Work found: " +work);\r
-                               if (work.equalsIgnoreCase("stop"))\r
+                               if (work.equalsIgnoreCase("stop")) {\r
+                                       idle=false;\r
                                        return;\r
+                               }\r
                                idle=false;\r
                                error=false;\r
-                               if (authRefreshNeeded == true || !isConnected) {\r
-                                       logger.log(logger.EXTREME, "Refreshing connection");\r
-                                       refreshConnection();\r
-                               }\r
                                if (syncNeeded) {\r
                                        logger.log(logger.EXTREME, "SyncNeeded is true");\r
                                        refreshNeeded=false;\r
@@ -201,13 +213,12 @@ public class SyncRunner extends QObject implements Runnable {
                                        updateSequenceNumber = conn.getSyncTable().getUpdateSequenceNumber();\r
                                        try {\r
                                                logger.log(logger.EXTREME, "Beginning sync");\r
-                                               evernoteSync();\r
+                                               evernoteSync(localNoteStore);\r
                                                logger.log(logger.EXTREME, "Sync finished");\r
                                        } catch (UnknownHostException e) {\r
                                                status.message.emit(e.getMessage());\r
                                        }\r
                                }\r
-                               dirtyNoteGuids = null;\r
                                idle=true;\r
                                logger.log(logger.EXTREME, "Signaling refresh finished.  refreshNeeded=" +refreshNeeded);\r
                                syncSignal.finished.emit(refreshNeeded);\r
@@ -215,6 +226,7 @@ public class SyncRunner extends QObject implements Runnable {
                                        syncSignal.errorDisconnect.emit();\r
                                        status.message.emit(tr("Error synchronizing - see log for details."));\r
                                }\r
+                               logger.log(logger.LOW, "Dirty Notes After Sync: " +new Integer(conn.getNoteTable().getDirtyCount()).toString());\r
                        }\r
                }       \r
                catch (InterruptedException e1) {\r
@@ -242,7 +254,7 @@ public class SyncRunner extends QObject implements Runnable {
        }\r
        public void setNoteStore(NoteStore.Client c) {\r
                logger.log(logger.EXTREME, "Setting NoteStore in sync thread");\r
-               noteStore = c;\r
+               localNoteStore = c;\r
        }\r
        public void setUserStore(UserStore.Client c) {\r
                logger.log(logger.EXTREME, "Setting UserStore in sync thread");\r
@@ -261,7 +273,7 @@ public class SyncRunner extends QObject implements Runnable {
     //***************************************************************\r
        // Synchronize changes with Evernote\r
        @SuppressWarnings("unused")\r
-       private void evernoteSync() throws java.net.UnknownHostException {\r
+       private void evernoteSync(Client noteStore) throws java.net.UnknownHostException {\r
                logger.log(logger.HIGH, "Entering SyncRunner.evernoteSync");\r
                \r
                // Rebuild list of tags & notebooks to ignore\r
@@ -353,6 +365,8 @@ public class SyncRunner extends QObject implements Runnable {
                        // If we need to do a full sync.\r
                        logger.log(logger.LOW, "Full Sequence Before: " +syncState.getFullSyncBefore());\r
                        logger.log(logger.LOW, "Last Sequence Date: " +sequenceDate);\r
+                       logger.log(logger.LOW, "Var Last Sequence Number: " +updateSequenceNumber);\r
+                       logger.log(logger.LOW, "DB Last Sequence Number: " + conn.getSyncTable().getUpdateSequenceNumber());\r
                        if (syncState.getFullSyncBefore() > sequenceDate) {\r
                                logger.log(logger.EXTREME, "Full sequence date has expired");\r
                                sequenceDate = 0;\r
@@ -362,17 +376,17 @@ public class SyncRunner extends QObject implements Runnable {
                        }\r
                        // Check for "special" sync instructions\r
                        String syncLinked = conn.getSyncTable().getRecord("FullLinkedNotebookSync");\r
-                       String syncShared = conn.getSyncTable().getRecord("FullLinkedNotebookSync");\r
+                       String syncShared = conn.getSyncTable().getRecord("FullSharedNotebookSync");\r
                        String syncNotebooks = conn.getSyncTable().getRecord("FullNotebookSync");\r
                        String syncInkNoteImages = conn.getSyncTable().getRecord("FullInkNoteImageSync");\r
                        if (syncLinked != null) {\r
-                               downloadAllLinkedNotebooks();\r
+                               downloadAllLinkedNotebooks(localNoteStore);\r
                        }\r
                        if (syncShared != null) {\r
-                               downloadAllSharedNotebooks();\r
+                               downloadAllSharedNotebooks(localNoteStore);\r
                        }\r
                        if (syncNotebooks != null) {\r
-                               downloadAllNotebooks();\r
+                               downloadAllNotebooks(localNoteStore);\r
                        }\r
                        \r
                        if (syncInkNoteImages != null) {\r
@@ -391,7 +405,7 @@ public class SyncRunner extends QObject implements Runnable {
                                logger.log(logger.EXTREME, "Refresh needed is true");\r
                                refreshNeeded = true;\r
                                logger.log(logger.EXTREME, "Downloading changes");\r
-                               syncRemoteToLocal();\r
+                               syncRemoteToLocal(localNoteStore);\r
                        }\r
                        \r
                        //*****************************************\r
@@ -403,19 +417,19 @@ public class SyncRunner extends QObject implements Runnable {
                                logger.log(logger.EXTREME, "Uploading changes");\r
                                // Synchronize remote changes\r
                                if (!error)\r
-                                       syncExpunged();\r
+                                       syncExpunged(localNoteStore);\r
                                if (!error)\r
-                                       syncLocalTags();\r
+                                       syncLocalTags(localNoteStore);\r
                                if (!error)\r
-                                       syncLocalNotebooks();\r
+                                       syncLocalNotebooks(localNoteStore);\r
                                if (!error)\r
-                                       syncLocalLinkedNotebooks();\r
+                                       syncLocalLinkedNotebooks(localNoteStore);\r
                                if (!error) \r
-                                       syncDeletedNotes();\r
+                                       syncDeletedNotes(localNoteStore);\r
                                if (!error)\r
                                        syncLocalNotes();\r
                                if (!error)\r
-                                       syncLocalSavedSearches();\r
+                                       syncLocalSavedSearches(localNoteStore);\r
                        }\r
                        \r
                        status.message.emit(tr("Cleaning up"));\r
@@ -431,7 +445,7 @@ public class SyncRunner extends QObject implements Runnable {
                                syncSignal.refreshLists.emit();\r
                        \r
                        if (!error) {\r
-                               logger.log(logger.EXTREME, "Sync completed.  Errors=" +error);\r
+                               logger.log(logger.LOW, "Sync completed.  Errors=" +error);\r
                                if (!disableUploads) \r
                                        status.message.emit(tr("Synchronizing complete"));\r
                                else\r
@@ -450,51 +464,56 @@ public class SyncRunner extends QObject implements Runnable {
        }\r
        \r
        // Sync deleted items with Evernote\r
-       private void syncExpunged() {\r
+       private void syncExpunged(Client noteStore) {\r
                logger.log(logger.HIGH, "Entering SyncRunner.syncExpunged");\r
                \r
                List<DeletedItemRecord> expunged = conn.getDeletedTable().getAllDeleted();\r
                boolean error = false;\r
                for (int i=0; i<expunged.size() && keepRunning; i++) {\r
-                       \r
-                       if (authRefreshNeeded)\r
-                               if (!refreshConnection())\r
-                                       return;\r
+\r
+//                     if (authRefreshNeeded)\r
+//                             if (!refreshConnection())\r
+//                                     return;\r
 \r
                        try {\r
                                if (expunged.get(i).type.equalsIgnoreCase("TAG")) {\r
                                        logger.log(logger.EXTREME, "Tag expunged");\r
                                        updateSequenceNumber = noteStore.expungeTag(authToken, expunged.get(i).guid);\r
                                        conn.getSyncTable().setUpdateSequenceNumber(updateSequenceNumber);\r
-                                       \r
+                                       conn.getSyncTable().setLastSequenceDate(sequenceDate);\r
+                                       conn.getSyncTable().setUpdateSequenceNumber(updateSequenceNumber);\r
+                                       conn.getDeletedTable().expungeDeletedItem(expunged.get(i).guid, "TAG");                                 \r
                                }\r
                                if      (expunged.get(i).type.equalsIgnoreCase("NOTEBOOK")) {\r
                                        logger.log(logger.EXTREME, "Notebook expunged");\r
                                        updateSequenceNumber = noteStore.expungeNotebook(authToken, expunged.get(i).guid);\r
                                        conn.getSyncTable().setLastSequenceDate(sequenceDate);\r
+                                       conn.getSyncTable().setUpdateSequenceNumber(updateSequenceNumber);\r
+                                       conn.getDeletedTable().expungeDeletedItem(expunged.get(i).guid, "NOTEBOOK");\r
                                }\r
                                if (expunged.get(i).type.equalsIgnoreCase("NOTE")) {\r
                                        logger.log(logger.EXTREME, "Note expunged");\r
                                        updateSequenceNumber = noteStore.deleteNote(authToken, expunged.get(i).guid);\r
+                                       refreshNeeded = true;\r
+                                       conn.getSyncTable().setLastSequenceDate(sequenceDate);\r
                                        conn.getSyncTable().setUpdateSequenceNumber(updateSequenceNumber);\r
+                                       conn.getDeletedTable().expungeDeletedItem(expunged.get(i).guid, "NOTE");\r
                                }\r
                                if (expunged.get(i).type.equalsIgnoreCase("SAVEDSEARCH")) {\r
                                        logger.log(logger.EXTREME, "saved search expunged");\r
                                        updateSequenceNumber = noteStore.expungeSearch(authToken, expunged.get(i).guid);\r
                                        conn.getSyncTable().setLastSequenceDate(sequenceDate);\r
+                                       conn.getSyncTable().setUpdateSequenceNumber(updateSequenceNumber);\r
+                                       conn.getDeletedTable().expungeDeletedItem(expunged.get(i).guid, "SAVEDSEARCH");\r
                                }\r
                        } catch (EDAMUserException e) {\r
-                               logger.log(logger.LOW, "EDAM User Excepton in syncExpunged: " +expunged.get(i).guid);\r
-                               logger.log(logger.LOW, e.getStackTrace());\r
-                               error = true;\r
+                               logger.log(logger.LOW, "EDAM User Excepton in syncExpunged: " +expunged.get(i).guid);   // This can happen if we try to delete a deleted note\r
                        } catch (EDAMSystemException e) {\r
                                logger.log(logger.LOW, "EDAM System Excepton in syncExpunged: "+expunged.get(i).guid);\r
                                logger.log(logger.LOW, e.getStackTrace());\r
                                error=true;\r
                        } catch (EDAMNotFoundException e) {\r
                                logger.log(logger.LOW, "EDAM Not Found Excepton in syncExpunged: "+expunged.get(i).guid);\r
-//                             logger.log(logger.LOW, e.getStackTrace());\r
-                               //error=true;\r
                        } catch (TException e) {\r
                                logger.log(logger.LOW, "EDAM TExcepton in syncExpunged: "+expunged.get(i).guid);\r
                                logger.log(logger.LOW, e.getStackTrace());\r
@@ -507,7 +526,7 @@ public class SyncRunner extends QObject implements Runnable {
                logger.log(logger.HIGH, "Leaving SyncRunner.syncExpunged");\r
 \r
        }\r
-       private void syncDeletedNotes() {\r
+       private void syncDeletedNotes(Client noteStore) {\r
                if (syncDeletedContent)\r
                        return;\r
                logger.log(logger.HIGH, "Entering SyncRunner.syncDeletedNotes");\r
@@ -517,9 +536,9 @@ public class SyncRunner extends QObject implements Runnable {
                // Sync the local notebooks with Evernote's\r
                for (int i=0; i<notes.size() && keepRunning; i++) {\r
                        \r
-                       if (authRefreshNeeded)\r
-                               if (!refreshConnection())\r
-                                       return;\r
+//                     if (authRefreshNeeded)\r
+//                             if (!refreshConnection())\r
+//                                     return;\r
                        \r
                        Note enNote = notes.get(i);\r
                        try {\r
@@ -569,38 +588,38 @@ public class SyncRunner extends QObject implements Runnable {
        // Sync notes with Evernote\r
        private void syncLocalNotes() {\r
                logger.log(logger.HIGH, "Entering SyncRunner.syncNotes");\r
+               logger.log(logger.LOW, "Dirty local notes found: " +new Integer(conn.getNoteTable().getDirtyCount()).toString());\r
                status.message.emit(tr("Sending local notes."));\r
 \r
                List<Note> notes = conn.getNoteTable().getDirty();\r
                // Sync the local notebooks with Evernote's\r
                for (int i=0; i<notes.size() && keepRunning; i++) {\r
-                       syncLocalNote(notes.get(i), authToken);\r
+                       syncLocalNote(localNoteStore, notes.get(i), authToken);\r
                }\r
-               logger.log(logger.HIGH, "Entering SyncRunner.syncNotes");\r
+               logger.log(logger.HIGH, "Leaving SyncRunner.syncNotes");\r
 \r
        }\r
        // Sync notes with Evernote\r
-       private void syncLocalNote(Note enNote, String token) {\r
+       private void syncLocalNote(Client noteStore, Note enNote, String token) {\r
                logger.log(logger.HIGH, "Entering SyncRunner.syncNotes");\r
                status.message.emit(tr("Sending local notes."));\r
-\r
-               if (authRefreshNeeded)\r
-                       if (!refreshConnection())\r
-                               return;\r
                        \r
                if (enNote.isActive()) {\r
                        try {\r
-                               logger.log(logger.EXTREME, "Active dirty note found - non new");\r
                                if (enNote.getUpdateSequenceNum() > 0) {\r
+                                       logger.log(logger.EXTREME, "Active dirty note found - non new - " +enNote.getGuid());\r
+                                       logger.log(logger.EXTREME, "Fetching note content");\r
                                        enNote = getNoteContent(enNote);\r
                                        logger.log(logger.MEDIUM, "Updating note : "+ enNote.getGuid() +" <title>" +enNote.getTitle()+"</title>");\r
                                        enNote = noteStore.updateNote(token, enNote);\r
                                } else { \r
-                                       logger.log(logger.EXTREME, "Active dirty found - new note");\r
-                                       logger.log(logger.MEDIUM, "Creating note : "+ enNote.getGuid() +" <title>" +enNote.getTitle()+"</title>");\r
+                                       logger.log(logger.EXTREME, "Active dirty found - new note " +enNote.getGuid());\r
                                        String oldGuid = enNote.getGuid();\r
+                                       logger.log(logger.MEDIUM, "Fetching note content");\r
                                        enNote = getNoteContent(enNote);\r
+                                       logger.log(logger.MEDIUM, "Creating note : "+ enNote.getGuid() +" <title>" +enNote.getTitle()+"</title>");\r
                                        enNote = noteStore.createNote(token, enNote);\r
+                                       logger.log(logger.MEDIUM, "New note Guid : "+ enNote.getGuid() +" <title>" +enNote.getTitle()+"</title>");\r
                                        noteSignal.guidChanged.emit(oldGuid, enNote.getGuid());\r
                                        conn.getNoteTable().updateNoteGuid(oldGuid, enNote.getGuid());\r
                                }\r
@@ -654,7 +673,7 @@ public class SyncRunner extends QObject implements Runnable {
        }\r
 \r
        // Sync Notebooks with Evernote\r
-       private void syncLocalNotebooks() {\r
+       private void syncLocalNotebooks(Client noteStore) {\r
                logger.log(logger.HIGH, "Entering SyncRunner.syncLocalNotebooks");\r
                \r
                status.message.emit(tr("Sending local notebooks."));\r
@@ -684,9 +703,9 @@ public class SyncRunner extends QObject implements Runnable {
                // Sync the local notebooks with Evernote's\r
                for (int i=0; i<notebooks.size() && keepRunning; i++) {\r
                        \r
-                       if (authRefreshNeeded)\r
-                               if (!refreshConnection())\r
-                                       return;\r
+//                     if (authRefreshNeeded)\r
+//                             if (!refreshConnection())\r
+//                                     return;\r
                        \r
                        Notebook enNotebook = notebooks.get(i);\r
                        try {\r
@@ -744,7 +763,7 @@ public class SyncRunner extends QObject implements Runnable {
 \r
        }\r
        // Sync Tags with Evernote\r
-       private void syncLocalTags() {\r
+       private void syncLocalTags(Client noteStore) {\r
                logger.log(logger.HIGH, "Entering SyncRunner.syncLocalTags");\r
                List<Tag> remoteList = new ArrayList<Tag>();\r
                status.message.emit(tr("Sending local tags."));\r
@@ -771,11 +790,24 @@ public class SyncRunner extends QObject implements Runnable {
                \r
                int sequence;\r
                \r
+               if (badTagSync == null)\r
+                       badTagSync = new HashMap<String,String>();\r
+               else\r
+                       badTagSync.clear();\r
+               \r
                Tag enTag = findNextTag();\r
-               while(enTag!=null) {\r
-                       if (authRefreshNeeded)\r
-                               if (!refreshConnection())\r
-                                       return;\r
+               \r
+               // This is a hack.  Sometimes this function goes flookey and goes into a \r
+               // perpetual loop.  This causes  NeverNote to flood Evernote's servers.\r
+               // This is a safety valve to prevent unlimited loops.\r
+               int maxCount = conn.getTagTable().getDirty().size()+10;\r
+               int loopCount = 0;\r
+               \r
+               while(enTag!=null && loopCount < maxCount) {\r
+                       loopCount++;\r
+//                     if (authRefreshNeeded)\r
+//                             if (!refreshConnection())\r
+//                                     return;\r
 \r
                        try {\r
                                if (enTag.getUpdateSequenceNum() > 0) {\r
@@ -814,20 +846,24 @@ public class SyncRunner extends QObject implements Runnable {
                                updateSequenceNumber = sequence;\r
                                conn.getSyncTable().setUpdateSequenceNumber(updateSequenceNumber);\r
                        } catch (EDAMUserException e) {\r
-                               logger.log(logger.LOW, "*** EDAM User Excepton syncLocalTags");\r
-                               logger.log(logger.LOW, e.toString());           \r
+                               logger.log(logger.LOW, "*** EDAM User Excepton syncLocalTags: " +enTag.getName());\r
+                               logger.log(logger.LOW, e.toString());\r
+                               badTagSync.put(enTag.getGuid(),null);\r
                                error = true;\r
                        } catch (EDAMSystemException e) {\r
-                               logger.log(logger.LOW, "** EDAM System Excepton syncLocalTags");\r
+                               logger.log(logger.LOW, "** EDAM System Excepton syncLocalTags: " +enTag.getName());\r
                                logger.log(logger.LOW, e.toString());   \r
+                               badTagSync.put(enTag.getGuid(),null);\r
                                error = true;\r
                        } catch (EDAMNotFoundException e) {\r
-                               logger.log(logger.LOW, "*** EDAM Not Found Excepton syncLocalTags");\r
-                               logger.log(logger.LOW, e.toString());   \r
+                               logger.log(logger.LOW, "*** EDAM Not Found Excepton syncLocalTags: " +enTag.getName());\r
+                               logger.log(logger.LOW, e.toString());\r
+                               badTagSync.put(enTag.getGuid(),null);\r
                                error = true;\r
                        } catch (TException e) {\r
-                               logger.log(logger.LOW, "*** EDAM TExcepton syncLocalTags");\r
-                               logger.log(logger.LOW, e.toString());           \r
+                               logger.log(logger.LOW, "*** EDAM TExcepton syncLocalTags: " +enTag.getName());\r
+                               logger.log(logger.LOW, e.toString());\r
+                               badTagSync.put(enTag.getGuid(),null);\r
                                error = true;\r
                        }       \r
                        \r
@@ -837,7 +873,7 @@ public class SyncRunner extends QObject implements Runnable {
                }\r
                logger.log(logger.HIGH, "Leaving SyncRunner.syncLocalTags");\r
        }\r
-       private void syncLocalLinkedNotebooks() {\r
+       private void syncLocalLinkedNotebooks(Client noteStore) {\r
                logger.log(logger.HIGH, "Entering SyncRunner.syncLocalLinkedNotebooks");\r
                \r
                List<String> list = conn.getLinkedNotebookTable().getDirtyGuids();\r
@@ -874,7 +910,7 @@ public class SyncRunner extends QObject implements Runnable {
                logger.log(logger.HIGH, "Leaving SyncRunner.syncLocalLinkedNotebooks");\r
        }\r
        // Sync Saved Searches with Evernote\r
-       private void syncLocalSavedSearches() {\r
+       private void syncLocalSavedSearches(Client noteStore) {\r
                logger.log(logger.HIGH, "Entering SyncRunner.syncLocalSavedSearches");\r
                List<SavedSearch> remoteList = new ArrayList<SavedSearch>();\r
                status.message.emit(tr("Sending saved searches."));\r
@@ -905,9 +941,9 @@ public class SyncRunner extends QObject implements Runnable {
                logger.log(logger.EXTREME, "Beginning to send saved searches");\r
                for (int i=0; i<searches.size() &&  keepRunning; i++) {\r
                        \r
-                       if (authRefreshNeeded)\r
-                               if (!refreshConnection())\r
-                                       return;\r
+//                     if (authRefreshNeeded)\r
+//                             if (!refreshConnection())\r
+//                                     return;\r
                        \r
                        SavedSearch enSearch = searches.get(i);\r
                        try {\r
@@ -933,7 +969,7 @@ public class SyncRunner extends QObject implements Runnable {
                                                enSearch = noteStore.createSearch(authToken, enSearch);\r
                                        sequence = enSearch.getUpdateSequenceNum();\r
                                        logger.log(logger.EXTREME, "Updating tag guid in local database");\r
-                                       conn.getTagTable().updateTagGuid(oldGuid, enSearch.getGuid());\r
+                                       conn.getSavedSearchTable().updateSavedSearchGuid(oldGuid, enSearch.getGuid());\r
                                }\r
                                logger.log(logger.EXTREME, "Updating tag sequence in local database");\r
                                conn.getSavedSearchTable().updateSavedSearchSequence(enSearch.getGuid(), sequence);\r
@@ -960,11 +996,12 @@ public class SyncRunner extends QObject implements Runnable {
                                error = true;\r
                        }               \r
                }\r
+\r
                logger.log(logger.HIGH, "Entering SyncRunner.syncLocalSavedSearches");\r
        }       \r
 \r
        // Sync evernote changes with local database\r
-       private void syncRemoteToLocal() {\r
+       private void syncRemoteToLocal(Client noteStore) {\r
                logger.log(logger.HIGH, "Entering SyncRunner.syncRemoteToLocal");\r
 \r
                List<Note> dirtyNotes = conn.getNoteTable().getDirty();\r
@@ -985,15 +1022,16 @@ public class SyncRunner extends QObject implements Runnable {
                \r
                while(more &&  keepRunning) {\r
                        \r
-                       if (authRefreshNeeded)\r
-                               if (!refreshConnection())\r
-                                       return;\r
+//                     if (authRefreshNeeded)\r
+//                             if (!refreshConnection())\r
+//                                     return;\r
                        \r
-                       chunk = null;\r
                        int sequence = updateSequenceNumber;\r
                        try {\r
+//                             conn.beginTransaction();\r
                                logger.log(logger.EXTREME, "Getting chunk from Evernote");\r
                                chunk = noteStore.getSyncChunk(authToken, sequence, chunkSize, fullSync);\r
+                               logger.log(logger.LOW, "Chunk High Sequence: " +chunk.getChunkHighUSN());\r
                        } catch (EDAMUserException e) {\r
                                error = true;\r
                                e.printStackTrace();\r
@@ -1015,9 +1053,13 @@ public class SyncRunner extends QObject implements Runnable {
                        syncRemoteTags(chunk.getTags());\r
                        syncRemoteSavedSearches(chunk.getSearches());\r
                        syncRemoteNotebooks(chunk.getNotebooks());\r
-                       syncRemoteNotes(chunk.getNotes(), fullSync, authToken);\r
-                       syncRemoteResources(chunk.getResources());\r
+                       syncRemoteNotes(noteStore, chunk.getNotes(), fullSync, authToken);\r
+                       syncRemoteResources(noteStore, chunk.getResources());\r
                        syncRemoteLinkedNotebooks(chunk.getLinkedNotebooks());\r
+                       \r
+                       // Signal about any updated notes to invalidate the cache\r
+                       for (int i=0; i<chunk.getNotesSize(); i++) \r
+                               noteSignal.noteChanged.emit(chunk.getNotes().get(i).getGuid(), null); \r
                        syncExpungedNotes(chunk);\r
                        \r
                        \r
@@ -1030,11 +1072,12 @@ public class SyncRunner extends QObject implements Runnable {
 \r
                        \r
                        // Save the chunk sequence number\r
-                       if (!error && chunk.getChunkHighUSN() > 0) {\r
+                       if (!error && chunk.getChunkHighUSN() > 0 && keepRunning) {\r
                                logger.log(logger.EXTREME, "emitting sequence number to main thread");\r
                                updateSequenceNumber = chunk.getChunkHighUSN();\r
                                conn.getSyncTable().setLastSequenceDate(chunk.getCurrentTime());\r
                                conn.getSyncTable().setUpdateSequenceNumber(updateSequenceNumber);\r
+//                             conn.commitTransaction();\r
                        }\r
                        \r
                        \r
@@ -1044,8 +1087,8 @@ public class SyncRunner extends QObject implements Runnable {
                                pct = pct/evernoteUpdateCount;\r
                                status.message.emit(tr("Downloading ") +new Long(pct).toString()+tr("% complete."));\r
                        }\r
+//                     conn.commitTransaction();\r
                }\r
-\r
                logger.log(logger.HIGH, "Leaving SyncRunner.syncRemoteToLocal");\r
        }\r
        // Sync expunged notes\r
@@ -1158,33 +1201,39 @@ public class SyncRunner extends QObject implements Runnable {
 //             List<SharedNotebook> books = noteStore.getSharedNotebookByAuth(authToken);\r
 //     }\r
        // Sync remote Resources\r
-       private void syncRemoteResources(List<Resource> resource) {\r
+       private void syncRemoteResources(Client noteStore, List<Resource> resource) {\r
                logger.log(logger.EXTREME, "Entering SyncRunner.syncRemoteResources");\r
                if (resource != null) {\r
                        for (int i=0; i<resource.size() && keepRunning; i++) {\r
-                               syncRemoteResource(resource.get(i), authToken);\r
+                               syncRemoteResource(noteStore, resource.get(i), authToken);\r
                        }\r
                }\r
                logger.log(logger.EXTREME, "Leaving SyncRunner.syncRemoteResources");\r
        }\r
        // Sync remote resource\r
-       private void syncRemoteResource(Resource resource, String authToken) {\r
+       private void syncRemoteResource(Client noteStore, Resource resource, String authToken) {\r
                // This is how the logic for this works.\r
                // 1.) If the resource is not in the local database, we add it.\r
                // 2.) If a copy of the resource is in the local database and the note isn't dirty, we update the local copy\r
                // 3.) If a copy of the resource is in the local databbase and it is dirty and the hash doesn't match, we ignore it because there\r
                //     is a conflict.  The note conflict should get a copy of the resource at that time.\r
                \r
+               Note n = conn.getNoteTable().getNote(resource.getNoteGuid(), false, false, false, false, false);\r
+               if (n!=null) {\r
+                       logger.log(logger.HIGH, "Resource for note " +n.getGuid() +" : " +n.getTitle());\r
+               }\r
                boolean saveNeeded = false;\r
-               /* #1 */                Resource r = getEvernoteResource(resource.getGuid(), true,true,true, authToken);\r
+               /* #1 */                Resource r = getEvernoteResource(noteStore, resource.getGuid(), true,true,true, authToken);\r
                                                Resource l = conn.getNoteTable().noteResourceTable.getNoteResource(r.getGuid(), false);\r
                                                if (l == null) {\r
+                                                       logger.log(logger.HIGH, "Local resource not found");\r
                                                        saveNeeded = true;\r
                                                } else {\r
                /* #2 */                        boolean isNoteDirty = conn.getNoteTable().isNoteDirty(r.getNoteGuid());\r
-                                                       if (!isNoteDirty)\r
+                                                       if (!isNoteDirty) {\r
+                                                               logger.log(logger.HIGH, "Local resource found, but is not dirty");\r
                                                                saveNeeded = true;\r
-                                                       else {\r
+                                                       else {\r
                /* #3 */                                String remoteHash = "";\r
                                                                if (r != null && r.getData() != null && r.getData().getBodyHash() != null)\r
                                                                        remoteHash = byteArrayToHexString(r.getData().getBodyHash());\r
@@ -1197,18 +1246,20 @@ public class SyncRunner extends QObject implements Runnable {
                                                        }\r
                                                }\r
                                                \r
+                                               logger.log(logger.HIGH, "Resource save needed: " +saveNeeded);\r
                                                if (saveNeeded) \r
                                                        conn.getNoteTable().noteResourceTable.updateNoteResource(r, false);\r
                                                if (r.getMime().equalsIgnoreCase("application/vnd.evernote.ink"))\r
                                                        downloadInkNoteImage(r.getGuid(), authToken);\r
+               \r
 \r
        }\r
        // Sync remote notes\r
-       private void syncRemoteNotes(List<Note> note, boolean fullSync, String token) {\r
+       private void syncRemoteNotes(Client noteStore, List<Note> note, boolean fullSync, String token) {\r
                logger.log(logger.EXTREME, "Entering SyncRunner.syncRemoteNotes");\r
                if (note != null) {\r
                        for (int i=0; i<note.size() && keepRunning; i++) {\r
-                               Note n = getEvernoteNote(note.get(i).getGuid(), true, fullSync, true,true, token);\r
+                               Note n = getEvernoteNote(noteStore, note.get(i).getGuid(), true, fullSync, true,true, token);\r
                                syncRemoteNote(n, fullSync, token);\r
                        }\r
                }\r
@@ -1247,8 +1298,10 @@ public class SyncRunner extends QObject implements Runnable {
                                \r
                        if ((conflictingNote || fullSync) && !ignoreNote) {\r
                                logger.log(logger.EXTREME, "Saving Note");\r
-                               conn.getNoteTable().syncNote(n, false);\r
-                               noteSignal.noteChanged.emit(n.getGuid(), null);   // Signal to ivalidate note cache\r
+                               conn.getNoteTable().syncNote(n);\r
+                               // The following was commented out because it caused a race condition on the database where resources \r
+                               // may be lost.  We do the same thing elsewhere;.\r
+//                             noteSignal.noteChanged.emit(n.getGuid(), null);   // Signal to ivalidate note cache \r
                                noteSignal.noteDownloaded.emit(n, true);                // Signal to add note to index\r
                                        logger.log(logger.EXTREME, "Note Saved");\r
                                if (fullSync && n.getResources() != null) {\r
@@ -1262,7 +1315,7 @@ public class SyncRunner extends QObject implements Runnable {
                        }\r
                }\r
        }\r
-       private Note getEvernoteNote(String guid, boolean withContent, boolean withResourceData, boolean withResourceRecognition, boolean withResourceAlternateData, String token) { \r
+       private Note getEvernoteNote(Client noteStore, String guid, boolean withContent, boolean withResourceData, boolean withResourceRecognition, boolean withResourceAlternateData, String token) { \r
                Note n = null;\r
                try {\r
                        logger.log(logger.EXTREME, "Retrieving note " +guid);\r
@@ -1291,7 +1344,7 @@ public class SyncRunner extends QObject implements Runnable {
                }\r
                return n;\r
        }\r
-       private Resource getEvernoteResource(String guid, boolean withData, boolean withRecognition, boolean withAttributes, String token) { \r
+       private Resource getEvernoteResource(Client noteStore, String guid, boolean withData, boolean withRecognition, boolean withAttributes, String token) { \r
                Resource n = null;\r
                try {\r
                        logger.log(logger.EXTREME, "Retrieving resource " +guid);\r
@@ -1437,14 +1490,16 @@ public class SyncRunner extends QObject implements Runnable {
                // Find the parent.  If the parent has a sequence > 0 then it is a good\r
                // parent.\r
                for (int i=0; i<tags.size() && keepRunning; i++) {\r
-                       if (tags.get(i).getParentGuid() == null) {\r
-                               logger.log(logger.HIGH, "Leaving SyncRunner.findNextTag - tag found without parent");\r
-                               return tags.get(i);\r
-                       }\r
-                       Tag parentTag = conn.getTagTable().getTag(tags.get(i).getParentGuid());\r
-                       if (parentTag.getUpdateSequenceNum() > 0) {\r
-                               logger.log(logger.HIGH, "Leaving SyncRunner.findNextTag - tag found");\r
-                               return tags.get(i);\r
+                       if (!badTagSync.containsKey(tags.get(i).getGuid())) {\r
+                               if (tags.get(i).getParentGuid() == null) {\r
+                                       logger.log(logger.HIGH, "Leaving SyncRunner.findNextTag - tag found without parent");\r
+                                       return tags.get(i);\r
+                               }\r
+                               Tag parentTag = conn.getTagTable().getTag(tags.get(i).getParentGuid());\r
+                               if (parentTag.getUpdateSequenceNum() > 0) {\r
+                                       logger.log(logger.HIGH, "Leaving SyncRunner.findNextTag - tag found");\r
+                                       return tags.get(i);\r
+                               }\r
                        }\r
                }\r
                \r
@@ -1457,6 +1512,7 @@ public class SyncRunner extends QObject implements Runnable {
     public boolean enConnect()  {\r
        try {\r
                        userStoreTrans = new THttpClient(userStoreUrl);\r
+                       userStoreTrans.setCustomHeader("User-Agent", userAgent);\r
                } catch (TTransportException e) {\r
                        QMessageBox mb = new QMessageBox(QMessageBox.Icon.Critical, "Transport Excepton", e.getLocalizedMessage());\r
                        mb.exec();\r
@@ -1466,9 +1522,10 @@ public class SyncRunner extends QObject implements Runnable {
            userStore = new UserStore.Client(userStoreProt, userStoreProt);\r
            syncSignal.saveUserStore.emit(userStore);\r
            try {\r
-                       authResult = userStore.authenticate(username, password, consumerKey, consumerSecret);\r
+                       //authResult = userStore.authenticate(username, password, consumerKey, consumerSecret);\r
+               user = userStore.getUser(authToken);\r
                } catch (EDAMUserException e) {\r
-                       QMessageBox mb = new QMessageBox(QMessageBox.Icon.Critical, "Error", "Incorrect username/password");\r
+                       QMessageBox mb = new QMessageBox(QMessageBox.Icon.Critical, "Error", "Invalid Authorization");\r
                        mb.exec();\r
                        isConnected = false;\r
                        return false;\r
@@ -1486,7 +1543,7 @@ public class SyncRunner extends QObject implements Runnable {
                \r
            boolean versionOk = false;\r
                try {\r
-                       versionOk = userStore.checkVersion("NeverNote", \r
+                       versionOk = userStore.checkVersion("NixNote", \r
                    com.evernote.edam.userstore.Constants.EDAM_VERSION_MAJOR, \r
                      com.evernote.edam.userstore.Constants.EDAM_VERSION_MINOR);\r
                } catch (TException e) {\r
@@ -1497,16 +1554,17 @@ public class SyncRunner extends QObject implements Runnable {
                System.err.println("Incomatible EDAM client protocol version"); \r
                isConnected = false;\r
            }\r
-           if (authResult != null) {\r
-               user = authResult.getUser(); \r
-               authToken = authResult.getAuthenticationToken(); \r
+           //if (authResult != null) {\r
+               //user = authResult.getUser(); \r
+               //authToken = authResult.getAuthenticationToken(); \r
                noteStoreUrl = noteStoreUrlBase + user.getShardId();\r
                syncSignal.saveAuthToken.emit(authToken);\r
-               syncSignal.saveNoteStore.emit(noteStore);\r
+               syncSignal.saveNoteStore.emit(localNoteStore);\r
                \r
         \r
                try {\r
                        noteStoreTrans = new THttpClient(noteStoreUrl);\r
+                       noteStoreTrans.setCustomHeader("User-Agent", userAgent);\r
                } catch (TTransportException e) {\r
                        QMessageBox mb = new QMessageBox(QMessageBox.Icon.Critical, "Transport Excepton", e.getLocalizedMessage());\r
                        mb.exec();\r
@@ -1514,12 +1572,12 @@ public class SyncRunner extends QObject implements Runnable {
                        isConnected = false;\r
                } \r
                noteStoreProt = new TBinaryProtocol(noteStoreTrans);\r
-               noteStore = \r
+               localNoteStore = \r
                        new NoteStore.Client(noteStoreProt, noteStoreProt); \r
                isConnected = true;\r
-               authTimeRemaining = authResult.getExpiration() - authResult.getCurrentTime();\r
-               authRefreshTime = authTimeRemaining / 2;\r
-           }\r
+               //authTimeRemaining = authResult.getExpiration() - authResult.getCurrentTime();\r
+               //authRefreshTime = authTimeRemaining / 2;\r
+           //}\r
            \r
                // Get user information\r
                try {\r
@@ -1539,8 +1597,11 @@ public class SyncRunner extends QObject implements Runnable {
     public void enDisconnect() {\r
        isConnected = false;\r
     }\r
+    \r
+    /*\r
     // Refresh the connection\r
     private synchronized boolean refreshConnection() {\r
+       \r
                logger.log(logger.EXTREME, "Entering SyncRunner.refreshConnection()");\r
 //        Calendar cal = Calendar.getInstance();\r
                \r
@@ -1618,6 +1679,8 @@ public class SyncRunner extends QObject implements Runnable {
                return true;\r
     }\r
     \r
+    */\r
+    \r
        public synchronized boolean addWork(String request) {\r
                if (workQueue.offer(request))\r
                        return true;\r
@@ -1625,7 +1688,9 @@ public class SyncRunner extends QObject implements Runnable {
        }\r
     \r
     private Note getNoteContent(Note n) {\r
-       n.setContent(conn.getNoteTable().getNoteContentBinary(n.getGuid()));\r
+               QTextCodec codec = QTextCodec.codecForLocale();\r
+               codec = QTextCodec.codecForName("UTF-8");\r
+       n.setContent(codec.toUnicode(new QByteArray(n.getContent())));\r
        return n;\r
     }\r
 \r
@@ -1634,7 +1699,7 @@ public class SyncRunner extends QObject implements Runnable {
     //*********************************************************\r
     //* Special download instructions.  Used for DB upgrades\r
     //*********************************************************\r
-    private void downloadAllSharedNotebooks() {\r
+    private void downloadAllSharedNotebooks(Client noteStore) {\r
        try {\r
                        List<SharedNotebook> books = noteStore.listSharedNotebooks(authToken);\r
                        logger.log(logger.LOW, "Shared notebooks found = " +books.size());\r
@@ -1663,7 +1728,7 @@ public class SyncRunner extends QObject implements Runnable {
                        logger.log(logger.LOW, e1.getMessage());\r
                }\r
     }\r
-    private void downloadAllNotebooks() {\r
+    private void downloadAllNotebooks(Client noteStore) {\r
        try {\r
                        List<Notebook> books = noteStore.listNotebooks(authToken);\r
                        logger.log(logger.LOW, "Shared notebooks found = " +books.size());\r
@@ -1688,7 +1753,7 @@ public class SyncRunner extends QObject implements Runnable {
                        return;\r
                }\r
     }\r
-    private void downloadAllLinkedNotebooks() {\r
+    private void downloadAllLinkedNotebooks(Client noteStore) {\r
        try {\r
                        List<LinkedNotebook> books = noteStore.listLinkedNotebooks(authToken);\r
                        logger.log(logger.LOW, "Linked notebooks found = " +books.size());\r
@@ -1737,7 +1802,6 @@ public class SyncRunner extends QObject implements Runnable {
             try {\r
                                post.setEntity(new UrlEncodedFormEntity(nvps, HTTP.UTF_8));\r
                        } catch (UnsupportedEncodingException e1) {\r
-                               // TODO Auto-generated catch block\r
                                e1.printStackTrace();\r
                        }\r
                try {\r
@@ -1790,40 +1854,72 @@ public class SyncRunner extends QObject implements Runnable {
        //* Begin syncing shared notebooks \r
        //******************************************\r
     private void syncLinkedNotebooks() {\r
-       logger.log(logger.MEDIUM, "Authenticating Shared Notebooks");\r
+       logger.log(logger.MEDIUM, "Authenticating linked Notebooks");\r
        status.message.emit(tr("Synchronizing shared notebooks."));\r
        List<LinkedNotebook> books = conn.getLinkedNotebookTable().getAll();\r
-\r
+       \r
+       errorSharedNotebooks.clear();\r
+               \r
        for (int i=0; i<books.size(); i++) {\r
+               if (errorSharedNotebooksIgnored.containsKey(books.get(i).getGuid()))\r
+                       break;\r
                try {\r
+                       logger.log(logger.EXTREME, "Checking notebook: " +books.get(i).getShareName());\r
                                long lastSyncDate = conn.getLinkedNotebookTable().getLastSequenceDate(books.get(i).getGuid());\r
                                int lastSequenceNumber = conn.getLinkedNotebookTable().getLastSequenceNumber(books.get(i).getGuid());\r
-                               linkedAuthResult = noteStore.authenticateToSharedNotebook(books.get(i).getShareKey(), authToken);\r
+\r
+                               logger.log(logger.EXTREME, "Last Sequence Number on file: " +lastSequenceNumber);\r
+                               \r
+                               // Authenticate to the owner's shard\r
+                               String linkedNoteStoreUrl       = noteStoreUrlBase + books.get(i).getShardId();\r
+                               logger.log(logger.EXTREME, "linkedNoteStoreURL: " +linkedNoteStoreUrl);\r
+                               THttpClient linkedNoteStoreTrans        = new THttpClient(linkedNoteStoreUrl);\r
+                               TBinaryProtocol linkedNoteStoreProt     = new TBinaryProtocol(linkedNoteStoreTrans);\r
+                               Client linkedNoteStore = new NoteStore.Client(linkedNoteStoreProt, linkedNoteStoreProt);        \r
+\r
+                               linkedAuthResult = null;\r
+                               if (books.get(i).getShareKey() != null) {\r
+                                       logger.log(logger.EXTREME, "Share Key Not Null: " +books.get(i).getShareKey());\r
+                                       linkedAuthResult = linkedNoteStore.authenticateToSharedNotebook(books.get(i).getShareKey(), authToken);\r
+                                       logger.log(logger.EXTREME, "Authentication Token" +linkedAuthResult.getAuthenticationToken());\r
+                               } else {\r
+                                       logger.log(logger.EXTREME, "Share key is null");\r
+                                       linkedAuthResult = new AuthenticationResult();\r
+                                       linkedAuthResult.setAuthenticationToken("");\r
+                               }\r
                                SyncState linkedSyncState = \r
-                                       noteStore.getLinkedNotebookSyncState(linkedAuthResult.getAuthenticationToken(), books.get(i));\r
+                                       linkedNoteStore.getLinkedNotebookSyncState(linkedAuthResult.getAuthenticationToken(), books.get(i));\r
                                if (linkedSyncState.getUpdateCount() > lastSequenceNumber) {\r
+                                       logger.log(logger.EXTREME, "Remote changes found");\r
                                        if (lastSyncDate < linkedSyncState.getFullSyncBefore()) {\r
                                                lastSequenceNumber = 0;\r
                                        } \r
-                                       syncLinkedNotebook(books.get(i), lastSequenceNumber, linkedSyncState.getUpdateCount());\r
+                                       logger.log(logger.EXTREME, "Calling syncLinkedNotebook for " +books.get(i).getShareName());\r
+                                       syncLinkedNotebook(linkedNoteStore, books.get(i), \r
+                                                       lastSequenceNumber, linkedSyncState.getUpdateCount(), authToken);\r
                                }\r
                        \r
                        // Synchronize local changes\r
-                       syncLocalLinkedNoteChanges(books.get(i));\r
+                       syncLocalLinkedNoteChanges(linkedNoteStore, books.get(i));\r
                                \r
                } catch (EDAMUserException e) {\r
-                       // TODO Auto-generated catch block\r
                        e.printStackTrace();\r
                } catch (EDAMNotFoundException e) {\r
                        status.message.emit(tr("Error synchronizing \" " +\r
                                        books.get(i).getShareName()+"\". Please verify you still have access to that shared notebook."));\r
+                       errorSharedNotebooks.add(books.get(i).getGuid());\r
+                       errorSharedNotebooksIgnored.put(books.get(i).getGuid(), books.get(i).getGuid());\r
+                       logger.log(logger.LOW, "Error synchronizing shared notebook.  EDAMNotFound: "+e.getMessage());\r
+                       logger.log(logger.LOW, e.getStackTrace());\r
                        error = true;\r
                        e.printStackTrace();\r
                } catch (EDAMSystemException e) {\r
-                       // TODO Auto-generated catch block\r
+                       error = true;\r
+                       logger.log(logger.LOW, "System error authenticating against shared notebook. "+\r
+                                       "Key: "+books.get(i).getShareKey() +" Error:" +e.getMessage());\r
                        e.printStackTrace();\r
                } catch (TException e) {\r
-                       // TODO Auto-generated catch block\r
+                       error = true;\r
                        e.printStackTrace();\r
                }\r
        }\r
@@ -1832,14 +1928,17 @@ public class SyncRunner extends QObject implements Runnable {
        conn.getTagTable().removeUnusedLinkedTags();\r
        conn.getTagTable().cleanupTags();\r
        tagSignal.listChanged.emit();\r
+       return;\r
        }\r
 \r
     \r
     //**************************************************************\r
     //* Linked notebook contents (from someone else's account)\r
     //*************************************************************\r
-       private void syncLinkedNotebook(LinkedNotebook book, int usn, int highSequence) {\r
-               \r
+       private void syncLinkedNotebook(Client linkedNoteStore, LinkedNotebook book, int usn, int highSequence, String token) {\r
+               logger.log(logger.EXTREME, "Entering syncLinkedNotebook");\r
+               if (ignoreLinkedNotebooks.contains(book.getGuid()))\r
+                       return;\r
                List<Note> dirtyNotes = conn.getNoteTable().getDirtyLinkedNotes();\r
                if (dirtyNoteGuids == null) \r
                        dirtyNoteGuids = new Vector<String>();\r
@@ -1850,23 +1949,35 @@ public class SyncRunner extends QObject implements Runnable {
                boolean fullSync = false;\r
                if (usn == 0)\r
                        fullSync = true;\r
-               while (usn < highSequence) {\r
+               boolean syncError = false;\r
+               while (usn < highSequence && !syncError) {\r
+                       refreshNeeded = true;\r
                        try {\r
                                SyncChunk chunk = \r
-                                       noteStore.getLinkedNotebookSyncChunk(authToken, book, usn, 10, fullSync);\r
+                                       linkedNoteStore.getLinkedNotebookSyncChunk(token, book, usn, 10, fullSync);\r
+                               \r
+                               // Expunge notes\r
+                               syncExpungedNotes(chunk);\r
 \r
-                               if (!ignoreLinkedNotebooks.contains(book.getGuid()))\r
-                                       syncRemoteNotes(chunk.getNotes(), fullSync, linkedAuthResult.getAuthenticationToken());\r
-                               findNewLinkedTags(chunk.getNotes(), linkedAuthResult.getAuthenticationToken());\r
+                               logger.log(logger.EXTREME, "Syncing remote notes: " +chunk.getNotesSize());\r
+                               syncRemoteNotes(linkedNoteStore, chunk.getNotes(), fullSync, linkedAuthResult.getAuthenticationToken());\r
+                               logger.log(logger.EXTREME, "Finding new linked tags");\r
+                               findNewLinkedTags(linkedNoteStore, chunk.getNotes(), linkedAuthResult.getAuthenticationToken());\r
+                               // Sync resources\r
+                               logger.log(logger.EXTREME, "Synchronizing tags: " +chunk.getTagsSize());\r
                                for (int i=0; i<chunk.getResourcesSize(); i++) {\r
-                                       syncRemoteResource(chunk.getResources().get(i), linkedAuthResult.getAuthenticationToken());\r
+                                       syncRemoteResource(linkedNoteStore, chunk.getResources().get(i), linkedAuthResult.getAuthenticationToken());\r
                                }\r
-                               syncRemoteLinkedNotebooks(chunk.getNotebooks(), false, book);\r
-                               SharedNotebook s = noteStore.getSharedNotebookByAuth(linkedAuthResult.getAuthenticationToken());\r
-                               syncLinkedTags(chunk.getTags(), s.getNotebookGuid());\r
+                               logger.log(logger.EXTREME, "Synchronizing linked notebooks: " +chunk.getNotebooksSize());\r
+                               syncRemoteLinkedNotebooks(linkedNoteStore, chunk.getNotebooks(), false, book);\r
+                               syncLinkedTags(chunk.getTags(), book.getGuid());\r
                                \r
-                               \r
-                               // Expunge records\r
+                               // Go through & signal any notes that have changed so we can refresh the user's view\r
+                               for (int i=0; i<chunk.getNotesSize(); i++) \r
+                                       noteSignal.noteChanged.emit(chunk.getNotes().get(i).getGuid(), null);\r
+\r
+                               // Expunge Notebook records\r
+                               logger.log(logger.EXTREME, "Expunging linked notebooks: " +chunk.getExpungedLinkedNotebooksSize());\r
                                for (int i=0; i<chunk.getExpungedLinkedNotebooksSize(); i++) {\r
                                        conn.getLinkedNotebookTable().expungeNotebook(chunk.getExpungedLinkedNotebooks().get(i), false);\r
                                }\r
@@ -1874,19 +1985,29 @@ public class SyncRunner extends QObject implements Runnable {
                                conn.getLinkedNotebookTable().setLastSequenceDate(book.getGuid(),chunk.getCurrentTime());\r
                                conn.getLinkedNotebookTable().setLastSequenceNumber(book.getGuid(),chunk.getChunkHighUSN());\r
                        } catch (EDAMUserException e) {\r
-                               // TODO Auto-generated catch block\r
+                               syncError = true;\r
+                               status.message.emit(tr("EDAM UserException synchronizing linked notbook.  See the log for datails."));\r
                                e.printStackTrace();\r
+                               logger.log(logger.LOW, tr("EDAM UserException synchronizing linked notbook ")+ e.getMessage());\r
                        } catch (EDAMSystemException e) {\r
-                               // TODO Auto-generated catch block\r
+                               syncError = true;\r
+                               status.message.emit(tr("EDAM SystemException synchronizing linked notbook.  See the log for datails."));\r
                                e.printStackTrace();\r
+                               logger.log(logger.LOW, tr("EDAM SystemException synchronizing linked notbook.  See the log for datails") +e.getMessage());\r
                        } catch (EDAMNotFoundException e) {\r
-                               // TODO Auto-generated catch block\r
-                               e.printStackTrace();\r
+                               syncError = true;\r
+                               status.message.emit(tr("Notebook URL not found. Removing notobook ") +book.getShareName());\r
+                               conn.getNotebookTable().deleteLinkedTags(book.getGuid());\r
+                               conn.getLinkedNotebookTable().expungeNotebook(book.getGuid(), false);\r
+                               logger.log(logger.LOW, tr("Notebook URL not found. Removing notobook ") +e.getMessage());\r
                        } catch (TException e) {\r
-                               // TODO Auto-generated catch block\r
+                               syncError = true;\r
+                               status.message.emit(tr("EDAM TException synchronizing linked notbook.  See the log for datails."));\r
                                e.printStackTrace();\r
+                               logger.log(logger.LOW, tr("EDAM TException synchronizing linked notbook.  See the log for datails." )+e.getMessage());\r
                        }\r
                }\r
+               logger.log(logger.EXTREME, "leaving syncLinkedNotebook");\r
        }\r
        // Sync remote tags\r
        private void syncLinkedTags(List<Tag> tags, String notebookGuid) {\r
@@ -1900,14 +2021,20 @@ public class SyncRunner extends QObject implements Runnable {
        }\r
        \r
        // Sync notebooks from a linked notebook\r
-       private void syncRemoteLinkedNotebooks(List<Notebook> notebooks, boolean readOnly, LinkedNotebook linked) {\r
+       private void syncRemoteLinkedNotebooks(Client noteStore, List<Notebook> notebooks, boolean readOnly, LinkedNotebook linked) {\r
                logger.log(logger.EXTREME, "Entering SyncRunner.syncRemoteNotebooks");\r
                if (notebooks != null) {\r
                        for (int i=0; i<notebooks.size() && keepRunning; i++) {\r
                                try {\r
-                                       SharedNotebook s = noteStore.getSharedNotebookByAuth(linkedAuthResult.getAuthenticationToken());\r
-                                       conn.getLinkedNotebookTable().setNotebookGuid(s.getShareKey(), s.getNotebookGuid());\r
-                                       readOnly = !s.isNotebookModifiable();\r
+                                       logger.log(logger.EXTREME, "auth token:" +linkedAuthResult.getAuthenticationToken());\r
+                                       if (!linkedAuthResult.getAuthenticationToken().equals("")) {\r
+                                               SharedNotebook s = noteStore.getSharedNotebookByAuth(linkedAuthResult.getAuthenticationToken());\r
+                                               logger.log(logger.EXTREME, "share key:"+s.getShareKey() +" notebookGuid" +s.getNotebookGuid());\r
+                                               conn.getLinkedNotebookTable().setNotebookGuid(s.getShareKey(), s.getNotebookGuid());\r
+                                               readOnly = !s.isNotebookModifiable();\r
+                                       } else {\r
+                                               readOnly = true;\r
+                                       }\r
                                        notebooks.get(i).setName(linked.getShareName());\r
                                        notebooks.get(i).setDefaultNotebook(false);\r
                                        conn.getNotebookTable().syncLinkedNotebook(notebooks.get(i), false, readOnly); \r
@@ -1930,7 +2057,7 @@ public class SyncRunner extends QObject implements Runnable {
                logger.log(logger.EXTREME, "Leaving SyncRunner.syncRemoteNotebooks");\r
        }\r
 \r
-       private void findNewLinkedTags(List<Note> newNotes, String token) {\r
+       private void findNewLinkedTags(Client noteStore, List<Note> newNotes, String token) {\r
                if (newNotes == null)\r
                        return;\r
                for (int i=0; i<newNotes.size(); i++) {\r
@@ -1943,16 +2070,12 @@ public class SyncRunner extends QObject implements Runnable {
                                                newTag = noteStore.getTag(token, tag);\r
                                                conn.getTagTable().addTag(newTag, false);\r
                                        } catch (EDAMUserException e) {\r
-                                               // TODO Auto-generated catch block\r
                                                e.printStackTrace();\r
                                        } catch (EDAMSystemException e) {\r
-                                               // TODO Auto-generated catch block\r
                                                e.printStackTrace();\r
                                        } catch (EDAMNotFoundException e) {\r
-                                               // TODO Auto-generated catch block\r
                                                e.printStackTrace();\r
                                        } catch (TException e) {\r
-                                               // TODO Auto-generated catch block\r
                                                e.printStackTrace();\r
                                        }\r
                                        \r
@@ -1962,12 +2085,17 @@ public class SyncRunner extends QObject implements Runnable {
        }\r
 \r
        // Synchronize changes locally done to linked notes\r
-       private void syncLocalLinkedNoteChanges(LinkedNotebook book) {\r
+       private void syncLocalLinkedNoteChanges(Client noteStore, LinkedNotebook book) {\r
+               logger.log(logger.EXTREME, "Entering SyncRunner.synclocalLinkedNoteChanges");\r
                String notebookGuid = conn.getLinkedNotebookTable().getNotebookGuid(book.getGuid());\r
+               logger.log(logger.EXTREME, "Finding changes for " +book.getShareName() +":" +book.getGuid() + ":" +notebookGuid);\r
                List<Note> notes = conn.getNoteTable().getDirtyLinked(notebookGuid);\r
+               logger.log(logger.EXTREME, "Number of changes found: " +notes.size());\r
                for (int i=0; i<notes.size(); i++) {\r
-                       syncLocalNote(notes.get(i), linkedAuthResult.getAuthenticationToken());\r
+                       logger.log(logger.EXTREME, "Calling syncLocalNote with key " +linkedAuthResult.getAuthenticationToken());\r
+                       syncLocalNote(noteStore, notes.get(i), linkedAuthResult.getAuthenticationToken());\r
                }\r
+               logger.log(logger.EXTREME, "Leaving SyncRunner.synclocalLinkedNoteChanges");\r
        }\r
 \r
 }\r