Hi All, My Last commit was implementation of AgAccountService object (GObject implementation)
Now, I am committing AccountService class under accounts-qt. (Wrapper class around AgAccountService) Clients will use AccountService class methods to access service settings of an account. Requesting to kindly provide comments on this commit too. https://gitorious.org/accounts-sso/accounts-qt/merge_requests/3 Wth Regards, Rama On Thu, Apr 7, 2011 at 7:05 PM, ramakrishna k n <[email protected]>wrote: > Hi All, > > I am Ramakrishna working as a developer in Nokia. > > I have added a new object - AgAccountService to accounts-glib. > Below is a short summary for need of this new object. > > Current APIs in accounts-glib to access account service setting could be > made much simpler. > Current API design: > > // get the username from the global settings > username = account->value("username"); > enabled = account->enabled(); > > // get some e-mail settings > account->selectService(Service("gmail")); > imapServer = account->value("server"); > imapPort = account->value("port"); > enabled = account->enabled(); > > With the above design, client has to select the service before it get/set > the value. Risk is if the client misses calling selectService() method, it > can get wrong value. > > The new AgAccountService class solves this overhead as shown below. > > accountService = new AccountService(account, Service("gmail")); > imapServer = accountService->value("server"); > imapPort = accountService->value("port"); > // the next line returns true only if the account is enabled *and* the > // "gmail" service is enabled on it > emailEnabled = accountService->enabled(); > > I have implemented the accounts-glib part of code to support the above > design. > Kindly review the code which is pushed on to a branch > "account-service-impl" and pls provide ur inputs. > > > https://gitorious.org/accounts-sso/accounts-glib/commits/account-service-impl > > With Regards, > Rama >
_______________________________________________ MeeGo-dev mailing list [email protected] http://lists.meego.com/listinfo/meego-dev http://wiki.meego.com/Mailing_list_guidelines
