Merge pull request 'handoperatable-feature' (#431) from handoperatable-feature into staging

Reviewed-on: #431
2024-04-24 12:19:06 +02:00 · 2024-04-24 12:19:06 +02:00 · 1412852087
commit 1412852087
parent 7a59c67763 a2b0146d6d
8 changed files with 48 additions and 34 deletions
--- a/2
+++ b/2
@ -1,5 +1,5 @@
 #!/bin/bash

 scp read@128.140.64.118:/home/rowing/db.sqlite db.sqlite
-sqlite3 db.sqlite < seeds.sql
+#sqlite3 db.sqlite < seeds.sql

--- a/frontend/main.ts
+++ b/frontend/main.ts
@ -118,6 +118,8 @@ interface ChoiceBoatEvent extends Event {
      default_destination: string;
      boat_in_ottensheim: boolean;
      boat_reserved_today: boolean;
+      default_handoperated: boolean;
+      convert_handoperated_possible: boolean;
    };
  };
 }
@ -147,6 +149,19 @@ function selectBoatChange() {
        const amount_seats = event.detail.customProperties.amount_seats;
        setMaxAmountRowers("newrower", amount_seats);

+	let only_steering =  <HTMLSelectElement>document.querySelector('#shipmaster_only_steering');
+	if (event.detail.customProperties.default_handoperated) {
+		only_steering.setAttribute('checked', 'true');
+	}else {
+		only_steering.removeAttribute('checked');
+	}
+
+	if (event.detail.customProperties.convert_handoperated_possible) {
+		only_steering.removeAttribute('disabled');
+	}else {
+		only_steering.setAttribute('disabled', 'disabled');
+	}
+
        const destination = <HTMLSelectElement>(
          document.querySelector("#destination")
        );
@ -159,9 +174,7 @@ function selectBoatChange() {

          if(event.detail.value === '36') {
              /** custom code for Etsch */
-              choiceObjects["newrower"].setChoiceByValue(
-                "81",
-              );
+              choiceObjects["newrower"].setChoiceByValue("81");
          }
        }

@ -250,22 +263,6 @@ function setMaxAmountRowers(name: string, rowers: number) {
      }
    }

-    //let only_steering =  <HTMLSelectElement>document.querySelector('#shipmaster_only_steering');
-    //if(only_steering) {
-    //  if(isShipmasterSteering == 'true') {
-    //    only_steering.removeAttribute('disabled');
-    //    only_steering.setAttribute('checked', 'true');
-    //    only_steering.parentElement?.parentElement?.parentElement?.classList.remove('hidden');
-    //    only_steering.parentElement?.parentElement?.parentElement?.classList.remove('md:block');
-    //    only_steering.parentElement?.parentElement?.parentElement?.classList.remove('opacity-50');
-    //  } else {
-    //    only_steering.setAttribute('disabled', 'disabled');
-    //    only_steering.removeAttribute('checked');
-    //    only_steering.parentElement?.parentElement?.parentElement?.classList.add('hidden');
-    //    only_steering.parentElement?.parentElement?.parentElement?.classList.add('md:block');
-    //    only_steering.parentElement?.parentElement?.parentElement?.classList.add('opacity-50');
-    //  }
-    //}
    let shipmaster = <HTMLElement>(
      document.querySelector("#shipmaster-" + name + "js")
    );
