summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
* | | Fix MapItemView removing wrong indices on model changesPaolo Angelelli2018-11-222-46/+51
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The incubating indices bookkeeping was wrong. This patch removes the bookkeeping and relies on the delegate model doing the right thing when a row is gone but the delegate still incubating. Fixes: QTBUG-71264 Change-Id: Ibf5e525aa7ac79faf2fa149b52def05893d0bcc0 Reviewed-by: Richard Moe Gustavsen <richard.gustavsen@qt.io>
* | | winrt: Fix msvc condition in configure testsOliver Wolff2018-11-221-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Versioned win32-msvc legacy scopes have been deprecated and might yield wrong results. As we do not support any versions older than the previously checked ones an unversioned check is fine. Change-Id: I234c5204c58e76a1a685f24c2aba316b1fe50508 Reviewed-by: Andy Shaw <andy.shaw@qt.io> Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@qt.io>
* | | Do not build geoclue plugins on platforms != linuxOliver Wolff2018-11-221-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | While dbus might be available on other platforms, the plugin might not be functional and thus should not be built on these. Change-Id: I7671a3ecaba6944f99317eee4f7738a7df98ba10 Reviewed-by: Andy Shaw <andy.shaw@qt.io> Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | | Fix GeoProjectionWebMercator and MapPolyline incorrect projectionPaolo Angelelli2018-11-192-5/+135
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch fixes rendering artifacts with polylines appearing when rendering large polylines and/or at high zoom levels. Two problems caused the artifacts: 1.A too close near plane when calculating the projectable region in QGeoProjectionWebMercator, which presumably introduced numerical errors when then using such a region to clip data to be projected using a projection transformation based on the same frustum. 2.Projected polylines too large for qTriangulatingStroker, that would then introduce artifacts at screen. To solve 1., as a temporary solution, the distance of the near plane has been increased to a value that seems safe for zoom levels < 19. This while a better formula that scales further is being researched. To solve 2., screen-space line clipping has been brought back from 5.8, and added on top of mercator-space clipping. This, in theory, should also increase the performance, allowing qTriangulatingStroker to process less data. Task-number: QTBUG-71607 Change-Id: Id774419dde819931e2fdd78b02081695a91302ef Reviewed-by: Eskil Abrahamsen Blomfeldt <eskil.abrahamsen-blomfeldt@qt.io>
* | | Doc: Make documented QML imports follow Qt versioningTopi Reinio2018-11-163-87/+113
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We can now pass parameters to qdoc commands - use them to automate the version bumps for documented QML imports. Also, improve the structure and language on QML module pages. Change-Id: Id56d36a9bc550ab9a1aea1af6ff8da83e5e1feb9 Reviewed-by: Paul Wicking <paul.wicking@qt.io> Reviewed-by: Paolo Angelelli <paolo.angelelli@qt.io>
* | | Improve error reporting in plugin loadingPaolo Angelelli2018-11-052-2/+12
| | | | | | | | | | | | | | | Change-Id: Ic49ab959a028adc9edd60018351386d51b2a9bca Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | | Remove dead codePaolo Angelelli2018-11-051-1/+0
| | | | | | | | | | | | | | | | | | | | | Removing a local variable never used. Change-Id: Id2517d0f326b16b4b4ad26e53e4115a33f44c7c2 Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | | Mapbox GL: Add mapboxgl.china plugin parameterBruno de Oliveira Abinader2018-10-315-28/+54
|/ / | | | | | | | | | | | | | | | | Adds support for setting up Mapbox China API base URL + China government-approved styles. Change-Id: I66d9e15f336804466285e0996b0e326ef288be44 Reviewed-by: Alex Blasche <alexander.blasche@qt.io> Reviewed-by: Paolo Angelelli <paolo.angelelli@qt.io>
* | Merge remote-tracking branch 'origin/5.11' into 5.12.0v5.12.0-beta4Liang Qi2018-10-302-0/+4
|\ \ | |/ | | | | Change-Id: I0f4a3b7bc7c5c7f8da5af631000a6d5156884efe
| * Fix crash when calling QGeoPath::length on empty QGeoPath instanceAlex Blasche2018-10-242-0/+4
| | | | | | | | | | | | | | Fixes: QTBUG-71355 Change-Id: I47b058891a47cd6d020c4e192d40233c57814313 Reviewed-by: Timur Pocheptsov <timur.pocheptsov@qt.io> Reviewed-by: Paolo Angelelli <paolo.angelelli@qt.io>
* | Move qquickcoordinateanimation to positioningquickPaolo Angelelli2018-10-296-9/+8
| | | | | | | | | | | | | | | | | | So plugins can use it, also stand alone ones, and no ugly "import ../" anymore. Change-Id: I4e53815e041e13d495026d60a826a31bccaef2b4 Reviewed-by: Alex Blasche <alexander.blasche@qt.io> Reviewed-by: Bramastyo Harimukti Santoso <bramastyo.harimukti.santoso@pelagicore.com>
* | update esri doc for place searches and creditsGuillaume Belz2018-10-251-9/+5
| | | | | | | | | | Change-Id: Ie8af7ed00a2f97439f6d801b284a49cfcf281ab0 Reviewed-by: Paolo Angelelli <paolo.angelelli@qt.io>
* | Doc: Fix \code command usageTopi Reinio2018-10-252-9/+24
| | | | | | | | | | | | | | | | | | | | | | Since Qt 5.12, \code command accepts parameters, and in-line usage no longer works (and was never documented to work). Also, use \badcode for qmake snippets to make it appear as plain text. Task-number: QTBUG-70980 Change-Id: I53cafdc912b4663d03c19b0a0003693330c94f15 Reviewed-by: Martin Smith <martin.smith@qt.io>
* | CoreLocation: fix unconditional (previously) method callsTimur Pocheptsov2018-10-221-8/+19
| | | | | | | | | | | | | | | | | | that result in a warning nowadays, saying we must have a description in Info.plist to be able to call these methods. Task-number: QTBUG-41827 Change-Id: Ib1997a609553345358aded419a403582ccfdc0fb Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | Esri: enable place searchv5.12.0-beta3v5.12.0-beta2Guillaume Belz2018-10-0931-31/+940
| | | | | | | | | | Change-Id: I136ef0a3125b42b484e36413d26aa49e59a43b57 Reviewed-by: Paolo Angelelli <paolo.angelelli@qt.io>
* | Update name of WebAssembly platformEskil Abrahamsen Blomfeldt2018-10-051-1/+1
| | | | | | | | | | | | | | | | The name was changed from html5 to wasm, so the condition needed to be updated. Change-Id: I94851dae5674c4bac7eb02f876a8251c09045f39 Reviewed-by: Paolo Angelelli <paolo.angelelli@qt.io>
* | Merge remote-tracking branch 'origin/5.11' into 5.12Liang Qi2018-10-041-1/+3
|\ \ | |/ | | | | | | | | | | Conflicts: src/imports/locationlabs/locationlabs.cpp Change-Id: I362b903966b5087022fd125b511b236323c265ed
| * Amend 8a7bde9b93 to correctly account for the UnspecifiedVisibility caseAndy Shaw2018-10-031-2/+2
| | | | | | | | | | Change-Id: I3813b65b18877f5030aea4511ca7cb850874d7c7 Reviewed-by: Paolo Angelelli <paolo.angelelli@qt.io>
| * If a category has UnspecifiedVisibility then we should still matchAndy Shaw2018-10-021-1/+3
| | | | | | | | | | | | | | | | | | Since it is possible that a category has unspecified visibility then when comparing we still want to match on those. Otherwise it can end up not finding any matches as a result. Change-Id: I586bc3c1c91502ff56ea2974ab87a5dddc9d1263 Reviewed-by: Paolo Angelelli <paolo.angelelli@qt.io>
| * Use the correct name for the resource when building staticallyAndy Shaw2018-09-271-1/+1
| | | | | | | | | | | | Task-number: QTBUG-69617 Change-Id: I98bd56c07a5897cae62a9c287dab1cacd5f8d07d Reviewed-by: Paolo Angelelli <paolo.angelelli@qt.io>
* | Fix HERE route parsingv5.12.0-beta1Paolo Angelelli2018-09-243-229/+51
| | | | | | | | | | | | | | | | | | | | We were previously not requesting all relevant attributes, and deriving some required info by ourself, in the wrong way. So just request what's needed and use it. Task-number: QTBUG-70499 Change-Id: I6e0d4b28da1412a4c13460d25e16e383f0aca9a0 Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | Update plugins.qmltypesAlex Blasche2018-09-245-26/+172
| | | | | | | | | | | | Task-number: QTBUG-70264 Change-Id: I2922b855d08e42bbd771cc959ea9b6e97eed60b9 Reviewed-by: Kai Koehne <kai.koehne@qt.io>
* | Revert "Mark QNmeaPositionInfoSource tests as insignificant on Windows."Tony Sarajärvi2018-09-215-35/+5
| | | | | | | | | | | | | | | | | | | | | | This reverts commit 63c8b3bca03586994c2b59c262e9cb3a9b12b363. With vSphere based VMs moved to KVM based, we can try if the networking issues have been solved. Change-Id: I41fd6845341df5722c1ba2e0cd76e01bf16df4fb Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org> Reviewed-by: Liang Qi <liang.qi@qt.io>
* | Check authorization in QGeoPositionInfoSourceCL::enableLocationManagerPaolo Angelelli2018-09-181-1/+33
| | | | | | | | | | | | Task-number: QTBUG-52660 Change-Id: I528bde71dd0ad5ad58e26a9d484364b35b6c4b62 Reviewed-by: Timur Pocheptsov <timur.pocheptsov@qt.io>
* | Improvements to geoclue2 pluginAki Koskinen2018-09-172-80/+48
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Relies on automatic service activation: https://dbus.freedesktop.org/doc/dbus-specification.html#message-bus-starting-services This means that the service isn't explicitly started but gets started automatically when a request to the service is made. It's also possible that the geoclue2 service gets shutdown during runtime (for example when no client is requesting updates for some time). Automatic service activation ensures, that the service gets (re)started whenever needed. Create the manager interface right in the constructor as there's no benefit in creating it lazily. Also, since automatic service activation is used, m_manager->isValid() can't be used, because it always returns false until the service has been (automatically) started. Always return correct values from supportedPositioningMethods(). The use of the m_manager->availableAccuracyLevel() shorthand helper doesn't tell the difference between an invalid property (for example if geoclue2 isn't available at all) and GCLUE_ACCURACY_LEVEL_NONE since both return 0. Getting the QVariant of the property is needed to distinguish those cases. An invalid QVariant means that we couldn't communicate to geoclue2. A QVariant that can be turned into 0 means GCLUE_ACCURACY_LEVEL_NONE. In this commit GCLUE_ACCURACY_LEVEL_NONE is also correctly turned into NoPositioningMethods. If client doesn't exist when startClient() is called, create a client first. It's also possible that a client is crerated more than once during runtime. For example, when the above mentioned automatic geoclue2 service shutdown takes place, the old client is no longer usable. Now in this case the old client gets discarded and a new client is created. Call startClient() in startUpdates() – because otherwise startUpdates() wouldn't really work. Remove the warning message from the beginning of configureClient(). It's quite OK that configureClient() gets called before m_client exists (it just doesn't do anything then). This was probably in place with the assumption that setUpdateInterval(), setPreferredPositioningMethods() and startUpdates() would get called in some specific order. But no such assumptions can be made. Delete client interface object once it's not usable anymore (either gives an error or is explicitly stopped). This way, a new client gets created once a client is needed again. Other minor changes: - In startClient(), check whether starting the client is needed before doing it. This removes the need to do any checks before calling this function. - In stopClient(), check whether stopping the client is OK before doing it. This removes the need to do any checks before calling this function. - Use QPointer for the D-Bus interface objects -> no need to set the pointers to nullptr after deletion. - Add explicit casts to few places that gave implicit signedness change warnings. Change-Id: If9db25775710fe64f272db222030e481533d9732 Reviewed-by: Volker Krause <volker.krause@kdab.com> Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | Add QGeoMapParameter::hasPropertyPaolo Angelelli2018-09-132-0/+7
| | | | | | | | | | | | | | Convenience method to test if a property is present. Change-Id: I01c6748e35aadfd89eecebf8b8cc64a278bba87f Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | Set bounds in qgeorouteparserosrmv5.cppPaolo Angelelli2018-09-131-0/+2
| | | | | | | | | | | | | | | | This has been missing until now. Change-Id: Ida8f7aa0d14e549d1ad1a04a85d096ff5801b71e Reviewed-by: BogDan Vatra <bogdan@kdab.com> Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | QGeoPath: fix clear APIv5.12.0-alpha1Samuel Gaist2018-09-123-3/+3
| | | | | | | | | | | | | | | | | | Change the clear method name from clearPath to just clear to follow more closely the other Qt APIs. Change-Id: I98d4ea43e69e3d365c2c470e9b8b14479f288fe6 Reviewed-by: Paolo Angelelli <paolo.angelelli@qt.io> Reviewed-by: Lars Knoll <lars.knoll@qt.io>
* | Merge remote-tracking branch 'origin/5.11' into 5.12Qt Forward Merge Bot2018-09-111-0/+38
|\ \ | |/ | | | | Change-Id: I6b28d02c3bb589b4ea6672693d4978a6c342d181
| * Merge remote-tracking branch 'origin/5.11.2' into 5.11Qt Forward Merge Bot2018-09-091-0/+38
| |\ | | | | | | | | | Change-Id: I1e2854bf5a4220d7c9252562c0583b704896df09
| | * Add changes file for Qt 5.11.2v5.11.25.11.2Paolo Angelelli2018-08-311-0/+38
| |/ | | | | | | | | Change-Id: I7045270108795be9d8a4dc1073748e1b9bb7d56f Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | Fix QGeoRouteSegment documentationPaolo Angelelli2018-09-051-2/+2
| | | | | | | | | | | | | | | | | | | | | | Up till now QGeoManeuvers inside QGeoRouteSegments were generated as starting maneuvers, that is, related to the beginning of the segment. But they were documented as being related to the end of the segment. This patch fixes the documentation. Change-Id: Ia08ed2bd597b4f63e6b8a7b3518755d895446a92 Reviewed-by: Alex Blasche <alexander.blasche@qt.io> Reviewed-by: Christopher Adams <chris.adams@jollamobile.com>
* | Fix HERE qgeoroutexmlparser and add legs supportPaolo Angelelli2018-09-053-75/+298
| | | | | | | | | | | | | | | | | | | | All route segments after the second waypoint contain broken paths. This patch fixes it refactoring the parsing logic, also adding support for route legs. Change-Id: I30ec9e889a5e16bf81c4a0caf1b59fc895b7ea5e Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org> Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | Allow injecting the qml engine into geoservice pluginsPaolo Angelelli2018-09-046-8/+53
| | | | | | | | | | | | | | | | With this geoservice plugins can register Image Providers. Change-Id: Ie2380d658758d2eba376035b4d0b5d5121cfcde2 Reviewed-by: Paolo Angelelli <paolo.angelelli@qt.io> Reviewed-by: Simon Hausmann <simon.hausmann@qt.io>
* | Safer and easier navigation APIs for pluginsBogDan Vatra2018-08-307-133/+153
| | | | | | | | | | | | | | | | | | | | Each QDeclarativeNavigator will create and manage its own QAbstractNavigator using the QNavigationManagerEngine, this way the QNavigationManagerEngine doesn't have to manage the sessions itself based on a const reference. Change-Id: Ibe006524969b4f8100e71ea188d951072b846c6c Reviewed-by: Paolo Angelelli <paolo.angelelli@qt.io>
* | Add "we mean it" warning to qplacesearchrequest_p.hPaolo Angelelli2018-08-301-0/+11
| | | | | | | | | | | | | | Previously missing. Change-Id: Id77a29d6d1fbcb93e265366828ab1203a18e09d7 Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | Set the request into QGeoRoute results in osm, mapbox pluginsPaolo Angelelli2018-08-272-0/+14
| | | | | | | | | | Change-Id: Iec8d1cb9ad369fe765e34bdcff2512e2726797ad Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | Add support for route legsPaolo Angelelli2018-08-2410-25/+432
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch adds support for route legs, that are the portions of a route between one waypoint and the next. QGeoRouteLeg in particular can be seen as an API addition to QGeoRoute in that it uses and exposes additional methods added to QGeoRoutePrivate but not used in QGeoRoute. Currently the request for legs is set to be the same as the request for the entire route. Finding the related bounding waypoints has to be done programmatically using the legIndex property. Change-Id: If462b1dc6348be16dc96b167db5500f079fe0a64 Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org> Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | Honor QGeoPolygon holes in the MapboxGL pluginPaolo Angelelli2018-08-221-9/+21
| | | | | | | | | | | | | | | | This since mapbox-gl-native is able to properly render these holes. Change-Id: I29ee9efd256c607ad36543a949b64699de310371 Reviewed-by: Bruno de Oliveira Abinader <brunoabinader@gmail.com> Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | Fix missing emits pre-map-initializationPaolo Angelelli2018-08-211-0/+15
| | | | | | | | | | | | | | | | | | | | This patch fixes a regression introduced with 367c49e91366aa9b13bc7d64209321168680841e, that doesn't make the Map emit property changed signals before it is initialized for camera data properties. Change-Id: I14ea0a08780d7b2ed79c72f37bc340a1780a0d78 Reviewed-by: BogDan Vatra <bogdan@kdab.com>
* | Add QGeoPolygon holes support to LocationSingletonJulian Sherollari2018-08-202-0/+35
| | | | | | | | | | Change-Id: I877e9c8e9d11c40ac302ceeaa9c8c85da958f038 Reviewed-by: Paolo Angelelli <paolo.angelelli@qt.io>
* | Add hole support in QGeoPolygonJulian Sherollari2018-08-204-0/+156
| | | | | | | | | | | | | | | | [ChangeLog][QtPositioning] Added holes in QGeoPolygons Change-Id: I490bcf8c4a6d9f35fa364072d8ea70b914bfc640 Reviewed-by: Alex Blasche <alexander.blasche@qt.io> Reviewed-by: Paolo Angelelli <paolo.angelelli@qt.io>
* | Move management of QDeclarativeGeoMap camera data into QGeoMapPaolo Angelelli2018-08-206-132/+109
| | | | | | | | | | | | | | | | | | | | | | | | | | This patch makes QDeclarativeGeoMap always refer to QGeoMap when it comes to setting and getting camera data properties. The signal emission for each of these properties is also deferred to the reception of QGeoMap::cameraDataChanged. This patch also enable plugins to enforce tilt ranges per zoom, updating the tilt value (as well as any other camera data value) on their own. Change-Id: Icc16645ea53fa2b53b33530f802ce390a2479d39 Reviewed-by: BogDan Vatra <bogdan@kdab.com>
* | Merge remote-tracking branch 'origin/5.11' into devQt Forward Merge Bot2018-08-181-2/+2
|\ \ | |/ | | | | Change-Id: I001477f8534dd8eb565a365143c5799834df56ff
| * Geoclue: respect minimum update interval for satellite info updatesChris Adams2018-08-171-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | If no update interval is specified for the Geoclue-backed satellite info source, it should use the minimum update interval (which is defined in the code as being one second) rather than zero, otherwise the satellite information is reset immediately after receiving an update from Geoclue. Change-Id: Ib48b8dfd600722f23c8bafc50adabfb5df546eaa Reviewed-by: Paolo Angelelli <paolo.angelelli@qt.io> Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | Long live for the geoclue2 position pluginDenis Shienkov2018-08-1612-0/+1149
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Geoclue2 requires the appropriate application's desktop id, to allow an access to the location service. This desktop id should be added to the /etc/geoclue/geoclue.conf file, e.g.: [my-app] allowed=true system=true users= Also, this desktop id should be imported to the Geoclue2 application via the QT_GEOCLUE_APP_DESKTOP_ID environment variable, e.g.: export QT_GEOCLUE_APP_DESKTOP_ID=my-app Note: Tested on stationar PC with the ArchLinux. Task-number: QTBUG-43435 Done-with: Volker Krause <vkrause@kde.org> Change-Id: I5cc204cca966cb23a59ccffbc83fd8dc7a7c4eb6 Reviewed-by: Denis Shienkov <denis.shienkov@gmail.com>
* | Remove src/locationlabsPaolo Angelelli2018-08-161-69/+0
| | | | | | | | | | | | | | Apparently some leftovers from the wip/navigation -> dev merge Change-Id: I52313ba215d656225b825d192af1db66d3fb2365 Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | Bump Mapbox GL to the latest versionSudarsana Babu Nagineni2018-08-142-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Bumps Mapbox GL to a new version that fixes the compilation errors on QNX 7. Also, enable Mapbox GL Native plugin for QNX. mapbox-gl-native @ 27b21363e62c105db0b040b4c5a5ef31170ebd30 Task-number: QTBUG-59685 Change-Id: Icc70ab53ba6f76e319feb53858f3614f3edca3e1 Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org> Reviewed-by: Paolo Angelelli <paolo.angelelli@qt.io>
* | Make QDeclarativeGeoMapItemBase::geoShape R/WPaolo Angelelli2018-08-1412-14/+90
| | | | | | | | | | | | | | | | This patch adds a setter for the geoShape property, overridden in the subclasses. Change-Id: I5fc4d412efee5e95c5650943ea5bf9b72dc40155 Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | Merge remote-tracking branch 'origin/5.11' into devQt Forward Merge Bot2018-08-080-0/+0
|\ \ | |/ | | | | | | | | | | Conflicts: .qmake.conf Change-Id: I46c40d4247bfc8d8310f1adc71bc9c6d225f26a0