Bug report #12153

QgsSnapper.snapPoint API changed

Added by Matthias Kuhn over 5 years ago. Updated over 5 years ago.

Status:Closed
Priority:Severe/Regression
Assignee:-
Category:Python plugins
Affected QGIS version:2.6.0 Regression?:No
Operating System: Easy fix?:No
Pull Request or Patch supplied:No Resolution:
Crashes QGIS or corrupts data:No Copied to github as #:20346

Description

QgsSnapper.snapPoint python binding changed from QPoint to QgsPoint.

This introduces the following error:

TypeError: QgsSnapper.snapPoint(QgsPoint, list-of-QgsSnappingResult, list-of-QgsPoint excludePoints=QList<QgsPoint>()): argument 1 has unexpected type 'QPoint'

It is very easy to maintain compatibility here as outlined in this comment:

#abd4a043998ae601835bd6749e5bcac5caa7a949#diff-a65ec2a8713673bd50b1ebc4958edc0eR85

I set the priority as a blocker so a decision needs to be taken if such an API change is desired or not.

Associated revisions

Revision 0593ef6a
Added by Jürgen Fischer over 5 years ago

rename new QgsSnapper::snapPoint method to snapMapPoint and restore python name
of deprecated old version (fixes #12153)

History

#1 Updated by Jürgen Fischer over 5 years ago

  • Status changed from Open to Closed

Also available in: Atom PDF