Esri / arcgis-maps-sdk-toolkit-qt

ArcGIS Maps SDK for Qt Toolkit
Apache License 2.0
132 stars 61 forks source link

modelData not listModelData #610

Closed bri12415 closed 8 months ago

bri12415 commented 8 months ago

This was mistakenly done by me from https://github.com/Esri/arcgis-maps-sdk-toolkit-qt/pull/605. I had thought that the data structure being referred to by controller.clientCertificateInfos was one of our ListModels, but it is not. It is just a QStringList, so we can use the modelData keyword.

Before: image

After: image

Thanks @ldanzinger for pointing this out.

ldanzinger commented 8 months ago

@bri12415 if you haven't done so already, it might be worth a scan to see if there's other areas where we made the same error?

bri12415 commented 8 months ago

@ldanzinger will do

Edit: I double checked, it looks like anywhere else we utilize the listModelData role it is in the context of a ListModel