summaryrefslogtreecommitdiff
path: root/src/location/places
Commit message (Collapse)AuthorAgeFilesLines
* Eradicate Java-style iterators and mark the module free of themMarc Mutz2019-08-031-4/+1
| | | | | | | | | | | | | | | ... and of QLinkedList Java-style iterators are going to be deprecated, or at the very least banned from use in Qt code. Ditto QLinkedList. Unfortunately, the module contains more than 120 uses of Q_FOREACH, even though, according to my sources, its use was banned in Qt implementation from the get-go. So QT_NO_FOREACH is currently not an option. Change-Id: I0f05e9c78dda259b0eac1bcdfc7dddfcddc4b908 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* Unify semicolon usage in Q_UNUSEDJesus Fernandez2019-01-231-16/+16
| | | | | Change-Id: I36dd6881b6f6f028869d63b6311cba7f52a99cc4 Reviewed-by: Paolo Angelelli <paolo.angelelli@qt.io>
* Allow searching along a QGeoRoutePaolo Angelelli2018-11-011-0/+2
| | | | | | | | | | | | | | | The result will be backend-dependent. If a backend does not support this, it will be the same as not specifying a search area. The current approach does not expose any new public API, but also does not allow specifying the distance from the route. If this will become necessary, public API might have to be extended. Task-number: QTBUG-69649 Change-Id: I9230d502a6cdb55c620e3a57a0e78cdb6f8955a6 Reviewed-by: Michael Brasser <michael.brasser@live.com> Reviewed-by: Alex Blasche <alexander.blasche@qt.io> Reviewed-by: BogDan Vatra <bogdan@kdab.com>
* 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>
* | 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>
* | Enable asynchronous incremental updates of QPlaceReplyPaolo Angelelli2018-07-272-0/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch adds a new signal to QPlaceReply, contentUpdated. This signal can be emitted in subclasses when the provider is capable of notifying partial updates of the content. The typical use case if fetching the data from multiple databases, also locally instead of over network, and incrementally populating the results, as opposed to the current way of operating, that is via previous page / next page, in which case it's the backend who provides the proper query parameters to obtain the previous or the next page. This signal is currently handled only for QPlaceSearchReplies in QDeclarativeSearchResultModel, where such a signal would trigger a layout update. Change-Id: I0aaba5aa9249b61c970d3a309d93fab7fc39d667 Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | Enable incremental updates in PlaceSearchModelPaolo Angelelli2018-07-054-23/+103
|/ | | | | | | | | | | This way pages can be changed without resetting the model. To achieve this, new members into QPlaceSearchRequestPrivate are introduced, to keep the relationship between a request for one page, and the previous or the next page. In this way sparse population of the model becomes possible. Change-Id: Ic8db0281408f3500ba83f78c7e152ee0b68cd099 Reviewed-by: Eskil Abrahamsen Blomfeldt <eskil.abrahamsen-blomfeldt@qt.io>
* Doc: Fix all documentation warningsTopi Reinio2018-03-273-5/+7
| | | | | | | | | | | | | | | | | | | | Fix multiple warnings caused by invalid documentation configuration, missing QML module specifiers, wrong QML type names, incorrect arguments for the \fn command, and missing documentation. Remove incorrect statement from the example usage of the Waypoint QML type. Use 'QtLocation' as the name for the module throughout the docs. Clang (used as the parser for QDoc in Qt 5.11) was unable to resolve the path for qlocation.h, causing QDoc to omit the QLocation namespace documentation. Fix this by including the file explicitly from qlocationglobal.h for documentation builds. Change-Id: I2bd790db1ff5ad96f0cdb415a682b97593b3a6dc Reviewed-by: Martin Smith <martin.smith@qt.io>
* Replace Q_NULLPTR with nullptrKevin Funk2018-02-059-10/+10
| | | | | Change-Id: Icbebfaae93cc5827d80ef09323ec82868789838a Reviewed-by: Lars Knoll <lars.knoll@qt.io>
* Replace Q_DECL_OVERRIDE with overrideKevin Funk2018-02-053-4/+4
| | | | | Change-Id: Iaa670de181255ea570fcf7bcbe465987ef7fc2fc Reviewed-by: Lars Knoll <lars.knoll@qt.io>
* Make QPlace extensiblePaolo Angelelli2018-01-274-194/+365
| | | | | | | | This change makes it possible to subclass QPlace with custom private implementations. Change-Id: I363c0e8b7db41d9a8400ce6dbddf5405c619eeef Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* Remove moc_* inclusions from source filesPaolo Angelelli2017-02-141-2/+0
| | | | | | Change-Id: I7385348d7b6ec22fa92ed1be65f89e262c38b5c0 Reviewed-by: Michal Klocek <michal.klocek@qt.io> Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* Fix export macro in private headersPaolo Angelelli2017-02-141-7/+8
| | | | | | | | Changing Q_LOCATION_EXPORT to Q_LOCATION_PRIVATE_EXPORT in all _p headers to avoid confusion. Change-Id: Ic42d1c59f0cd68c27bcf40296c612d005da11efd Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* Doc: Location *Reply classes: Document aborted() signalTopi Reinio2017-01-301-1/+12
| | | | | | | | | | | | | | | Fixes the following qdoc warnings: src/location/maps/qgeocodereply.h:83: warning: No documentation for 'QGeoCodeReply::aborted()' src/location/maps/qgeoroutereply.h:77: warning: No documentation for 'QGeoRouteReply::aborted()' src/location/places/qplacereply.h:89: warning: No documentation for 'QPlaceReply::aborted()' Change-Id: Id8c871abf2a197e41042bfa74e1abf9db94a65e4 Reviewed-by: Paolo Angelelli <paolo.angelelli@qt.io> Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* Refactoring: removing m_reply from GeoReply classesPaolo Angelelli2016-11-292-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch tries to simplify the code removing the contained m_reply from all the Geo[Map,Route,Geocode,Places]Reply classes. The need for m_reply was associated to the "abort" method, but this can be solved by emitting a signal in the superclass abort() method, and connecting that to QNetworkReply::abort() in the constructor. Since QNetworkReplyHttpImpl always sends an OperationCanceledError it should then be safe to call deleteLater() on the network reply in the slot connected to QNetworkReply::error This patch also prevents the series of "QCoreApplication::postEvent: Unexpected null receiver" warnings that are generated due to deletingLater already deleted objects (abort() emits an error, the reply is destroyed inside the onError slot, but also in the abort() method). Finally, this patch removes the setFinished() call in QGeoRouteReply::abort() since the documentation does not mention this, and all the subclasses do not perform this operation and emit the corresponding signal. tst_qgeoroutereply has been adapted accordingly. Change-Id: I226ee163e7bed784dd7f0da1522e651459543bca Reviewed-by: Alex Blasche <alexander.blasche@qt.io> Reviewed-by: Edward Welbourne <edward.welbourne@qt.io>
* Add explicit and make public headers compile with ↵Marc Mutz2016-03-039-10/+10
| | | | | | | | | | | | | | | | | -Wzero-as-null-pointer-constant ... or equivalent. QtBase 5.6 headers already compile that way, so let the other modules follow suit. Added explicit where it was missing. This is not a source- incompatible change, because code that breaks by this is a bug. Let's not have this sitting around in an LTS. Task-number: QTBUG-45291 Change-Id: I279fd100b61d3e2b2e46e7cbd30cc6b4bfb4d10e Reviewed-by: Alex Blasche <alexander.blasche@theqtcompany.com>
* Add missing "We mean it" comments to private headers.Friedemann Kleint2015-10-0812-0/+132
| | | | | Change-Id: Id3ef277b5610a45ffa42850dd9415e963ed45285 Reviewed-by: Alex Blasche <alexander.blasche@theqtcompany.com>
* Bump all QtLocation 5.5 tags to 5.6Alex Blasche2015-09-2325-25/+25
| | | | | | | | | | | | | | | 5.5 was a tech preview release. The real release is 5.6. Some 5.5 interfaces did change. It is possible that there are apps in existence which are based on the 5.5 API's. To avoid miscommunication we bump all version tags to the real first release version. The existing 5.5 QML imports will continue to work but apps may encounter API changes when they move from 5.5 to 5.6. The API changes are listed in the QtLocation documentation. Change-Id: Ia7000e871a5ee6b09234e5d58d4fc8927e93b154 Reviewed-by: Laszlo Agocs <laszlo.agocs@theqtcompany.com>
* Fix remaining QtLocation qdoc warningsAlex Blasche2015-08-281-1/+1
| | | | | | | | | | | /home/bb/dev/qt/qt56/qtlocation/src/imports/location/qquickgeomapgesturearea.cpp:145: warning: C++ class QDeclarativeGeoMapGestureArea not found: \instantiates QDeclarativeGeoMapGestureArea /home/bb/dev/qt/qt56/qtlocation/src/location/places/qplaceeditorial.cpp:99: warning: Cannot find 'QPlaceEditorial(...)' in '\fn' QPlaceEditorial::QPlaceEditorial(const QPlaceContent &other) [I cannot find any function of that name with the specified signature. Make sure that the signature is identical to the declaration, including 'const' qualifiers.] Change-Id: I10c2326364c8e8d29ddf641f42267302b05b9379 Reviewed-by: Aaron McCarthy <mccarthy.aaron@gmail.com>
* Fix QtLocation \since tags in QML and C++Alex Blasche2015-05-2925-25/+25
| | | | | | | | The first release is 5.5 and not 5.0 Change-Id: Ib659adf77293e10e8f3d117ce16bbc25858d6ab2 Reviewed-by: Michal Klocek <michal.klocek@theqtcompany.com> Reviewed-by: Alex Blasche <alexander.blasche@theqtcompany.com>
* Changes key of "nokia" geoservices plugin to "here"Alex Blasche2015-03-252-2/+2
| | | | | | | | | | | | The old name is not advertised anymore but the plugin can still be loaded using the "nokia" key. This will be removed by later versions of Qt. Adds a QGeoServiceProvider unit tests Change-Id: I01f36f1bb19d31e0855e90c1605b99e61ee4450b Reviewed-by: Michal Klocek <michal.klocek@theqtcompany.com> Reviewed-by: Alex Blasche <alexander.blasche@theqtcompany.com> Reviewed-by: Aaron McCarthy <mccarthy.aaron@gmail.com>
* Remove LGPLv2.1 and add GPLv2+ from/to QtLocationAlex Blasche2015-02-1974-1033/+1255
| | | | | | Change-Id: I3007aa04d3dbc0e00cb0a43987d64d01ddf91edd Reviewed-by: Lorn Potter <lorn.potter@gmail.com> Reviewed-by: Maurice Kalinowski <maurice.kalinowski@theqtcompany.com>
* Export the *Unsupported classes.Friedemann Kleint2015-02-191-7/+7
| | | | | | | Fix linking errors of the OSM plugin on Windows. Change-Id: I8bfe73266665fd83281ec96dbca05faaa34b184d Reviewed-by: Alex Blasche <alexander.blasche@theqtcompany.com>
* Update contact URL in all license headers to qt.ioAlex Blasche2015-02-1874-74/+74
| | | | | Change-Id: Ia1e333be7fec8898de609d9b9303b1ad7687632e Reviewed-by: Maurice Kalinowski <maurice.kalinowski@theqtcompany.com>
* Add Q_DECL_OVERRIDE where missingAlex Blasche2015-02-171-3/+12
| | | | | | | | | | | | | | | | clang 3.6 and later print a warning for such cases. This patch fixes all override warnings caused by the content of the qtlocation itself. The remaining cases are triggered by headers outside of the qtlocation repo. This patch only fixes cases which clang on Linux can reach. It is possible that some of the platform specific plugins still have some ovveride related cases. Change-Id: I9f93062511d38cba4449b7fc70e65c93d4251a8b Reviewed-by: Michal Klocek <michal.klocek@theqtcompany.com> Reviewed-by: Aaron McCarthy <mccarthy.aaron@gmail.com> Reviewed-by: Alex Blasche <alexander.blasche@theqtcompany.com>
* Merge remote-tracking branch 'gerrit/5.4' into devAlex Blasche2015-02-118-20/+20
|\ | | | | | | Change-Id: I5ef7649f02455269a8e2caa2f2d2241503586be3
| * Add the QtCore/ library prefix in public headersSergio Ahumada2015-02-098-20/+20
| | | | | | | | | | | | Change-Id: I538f0de4d2fca50e42114934f12dfd0ee51088e9 Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@theqtcompany.com> Reviewed-by: Alex Blasche <alexander.blasche@theqtcompany.com>
* | Add support for places to Open Street Map plugin.Aaron McCarthy2015-02-023-7/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | Implement basic places support based on the Open Street Map Nominatim service. Support for read only categories and place searching is supported. The plugin does not support getting place details, getting place content, search suggestions, saving/removing places or saving/removing categories. Change-Id: I5a185cdf25b50d5b377be4d2c3c53c8f1e807288 Reviewed-by: Alex Blasche <alexander.blasche@theqtcompany.com>
* | Cleanup qRegisterMetaType<T>() callsAlex Blasche2015-01-152-3/+3
|/ | | | | | | The non-parameter version is much more secure. Change-Id: I70612c60d780a972bbf9c852f50f7b0c6f2a0402 Reviewed-by: Jędrzej Nowacki <jedrzej.nowacki@theqtcompany.com>
* Doc: Applied title case to section1 titlesv5.4.0-beta1Nico Vertriest2014-10-091-1/+1
| | | | | | Change-Id: I7ad07daac92e1af8c7daa4f59b948692c03a85e3 Task-number: QTBUG-41250 Reviewed-by: Topi Reiniö <topi.reinio@digia.com>
* Update license headers and add new license filesAntti Kokko2014-08-2474-1403/+811
| | | | | | | | | - Renamed LICENSE.LGPL to LICENSE.LGPLv21 - Added LICENSE.LGPLv3 & LICENSE.GPLv2 - Removed LICENSE.GPL Change-Id: Iedee2507ab1bd46788409a98505d7be510253110 Reviewed-by: Jani Heikkinen <jani.heikkinen@digia.com>
* Fix typoSergio Ahumada2014-04-021-1/+1
| | | | | Change-Id: I04183182a9db9dc169390fe5c77a92588f40347b Reviewed-by: Mitch Curtis <mitch.curtis@digia.com>
* Fix some typosSergio Ahumada2014-03-033-3/+3
| | | | | Change-Id: If46057e48040fda7b61b81e735c171daf2cc7d06 Reviewed-by: Jerome Pasion <jerome.pasion@digia.com>
* Remove offset attribute from place content requests.Aaron McCarthy2014-01-229-41/+113
| | | | | | | | | | | | This complements 1d0966c6924876655c52725e779a7aa24866ff60 Not all service providers support arbitrary offsets when requesting place content. The offset attribute has been removed, instead service providers can supply a previous and next content request query in the form of a QPlaceContentRequest. Change-Id: I6239b42c31bed5dcd6645d86d5a48d7e6a667f46 Reviewed-by: Alex Blasche <alexander.blasche@digia.com>
* Remove offset attribute from place search requests.Aaron McCarthy2014-01-204-37/+64
| | | | | | | | | Not all service providers support arbitrary search offsets. The offset attribute has been removed, instead service providers can supply a previous and next search query in the form of a QPlaceSearchRequest. Change-Id: If68e969b06f909f3fa62718656cbef70d1bf84a6 Reviewed-by: Aaron McCarthy <mccarthy.aaron@gmail.com>
* Use QVariantMap instead of QMap<QString, QVariant>.Aaron McCarthy2014-01-082-2/+2
| | | | | | | | It is shorter, easier to read, easier to type and it doesn't affect the ABI. Change-Id: Ic4d8a371d00b434ac84382dd75d8d82a963089cb Reviewed-by: Alex Blasche <alexander.blasche@digia.com>
* Fix some name inconsistencies in QtLocationAlex Blasche2013-09-034-35/+35
| | | | | | | | | | | | | 1. Rename QGeocoding* classes to QGeoCoding*. This emphasizes the QGeo prefix rather than the geocing word. The QML interfaces continues to use the Geocoding* as they don't a prefix as such. 2. Rename QProposedSearchResult to QPlaceProposedSearchResult. The class is place related and should carry the QPlace name prefix Change-Id: I428c7e6368774a9f199db5c113de11649e4a665c Reviewed-by: Kurt Pattyn <pattyn.kurt@gmail.com> Reviewed-by: Aaron McCarthy <mccarthy.aaron@gmail.com>
* Split QtLocation into QtPositioning and QtLocationAlex Blasche2013-09-021-4/+4
| | | | | | | | | | | | QtPositioning is not dependent on any gui component and mostly covers everything around the retrieval of position information. QtLocation covers place, map and navigation APIs. Several documentation issues were fixed during the process. Change-Id: I98e2a53065549a2fc43f93a75b4f65b5bc884c92 Reviewed-by: Alex Blasche <alexander.blasche@digia.com>
* Support explore functionality.Aaron McCarthy2013-07-2610-5/+318
| | | | | | | | | | | | | | | | | | | | | Add support for Nokia's explore functionality in places. This adds the ProposedSearchResult search result type. The proposed search result has a QPlaceSearchRequest, which can be submitted to QPlaceManager::search() to perform the proposed search. This has the added advantage that any arbitrary search request can be returned as a search result. To facilitate this further a search context field has been added to the place search request. This field can be used by backends to store additional search context. Other relevant fields should also be filled in. This allows additional search context to be kept. The Nokia v2 places implementation has be updated to use the explore feature and to return ProposedSearchResults as well as place results. Task-number: QTBUG-24874 Change-Id: Ib20c5a8c8a60e6bf16f38dcd75961a752b8b1b32 Reviewed-by: Alex <alexander.blasche@digia.com>
* Fix QtLocation documentationAlex Blasche2013-07-1210-20/+19
| | | | | | | | | | This moves the docs to its proper place and fixes content as well as broken links. There are still some warnings left. Change-Id: Ie83086f4feabab5f3b3d6c92eb6b401a5ff43e29 Reviewed-by: Alex <alexander.blasche@digia.com>
* Update copyright year to 2013Sergio Ahumada2013-06-1326-26/+26
| | | | | Change-Id: I5b0cb7d56997ad7b91c073b3e02b31d0368fdcfa Reviewed-by: Alex <alexander.blasche@digia.com>
* QtLocation header check: compile in pedantic modeThiago Macieira2013-02-241-2/+2
| | | | | | | | qgeopositioninfosourcefactory.h:62:68: error: extra ‘;’ [-pedantic] Change-Id: Ib024d4898ae52a181133d975d09e1381f231eb7b Reviewed-by: Laszlo Papp <lpapp@kde.org> Reviewed-by: Aaron McCarthy <mccarthy.aaron@gmail.com>
* Enable the "#include <QLocation>" camel-case headerSze Howe Koh2013-02-104-4/+4
| | | | | | | | | | | | Rename qtlocation.h/cpp -> qlocation.h/cpp, and use #pragma qt_class() to generate the camel-case header during compilation. Part of the Header Consistency Project (http://lists.qt-project.org/pipermail/development/2012-October/007570.html) Change-Id: I2899c9ca0969c37365bd7e1da5d5a15e9632f5f0 Reviewed-by: Laszlo Papp <lpapp@kde.org> Reviewed-by: Stephen Kelly <stephen.kelly@kdab.com>
* Rename namespace QtLocation -> QLocationSze Howe Koh2013-02-058-22/+22
| | | | | | | | | | | | | | Main code, tests and docs updated. For consistency (with minimal disruption to the Qt 5.0.0 release), namespaces with a "Qt" prefix were renamed. Part of the Header Consistency Project (http://lists.qt-project.org/pipermail/development/2012-October/007570.html) Change-Id: Ic728cbbe39e0140dca1cdee24257dda095c9a478 Reviewed-by: Laszlo Papp <lpapp@kde.org> Reviewed-by: Aaron McCarthy <mccarthy.aaron@gmail.com>
* Remove use of QT_{BEGIN,END}_HEADER macros.Aaron McCarthy2013-02-0327-130/+0
| | | | | | | | | These macros were removed from Qt Core and they are no longer defined causing Qt Location to fail to build. Change-Id: Ibed30daa1a802957cebe95a577dfe86f0f643970 Reviewed-by: Laszlo Papp <lpapp@kde.org> Reviewed-by: Lorn Potter <lorn.potter@jollamobile.com>
* Change copyright from 2012 to 2013Rainer Keller2013-01-1745-45/+45
| | | | | Change-Id: I61ae1bd1a9f68d120d4e11635f306374b1f494d0 Reviewed-by: Aaron McCarthy <mccarthy.aaron@gmail.com>
* Fix qmake warningsRainer Keller2013-01-1713-43/+44
| | | | | Change-Id: I17265b37a557472fc55fec7e25cd2bf2b3a6dc9d Reviewed-by: Aaron McCarthy <mccarthy.aaron@gmail.com>
* Change copyrights from Nokia to DigiaIikka Eklund2012-09-2471-1704/+1704
| | | | | | | | Change copyrights and license headers from Nokia to Digia Change-Id: Ic1390da64a219642fa1e1482a7677a4c2db15cbc Reviewed-by: Janne Anttila <janne.anttila@digia.com> Reviewed-by: Lars Knoll <lars.knoll@digia.com>