/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

TODO
1
 
* VcardImporter.Vcard.finaliseParsing() should be renamed to finalise() to
2
 
        override Importer.ContactData.finalise(), if Java will permit us to do that
3
 
        (fnaliseParsing() throws stuff that finalise() doesn't), and should call
4
 
        super.finalise().
5
 
* rename Importer.isImportRequired() to checkForDuplicate() and make it private.
6
 
        This should be called from Importer.importContact(), which should then call
7
 
        skipContact() to update the UI and return if checkForDuplicate() returns
8
 
        false.
9
 
* ContactData.finalise should create a ContactsCache.CacheIdentifier and throw
10
 
        ContactCannotBeIdentifiedException() if that fails, which can be picked up
11
 
        in Vcard.finalise(). Importer.importContact() and the renamed
12
 
        Importer.checkForDuplicates() should use the ContactData's CacheIdentifier
13
 
        and not create their own. This will also ensure that ContactData.finalise()
14
 
        has been correctly called (or the identifier would be null).
15
1
* import a crap-load of contacts before release!
16
2
* change some public final static ints, that aren't really consts, in to enums
17
3
* create a back-end object to read/write to android. This can be overridden to