about summary refs log tree commit diff
path: root/uploader/samples
diff options
context:
space:
mode:
Diffstat (limited to 'uploader/samples')
-rw-r--r--uploader/samples/views.py72
1 files changed, 20 insertions, 52 deletions
diff --git a/uploader/samples/views.py b/uploader/samples/views.py
index 95a6f8c..27e5d3c 100644
--- a/uploader/samples/views.py
+++ b/uploader/samples/views.py
@@ -16,16 +16,15 @@ from uploader import jobs
 from uploader.files import save_file
 from uploader.ui import make_template_renderer
 from uploader.authorisation import require_login
-from uploader.request_checks import with_population
 from uploader.input_validation import is_integer_input
-from uploader.datautils import safe_int, order_by_family, enumerate_sequence
-from uploader.population.models import population_by_id, populations_by_species
+from uploader.population.models import population_by_id
+from uploader.route_utils import generic_select_population
+from uploader.datautils import safe_int, enumerate_sequence
+from uploader.species.models import all_species, species_by_id
+from uploader.request_checks import with_species, with_population
 from uploader.db_utils import (with_db_connection,
                                database_connection,
                                with_redis_connection)
-from uploader.species.models import (all_species,
-                                     species_by_id,
-                                     order_species_by_family)
 
 from .models import samples_by_species_and_population
 
@@ -59,62 +58,31 @@ def index():
 
 @samplesbp.route("<int:species_id>/samples/select-population", methods=["GET"])
 @require_login
-def select_population(species_id: int):
+@with_species(redirect_uri="species.populations.samples.index")
+def select_population(species: dict, **kwargs):# pylint: disable=[unused-argument]
     """Select the population to use for the samples."""
-    with database_connection(app.config["SQL_URI"]) as conn:
-        species = species_by_id(conn, species_id)
-        if not bool(species):
-            flash("Invalid species!", "alert-danger")
-            return redirect(url_for("species.populations.samples.index"))
-
-        if not bool(request.args.get("population_id")):
-            return render_template("samples/select-population.html",
-                                   species=species,
-                                   populations=populations_by_species(
-                                       conn,
-                                       species_id),
-                                   activelink="samples")
-
-        population_id = request.args["population_id"]
-        if population_id == "CREATE-POPULATION":
-            return redirect(url_for(
-                "species.populations.create_population",
-                species_id=species["SpeciesId"],
-                return_to="species.populations.samples.list_samples"))
-
-        population = population_by_id(conn, request.args.get("population_id"))
-        if not bool(population):
-            flash("Population not found!", "alert-danger")
-            return redirect(url_for(
-                "species.populations.samples.select_population",
-                species_id=species_id))
-
-        return redirect(url_for("species.populations.samples.list_samples",
-                                species_id=species_id,
-                                population_id=population["Id"]))
+    return generic_select_population(
+        species,
+        "samples/select-population.html",
+        request.args.get("population_id") or "",
+        "species.populations.samples.select_population",
+        "species.populations.samples.list_samples",
+        "samples",
+        "Population not found!")
 
 @samplesbp.route("<int:species_id>/populations/<int:population_id>/samples")
 @require_login
-def list_samples(species_id: int, population_id: int):
+@with_population(
+    species_redirect_uri="species.populations.samples.index",
+    redirect_uri="species.populations.samples.select_population")
+def list_samples(species: dict, population: dict, **kwargs):# pylint: disable=[unused-argument]
     """
     List the samples in a particular population and give the ability to upload
     new ones.
     """
     with database_connection(app.config["SQL_URI"]) as conn:
-        species = species_by_id(conn, species_id)
-        if not bool(species):
-            flash("Invalid species!", "alert-danger")
-            return redirect(url_for("species.populations.samples.index"))
-
-        population = population_by_id(conn, population_id)
-        if not bool(population):
-            flash("Population not found!", "alert-danger")
-            return redirect(url_for(
-                "species.populations.samples.select_population",
-                species_id=species_id))
-
         all_samples = enumerate_sequence(samples_by_species_and_population(
-            conn, species_id, population_id))
+            conn, species["SpeciesId"], population["Id"]))
         total_samples = len(all_samples)
         offset = max(safe_int(request.args.get("from") or 0), 0)
         count = int(request.args.get("count") or 20)