[Owncloud] Scratching an itch. Was: Re: Maintainers for contacts app needed

[Thomas Olsen]

On Thursday 01 December 2011 21:35 Bart Visscher wrote:
> On Thu, Dec 01, 2011 at 10:51:54AM +0100, Thomas Olsen wrote:
> > On Thursday 01 December 2011 10:38 Jakob Sack wrote:
> > > Apart from a good layout the most important thing missing is the
> > > creation of a Name field (N). According to the RFC the name field is
> > > required but ownCloud does not provide a way to add/edit this.
> > 
> > I'm just starting to get my hands dirty in the code and am still taking
> > baby steps ;-)
> I have been working on the contacts app beside the calender app. I definitly
> can help maintaining the contacts app.

As I like to run regular, automated backups of my PIM data I looked into an 
easy way of getting the direct link to an address book, and noticed the 
elegant way it was possible in the calendar, where you can click on the 
'Calendars' button and download it directly.
I modified the export.php script for the contacts app and can now run backups 
with wget and http authentication.
Then I implemented the nice popup window function to get an UI for it and one 
thing lead to another ;-)

Status now:

- UI: Same as for the Calendar app with to activation, CardDAV link, export, 
edit, delete and add.

Functionality: CardDAV link and export implemented. Edit, add and delete is 
still WiP but I think I've got a pretty good idea on how it should be done.

I need to add an integer (or boolean) field 'active' to the 
'contacts_addressbooks' table. I can see that I need to edit database.xml but 
how do I make an automatic update of existing installations?

I need to add a method (or modify existing) to 'OC_Contacts_Addressbook' for 
getting only active address books.

I need to make sure the underlying UI is updated on changes.

I probably need several other things that I have forgotten or not yet 
considered ;-)

Are there any objections on me going ahead with this?

> BTW. For anyone looking for a nice task, look at the json-notification
> branch. A lot of the error handling still needs to be moved to use the
> notification system in that branch.

I got carried away with this one and it's a nice way to get familiar with the 
code.

-- 
Med venlig hilsen / Best Regards

Thomas Tanghus Olsen