mirror of
https://github.com/jberkel/sms-backup-plus.git
synced 2026-04-26 01:15:58 +03:00
[PR #269] [CLOSED] Inverse restore order #939
Labels
No labels
AM+RCS
FAQ
awaiting response
backup
bespoke
bug
calendar
call log
cannot reproduce
cloudless
device-specific
documentation
dual- & multi-SIM
duplicate
feature-request
fixed in beta
good first issue
half-missing
help wanted
helpful
meta
misattribution
mms
other message sources
pull-request
question
rejuvenation
restore
schedule
security
stale
task
thanks
v1.5.1
v1.5.10
v1.5.11
v1.5.2
v1.5.3
v1.5.3
v1.5.4
v1.5.4
v1.5.5
v1.5.5
v1.5.6
v1.5.7
v1.5.8
v1.5.9
v1.6β
xoauth
~$ bounty $~
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/sms-backup-plus-jberkel#939
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/jberkel/sms-backup-plus/pull/269
Author: @miniyou
Created: 10/18/2012
Status: ❌ Closed
Base:
master← Head:restore-order📝 Commits (2)
0d15e90Removing unnecessary @override annotations causing errorseda0822Restoring most recent items first📊 Changes
5 files changed (+1181 additions, -1078 deletions)
View changed files
📝
src/com/zegoggles/smssync/AuthActivity.java(+0 -1)📝
src/com/zegoggles/smssync/BackupImapStore.java(+3 -1)📝
src/com/zegoggles/smssync/CalendarAccessorPost40.java(+40 -42)📝
src/com/zegoggles/smssync/SmsSync.java(+1138 -1033)📝
src/com/zegoggles/smssync/Utils.java(+0 -1)📄 Description
The most recent commit is the most relevant one, however they are both worth pulling.
I only added one line of code as a partial fix for issue #267: while the most recent version seems more stable than the previous, it still restores old messages first. I have ~25k messages backed up but only interested in the latest 2k or so.
I hope this is the preferred place to put this fix in. It seems to work for me - it has restored a bit over 1000 messages at the time of writing, and I get the most recent ones first.
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.