[PATCH] KSecretsService Collection and Item property names

[Valentin Rusu]

Hello Jakub, Kevin,

Thanks for the patch. I'm now working on it. I'll also adjuste the 
client API then I'll commit all the changes asap for me.

Cheers,
Valentin

On 08/07/2012 03:50 PM, Kevin Krammer wrote:
> 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


-- 
Valentin Rusu (IRC valir, KDE vrusu)
KSecretsService (former KSecretService, KWallet replacement)