[PATCH] KSecretsService Collection and Item property names

[Kevin Krammer]

Hi Jakub,

thank you caring about our client implementation.
The macro usage looks a bit weird to me, but it is Valentin's call :)

Cheers,
Kevin

On Sunday, 2012-08-05, Jakub Filak wrote:
> The current implementation of KSecretsService accepts property names of
> Collection and Item without interface name. The Secret Service API standard
> says "Specify the property names in full interface.Property form" [1]
> 
> The form required by the standard:
> 	"org.freedesktop.Secret.Item.Label"
> 
> The accepted form in KSecretsService:
> 	"Label"
> 
> (gnome-keyring accepts the properties only in full interface form)
> 
> The patch changes the accepted name form from single name form to full
> interface name form. The patch simply adds an interface prefix to each
> occurrence of a property name. (I used a helper macros because of DRY.)
> 
> The patch applies to the files:
>  ksecretsserviced/frontend/secret/collection.cpp
>  ksecretsserviced/frontend/secret/service.cpp
>  ksecretsserviced/frontend/tests/servicetest.cpp
> 
> 
> Regards,
> Jakub
> 
> 
> [1] : http://standards.freedesktop.org/secret-service/re02.html

-- 
Kevin Krammer, KDE developer, xdg-utils developer
KDE user support, developer mentoring
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 190 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20120807/39a4d9c9/attachment.sig>