reimplement Object as slice type

12 files changed, 228 insertions, 225 deletions

diff --git a/server/src/auth.rs b/server/src/auth.rs
index d973dfc..0174e13 100644
--- a/server/src/auth.rs
+++ b/server/src/auth.rs
@@ -7,13 +7,10 @@
 use crate::State;
 use anyhow::{Result, anyhow, bail};
 use argon2::{Argon2, PasswordHasher, password_hash::Salt};
-use jellycommon::{
-    jellyobject::{ObjectBuffer, Path},
-    *,
-};
+use jellycommon::{jellyobject::Path, *};
 use jellydb::{Filter, Query};
 
-pub fn token_to_user(state: &State, token: &str) -> Result<ObjectBuffer> {
+pub fn token_to_user(state: &State, token: &str) -> Result<Box<User>> {
     let user_row = token::validate(&state.session_key, token)?;
 
     let mut user = None;
@@ -49,7 +46,7 @@ pub fn login(
     let (Some(user_row), Some(user)) = (user_row, user) else {
         bail!("unknown user");
     };
-    let Some(correct_pw) = user.as_object().get(USER_PASSWORD) else {
+    let Some(correct_pw) = user.get(USER_PASSWORD) else {
         bail!("password login is disabled")
     };
     if password != correct_pw {
@@ -62,7 +59,7 @@ pub fn login(
             user_row,
             expire.unwrap_or(60 * 60 * 24 * 30),
         ),
-        user.as_object().has(USER_PASSWORD_REQUIRE_CHANGE.0),
+        user.has(USER_PASSWORD_REQUIRE_CHANGE.0),
     ))
 }
 
diff --git a/server/src/compat/youtube.rs b/server/src/compat/youtube.rs
index 4ba3dc8..917adc0 100644
--- a/server/src/compat/youtube.rs
+++ b/server/src/compat/youtube.rs
@@ -30,10 +30,7 @@ pub fn r_youtube_watch(ri: RequestInfo<'_>, v: &str) -> MyResult<Redirect> {
         Ok(())
     })?;
     let node = res.ok_or(anyhow!("video not found"))?;
-    let slug = node
-        .as_object()
-        .get(NO_SLUG)
-        .ok_or(anyhow!("node has no slug"))?;
+    let slug = node.get(NO_SLUG).ok_or(anyhow!("node has no slug"))?;
     Ok(Redirect::found(u_node_id(slug)))
 }
 
diff --git a/server/src/logic/stream.rs b/server/src/logic/stream.rs
index e332811..38d2b7b 100644
--- a/server/src/logic/stream.rs
+++ b/server/src/logic/stream.rs
@@ -65,7 +65,7 @@ pub async fn r_stream(
         Ok(())
     })?;
 