--- a/migration.sql
+++ b/migration.sql
@ -99,6 +99,7 @@ CREATE TABLE IF NOT EXISTS "boat" (
 	"year_built" INTEGER,
 	"boatbuilder" TEXT,
 	"default_shipmaster_only_steering" boolean default false not null,
+	"convert_handoperated_possible" boolean default false not null,
 	"default_destination" text,
 	"skull" boolean default true NOT NULL, -- false => riemen
 	"external" boolean default false NOT NULL, -- false => owned by different club
--- a/src/model/boat.rs
+++ b/src/model/boat.rs
@ -21,6 +21,8 @@ pub struct Boat {
    pub boatbuilder: Option<String>,
    pub default_destination: Option<String>,
    #[serde(default = "bool::default")]
+    convert_handoperated_possible: bool,
+    #[serde(default = "bool::default")]
    default_shipmaster_only_steering: bool,
    #[serde(default = "bool::default")]
    skull: bool,
@ -53,6 +55,7 @@ pub struct BoatToAdd<'r> {
    pub year_built: Option<i64>,
    pub boatbuilder: Option<&'r str>,
    pub default_shipmaster_only_steering: bool,
+    pub convert_handoperated_possible: bool,
    pub default_destination: Option<&'r str>,
    pub skull: bool,
    pub external: bool,
@ -69,6 +72,7 @@ pub struct BoatToUpdate<'r> {
    pub default_shipmaster_only_steering: bool,
    pub default_destination: Option<&'r str>,
    pub skull: bool,
+    pub convert_handoperated_possible: bool,
    pub external: bool,
    pub location_id: i64,
    pub owner: Option<i64>,
@ -189,7 +193,7 @@ AND date('now') BETWEEN start_date AND end_date;",
        let boats = sqlx::query_as!(
            Boat,
            "
-SELECT id, name, amount_seats, location_id, owner, year_built, boatbuilder, default_shipmaster_only_steering, default_destination, skull, external, deleted 
+SELECT id, name, amount_seats, location_id, owner, year_built, boatbuilder, default_shipmaster_only_steering, default_destination, skull, external, deleted, convert_handoperated_possible 
 FROM boat 
 WHERE deleted=false
 ORDER BY amount_seats DESC
@ -218,7 +222,8 @@ SELECT
    b.default_destination, 
    b.skull, 
    b.external,
-    b.deleted
+    b.deleted,
+    b.convert_handoperated_possible
 FROM 
    boat AS b
 WHERE 
@ -244,7 +249,7 @@ ORDER BY
            sqlx::query_as!(
            Boat,
            "
-SELECT id, name, amount_seats, location_id, owner, year_built, boatbuilder, default_shipmaster_only_steering, default_destination, skull, external, deleted 
+SELECT id, name, amount_seats, location_id, owner, year_built, boatbuilder, default_shipmaster_only_steering, default_destination, skull, external, deleted, convert_handoperated_possible
 FROM boat 
 WHERE owner is null or owner = ?
 ORDER BY amount_seats DESC
@ -258,7 +263,7 @@ ORDER BY amount_seats DESC
            sqlx::query_as!(
            Boat,
            "
-SELECT id, name, amount_seats, location_id, owner, year_built, boatbuilder, default_shipmaster_only_steering, default_destination, skull, external, deleted 
+SELECT id, name, amount_seats, location_id, owner, year_built, boatbuilder, default_shipmaster_only_steering, default_destination, skull, external, deleted, convert_handoperated_possible 
 FROM boat 
 WHERE owner = ? OR (owner is null and amount_seats = 1)
 ORDER BY amount_seats DESC
@ -276,7 +281,7 @@ ORDER BY amount_seats DESC
                .unwrap();
            let boats_in_ottensheim = sqlx::query_as!(
            Boat,
-            "SELECT id, name, amount_seats, location_id, owner, year_built, boatbuilder, default_shipmaster_only_steering, default_destination, skull, external, deleted 
+            "SELECT id, name, amount_seats, location_id, owner, year_built, boatbuilder, default_shipmaster_only_steering, default_destination, skull, external, deleted, convert_handoperated_possible 
 FROM boat 
 WHERE owner is null and location_id = ? 
 ORDER BY amount_seats DESC
@ -295,7 +300,7 @@ ORDER BY amount_seats DESC
        let boats = sqlx::query_as!(
            Boat,
            "
-SELECT boat.id, boat.name, amount_seats, location_id, owner, year_built, boatbuilder, default_shipmaster_only_steering, default_destination, skull, external, deleted
+SELECT boat.id, boat.name, amount_seats, location_id, owner, year_built, boatbuilder, default_shipmaster_only_steering, default_destination, skull, external, deleted, convert_handoperated_possible
 FROM boat 
 INNER JOIN location ON boat.location_id = location.id
 WHERE location.name=?
@ -312,26 +317,7 @@ ORDER BY amount_seats DESC

    pub async fn create(db: &SqlitePool, boat: BoatToAdd<'_>) -> Result<(), String> {
        sqlx::query!(
-            "INSERT INTO boat(name, amount_seats, year_built, boatbuilder, default_shipmaster_only_steering, default_destination, skull, external, location_id, owner) VALUES (?,?,?,?,?,?,?,?,?,?)",
-            boat.name,
-            boat.amount_seats,
-            boat.year_built,
-            boat.boatbuilder,
-            boat.default_shipmaster_only_steering,
-            boat.default_destination,
-            boat.skull,
-            boat.external,
-            boat.location_id,
-            boat.owner
-        )
-        .execute(db)
-        .await.map_err(|e| e.to_string())?;
-        Ok(())
-    }
-
-    pub async fn update(&self, db: &SqlitePool, boat: BoatToUpdate<'_>) -> Result<(), String> {
-        sqlx::query!(
-            "UPDATE boat SET name=?, amount_seats=?, year_built=?, boatbuilder=?, default_shipmaster_only_steering=?, default_destination=?, skull=?, external=?, location_id=?, owner=? WHERE id=?",
+            "INSERT INTO boat(name, amount_seats, year_built, boatbuilder, default_shipmaster_only_steering, default_destination, skull, external, location_id, owner, convert_handoperated_possible) VALUES (?,?,?,?,?,?,?,?,?,?,?)",
            boat.name,
            boat.amount_seats,
            boat.year_built,
@ -342,6 +328,27 @@ ORDER BY amount_seats DESC
            boat.external,
            boat.location_id,
            boat.owner,
+            boat.convert_handoperated_possible
+        )
+        .execute(db)
+        .await.map_err(|e| e.to_string())?;
+        Ok(())
+    }
+
+    pub async fn update(&self, db: &SqlitePool, boat: BoatToUpdate<'_>) -> Result<(), String> {
+        sqlx::query!(
+            "UPDATE boat SET name=?, amount_seats=?, year_built=?, boatbuilder=?, default_shipmaster_only_steering=?, default_destination=?, skull=?, external=?, location_id=?, owner=?, convert_handoperated_possible=? WHERE id=?",
+        boat.name,
+        boat.amount_seats,
+        boat.year_built,
+        boat.boatbuilder,
+        boat.default_shipmaster_only_steering,
+        boat.default_destination,
+        boat.skull,
+        boat.external,
+        boat.location_id,
+        boat.owner,
+        boat.convert_handoperated_possible,
            self.id
        )
        .execute(db)
@ -421,6 +428,7 @@ mod test {
                    year_built: None,
                    boatbuilder: "Best Boatbuilder".into(),
                    default_shipmaster_only_steering: true,
+                    convert_handoperated_possible: false,
                    skull: true,
                    external: false,
                    location_id: Some(1),
@ -446,6 +454,7 @@ mod test {
                    year_built: None,
                    boatbuilder: "Best Boatbuilder".into(),
                    default_shipmaster_only_steering: true,
+                    convert_handoperated_possible: false,
                    skull: true,
                    external: false,
                    location_id: Some(1),
@ -548,6 +557,7 @@ mod test {
            year_built: None,
            boatbuilder: None,
            default_shipmaster_only_steering: false,
+            convert_handoperated_possible: false,
            skull: true,
            external: false,
            location_id: 1,
@ -571,6 +581,7 @@ mod test {
            year_built: None,
            boatbuilder: None,
            default_shipmaster_only_steering: false,
+            convert_handoperated_possible: false,
            skull: true,
            external: false,
            location_id: 999,
--- a/staging-diff.sql
+++ b/staging-diff.sql
@ -1,3 +1,6 @@
+ALTER TABLE boat ADD COLUMN convert_handoperated_possible BOOLEAN DEFAULT false NOT NULL;
+
+
 -- test user
 INSERT INTO user(name) VALUES('Marie');
 INSERT INTO "user_role" (user_id, role_id) VALUES((SELECT id from user where name = 'Marie'),(SELECT id FROM role where name = 'Donau Linz'));
--- a/templates/includes/forms/boat.html.tera
+++ b/templates/includes/forms/boat.html.tera
@ -24,6 +24,7 @@
                    {{ macros::input(label='Default destination', name='default_destination', type='text') }}
                    <div>
                        {{ macros::checkbox(label="handgesteuert", name="default_shipmaster_only_steering") }}
+                        {{ macros::checkbox(label="Umbau Hand-/Fußsteuerung möglich", name="convert_handoperated_possible") }}
                        {{ macros::checkbox(label="Skull", name="skull", checked=true) }}
                        {{ macros::checkbox(label="Externes Boot (anderer Verein)", name="external") }}
                    </div>
@ -66,6 +67,7 @@
                    {{ macros::input(label='Baujahr', name='year_built', type='number', min=1950, value=boat.year_built) }}
                    {{ macros::input(label='Bootsbauer', name='boatbuilder', type='text', value=boat.boatbuilder) }}
                    {{ macros::checkbox(label='handgesteuert', name='default_shipmaster_only_steering', id=uuid , checked=boat.default_shipmaster_only_steering) }}
+                    {{ macros::checkbox(label='Umbau Hand-/Fußsteuerung möglich', name='convert_handoperated_possible', id=uuid , checked=boat.convert_handoperated_possible) }}
                    {{ macros::input(label='Default destination', name='default_destination', type='text', value=boat.default_destination) }}
                    {{ macros::checkbox(label='Skull', name='skull', id=uuid , checked=boat.skull) }}
                    {{ macros::checkbox(label='Externes Boot', name='external', id=uuid , checked=boat.external) }}
--- a/templates/includes/forms/log.html.tera
+++ b/templates/includes/forms/log.html.tera
@ -7,12 +7,12 @@
        <details>
            <summary class="font-bold cursor-pointer text-primary-900 dark:text-white border-t p-3 hover:bg-gray-100 dark:hover:bg-primary-950">
                <span>
-                    {% if grouped_boats[0].amount_seats < 9 or grouped_boats[0].amount_seats == 24 %}
-                        {{ amount_seats }}x
-                    {% elif grouped_boats[0].amount_seats == 9 %}
+                    {% if grouped_boats[0].external %}
+                        Vereinsfremde Boote
+                    {% elif grouped_boats[0].default_shipmaster_only_steering %}
                        {{ grouped_boats[0].amount_seats - 1 }}+
                    {% else %}
-                        Vereinsfremde Boote
+                        {{ amount_seats }}x
                    {% endif %}
                </span>
                <small class="text-gray-500 dark:text-gray-100">({{ grouped_boats | length }})</small>
--- a/templates/includes/macros.html.tera
+++ b/templates/includes/macros.html.tera
@ -159,7 +159,7 @@
                {% if required %}required="required"{% endif %}>
            {% if default %}<option selected value>{{ default }}</option>{% endif %}
            {% for d in data %}
-                <option value="{{ d.id }}" {% if d.id == selected_id %}selected{% endif %} {% if extras != '' %} {% for extra in extras %} {% if extra != 'on_water' and d[extra] %} data- {{ extra }}={{ d[extra] }} {% else %} {% if d[extra] %}disabled{% endif %} {% endif %} {% endfor %} {% endif %} {% if show_seats %} data-custom-properties='{"amount_seats": {{ d["amount_seats"] }}, "owner": "{{ d["owner"] }}", "default_destination": "{{ d["default_destination"] }}", "boat_in_ottensheim": {{ d["location_id"] == 2 }}, "boat_reserved_today": {{ d["reserved_today"] }}}' {% endif %}>
+                <option value="{{ d.id }}" {% if d.id == selected_id %}selected{% endif %} {% if extras != '' %} {% for extra in extras %} {% if extra != 'on_water' and d[extra] %} data- {{ extra }}={{ d[extra] }} {% else %} {% if d[extra] %}disabled{% endif %} {% endif %} {% endfor %} {% endif %} {% if show_seats %} data-custom-properties='{"amount_seats": {{ d["amount_seats"] }}, "owner": "{{ d["owner"] }}", "default_destination": "{{ d["default_destination"] }}", "boat_in_ottensheim": {{ d["location_id"] == 2 }}, "boat_reserved_today": {{ d["reserved_today"] }}, "convert_handoperated_possible": {{ d["convert_handoperated_possible"] }}, "default_handoperated": {{ d["default_shipmaster_only_steering"] }}}' {% endif %}>
                    {% for displa in display -%}
                        {%- if d[displa] -%}
                            {{- d[displa] -}}