Bug report #11976

qgis:grid parameter order changed breaking dependent scripts

Added by Anita Graser over 5 years ago. Updated over 5 years ago.

Status:Closed
Priority:Low
Assignee:Victor Olaya
Category:Processing/QGIS
Affected QGIS version:master Regression?:No
Operating System: Easy fix?:No
Pull Request or Patch supplied:No Resolution:wontfix
Crashes QGIS or corrupts data:No Copied to github as #:20183

Description

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

See https://github.com/qgis/QGIS-Processing/issues/8

History

#1 Updated by Anita Graser over 5 years ago

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

#2 Updated by Giovanni Manghi over 5 years ago

  • Target version changed from Version 2.8 to Version 2.8.2

#3 Updated by Giovanni Manghi over 5 years ago

  • Target version changed from Version 2.8.2 to Version 2.10

#4 Updated by Victor Olaya over 5 years ago

  • Priority changed from High to Low

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

#5 Updated by Anita Graser over 5 years ago

  • Status changed from Open to Closed

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.

#6 Updated by Anita Graser over 5 years ago

  • Resolution set to wontfix

Also available in: Atom PDF