about summary refs log tree commit diff
path: root/uploader/species
diff options
context:
space:
mode:
Diffstat (limited to 'uploader/species')
-rw-r--r--uploader/species/models.py10
-rw-r--r--uploader/species/views.py20
2 files changed, 23 insertions, 7 deletions
diff --git a/uploader/species/models.py b/uploader/species/models.py
index 51f941c..acfa51e 100644
--- a/uploader/species/models.py
+++ b/uploader/species/models.py
@@ -58,7 +58,8 @@ def save_species(conn: mdb.Connection,
     common_name: The species' common name.
     scientific_name; The species' scientific name.
     """
-    genus, species_name = scientific_name.split(" ")
+    genus, *species_parts = scientific_name.split(" ")
+    species_name: str = " ".join(species_parts)
     families = species_families(conn)
     with conn.cursor() as cursor:
         cursor.execute("SELECT MAX(OrderId) FROM Species")
@@ -68,7 +69,7 @@ def save_species(conn: mdb.Connection,
             "menu_name": f"{common_name} ({genus[0]}. {species_name.lower()})",
             "scientific_name": scientific_name,
             "family": family,
-            "family_order": families[family],
+            "family_order": families.get(family, 999999),
             "taxon_id": taxon_id,
             "species_order": cursor.fetchone()[0] + 5
         }
@@ -91,7 +92,7 @@ def save_species(conn: mdb.Connection,
         }
 
 
-def update_species(# pylint: disable=[too-many-arguments]
+def update_species(# pylint: disable=[too-many-arguments, too-many-positional-arguments]
         conn: mdb.Connection,
         species_id: int,
         common_name: str,
@@ -116,7 +117,8 @@ def update_species(# pylint: disable=[too-many-arguments]
     species_order: The ordering of this species in relation to others
     """
     with conn.cursor(cursorclass=DictCursor) as cursor:
-        genus, species_name = scientific_name.split(" ")
+        genus, *species_parts = scientific_name.split(" ")
+        species_name = " ".join(species_parts)
         species = {
             "species_id": species_id,
             "common_name": common_name,
diff --git a/uploader/species/views.py b/uploader/species/views.py
index 08d3728..cea2f68 100644
--- a/uploader/species/views.py
+++ b/uploader/species/views.py
@@ -1,5 +1,7 @@
 """Endpoints handling species."""
+from markupsafe import escape
 from pymonad.either import Left, Right, Either
+from gn_libs.mysqldb import database_connection
 from flask import (flash,
                    request,
                    url_for,
@@ -8,8 +10,8 @@ from flask import (flash,
                    current_app as app)
 
 from uploader.population import popbp
+from uploader.platforms import platformsbp
 from uploader.ui import make_template_renderer
-from uploader.db_utils import database_connection
 from uploader.oauth2.client import oauth2_get, oauth2_post
 from uploader.authorisation import require_login, require_token
 from uploader.datautils import order_by_family, enumerate_sequence
@@ -23,10 +25,12 @@ from .models import (all_species,
 
 speciesbp = Blueprint("species", __name__)
 speciesbp.register_blueprint(popbp, url_prefix="/")
+speciesbp.register_blueprint(platformsbp, url_prefix="/")
 render_template = make_template_renderer("species")
 
 
 @speciesbp.route("/", methods=["GET"])
+@require_login
 def list_species():
     """List and display all the species in the database."""
     with database_connection(app.config["SQL_URI"]) as conn:
@@ -59,6 +63,8 @@ def create_species():
         if request.method == "GET":
             return render_template("species/create-species.html",
                                    families=species_families(conn),
+                                   return_to=(
+                                       request.args.get("return_to") or ""),
                                    activelink="create-species")
 
         error = False
@@ -76,7 +82,7 @@ def create_species():
             error = True
 
         parts = tuple(name.strip() for name in scientific_name.split(" "))
-        if len(parts) != 2 or not all(bool(name) for name in parts):
+        if (len(parts) != 2 and len(parts) != 3) or not all(bool(name) for name in parts):
             flash("The scientific name you provided is invalid.", "alert-danger")
             error = True
 
@@ -110,7 +116,15 @@ def create_species():
 
         species = save_species(
             conn, common_name, scientific_name, family, taxon_id)
-        flash("Species saved successfully!", "alert-success")
+        flash(
+            f"You have successfully added species "
+            f"'{escape(species['scientific_name'])} "
+            f"({escape(species['common_name'])})'.",
+            "alert-success")
+
+        return_to = request.form.get("return_to").strip()
+        if return_to:
+            return redirect(url_for(return_to, species_id=species["species_id"]))
         return redirect(url_for("species.view_species", species_id=species["species_id"]))