Bug report #16407

Expression builder: Rename simplify and simplify_vw

Added by Harrissou Santanna over 2 years ago. Updated about 2 years ago.

Status:Closed
Priority:Normal
Assignee:-
Category:Expressions
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 #:24316

Description

Given their description, what about renaming:
  • simplify function into simplify_(by_)distance
  • simplify_vw function into simplify_(by_)area. Actually this is the one that bothers me: I wonder if adding "by area" wouldn't be more meaningful and remembered by user than vw (Viswalingam-Whyatt?).

(I set it as a bug report though it's not, but not feel that it's also a feature request)

History

#1 Updated by Giovanni Manghi over 2 years ago

  • Easy fix? set to No
  • Regression? set to No

#2 Updated by Jürgen Fischer about 2 years ago

  • Category set to Unknown

#3 Updated by Harrissou Santanna about 2 years ago

  • Resolution set to wontfix
  • Description updated (diff)
  • Status changed from Open to Closed
  • Category changed from Unknown to Expressions

We are following PostgreSQL/PostGIS functions convention naming which use st_simplify and st_simplifyvw
Note: A PR was proposed at https://github.com/qgis/QGIS/pull/5274

Also available in: Atom PDF