[PATCH yew-comp] user panel: handle expire property on edit properly

[PATCH yew-comp] user panel: handle expire property on edit properly

[Shannon Sterz]

setting expire to "0" is a special case meaning "never expires". while
this was honored in the user management overview, the edit panel
rendered this as 01.01.1970 00:00:00 UTC, or the Unix epoch. this had
minimal functional impact, but it may be confusing to users. set
expire to `Value::Null` in this special case, rendering a cleared date
picker.

also the expire field is typically an `Option<i64>` [1]. so instead of
interpreting it as a `f64` and then casting it back to an `i64` when
rendering the input date, interpret it as an `i64` straight away.

[1]:
https://git.proxmox.com/?p=proxmox.git;a=blob;f=proxmox-access-control/src/types.rs;h=78eafb4c9d1e652f4d69c203aa2117cfe65544c2;hb=HEAD#l218

Signed-off-by: Shannon Sterz <s.sterz@proxmox.com>
---
 src/user_panel.rs | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/src/user_panel.rs b/src/user_panel.rs
index 547fdfe..75a442f 100644
--- a/src/user_panel.rs
+++ b/src/user_panel.rs
@@ -40,8 +40,12 @@ async fn load_user(userid: Key) -> Result<ApiResponseData<Value>, Error> {
     let mut resp: ApiResponseData<Value> = crate::http_get_full(&url, None).await?;
 
     if let Value::Number(number) = &resp.data["expire"] {
-        if let Some(epoch) = number.as_f64() {
-            resp.data["expire"] = Value::String(epoch_to_input_value(epoch as i64));
+        if let Some(epoch) = number.as_i64() {
+            if epoch == 0 {
+                resp.data["expire"] = Value::Null;
+            } else {
+                resp.data["expire"] = Value::String(epoch_to_input_value(epoch));
+            }
         }
     }
 
-- 
2.47.3

applied: [PATCH yew-comp] user panel: handle expire property on edit properly

[Thomas Lamprecht]

On Wed, 25 Mar 2026 13:15:16 +0100, Shannon Sterz wrote:
> setting expire to "0" is a special case meaning "never expires". while
> this was honored in the user management overview, the edit panel
> rendered this as 01.01.1970 00:00:00 UTC, or the Unix epoch. this had
> minimal functional impact, but it may be confusing to users. set
> expire to `Value::Null` in this special case, rendering a cleared date
> picker.
> 
> [...]

Applied, thanks!

[1/1] user panel: handle expire property on edit properly
      commit: fc2c392666fa56f005dbd3cbf4af100dc6344be9