-    let Some(node) = node.as_ref().map(|n| n.as_object()) else {
+    let Some(node) = node else {
         Err(anyhow!("node not found"))?
     };
 
diff --git a/server/src/main.rs b/server/src/main.rs
index 8209879..d6f1e1e 100644
--- a/server/src/main.rs
+++ b/server/src/main.rs
@@ -15,7 +15,7 @@ use anyhow::{Result, anyhow};
 use jellycache::Cache;
 use jellycommon::{
     USER_ADMIN, USER_LOGIN, USER_PASSWORD,
-    jellyobject::{ObjectBuffer, Path},
+    jellyobject::{OBB, Path},
 };
 use jellydb::{Database, Filter, Query};
 use log::{error, info};
@@ -105,11 +105,13 @@ fn create_admin_user(state: &State) -> Result<()> {
         if admin_row.is_none() {
             info!("Creating new admin user");
             let pwhash = hash_password("admin", &state.config.admin_password);
-            txn.insert(ObjectBuffer::new(&mut [
-                (USER_LOGIN.0, &"admin"),
-                (USER_PASSWORD.0, &pwhash.as_slice()),
-                (USER_ADMIN.0, &()),
-            ]))?;
+            txn.insert(
+                OBB::new()
+                    .with(USER_LOGIN, "admin")
+                    .with(USER_PASSWORD, &pwhash)
+                    .with(USER_ADMIN, ())
+                    .finish(),
+            )?;
         }
 
         Ok(())
diff --git a/server/src/request_info.rs b/server/src/request_info.rs
index 442c7fb..0adfe96 100644
--- a/server/src/request_info.rs
+++ b/server/src/request_info.rs
@@ -10,10 +10,7 @@ use crate::{
     ui::error::{MyError, MyResult},
 };
 use anyhow::anyhow;
-use jellycommon::{
-    USER_ADMIN,
-    jellyobject::{Object, ObjectBuffer},
-};
+use jellycommon::{USER_ADMIN, User, jellyobject::Object};
 use jellyui::{Page, RenderInfo, Scaffold};
 use rocket::{
     Request, async_trait,
@@ -27,7 +24,7 @@ pub struct RequestInfo<'a> {
     pub lang: &'a str,
     pub accept: Accept,
     pub debug: &'a str,
-    pub user: Option<ObjectBuffer>,
+    pub user: Option<Box<User>>,
     pub state: Arc<State>,
     pub flash: Option<FlashMessage<'a>>,
 }
@@ -59,13 +56,12 @@ impl<'a> RequestInfo<'a> {
             flash: FlashMessage::from_request(request).await.succeeded(),
         })
     }
