http://bugzilla.moblin.org/show_bug.cgi?id=3604





--- Comment #7 from pohly <[email protected]>  2009-09-01 06:21:25 ---
(In reply to comment #6)
> > Can we add a UI operation that is complementary to askPassword()?
> > savePassword()? The config system could call it whenever it has a password
> > value which needs to be stored. If the UI saves the value or the value was 
> > "-"
> > to start with, then the config itself saves "-".
> You mean adding 'savePassword' function in the ConfigUserInterface?
> 
> I summary up the discussion. Patrick, please correct me if I get mistakes.
> 1) for retrieving password, write a CmdlineSyncClient derived from
> EvolutionSyncClient to implement 'checkPassword' to retrive password from
> keyring.

Correct.

> 2)for storing password, first add a virtual function in the 
> ConfigUserInterface
> and implement it in the CmdlineSyncClient. In the EvolutionSyncConfig, check
> the property whether it is 'password'. If yes, invoke ConfigUserInterface's
> function 'savePassword' to store it. I'd like to check this in the
> EvolutionSyncConfig directly instead of writing a new class. make sense?

Not quite: don't check for the 'password' property name. That would enabled the
keyring only for that single password. Instead apply askPassword and
savePassword to all properties which are a PasswordConfigProperty. This is
analogous to how PasswordConfigProperty::checkPassword() calls
ui.askPassword().

-- 
Configure bugmail: http://bugzilla.moblin.org/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