Automattic / Automattic/jetpack

VideoPress: local video marked as uploaded to VideoPress after being deleted

Open
#28,851 1 comment 0 reactions 0 assignees View on GitHub
[Package] VideoPress [Pri] Normal [Status] Stale Bug
Dominant language
PHP
Stars
1.8k
Forks
898
Avg merge
1d 18h
Merged PRs (30d)
774

Description

When a local video is uploaded to VideoPress using the dashboard button, and then is deleted from VideoPress, it's still showing the uploaded status on the dashboard:

![2023-02-08_15-25-14](https://user-images.githubusercontent.com/6760046/217625940-fe72c79a-06a3-418f-96da-a9efcd9558f8.png)

Checking the video data, the `is_uploaded ` field is still set to true.

This field is defined by checking if a given meta key is defined on the attachment: https://github.com/Automattic/jetpack/blob/trunk/projects/packages/videopress/src/class-uploader.php#L174-L185

We need to make sure we are deleting this key after the video is deleted from VideoPress.

Contributor guide

Open the contributing guide

Research direction

Start in projects/packages/videopress/src/class-uploader.php, especially lines 174-185 where the is_uploaded value is determined, then trace the VideoPress deletion flow. Ensure the attachment meta key used for that status is removed when the video is deleted, so the dashboard no longer reports it as uploaded.

Written by the indexing model from the issue text.

Assessment

Tech stack
php
Domain
backend
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.