http://bugs.meego.com/show_bug.cgi?id=6898

--- Comment #6 from pohly <[email protected]> 2010-09-29 04:53:37 PDT ---
(In reply to comment #2)
> I pushed notebook name implementation into kcal-notebook branch. Patrick, 
> could
> you help to review and merge to master.

I don't think we should create Notebooks inside of our backend. Every small
typo would lead to the creation of a new Notebook.

Traditionally, "evolutionsource" has only created something if there was an
explicit file:// prefix. If supported, the result was a database file at the
location following the prefix.

I suggest to do the same with the KCalExtended backend: with file://,
instantiate a SqliteStorage at that place. Without file://, search for an
existing Notebook with the given name in the default storage. If empty, use
default Notebook in default storage. This behavior should fit the needs of most
users and of the automated testing (which uses the file:// prefix).

This needs to be documented in the description of mkcal
(KCalExtendedRegister.cpp):

                  "mkcal = KCalExtended = calendar\n"
                  "   iCalendar 2.0 = text/calendar\n"
                  "   \"evolutionsource\" either refers to an existing\n"
                  "   calendar inside the default calendar storage or,\n"
                  "   if it starts with the file:// prefix, to the\n"
                  "   default calendar in that file. The file will be\n"
                  "   created if it does not exist yet.\n",

Also please implement listing of existing Notebooks ("syncevolution" without
arguments, KCalExtendedSource::getDatabases()).

-- 
Configure bugmail: http://bugs.meego.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching someone on the CC list of the bug.
_______________________________________________
Syncevolution-issues mailing list
[email protected]
http://lists.syncevolution.org/listinfo/syncevolution-issues

Reply via email to