WordPress.org

Making WordPress.org

Opened 14 months ago

Closed 5 months ago

Last modified 5 months ago

#4667 closed enhancement (fixed)

Rename video title upon publishing or update

Reported by: casiepa Owned by: dd32
Milestone: Priority: normal
Component: WordPress.tv Keywords:
Cc:

Description

When a video is submitted to WordPress.tv, it creates a new post with all the data (e.g. https://wptv.wordpress.com/wp-admin/post.php?post=94816&action=edit) and also the (video)attachment linked to that post (e.g. https://wptv.wordpress.com/wp-admin/post.php?post=94817&action=edit). The title of the video-attachment is shown when a viewer hovers over the video on the frontend.

A patch is needed to automatically update the title the of the video-attachement with the title of the post when the post is 'published' or 'updated'.

Attachments (1)

Capture.PNG (64.8 KB) - added by casiepa 14 months ago.
View on frontend

Download all attachments as: .zip

Change History (10)

@casiepa
14 months ago

View on frontend

#1 @dd32
11 months ago

  • Owner set to dd32
  • Resolution set to fixed
  • Status changed from new to closed

In 9176:

WordPress.TV: Upon updating the post title, update the respective video title to match.

This is done, as the attachment title is used as the VideoPress embed title.

Fixes #4667.

This ticket was mentioned in Slack in #wptv by casiepa. View the logs.


11 months ago

#3 @casiepa
10 months ago

@dd32 Looks like it's no longer working...

#4 follow-up: @casiepa
5 months ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

It's not updating for e.g. https://wptv.wordpress.com/wp-admin/post.php?post=102131&action=edit

@dd32 FYI reopening ticket.

#5 in reply to: ↑ 4 @dd32
5 months ago

Replying to casiepa:

It's not updating for e.g. https://wptv.wordpress.com/wp-admin/post.php?post=102131&action=edit

Is there anything special about that upload? I mean, like how it was uploaded?

Looking at https://wptv.wordpress.com/wp-admin/upload.php it's working for most uploads, except for that one and another one (C1_08_David Stoeckl_NET-c4.9.mp4 and C1_03_Tamara Zimmermann_NET-c4.9.mp4)

I'll take a look this week.

#6 @casiepa
5 months ago

Hi @dd32
Since we saw that it was not updating anymore, we are manually updating the title. But I left Tamara as it was and David we still need to work on that video.
Pascal.

#7 @dd32
5 months ago

  • Resolution set to fixed
  • Status changed from reopened to closed

In 9744:

WordPress.TV: Set the initial title for video files to that of the parent post.

Fixes #4667.

#8 @dd32
5 months ago

It looks like this was affecting any uploads with a space in the filename, or where core changes to sanitize_file_name() were stripping out extra characters. [9744] should avoid that I think, or at least has worked with my test uploads.

#9 @dd32
5 months ago

In 9745:

WordPress.TV: Set the default post title to Speakers: Video Title to match the WordPress.tv naming.

See https://wordpress.slack.com/archives/C037W5MBT/p1587349553150700
See #4667.

Note: See TracTickets for help on using tickets.