Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Merge pull request #42869 from m-kuhn/statsbycaterror
Add safety check for field name in StatisticsByCategories
  • Loading branch information
m-kuhn authored and github-actions[bot] committed Apr 21, 2021
1 parent 21fab96 commit 3a25f8a
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions python/plugins/processing/algs/qgis/StatisticsByCategories.py
Expand Up @@ -104,11 +104,15 @@ def processAlgorithm(self, parameters, context, feedback):
value_field = source.fields().at(value_field_index)
else:
value_field = None
category_field_indexes = [source.fields().lookupField(n) for n in category_field_names]
category_field_indexes = list()

# generate output fields
fields = QgsFields()
for c in category_field_indexes:
for field_name in category_field_names:
c = source.fields().lookupField(field_name)
if c == -1:
raise QgsProcessingException(self.tr('Field "{field_name}" does not exist.').format(field_name=field_name))
category_field_indexes.append(c)
fields.append(source.fields().at(c))

def addField(name):
Expand Down

0 comments on commit 3a25f8a

Please sign in to comment.