Bug 251914

Summary: KNotes dbus interface missing notes() as compared to old dcop interface
Product: [Applications] knotes Reporter: Tejas Guruswamy <masterpatricko>
Component: generalAssignee: kdepim bugs <kdepim-bugs>
Status: RESOLVED FIXED    
Severity: normal    
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: openSUSE   
OS: Linux   
Latest Commit: Version Fixed In:
Attachments: Patch restoring notes() method to knotes

Description Tejas Guruswamy 2010-09-21 12:25:18 UTC 
Version:           unspecified (using KDE 4.5.1) 
OS:                Linux

KNotes dbus interface is missing the notes() method as compared to the old dcop interface, making it impossible to get note id's.
This is probably because it was returning a QMap<QString,QString> which is not supported by DBus Adaptors.
Attached patch returns this functionality by using a QVariantMap instead.

Reproducible: Always

Steps to Reproduce:
Inspect KNotes dbus interface

Actual Results:  
No notes() method

Expected Results:  
notes() method returning a map of note id's to titles
Comment 1 Tejas Guruswamy 2010-09-21 12:26:30 UTC 
Created attachment 51852 [details]
Patch restoring notes() method to knotes
Comment 2 Allen Winter 2010-09-27 00:53:19 UTC 
SVN commit 1180082 by winterz:

restore notes() dbus interface
BUG: 251914
thanks for the patch Tejas.



 M  +2 -2      knotesapp.cpp  
 M  +1 -1      knotesapp.h  
 M  +4 -0      org.kde.KNotes.xml  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1180082
Comment 3 Allen Winter 2010-09-27 00:54:52 UTC 
SVN commit 1180083 by winterz:

merge forward SVN commit 1180082 by winterz:
restore notes() dbus interface
CCBUG: 251914
thanks for the patch Tejas.



 M  +2 -2      knotesapp.cpp  
 M  +1 -1      knotesapp.h  
 M  +4 -0      org.kde.KNotes.xml  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1180083
Comment 4 Allen Winter 2010-09-27 00:56:12 UTC 
SVN commit 1180084 by winterz:

merge forward SVN commit 1180082 by winterz:
restore notes() dbus interface
CCBUG: 251914
thanks for the patch Tejas. 




 M  +2 -2      knotesapp.cpp  
 M  +1 -1      knotesapp.h  
 M  +4 -0      org.kde.KNotes.xml  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1180084