By default, Bugzilla does not search the list of RESOLVED bugs.
You can force it to do so by putting the upper-case word ALL in front of your search query, e.g.: ALL tdelibs
We recommend searching for bugs this way, as you may discover that your bug has already been resolved and fixed in a later release.

Bug 2925

Summary: dbus-1-tqt returning only last map for array of maps
Product: TDE Reporter: deloptes
Component: tqtinterfaceAssignee: Timothy Pearson <kb9vqf>
Status: RESOLVED FIXED    
Severity: major CC: bugwatch, deloptes, michele.calgaro
Priority: P5    
Version: R14.0.x [Trinity]   
Hardware: Other   
OS: Linux   
Compiler Version: TDE Version String:
Application Version: Application Name:
Bug Depends on:    
Bug Blocks: 2885    
Attachments: add a function to handle maps of objectpaths
add a function to handle maps of objectpaths

Description deloptes 2018-05-31 21:16:16 CDT
Hi,
I spent like 3 days chasing this until I came to the bottom of it.
I have not yet found a solution, but I found the place where it possibly fails.

When trying to parse data with signature a{oa{sa{sv}}} the flow is something like this:

TQT_DBusProxy::sendWithReply 
	-> TQT_DBusConnection::sendWithReply 
		-> TQT_DBusMessage::fromDBusMessage(reply)
			-> TQT_DBusMarshall::messageToList(message, dmsg)
			-> qFetchParameter(&it)
			-> TQT_DBusData qFetchMap
			-> qFetchStringKeyMapEntry


It turned out that it does not support array of objectpathkeymaps and it was returning a null key, which explains why it was returning always the last entry
Comment 1 deloptes 2018-05-31 21:18:22 CDT
Created attachment 2856 [details]
add a function to handle maps of objectpaths
Comment 2 Michele Calgaro 2018-06-01 08:19:42 CDT
To consider as addition for R14.0.6.
Thanks for the work done.
Comment 3 deloptes 2018-06-01 10:27:57 CDT
There is one tqWanring that I have overseen to remove from the patch - I could upload a clean patch later

regards
Comment 4 deloptes 2018-06-02 19:48:12 CDT
Created attachment 2857 [details]
add a function to handle maps of objectpaths

cleaned up the left over debug/warning line
Comment 6 Michele Calgaro 2018-12-19 05:46:30 CST
PR has been merged to main repo. Thanks!