Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

qgis:grid parameter order changed breaking dependent scripts #20183

Closed
qgib opened this issue Jan 11, 2015 · 6 comments
Closed

qgis:grid parameter order changed breaking dependent scripts #20183

qgib opened this issue Jan 11, 2015 · 6 comments
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter! Processing Relating to QGIS Processing framework or individual Processing algorithms
Milestone

Comments

@qgib
Copy link
Contributor

qgib commented Jan 11, 2015

Author Name: Anita Graser (@anitagraser)
Original Redmine Issue: 11976
Affected QGIS version: master
Redmine category:processing/qgis
Assignee: Victor Olaya


Most likely qgis:grid is not the only algorithm which is affected by problem.

See qgis/QGIS-Processing#8

@qgib
Copy link
Contributor Author

qgib commented Jan 11, 2015

Author Name: Anita Graser (@anitagraser)


I fixed the two scripts in the QGIS Processing repository but I'm concerned why this happened. Was the parameter order changed on purpose?

@qgib
Copy link
Contributor Author

qgib commented May 9, 2015

Author Name: Giovanni Manghi (@gioman)


  • fixed_version_id was changed from Version 2.8 to Version 2.8.2

@qgib
Copy link
Contributor Author

qgib commented May 14, 2015

Author Name: Giovanni Manghi (@gioman)


  • fixed_version_id was changed from Version 2.8.2 to Version 2.10

@qgib
Copy link
Contributor Author

qgib commented Jun 21, 2015

Author Name: Victor Olaya (@volaya)


Not sure we should still consider this a problem

The change was a minor API break, but it was already sometime ago and no one complained...so I guess that reverting it now makes no sense. Scripts in the repo are fixed, so we should leave it like that.

It is not a change that affects other algorithms, it was just a specific change of syntax

Please, close the ticket if you agree with this


  • priority_id was changed from High to Low

@qgib
Copy link
Contributor Author

qgib commented Jun 21, 2015

Author Name: Anita Graser (@anitagraser)


I guess it's fine if the scripts in the repo are fixed. I hope this kind of issue does not happen on a regular basis.


  • status_id was changed from Open to Closed

@qgib
Copy link
Contributor Author

qgib commented Jun 21, 2015

Author Name: Anita Graser (@anitagraser)


  • resolution was changed from to wontfix

@qgib qgib added Bug Either a bug report, or a bug fix. Let's hope for the latter! Processing Relating to QGIS Processing framework or individual Processing algorithms labels May 25, 2019
@qgib qgib added this to the Version 2.10 milestone May 25, 2019
@qgib qgib closed this as completed May 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter! Processing Relating to QGIS Processing framework or individual Processing algorithms
Projects
None yet
Development

No branches or pull requests

1 participant