Move thumbnail back to synced data

backend/src/api/common.rs

@@ -7,9 +7,9 @@ use crate::{
         Context,
         err::{self, ApiResult},
         model::{
-            event::AuthorizedEvent, 
-            series::Series, 
-            realm::Realm, 
+            event::AuthorizedEvent,
+            series::Series,
+            realm::Realm,
             playlist::AuthorizedPlaylist,
             search::{SearchEvent, SearchRealm, SearchSeries},
         },

backend/src/api/model/event.rs

@@ -51,6 +51,7 @@ pub(crate) struct SyncedEventData {
     updated: DateTime<Utc>,
     start_time: Option<DateTime<Utc>>,
     end_time: Option<DateTime<Utc>>,
+    thumbnail: Option<String>,
 
     /// Duration in milliseconds
     duration: i64,
@@ -59,7 +60,6 @@ pub(crate) struct SyncedEventData {
 #[derive(Debug)]
 pub(crate) struct AuthorizedEventData {
     tracks: Vec<Track>,
-    thumbnail: Option<String>,
     captions: Vec<Caption>,
     segments: Vec<Segment>,
 }
@@ -98,12 +98,12 @@ impl_from_db!(
                     start_time: row.start_time(),
                     end_time: row.end_time(),
                     duration: row.duration(),
+                    thumbnail: row.thumbnail(),
                 }),
                 EventState::Waiting => None,
             },
             authorized_data: match row.state::<EventState>() {
                 EventState::Ready => Some(AuthorizedEventData {
-                    thumbnail: row.thumbnail(),
                     tracks: row.tracks::<Vec<EventTrack>>().into_iter().map(Track::from).collect(),
                     captions: row.captions::<Vec<EventCaption>>()
                         .into_iter()
@@ -165,6 +165,9 @@ impl SyncedEventData {
     fn duration(&self) -> f64 {
         self.duration as f64
     }
+    fn thumbnail(&self) -> Option<&str> {
+        self.thumbnail.as_deref()
+    }
 }
 
 /// Represents event data that is only accessible for users with read access
@@ -174,9 +177,6 @@ impl AuthorizedEventData {
     fn tracks(&self) -> &[Track] {
         &self.tracks
     }
-    fn thumbnail(&self) -> Option<&str> {
-        self.thumbnail.as_deref()
-    }
     fn captions(&self) -> &[Caption] {
         &self.captions
     }
@@ -231,8 +231,8 @@ impl AuthorizedEvent {
     /// Returns the authorized event data if the user has read access or is authenticated for the event.
     async fn authorized_data(
         &self,
-        context: &Context, 
-        user: Option<String>, 
+        context: &Context,
+        user: Option<String>,
         password: Option<String>,
     ) -> Option<&AuthorizedEventData> {
         let sha1_matches = |input: &str, encoded: &str| {
@@ -319,6 +319,18 @@ impl AuthorizedEvent {
             .get::<_, bool>(0)
             .pipe(Ok)
     }
+
+    async fn audio_only(&self, context: &Context) -> ApiResult<bool> {
+        let query = "select not exists ( \
+            select from events, unnest(tracks) as t \
+            where id = $1 and t.resolution is not null \
+        )";
+
+        context.db.query_one(&query, &[&self.key])
+            .await?
+            .get::<_, bool>(0)
+            .pipe(Ok)
+    }
 }
 
 #[derive(juniper::GraphQLUnion)]

backend/src/api/model/search/event.rs

@@ -116,7 +116,7 @@ impl SearchEvent {
             title: src.title,
             description: src.description,
             creators: src.creators,
-            thumbnail: if user_can_read { src.thumbnail } else { None },
+            thumbnail: src.thumbnail,
             duration: src.duration as f64,
             created: src.created,
             start_time: src.start_time,

frontend/src/routes/Embed.tsx

@@ -123,6 +123,7 @@ const embedEventFragment = graphql`
                 startTime
                 endTime
                 duration
+                thumbnail
             }
             ... VideoPageAuthorizedData
                 @arguments(eventUser: $eventUser, eventPassword: $eventPassword)

frontend/src/routes/Search.tsx

@@ -161,7 +161,6 @@ const query = graphql`
                         startTime
                         endTime
                         created
-                        userIsAuthorized
                         hostRealms { path ancestorNames }
                         textMatches {
                             start
@@ -521,7 +520,6 @@ const SearchEvent: React.FC<EventItem> = ({
     hostRealms,
     textMatches,
     matches,
-    userIsAuthorized,
 }) => {
     // TODO: decide what to do in the case of more than two host realms. Direct
     // link should be avoided.
@@ -534,21 +532,15 @@ const SearchEvent: React.FC<EventItem> = ({
             image: <Link to={link} tabIndex={-1}>
                 <Thumbnail
                     event={{
-                        id,
                         title,
                         isLive,
                         created,
-                        series: seriesId ? {
-                            id: seriesId,
-                        } : null,
+                        audioOnly,
                         syncedData: {
                             duration,
                             startTime,
                             endTime,
-                        },
-                        authorizedData: !userIsAuthorized ? null : {
                             thumbnail,
-                            audioOnly,
                         },
                     }}
                 />

frontend/src/routes/Video.tsx

@@ -461,6 +461,7 @@ const eventFragment = graphql`
                 duration
                 startTime
                 endTime
+                thumbnail
             }
             ... VideoPageAuthorizedData
                 @arguments(eventUser: $eventUser, eventPassword: $eventPassword)
@@ -486,13 +487,12 @@ const authorizedDataFragment = graphql`
             tracks { uri flavor mimetype resolution isMaster }
             captions { uri lang }
             segments { uri startTime }
-            thumbnail
         }
     }
 `;
 
 // Custom query to refetch authorized event data manually. Unfortunately, using
-// the fragment here is not enough, we need to also selected `authorizedData`
+// the fragment here is not enough, we need to also select `authorizedData`
 // manually. Without that, we could not access that field below to check if the
 // credentials were correct. Normally, adding `@relay(mask: false)` to the
 // fragment should also fix that, but that does not work for some reason.
@@ -509,7 +509,7 @@ export const authorizedDataQuery = graphql`
                 @arguments(eventUser: $eventUser, eventPassword: $eventPassword)
             ...on AuthorizedEvent {
                 authorizedData(user: $eventUser, password: $eventPassword) {
-                    thumbnail
+                    __id
                 }
             }
         }
@@ -558,7 +558,7 @@ const VideoPage: React.FC<Props> = ({ eventRef, realmRef, playlistRef, basePath
         "@type": "VideoObject",
         name: event.title,
         description: event.description ?? undefined,
-        thumbnailUrl: event.authorizedData?.thumbnail ?? undefined,
+        thumbnailUrl: event.syncedData?.thumbnail ?? undefined,
         uploadDate: event.created,
         duration: toIsoDuration(event.syncedData.duration),
         ...event.isLive && event.syncedData.startTime && event.syncedData.endTime && {

frontend/src/routes/manage/Realm/Content/Edit/EditMode/Video.tsx

@@ -47,8 +47,8 @@ export const EditVideoBlock: React.FC<EditVideoBlockProps> = ({ block: blockRef
                     created
                     isLive
                     creators
+                    audioOnly
                     syncedData { duration startTime endTime }
-                    authorizedData { thumbnail }
                 }
             }
             showTitle
@@ -152,6 +152,7 @@ const EventSelector: React.FC<EventSelectorProps> = ({ onChange, onBlur, default
                         duration
                         startTime
                         endTime
+                        audioOnly
                     }
                 }
             }
@@ -204,16 +205,7 @@ const formatOption = (event: Option, t: TFunction) => (
             ? <MovingTruck />
             : <Thumbnail
                 css={{ width: 120, minWidth: 120 }}
-                event={{
-                    ...event,
-                    syncedData: event.syncedData && {
-                        ...event.syncedData,
-                    },
-                    authorizedData: event.authorizedData && {
-                        ...event.authorizedData,
-                        audioOnly: false, // TODO
-                    },
-                }}
+                event={event}
             />}
         <div>
             <div>{event.title}</div>

frontend/src/routes/manage/Video/Shared.tsx

@@ -86,15 +86,16 @@ const query = graphql`
                 created
                 canWrite
                 isLive
+                audioOnly
                 acl { role actions info { label implies large } }
                 syncedData {
                     duration
                     updated
                     startTime
                     endTime
+                    thumbnail
                 }
                 authorizedData {
-                    thumbnail
                     tracks { flavor resolution mimetype uri }
                 }
                 series {

frontend/src/routes/manage/Video/index.tsx

@@ -76,13 +76,18 @@ const query = graphql`
                     startIndex endIndex
                 }
                 items {
-                    id title created description isLive tobiraDeletionTimestamp
+                    id
+                    title
+                    created
+                    description
+                    isLive
+                    tobiraDeletionTimestamp
+                    audioOnly
                     series { id }
                     syncedData {
-                        duration updated startTime endTime
+                        duration thumbnail updated startTime endTime
                     }
                     authorizedData {
-                        thumbnail
                         tracks { resolution }
                     }
                 }

frontend/src/schema.graphql

@@ -215,6 +215,7 @@ type SyncedEventData implements Node {
   endTime: DateTimeUtc
   "Duration in ms."
   duration: Float!
+  thumbnail: String
 }
 
 input NewPlaylistBlock {
@@ -335,6 +336,7 @@ type AuthorizedEvent implements Node {
     Otherwise, `false` is returned.
   """
   isReferencedByRealm(path: String!): Boolean!
+  audioOnly: Boolean!
 }
 
 "A block just showing some title."
@@ -766,7 +768,6 @@ union RemoveMountedSeriesOutcome = RemovedRealm | RemovedBlock
 """
 type AuthorizedEventData implements Node {
   tracks: [Track!]!
-  thumbnail: String
   captions: [Caption!]!
   segments: [Segment!]!
 }

frontend/src/ui/Blocks/Video.tsx

@@ -44,6 +44,7 @@ export const VideoBlock: React.FC<Props> = ({ fragRef, basePath }) => {
                         updated
                         startTime
                         endTime
+                        thumbnail
                     }
                     ... VideoPageAuthorizedData
                 }
@@ -65,7 +66,6 @@ export const VideoBlock: React.FC<Props> = ({ fragRef, basePath }) => {
         return unreachable();
     }
 
-
     return <div css={{ maxWidth: 800 }}>
         {showTitle && <Title title={event.title} />}
         <PlayerContextProvider>

frontend/src/ui/Blocks/VideoList.tsx

@@ -55,10 +55,10 @@ export const videoListEventFragment = graphql`
         creators
         isLive
         description
+        audioOnly
         series { title id }
-        syncedData { duration startTime endTime }
+        syncedData { thumbnail duration startTime endTime }
         authorizedData {
-            thumbnail
             tracks { resolution }
         }
     }