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 - dbus-1-tqt returning only last map for array of maps
Summary: dbus-1-tqt returning only last map for array of maps
Status: RESOLVED FIXED
Alias: None
Product: TDE
Classification: Unclassified
Component: tqtinterface (show other bugs)
Version: R14.0.x [Trinity]
Hardware: Other Linux
: P5 major
Assignee: Timothy Pearson
URL:
Depends on:
Blocks: R14.0.6
  Show dependency tree / graph
 
Reported: 2018-05-31 21:16 CDT by deloptes
Modified: 2018-12-19 05:48 CST (History)
3 users (show)

See Also:
Compiler Version:
TDE Version String:
Application Version:
Application Name:

Attachments
add a function to handle maps of objectpaths (2.65 KB, patch)
2018-05-31 21:18 CDT, deloptes 		Details | Diff
add a function to handle maps of objectpaths (2.44 KB, patch)
2018-06-02 19:48 CDT, deloptes 		Details | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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!