Bug report #21264

bug in saga algorithm python

Added by Wim Dedulle almost 6 years ago. Updated almost 6 years ago.

Status:Closed
Priority:Normal
Assignee:-
Category:Processing/SAGA
Affected QGIS version:3.4.4 Regression?:No
Operating System: Easy fix?:No
Pull Request or Patch supplied:Yes Resolution:
Crashes QGIS or corrupts data:No Copied to github as #:29082

Description

apps\qgis\python\plugins\processing\algs\saga\SagaAlgorithm.py
While checking QgsProcessingParameterMultipleLayers param a param XGRIDS was attempted to be added as a file
Should not get from parameters but from the layer source itself

From line 217 till 219

if layer.source().lower().endswith('sdat'):
files.append(parameters[param.name()].source()[:-4] + 'sgrd')
if layer.source().lower().endswith('sgrd'):
files.append(parameters[param.name()].source())

change to

if layer.source().lower().endswith('sdat'):
files.append(layer.source()[:-4] + 'sgrd')
if layer.source().lower().endswith('sgrd'):
files.append(layer.source())
else:

Associated revisions

Revision 6872ce7f
Added by Luigi Pirelli almost 6 years ago

Merge pull request #9194 from wdedulle/master

Fix saga algorithm when QgsProcessingParameterMultipleLayers. Fixes #21264

History

#1 Updated by Giovanni Manghi almost 6 years ago

  • Resolution deleted (fixed/implemented)
  • Category changed from Python bindings / sipify to Processing/SAGA
  • Crashes QGIS or corrupts data changed from Yes to No
  • Easy fix? changed from Yes to No

#2 Updated by Giovanni Manghi almost 6 years ago

  • Status changed from Open to Feedback

Can you submit a pull request on Github? thanks.

#4 Updated by Giovanni Manghi almost 6 years ago

  • Status changed from Feedback to Open
  • Pull Request or Patch supplied changed from No to Yes

Thanks!

#5 Updated by Luigi Pirelli almost 6 years ago

  • % Done changed from 0 to 100
  • Status changed from Open to Closed

Also available in: Atom PDF