Opened 3 years ago

Last modified 5 weeks ago

#2809 new defect

[raster] AddRasterConstraint does not check arguments upfront

Reported by: strk Owned by: dustymugs
Priority: high Milestone: PostGIS 2.2.7
Component: raster Version: trunk
Keywords: Cc:

Description

I was confused by AddRasterConstraint? failing with a vague message "could not add any constraint". Improving the error messages with r12722 I found out I had a typo in the column name, but still the AddRasterConstraint? function kept calling each constraint-specific method rather than immediately giving up.

It would be nicer if the existance of the target table/column and its readability would be checked upfront instead, giving a more human-readable message on failure.

Dunno how safe it would be to backport this (togheter with r12722) back till the 2.0 branch

Change History (6)

comment:1 Changed 3 years ago by dustymugs

Priority: mediumhigh

comment:2 Changed 2 years ago by robe

Type: enhancementdefect

comment:3 Changed 2 years ago by robe

Milestone: PostGIS 2.2.0PostGIS 2.3.0

comment:4 Changed 2 years ago by robe

Milestone: PostGIS 2.3.0PostGIS 2.1.9

strk actually is this a 2.1 issue also I suspect so so switching this to 2.1.9. We can switch to 2.2.1 (after we relase 2.2.0) if it only affects 2.2+

comment:5 Changed 7 weeks ago by pramsey

Milestone: PostGIS 2.1.9PostGIS 2.2.6

comment:6 Changed 5 weeks ago by pramsey

Milestone: PostGIS 2.2.6PostGIS 2.2.7
Note: See TracTickets for help on using tickets.