Revert "「前データを取得」で記録される最後に取得したツイートのIDが、通常の更新によって上書きされる不具合を修正"

スリープからの復帰時など、通常の「更新」を行った場合で一度の更新で新着ツイートを
取得しきれなかった場合に、欠けた区間を「前データを取得」で取得できなくなっていた。
そのため、v1.3.7 で変更した「前データを取得」の動作を一旦以前の挙動に戻す修正を行う

8a32c84d で解決しようとした問題については、上記の状況を考慮した策を練り直す必要がある

This reverts commit 8a32c84d05c5e654d87f1881409b614eb56dc7fd.

diff --git a/OpenTween/Resources/ChangeLog.txt b/OpenTween/Resources/ChangeLog.txt
index 38768a9..77a9c8d 100644 (file)
--- a/OpenTween/Resources/ChangeLog.txt
+++ b/OpenTween/Resources/ChangeLog.txt
@@ -1,6 +1,8 @@
 更新履歴
 
 ==== Ver 1.3.9-dev(2017/xx/xx)
+ * CHG: 「前データを取得」を v1.3.7 より前と同様の動作となるように戻しました
+  - スリープからの復帰時など、一度の更新で取得しきれなかった分のツイートを取得したい場合に期待した動作とならないため
 
 ==== Ver 1.3.8(2017/06/05)
  * NEW: bit.ly の認証方式が変更されました

diff --git a/OpenTween/Twitter.cs b/OpenTween/Twitter.cs
index 990711d..e6463e9 100644 (file)
--- a/OpenTween/Twitter.cs
+++ b/OpenTween/Twitter.cs
@@ -601,8 +601,7 @@ namespace OpenTween
             }
 
             var minimumId = this.CreatePostsFromJson(statuses, MyCommon.WORKERTYPE.Timeline, tab, read);
-
-            if (minimumId != null && minimumId.Value < tab.OldestId)
+            if (minimumId != null)
                 tab.OldestId = minimumId.Value;
         }
 
@@ -625,8 +624,7 @@ namespace OpenTween
             }
 
             var minimumId = this.CreatePostsFromJson(statuses, MyCommon.WORKERTYPE.Reply, tab, read);
-
-            if (minimumId != null && minimumId.Value < tab.OldestId)
+            if (minimumId != null)
                 tab.OldestId = minimumId.Value;
         }
 
@@ -661,7 +659,7 @@ namespace OpenTween
 
             var minimumId = CreatePostsFromJson(statuses, MyCommon.WORKERTYPE.UserTimeline, tab, read);
 
-            if (minimumId != null && minimumId.Value < tab.OldestId)
+            if (minimumId != null)
                 tab.OldestId = minimumId.Value;
         }
 
@@ -994,7 +992,7 @@ namespace OpenTween
 
             var minimumId = CreatePostsFromJson(statuses, MyCommon.WORKERTYPE.List, tab, read);
 
-            if (minimumId != null && minimumId.Value < tab.OldestId)
+            if (minimumId != null)
                 tab.OldestId = minimumId.Value;
         }
 
@@ -1136,7 +1134,7 @@ namespace OpenTween
 
             var minimumId = this.CreatePostsFromSearchJson(searchResult, tab, read, more);
 
-            if (minimumId != null && minimumId.Value < tab.OldestId)
+            if (minimumId != null)
                 tab.OldestId = minimumId.Value;
         }
 
@@ -1312,7 +1310,7 @@ namespace OpenTween
 
             var minimumId = this.CreateFavoritePostsFromJson(statuses, read);
 
-            if (minimumId != null && minimumId.Value < tab.OldestId)
+            if (minimumId != null)
                 tab.OldestId = minimumId.Value;
         }