/android/import-contacts

To get this branch, use:
bzr branch http://bzr.ed.am/android/import-contacts

« back to all changes in this revision

Viewing changes to TODO

  • Committer: edam
  • Date: 2011-05-30 15:33:01 UTC
  • Revision ID: edam@waxworlds.org-20110530153301-oor6ci9b3hf9clul
- refactored some code to do with how contacts are imported
- Vcards (and ContactData) instances now generate a CacheIdentifier when they are finalised so that ContactData instances that do not have enough information to identify them can be discovered then
- importContact() now calls the private method checkForDuplicate(), renamed from isImportRequired(), and return if it is not
- importContact() and checkForDuplicate() now use the ContactData's generated CacheIdentifier

Show diffs side-by-side

added added

removed removed

Lines of Context:
TODO
1
 
* rename VCFImporter to VcardImporter and VCard to Vcard
2
 
* VcardImporter.Vcard.finaliseParsing() should be renamed to finalise() to
3
 
        override Importer.ContactData.finalise(), if Java will permit us to do that
4
 
        (fnaliseParsing() throws stuff that finalise() doesn't), and should call
5
 
        super.finalise().
6
 
* rename Importer.isImportRequired() to checkForDuplicate() and make it private.
7
 
        This should be called from Importer.importContact(), which should then call
8
 
        skipContact() to update the UI and return if checkForDuplicate() returns
9
 
        false.
10
 
* ContactData.finalise should create a ContactsCache.CacheIdentifier and throw
11
 
        ContactCannotBeIdentifiedException() if that fails, which can be picked up
12
 
        in Vcard.finalise(). Importer.importContact() and the renamed
13
 
        Importer.checkForDuplicates() should use the ContactData's CacheIdentifier
14
 
        and not create their own. This will also ensure that ContactData.finalise()
15
 
        has been correctly called (or the identifier would be null).
16
1
* import a crap-load of contacts before release!
17
2
* change some public final static ints, that aren't really consts, in to enums
18
3
* create a back-end object to read/write to android. This can be overridden to
23
8
* possible bug: when displaying "aborting import" toaster, make sure the back
24
9
        button is enabled and the abort button is hidden (as if abort were pressed)
25
10
        since this can happen if, say, the alarm goes off.
 
11
* add facility to be asked whether to import all contacts, not just those that
 
12
        are duplicates.
26
13
* in the long term, we need to save/restore state from doit/importer