Upload Custom Thumbnail for a Lecture #1479
Conversation
There was a problem hiding this comment.
Looks like a good start! This is only a draft but I thought I could leave some input already.
In addition to my thoughts, we need a way during livestreams, to prevent the thumbnail from being overwritten by the automatomatically generated one. My suggestion is, creating a new FileType for costum thumbnails, which is preferred over automatically generated ones.
api/stream.go
Outdated
| @@ -50,6 +50,8 @@ func configGinStreamRestRouter(router *gin.Engine, daoWrapper dao.DaoWrapper) { | |||
| thumbs.GET(":fid", routes.getThumbs) | |||
| thumbs.GET("/live", routes.getLiveThumbs) | |||
| thumbs.GET("/vod", routes.getVODThumbs) | |||
| thumbs.POST("/customlive", routes.putCustomLiveThumbnail) | |||
There was a problem hiding this comment.
Why do we need two routes to do the same thing? I'd argue there's no need to distinguish livestreams from vods when uploading thumbnails
There was a problem hiding this comment.
yes the two routes were originally there in case a distinction was necessary but now of course one route is enough.
| return | ||
| } | ||
|
|
||
| filename := file.Filename |
There was a problem hiding this comment.
remove the filename if it's unused
There was a problem hiding this comment.
filename is needed to create an object of type File as it has the element Filename as part of it.
api/stream.go
Outdated
| course := tumLiveContext.Course | ||
| file, err := c.FormFile("file") | ||
| if err != nil { | ||
| c.JSON(http.StatusBadRequest, gin.H{"error": "Invalid file"}) |
There was a problem hiding this comment.
you can use our RequestError to abort:
_ = c.AbortWithError(http.StatusForbidden, tools.RequestError{
Status: http.StatudBadRequest,
CustomMessage: "Couldn't read file",
Err: err,
})
api/stream.go
Outdated
| filesFolder := fmt.Sprintf("%s/%s.%d/%s.%s/files", | ||
| tools.Cfg.Paths.Mass, | ||
| course.Name, course.Year, | ||
| course.Name, course.TeachingTerm) | ||
|
|
||
| path := fmt.Sprintf("%s/%s%s", filesFolder, fileUuid, filepath.Ext(filename)) |
There was a problem hiding this comment.
please use path.Join from the standard library to construct this filepath:
https://pkg.go.dev/path#example-Join
api/stream.go
Outdated
|
|
||
| //tempFilePath := pathprovider.LiveThumbnail(strconv.Itoa(int(streamID))) | ||
| if err := c.SaveUploadedFile(file, path); err != nil { | ||
| c.JSON(http.StatusInternalServerError, gin.H{"error": "Failed to save file"}) |
There was a problem hiding this comment.
Use RequestError here too
api/stream.go
Outdated
| StreamID: streamID, | ||
| Path: path, | ||
| Filename: file.Filename, | ||
| Type: model.FILETYPE_THUMB_CAM, |
There was a problem hiding this comment.
iirc this filetype is for the scrubbar, not the preview. To be sure, use FILETYPE_THUMB_LG_CAM_PRES, which is always elected default
There was a problem hiding this comment.
I will change the type now temporarily but the idea is as you suggested to have a new type for custom thumbnail
api/stream.go
Outdated
| Type: model.FILETYPE_THUMB_CAM, | ||
| } | ||
|
|
||
| fileDao := dao.NewFileDao() |
There was a problem hiding this comment.
no need to create a new DAO, just use r.DaoWrapper.FileDao
changed route for thumbnail posting Co-authored-by: Joscha Henningsen <[email protected]>
Motivation and Context
resolves #1155
Description
Currently the thumbnail for the videos is generated automatically. The new Upload Custom Thumbnail button allows the admin users to be able to drag and drop an image file as a custom thumbnail for a lecture. The UI is still a work in progress and the file upload without drag and drop will be supported for better support on other devices such as mobiles.
The current to-dos:
Steps for Testing
Prerequisites:
Screenshots