From eaea20ae7ab3705253423aced10399b0a15cc59e Mon Sep 17 00:00:00 2001
From: Michael Geers <michael@geers.tv>
Date: Fri, 20 Oct 2023 19:53:23 +0200
Subject: [PATCH] Planner: add message for soc vehicles without capacity
 (#10320)

---
 assets/js/components/ChargingPlan.vue     |  1 +
 assets/js/components/TargetCharge.vue     | 25 +++++++++++++++++++----
 assets/js/components/TargetChargePlan.vue |  2 +-
 i18n/de.toml                              |  2 ++
 i18n/en.toml                              |  2 ++
 5 files changed, 27 insertions(+), 5 deletions(-)

diff --git a/assets/js/components/ChargingPlan.vue b/assets/js/components/ChargingPlan.vue
index 18874ad72..b9a3414d6 100644
--- a/assets/js/components/ChargingPlan.vue
+++ b/assets/js/components/ChargingPlan.vue
@@ -114,6 +114,7 @@ export default {
 		smartCostType: String,
 		currency: String,
 		mode: String,
+		vehicleCapacity: Number,
 	},
 	emits: ["target-time-updated", "target-time-removed", "minsoc-updated"],
 	data: function () {
diff --git a/assets/js/components/TargetCharge.vue b/assets/js/components/TargetCharge.vue
index aa83850ce..47382347d 100644
--- a/assets/js/components/TargetCharge.vue
+++ b/assets/js/components/TargetCharge.vue
@@ -1,7 +1,7 @@
 <template>
 	<form @submit.prevent="setTargetTime">
 		<div class="mt-4">
-			<div class="form-group d-lg-flex align-items-baseline mb-2 justify-content-between">
+			<div class="form-group d-lg-flex align-items-baseline mb-3 justify-content-between">
 				<!-- eslint-disable vue/no-v-html -->
 				<label :for="`targetTimeLabel${id}`" class="mb-3 me-3">
 					<span v-if="socBasedCharging">
@@ -41,13 +41,23 @@
 					/>
 				</div>
 			</div>
-			<p class="mb-0">
+			<p class="my-3 pb-1">
 				<span v-if="timeInThePast" class="d-block text-danger mb-1">
 					{{ $t("main.targetCharge.targetIsInThePast") }}
 				</span>
 				<span v-if="!socBasedCharging && !targetEnergy" class="d-block text-danger mb-1">
 					{{ $t("main.targetCharge.targetEnergyRequired") }}
 				</span>
+				<span v-if="vehicleCapacityRequired" class="d-block text-danger mb-1">
+					{{ $t("main.targetCharge.vehicleCapacityRequired") }}
+					<a
+						class="d-inline text-danger"
+						href="https://docs.evcc.io/en/docs/reference/configuration/vehicles/#capacity"
+						target="_blank"
+					>
+						{{ $t("main.targetCharge.vehicleCapacityDocs") }}
+					</a>
+				</span>
 				<span v-if="timeTooFarInTheFuture" class="d-block text-secondary mb-1">
 					{{ $t("main.targetCharge.targetIsTooFarInTheFuture") }}
 				</span>
@@ -61,7 +71,6 @@
 				<span v-if="['off', 'now'].includes(mode)" class="d-block text-secondary mb-1">
 					{{ $t("main.targetCharge.onlyInPvMode") }}
 				</span>
-				&nbsp;
 			</p>
 			<TargetChargePlan v-if="targetChargePlanProps" v-bind="targetChargePlanProps" />
 			<div class="d-flex justify-content-between mt-3">
@@ -73,7 +82,11 @@
 				>
 					{{ $t("main.targetCharge.remove") }}
 				</button>
-				<button type="submit" class="btn btn-primary" :disabled="timeInThePast">
+				<button
+					type="submit"
+					class="btn btn-primary"
+					:disabled="timeInThePast || vehicleCapacityRequired"
+				>
 					<span v-if="targetTime">
 						{{ $t("main.targetCharge.update") }}
 					</span>
@@ -114,6 +127,7 @@ export default {
 		smartCostType: String,
 		currency: String,
 		mode: String,
+		vehicleCapacity: Number,
 	},
 	emits: ["target-time-updated", "target-time-removed"],
 	data: function () {
@@ -189,6 +203,9 @@ export default {
 		isCo2() {
 			return this.smartCostType === CO2_TYPE;
 		},
+		vehicleCapacityRequired() {
+			return this.socBasedCharging && !this.vehicleCapacity;
+		},
 	},
 	watch: {
 		targetTime() {
diff --git a/assets/js/components/TargetChargePlan.vue b/assets/js/components/TargetChargePlan.vue
index b17c41216..066b513e9 100644
--- a/assets/js/components/TargetChargePlan.vue
+++ b/assets/js/components/TargetChargePlan.vue
@@ -1,5 +1,5 @@
 <template>
-	<div class="plan pt-2">
+	<div class="plan my-3">
 		<div class="justify-content-between mb-2 d-flex justify-content-between">
 			<div class="text-start">
 				<div class="label">{{ $t("main.targetChargePlan.chargeDuration") }}</div>
diff --git a/i18n/de.toml b/i18n/de.toml
index ac005704d..3c64498f9 100644
--- a/i18n/de.toml
+++ b/i18n/de.toml
@@ -248,6 +248,8 @@ title = "Zielzeit"
 today = "heute"
 tomorrow = "morgen"
 update = "Aktualisieren"
+vehicleCapacityDocs = "Erfahre, wie du sie konfigurierst."
+vehicleCapacityRequired = "Die Batteriekapazität des Fahrzeugs wird benötigt, um die Ladedauer zu schätzen."
 
 [main.targetChargePlan]
 chargeDuration = "Ladedauer"
diff --git a/i18n/en.toml b/i18n/en.toml
index e78f176b5..56e234b1e 100644
--- a/i18n/en.toml
+++ b/i18n/en.toml
@@ -245,6 +245,8 @@ title = "Target Time"
 today = "today"
 tomorrow = "tomorrow"
 update = "Update"
+vehicleCapacityDocs = "Learn how to configure it."
+vehicleCapacityRequired = "The vehicle battery capacity is required to estimate the charging time."
 
 [main.targetChargePlan]
 chargeDuration = "Charging time"