>            name of the source configuration which defines
>            the backend ("type" property); a temporary config
>            is allowed here
Since a source won't have many 'evolutionsource' properties in its config.ini, 
I guess
here we get the available databases by its 'type' properties in its config.ini.
For example, a 'vcard_test' source configuration defines its 'type' as 
'Evolution Address Book'. 
Then this method returns all available databases for 'Evolution Address Book' 
in the sync source registry .
Is anything wrong?

For CheckSource, why preferring 'throwing exception' to return values to 
indicate results?

Cheers,
Yongsheng


-----Original Message-----
From: Ohly, Patrick 
Sent: Wednesday, September 23, 2009 9:44 PM
To: Zhu, Yongsheng
Cc: SyncEvolution
Subject: Re: [SyncEvolution] D-Bus API: availability of local sources (was: 
syncevo-dbus-server implementation)

On Fri, 2009-09-18 at 09:14 +0100, Patrick Ohly wrote:
> On Fri, 2009-09-18 at 02:19 +0100, Zhu, Yongsheng wrote:
> > So we define 2 new APIs for this issue instead of doing them in 
> > getConfig/setConfig?
> 
> Yes.

Here's a first draft for this. Does it make sense?

    <method name="GetDatabases">
      <doc:doc>
        <doc:description>
          Get list of available databases that can be synchronized
          by a source backend.
        </doc:description>
      </doc:doc>
      <arg type="s" name="source" direction="in">
        <doc:doc>
          <doc:summary>
            name of the source configuration which defines
            the backend ("type" property); a temporary config
            is allowed here
          </doc:summary>
        </doc:doc>
      </arg>
      <arg type="a(ssb)" name="databases" direction="out">
        <doc:doc><doc:summary>information about all available 
databases</doc:summary></doc:doc>
        <doc:doc>
          <doc:description>
            each entry contains in this order:
            an optional name that can be shown to the user
            (already localized or chosen by the user, empty if unavailable),
            a unique value for the "evolutionSource" property,
            a boolean which is true at most once for the default source
          </doc:description>
        </doc:doc>
      </arg>
    </method>

    <method name="CheckSource">
      <doc:doc>
        <doc:description>Tests whether the source configuration
          is correct. Raises an exception if not.
        </doc:description>
      </doc:doc>
      <arg type="s" name="source" direction="in">
        <doc:doc>
          <doc:summary>
            name of the source configuration which is to be tested;
            a temporary config is allowed here
          </doc:summary>
        </doc:doc>
      </arg>
    </method>



-- 
Best Regards, Patrick Ohly

The content of this message is my personal opinion only and although
I am an employee of Intel, the statements I make here in no way
represent Intel's position on the issue, nor am I authorized to speak
on behalf of Intel on this matter.


_______________________________________________
SyncEvolution mailing list
[email protected]
http://lists.syncevolution.org/listinfo/syncevolution

Reply via email to