-    pub fn require_user(&'a self) -> MyResult<Object<'a>> {
+    pub fn require_user(&'a self) -> MyResult<&'a Object> {
         self.user
-            .as_ref()
-            .map(|u| u.as_object())
+            .as_deref()
             .ok_or(MyError(anyhow!("user required")))
     }
-    pub fn require_admin(&'a self) -> MyResult<Object<'a>> {
+    pub fn require_admin(&'a self) -> MyResult<&'a Object> {
         let user = self.require_user()?;
         if !user.has(USER_ADMIN.0) {
             Err(anyhow!("admin required"))?
@@ -76,7 +72,7 @@ impl<'a> RequestInfo<'a> {
         RenderInfo {
             lang: self.lang,
             status_message: None,
-            user: self.user.as_ref().map(|u| u.as_object()),
+            user: self.user.as_deref(),
             config: &self.state.config.ui,
             message: self.flash.as_ref().map(|f| (f.kind(), f.message())),
         }
@@ -129,7 +125,7 @@ fn accept_language<'a>(request: &'a Request<'_>) -> &'a str {
         .unwrap_or("en")
 }
 
-fn user_from_request(state: &State, req: &Request<'_>) -> Result<Option<ObjectBuffer>, MyError> {
+fn user_from_request(state: &State, req: &Request<'_>) -> Result<Option<Box<User>>, MyError> {
     let Some(token) = req
         .query_value("session")
         .map(|e| e.unwrap())
diff --git a/server/src/ui/account/mod.rs b/server/src/ui/account/mod.rs
index b106fc7..9e5fa39 100644
--- a/server/src/ui/account/mod.rs
+++ b/server/src/ui/account/mod.rs
@@ -82,10 +82,10 @@ pub fn r_account_login_post(
                 })?;
                 if let Some(ur) = user_row {
                     let mut user = txn.get(ur)?.unwrap();
-                    user = user.as_object().remove(USER_PASSWORD_REQUIRE_CHANGE);
-                    user = user.as_object().insert(USER_PASSWORD, &password_hash);
+                    user = user.remove(USER_PASSWORD_REQUIRE_CHANGE);
+                    user = user.insert(USER_PASSWORD, &password_hash);
                     if let Some(name) = &form.display_name {
-                        user = user.as_object().insert(USER_NAME, &name);
+                        user = user.insert(USER_NAME, &name);
                     }
                     txn.update(ur, user)?;
                 }
diff --git a/server/src/ui/account/settings.rs b/server/src/ui/account/settings.rs
index 9a98c09..2585061 100644
--- a/server/src/ui/account/settings.rs
+++ b/server/src/ui/account/settings.rs
@@ -7,7 +7,7 @@ use super::format_form_error;
 use crate::{auth::hash_password, request_info::RequestInfo, ui::error::MyResult};
 use anyhow::anyhow;
 use jellycommon::{
-    jellyobject::{Object, ObjectBuffer, Path, Tag},
+    jellyobject::{Object, Path, Tag},
     routes::u_account_settings,
     *,
 };
@@ -104,7 +104,7 @@ pub fn r_account_settings_post(
     Ok(Flash::success(Redirect::to(u_account_settings()), out))
 }
 
-fn update_user(ri: &RequestInfo, update: impl Fn(Object) -> ObjectBuffer) -> MyResult<()> {
+fn update_user(ri: &RequestInfo, update: impl Fn(&Object) -> Box<Object>) -> MyResult<()> {
     let login = ri
         .require_user()?
         .get(USER_LOGIN)
@@ -118,7 +118,7 @@ fn update_user(ri: &RequestInfo, update: impl Fn(Object) -> ObjectBuffer) -> MyR
             .ok_or(anyhow!("user vanished"))?;
 
         let user = txn.get(user_row)?.unwrap();
-        let new_user = update(user.as_object());
+        let new_user = update(&user);
         txn.update(user_row, new_user)?;
 
         Ok(())
diff --git a/server/src/ui/admin/users.rs b/server/src/ui/admin/users.rs
index 85f241b..4b11279 100644
--- a/server/src/ui/admin/users.rs
+++ b/server/src/ui/admin/users.rs
@@ -46,7 +46,7 @@ pub fn r_admin_users(ri: RequestInfo) -> MyResult<RawHtml<String>> {
 
     Ok(ri.respond_ui(&AdminUserList {
         ri: &ri.render_info(),
-        users: &users.iter().map(|u| u.as_object()).collect::<Vec<_>>(),
+        users: &users.iter().map(|u| &**u).collect::<Vec<_>>(),
     }))
 }
 
@@ -97,7 +97,7 @@ pub fn r_admin_user(ri: RequestInfo<'_>, name: &str) -> MyResult<RawHtml<String>
 
     Ok(ri.respond_ui(&AdminUser {
         ri: &ri.render_info(),
-        user: user.as_object(),
+        user: &user,
     }))
 }
 
diff --git a/server/src/ui/home.rs b/server/src/ui/home.rs
index 6cb6a77..17cac83 100644
--- a/server/src/ui/home.rs
+++ b/server/src/ui/home.rs
@@ -7,11 +7,11 @@
 use super::error::MyResult;
 use crate::request_info::RequestInfo;
 use anyhow::{Context, Result};
-use jellycommon::jellyobject::{Object, ObjectBuffer};
-use jellydb::Query;
-use jellyui::components::home::HomeRow;
+use jellycommon::{Nku, jellyobject::EMPTY};
+use jellydb::{Query, helper::DatabaseReturnExt};
+use jellyui::components::home::{Home, HomeRow};
 use rocket::{get, response::content::RawHtml};
-use std::str::FromStr;
+use std::{borrow::Cow, str::FromStr};
 
 #[get("/home")]
 pub fn r_home(ri: RequestInfo<'_>) -> MyResult<RawHtml<String>> {
@@ -28,7 +28,7 @@ pub fn r_home(ri: RequestInfo<'_>) -> MyResult<RawHtml<String>> {
         "home.bin.latest_music",
         "FILTER (visi = visi AND kind = musi) SORT DESCENDING BY FIRST rldt",
     )?);
-    rows.push(home_row_highlight(
+    rows.extend(home_row_highlight(
         &ri,
         "home.bin.daily_random",
         "FILTER (visi = visi AND kind = movi) SORT RANDOM",
@@ -38,51 +38,59 @@ pub fn r_home(ri: RequestInfo<'_>) -> MyResult<RawHtml<String>> {
         "home.bin.max_rating",
         "SORT DESCENDING BY FIRST rtng.imdb",
     )?);
-    rows.push(home_row_highlight(
+    rows.extend(home_row_highlight(
         &ri,
         "home.bin.daily_random",
         "FILTER (visi = visi AND kind = show) SORT RANDOM",
     )?);
 
-    Ok(ri.respond_ui(rows))
+    Ok(ri.respond_ui(&Home {
+        ri: &ri.render_info(),
+        rows: &rows,
+    }))
 }
 
-fn home_row(ri: &RequestInfo<'_>, title: &str, query: &str) -> Result<HomeRow> {
+fn home_row(
+    ri: &RequestInfo<'_>,
+    title: &'static str,
+    query: &str,
+) -> Result<(&'static str, HomeRow<'static>)> {
     let q = Query::from_str(query).context("parse query")?;
-    let mut res = ObjectBuffer::empty();
-    ri.state.database.transaction(&mut |txn| {
+    ri.state.database.transaction_ret(|txn| {
         let rows = txn.query(q.clone())?.take(16).collect::<Result<Vec<_>>>()?;
 
         let mut nkus = Vec::new();
         for (row, _) in rows {
             let node = txn.get(row)?.unwrap();
-            nkus.push(ObjectBuffer::new(&mut [(NKU_NODE.0, &node.as_object())]));
+            nkus.push(Nku {
+                node: Cow::Owned(node),
+                role: None,
+                userdata: Cow::Borrowed(EMPTY),
+            });
         }
-        let nkus = nkus.iter().map(|n| n.as_object()).collect::<Vec<_>>(); // TODO -_-
-
-        res = Object::EMPTY.insert(NODELIST_DISPLAYSTYLE, NLSTYLE_INLINE);
-        res = res.as_object().insert(NODELIST_TITLE, title);
-        res = res.as_object().insert_multi(NODELIST_ITEM, &nkus);
-
-        Ok(())
-    })?;
-    Ok(res)
+        Ok((title, HomeRow::Inline(nkus)))
+    })
 }
 
-fn home_row_highlight(ri: &RequestInfo<'_>, title: &str, query: &str) -> Result<HomeRow> {
+fn home_row_highlight(
+    ri: &RequestInfo<'_>,
+    title: &'static str,
+    query: &str,
+) -> Result<Option<(&'static str, HomeRow<'static>)>> {
     let q = Query::from_str(query).context("parse query")?;
-    let mut res = ObjectBuffer::empty();
-    ri.state.database.transaction(&mut |txn| {
+    ri.state.database.transaction_ret(|txn| {
         let Some(row) = txn.query(q.clone())?.next() else {
-            return Ok(());
+            return Ok(None);
         };
         let row = row?.0;
         let node = txn.get(row)?.unwrap();
-        let nku = ObjectBuffer::new(&mut [(NKU_NODE.0, &node.as_object())]);
-        res = Object::EMPTY.insert(NODELIST_DISPLAYSTYLE, NLSTYLE_HIGHLIGHT);
-        res = res.as_object().insert(NODELIST_TITLE, title);
-        res = res.as_object().insert(NODELIST_ITEM, nku.as_object());
-        Ok(())
-    })?;
-    Ok(res)
+        Ok(Some((
+            title,
+            HomeRow::Highlight(Nku {
+                node: Cow::Owned(node),
+                role: None,
+                userdata: Cow::Borrowed(EMPTY),
+            }),
+        )))
+    })
 }
diff --git a/server/src/ui/items.rs b/server/src/ui/items.rs
index b800914..6071bcc 100644
--- a/server/src/ui/items.rs
+++ b/server/src/ui/items.rs
@@ -8,12 +8,13 @@ use crate::{request_info::RequestInfo, ui::error::MyResult};
 use anyhow::anyhow;
 use base64::{Engine, prelude::BASE64_URL_SAFE};
 use jellycommon::{
-    jellyobject::{Object, Path},
+    jellyobject::{EMPTY, Path},
     *,
 };
 use jellydb::{Filter, Query};
 use jellyui::components::items::Items;
 use rocket::{get, response::content::RawHtml};
+use std::borrow::Cow;
 
 #[get("/items?<cont>")]
 pub fn r_items(ri: RequestInfo, cont: Option<&str>) -> MyResult<RawHtml<String>> {
@@ -49,8 +50,8 @@ pub fn r_items(ri: RequestInfo, cont: Option<&str>) -> MyResult<RawHtml<String>>
         items: &items
             .iter()
             .map(|node| Nku {
-                node: node.as_object(),
-                userdata: Object::EMPTY,
+                node: Cow::Borrowed(&node),
+                userdata: Cow::Borrowed(EMPTY),
                 role: None,
             })
             .collect::<Vec<_>>(),
diff --git a/server/src/ui/node.rs b/server/src/ui/node.rs
index 55a1d09..ca07bac 100644
--- a/server/src/ui/node.rs
+++ b/server/src/ui/node.rs
@@ -5,186 +5,183 @@
 */
 
 use super::error::MyResult;
-use crate::{request_info::RequestInfo, ui_responder::UiResponse};
-use anyhow::Result;
+use crate::request_info::RequestInfo;
+use anyhow::anyhow;
 use jellycommon::{
-    jellyobject::{OBB, Object, ObjectBuffer, ObjectBufferBuilder, Path},
+    jellyobject::{EMPTY, Path},
     *,
 };
-use jellydb::{Filter, MultiBehaviour, Query, Sort, SortOrder, Transaction, ValueSort};
-use rocket::get;
-use std::collections::BTreeMap;
+use jellydb::{Filter, Query};
+use jellyui::components::node_page::NodePage;
+use rocket::{get, response::content::RawHtml};
+use std::borrow::Cow;
 
 #[get("/n/<slug>")]
-pub fn r_node(ri: RequestInfo<'_>, slug: &str) -> MyResult<UiResponse> {
+pub fn r_node(ri: RequestInfo<'_>, slug: &str) -> MyResult<RawHtml<String>> {
     ri.require_user()?;
 
-    let mut page = OBB::new();
+    let mut nku = None;
     ri.state.database.transaction(&mut |txn| {
         if let Some(row) = txn.query_single(Query {
             filter: Filter::Match(Path(vec![NO_SLUG.0]), slug.into()),
             ..Default::default()
         })? {
             let n = txn.get(row)?.unwrap();
-            let nku = Object::EMPTY.insert(NKU_NODE, n.as_object());
-            let nku = nku.as_object();
-
-            page = OBB::new();
-            let title = nku
-                .get(NKU_NODE)
-                .unwrap_or_default()
-                .get(NO_TITLE)
-                .unwrap_or_default();
-
-            page.push(VIEW_TITLE, title);
-            page.push(VIEW_NODE_PAGE, nku);
-            c_children(&mut page, txn, row, &nku)?;
-            c_credits(&mut page, txn, &nku)?;
-            c_credited(&mut page, txn, row)?;
+            nku = Some(Nku {
+                node: Cow::Owned(n),
+                userdata: Cow::Borrowed(EMPTY),
+                role: None,
+            });
         }
         Ok(())
     })?;
+    let Some(nku) = nku else {
+        Err(anyhow!("no such node"))?
+    };
 
-    Ok(ri.respond_ui(page))
+    Ok(ri.respond_ui(&NodePage {
+        ri: &ri.render_info(),
+        nku,
+    }))
 }
 
-fn c_children(
-    page: &mut ObjectBufferBuilder,
-    txn: &mut dyn Transaction,
-    row: u64,
-    nku: &Object,
-) -> Result<()> {
-    let kind = nku
-        .get(NKU_NODE)
-        .unwrap_or_default()
-        .get(NO_KIND)
-        .unwrap_or(KIND_COLLECTION);
+// fn c_children(
+//     page: &mut ObjectBufferBuilder,
+//     txn: &mut dyn Transaction,
+//     row: u64,
+//     nku: &Object,
+// ) -> Result<()> {
+//     let kind = nku
+//         .get(NKU_NODE)
+//         .unwrap_or_default()
+//         .get(NO_KIND)
+//         .unwrap_or(KIND_COLLECTION);
 
-    let (order, path) = match kind {
-        KIND_CHANNEL => (SortOrder::Descending, Path(vec![NO_RELEASEDATE.0])),
-        KIND_SEASON | KIND_SHOW => (SortOrder::Ascending, Path(vec![NO_INDEX.0])),
-        _ => (SortOrder::Ascending, Path(vec![NO_TITLE.0])),
-    };
+//     let (order, path) = match kind {
+//         KIND_CHANNEL => (SortOrder::Descending, Path(vec![NO_RELEASEDATE.0])),
+//         KIND_SEASON | KIND_SHOW => (SortOrder::Ascending, Path(vec![NO_INDEX.0])),
+//         _ => (SortOrder::Ascending, Path(vec![NO_TITLE.0])),
+//     };
 
-    let children_rows = txn
-        .query(Query {
-            sort: Sort::Value(ValueSort {
-                multi: MultiBehaviour::First,
-                offset: None,
-                order,
-                path,
-            }),
-            filter: Filter::All(vec![
-                Filter::Match(Path(vec![NO_VISIBILITY.0]), VISI_VISIBLE.into()),
-                Filter::Match(Path(vec![NO_PARENT.0]), row.into()),
-            ]),
-            ..Default::default()
-        })?
-        .collect::<Result<Vec<_>>>()?;
+//     let children_rows = txn
+//         .query(Query {
+//             sort: Sort::Value(ValueSort {
+//                 multi: MultiBehaviour::First,
+//                 offset: None,
+//                 order,
+//                 path,
+//             }),
+//             filter: Filter::All(vec![
+//                 Filter::Match(Path(vec![NO_VISIBILITY.0]), VISI_VISIBLE.into()),
+//                 Filter::Match(Path(vec![NO_PARENT.0]), row.into()),
+//             ]),
+//             ..Default::default()
+//         })?
+//         .collect::<Result<Vec<_>>>()?;
 
-    if children_rows.is_empty() {
-        return Ok(());
-    }
+//     if children_rows.is_empty() {
+//         return Ok(());
+//     }
 
-    let mut list = ObjectBufferBuilder::default();
+//     let mut list = ObjectBufferBuilder::default();
 
-    list.push(
-        NODELIST_DISPLAYSTYLE,
-        match kind {
-            KIND_SEASON | KIND_SHOW => NLSTYLE_LIST,
-            _ => NLSTYLE_GRID,
-        },
-    );
+//     list.push(
+//         NODELIST_DISPLAYSTYLE,
+//         match kind {
+//             KIND_SEASON | KIND_SHOW => NLSTYLE_LIST,
+//             _ => NLSTYLE_GRID,
+//         },
+//     );
 
-    for (row, _) in children_rows {
-        list.push(
-            NODELIST_ITEM,
-            Object::EMPTY
-                .insert(NKU_NODE, txn.get(row)?.unwrap().as_object())
-                .as_object(),
-        );
-    }
+//     for (row, _) in children_rows {
+//         list.push(
+//             NODELIST_ITEM,
+//             Object::EMPTY
+//                 .insert(NKU_NODE, txn.get(row)?.unwrap().as_object())
+//                 .as_object(),
+//         );
+//     }
 
-    page.push(VIEW_NODE_LIST, list.finish().as_object());
-    Ok(())
-}
+//     page.push(VIEW_NODE_LIST, list.finish().as_object());
+//     Ok(())
+// }
 
-fn c_credits(
-    page: &mut ObjectBufferBuilder,
-    txn: &mut dyn Transaction,
-    nku: &Object,
-) -> Result<()> {
-    if !nku.get(NKU_NODE).unwrap_or_default().has(NO_CREDIT.0) {
-        return Ok(());
-    }
+// fn c_credits(
+//     page: &mut ObjectBufferBuilder,
+//     txn: &mut dyn Transaction,
+//     nku: &Object,
+// ) -> Result<()> {
+//     if !nku.get(NKU_NODE).unwrap_or_default().has(NO_CREDIT.0) {
+//         return Ok(());
+//     }
 
-    let mut cats = BTreeMap::<_, Vec<_>>::new();
-    for cred in nku.get(NKU_NODE).unwrap_or_default().iter(NO_CREDIT) {
-        let mut o = ObjectBuffer::empty();
-        if let Some(row) = cred.get(CR_NODE) {
-            let node = txn.get(row)?.unwrap();
-            o = o.as_object().insert(NKU_NODE, node.as_object());
-        }
-        if let Some(role) = cred.get(CR_ROLE) {
-            o = o.as_object().insert(NKU_ROLE, role)
-        }
-        cats.entry(cred.get(CR_KIND).unwrap_or(CRCAT_CREW))
-            .or_default()
-            .push(o);
-    }
-    let mut cats = cats.into_iter().collect::<Vec<_>>();
-    cats.sort_by_key(|(c, _)| match *c {
-        CRCAT_CAST => 0,
-        CRCAT_CREW => 1,
-        _ => 100,
-    });
-    for (cat, elems) in cats {
-        let mut list = ObjectBufferBuilder::default();
-        list.push(NODELIST_DISPLAYSTYLE, NLSTYLE_INLINE);
-        list.push(NODELIST_TITLE, &format!("tag.cred.kind.{cat}"));
-        for item in elems {
-            list.push(NODELIST_ITEM, item.as_object());
-        }
-        page.push(VIEW_NODE_LIST, list.finish().as_object());
-    }
+//     let mut cats = BTreeMap::<_, Vec<_>>::new();
+//     for cred in nku.get(NKU_NODE).unwrap_or_default().iter(NO_CREDIT) {
+//         let mut o = ObjectBuffer::empty();
+//         if let Some(row) = cred.get(CR_NODE) {
+//             let node = txn.get(row)?.unwrap();
+//             o = o.as_object().insert(NKU_NODE, node.as_object());
+//         }
+//         if let Some(role) = cred.get(CR_ROLE) {
+//             o = o.as_object().insert(NKU_ROLE, role)
+//         }
+//         cats.entry(cred.get(CR_KIND).unwrap_or(CRCAT_CREW))
+//             .or_default()
+//             .push(o);
+//     }
+//     let mut cats = cats.into_iter().collect::<Vec<_>>();
+//     cats.sort_by_key(|(c, _)| match *c {
+//         CRCAT_CAST => 0,
+//         CRCAT_CREW => 1,
+//         _ => 100,
+//     });
+//     for (cat, elems) in cats {
+//         let mut list = ObjectBufferBuilder::default();
+//         list.push(NODELIST_DISPLAYSTYLE, NLSTYLE_INLINE);
+//         list.push(NODELIST_TITLE, &format!("tag.cred.kind.{cat}"));
+//         for item in elems {
+//             list.push(NODELIST_ITEM, item.as_object());
+//         }
+//         page.push(VIEW_NODE_LIST, list.finish().as_object());
+//     }
 
-    Ok(())
-}
+//     Ok(())
+// }
 
-fn c_credited(page: &mut ObjectBufferBuilder, txn: &mut dyn Transaction, row: u64) -> Result<()> {
-    let children_rows = txn
-        .query(Query {
-            sort: Sort::Value(ValueSort {
-                multi: MultiBehaviour::First,
-                offset: None,
-                order: SortOrder::Ascending,
-                path: Path(vec![NO_TITLE.0]),
-            }),
-            filter: Filter::All(vec![
-                Filter::Match(Path(vec![NO_VISIBILITY.0]), VISI_VISIBLE.into()),
-                Filter::Match(Path(vec![NO_CREDIT.0, CR_NODE.0]), row.into()),
-            ]),
-            ..Default::default()
-        })?
-        .collect::<Result<Vec<_>>>()?;
+// fn c_credited(page: &mut ObjectBufferBuilder, txn: &mut dyn Transaction, row: u64) -> Result<()> {
+//     let children_rows = txn
+//         .query(Query {
+//             sort: Sort::Value(ValueSort {
+//                 multi: MultiBehaviour::First,
+//                 offset: None,
+//                 order: SortOrder::Ascending,
+//                 path: Path(vec![NO_TITLE.0]),
+//             }),
+//             filter: Filter::All(vec![
+//                 Filter::Match(Path(vec![NO_VISIBILITY.0]), VISI_VISIBLE.into()),
+//                 Filter::Match(Path(vec![NO_CREDIT.0, CR_NODE.0]), row.into()),
+//             ]),
+//             ..Default::default()
+//         })?
+//         .collect::<Result<Vec<_>>>()?;
 
-    if children_rows.is_empty() {
-        return Ok(());
-    }
+//     if children_rows.is_empty() {
+//         return Ok(());
+//     }
 
-    let mut list = ObjectBufferBuilder::default();
-    list.push(NODELIST_DISPLAYSTYLE, NLSTYLE_GRID);
-    list.push(NODELIST_TITLE, "node.credited");
+//     let mut list = ObjectBufferBuilder::default();
+//     list.push(NODELIST_DISPLAYSTYLE, NLSTYLE_GRID);
+//     list.push(NODELIST_TITLE, "node.credited");
 
-    for (row, _) in children_rows {
-        list.push(
-            NODELIST_ITEM,
-            Object::EMPTY
-                .insert(NKU_NODE, txn.get(row)?.unwrap().as_object())
-                .as_object(),
-        );
-    }
+//     for (row, _) in children_rows {
+//         list.push(
+//             NODELIST_ITEM,
+//             Object::EMPTY
+//                 .insert(NKU_NODE, txn.get(row)?.unwrap().as_object())
+//                 .as_object(),
+//         );
+//     }
 
-    page.push(VIEW_NODE_LIST, list.finish().as_object());
-    Ok(())
-}
+//     page.push(VIEW_NODE_LIST, list.finish().as_object());
+//     Ok(())
+// }
diff --git a/server/src/ui/player.rs b/server/src/ui/player.rs
index 1050abb..c6c177e 100644
--- a/server/src/ui/player.rs
+++ b/server/src/ui/player.rs
@@ -5,13 +5,15 @@
 */
 use super::error::MyResult;
 use crate::request_info::RequestInfo;
+use anyhow::anyhow;
 use jellycommon::{
-    jellyobject::{Object, ObjectBuffer, Path},
+    jellyobject::{EMPTY, Path},
     *,
 };
 use jellydb::{Filter, Query};
 use jellyui::components::node_page::Player;
 use rocket::{get, response::content::RawHtml};
+use std::borrow::Cow;
 
 // fn jellynative_url(action: &str, seek: f64, secret: &str, node: &str, session: &str) -> String {
 //     let protocol = if CONF.tls { "https" } else { "http" };
@@ -31,22 +33,25 @@ pub fn r_player(ri: RequestInfo<'_>, t: Option<f64>, slug: &str) -> MyResult<Raw
     ri.require_user()?;
     let _ = t;
 
-    let mut node = ObjectBuffer::empty();
+    let mut node = None;
     ri.state.database.transaction(&mut |txn| {
         if let Some(row) = txn.query_single(Query {
             filter: Filter::Match(Path(vec![NO_SLUG.0]), slug.into()),
             ..Default::default()
         })? {
-            node = txn.get(row)?.unwrap();
+            node = Some(txn.get(row)?.unwrap());
         }
         Ok(())
     })?;
+    let Some(node) = node else {
+        Err(anyhow!("no such node"))?
+    };
 
     Ok(ri.respond_ui(&Player {
         ri: &ri.render_info(),
         nku: Nku {
-            node: node.as_object(),
-            userdata: Object::EMPTY,
+            node: Cow::Borrowed(&node),
+            userdata: Cow::Borrowed(EMPTY),
             role: None,
         },
     }))