Mercurial > gemma

diff pkg/controllers/surveys.go @ 5574:271888ef85bc surveysperbottleneckid

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
Finalize use of bottleneck id instead of name. Main feaure is the restructuring of the /surveys endpoint. The endpoint now takes queryparameters as qualifiers. /surveys?id={id} returns surveys to a given bottleck {id} /surveys?name={name} & date={date} returns surveys for given {name} of a bottleneck and {date} for a surveydate. This is needed mainly because there is some backwards incompatibility when reviewing sounding results where the summary of a sr import contains only the name instead of the id of the according bottleneck. To bridge this mismatch the survey endpoint could be queried for the sr to review given the surveydate and the name of the bottleneck. Premise is here, that a date and a bottleneck's name is specific enough to identify the correct survey with the correct bottleneck.
author Thomas Junk <thomas.junk@intevation.de>
date Wed, 21 Jul 2021 14:33:35 +0200
parents 3b842e951317
children 6709530f002e
line wrap: on
line diff
--- a/pkg/controllers/surveys.go	Tue Jul 20 18:56:18 2021 +0200
+++ b/pkg/controllers/surveys.go	Wed Jul 21 14:33:35 2021 +0200
@@ -19,15 +19,13 @@
 	"database/sql"
 	"net/http"
 
-	"github.com/gorilla/mux"
-
 	"gemma.intevation.de/gemma/pkg/models"
 
 	mw "gemma.intevation.de/gemma/pkg/middleware"
 )
 
 const (
-	listSurveysSQL = `
+	listSurveysByIdSQL = `
 SELECT DISTINCT
   s.bottleneck_id,
   s.date_info::text,
@@ -43,22 +41,50 @@
     ON s.depth_reference = r.depth_reference
       AND g.location = r.location AND g.validity = r.validity
 WHERE b.bottleneck_id = $1`
+
+	listSurveysByNameDateSQL = `
+SELECT DISTINCT
+	s.bottleneck_id,
+	s.date_info::text,
+	s.depth_reference,
+	COALESCE(g.objname, 'ERROR: MISSING GAUGE') AS gauge_objname,
+	r.value AS waterlevel_value,
+	COALESCE(s.surtyp, 'ERROR: MISSING SURVEY TYPE') AS surtype
+FROM waterway.bottlenecks AS b
+	JOIN waterway.sounding_results AS s ON b.bottleneck_id = s.bottleneck_id
+	LEFT JOIN waterway.gauges AS g
+	  ON b.gauge_location = g.location AND s.date_info::timestamptz <@ g.validity
+	LEFT JOIN waterway.gauges_reference_water_levels AS r
+	  ON s.depth_reference = r.depth_reference
+		AND g.location = r.location AND g.validity = r.validity
+WHERE b.objnam = $1 and s.date_info = $2`
 )
 
 func listSurveys(req *http.Request) (jr mw.JSONResult, err error) {
 
-	bottleneckName := mux.Vars(req)["bottleneck"]
-
+	v := req.URL.Query()
+	bottleneckName := v.Get("name")
+	date := v.Get("date")
+	id := v.Get("id")
 	var rows *sql.Rows
+	surveys := []*models.Survey{}
 
-	rows, err = mw.JSONConn(req).QueryContext(req.Context(), listSurveysSQL, bottleneckName)
+	if date == "" && bottleneckName == "" && id != "" {
+		rows, err = mw.JSONConn(req).QueryContext(req.Context(), listSurveysByIdSQL, id)
+	} else if date != "" && bottleneckName != "" && id == "" {
+		rows, err = mw.JSONConn(req).QueryContext(req.Context(), listSurveysByNameDateSQL, bottleneckName, date)
+	} else {
+		jr = mw.JSONResult{
+			Result: struct {
+				Surveys []*models.Survey `json:"surveys"`
+			}{surveys},
+		}
+		return
+	}
 	if err != nil {
 		return
 	}
 	defer rows.Close()
-
-	surveys := []*models.Survey{}
-
 	// as we do not use the values here, we could simply the code here
 	// to work without an explicit mdels/surverys.go
 	// (like done in controllers/search.go)