Bug 19428 - Account: [impl needed] add CreateWithProperties or equivalent
Summary: Account: [impl needed] add CreateWithProperties or equivalent
Status: RESOLVED FIXED
Alias: None
Product: Telepathy
Classification: Unclassified
Component: tp-spec (show other bugs)
Version: unspecified
Hardware: Other All
: medium normal
Assignee: Simon McVittie
QA Contact: Telepathy bugs list
URL: http://git.collabora.co.uk/?p=user/sm...
Whiteboard:
Keywords:
Depends on: 21170 21171
Blocks:
  Show dependency treegraph
 
Reported: 2009-01-06 11:42 UTC by Simon McVittie
Modified: 2009-06-04 07:08 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments

Description Simon McVittie 2009-01-06 11:42:28 UTC
mardy's CreateWithProperties method should probably just be added to the core Account interface.
Comment 1 Simon McVittie 2009-04-10 09:43:36 UTC
Incomplete spec wording:

http://git.collabora.co.uk/?p=user/smcv/telepathy-spec-smcv.git;a=shortlog;h=refs/heads/account-creation

Unresolved:

* how does the caller tell which properties this AM will accept "at construct time"?

* what happens if the caller gets a property value wrong? As currently implemented in MC, it's silently ignored

* what happens if the caller passes an unsupported property name?
Comment 2 Simon McVittie 2009-04-13 06:52:28 UTC
Sortable summary
Comment 3 Simon McVittie 2009-04-13 10:46:48 UTC
Resolution of three fixmes:

* SupportedAccountProperties property

* NotImplemented if name is unknown to the AM

* InvalidArgument if value or type is nonsensical

* (bonus fixme) MC should actually do this, and actually be atomic
Comment 4 Simon McVittie 2009-06-04 07:08:08 UTC
Fixed in 0.17.24


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.