silviapfeiffer / external-videos Goto Github PK

With the large overhaul, we've removed the ability to delete channels from being aggregated. The "delete_author" functionality from the old version has gone missing.

for convenience...

Need to sanitize video post data in the SP_EV_Admin::save_video() function prior to saving video posts. Also need to change this function to use the embed_url as the video's URL in post content, rather than the video_url (for viewing on host). The latter works as an embed url on some hosts, but not on others. I believe the embed url will always work if the format is correct.

In the host classes fetch loops I would like to break out the $video array standardization functions into separate functions (compose_video($vid)) so they are easier to read and maintain.

The upcoming "Media Explorer" extensions will need analagous functions which can be defined in the host classes at a later time.

Currently, when videos are pulled from the hosting services to the WP instance, they are automatically published.

However, at a service with higher privacy like Vimeo, users and others can view draft videos that are not ready for publication. These videos would still appear in an authenticated request to the user's upload channel and be published on WP - not ideal.

The proposed setting would govern whether posts pulled in by the cron job, or otherwise, are automatically published, or set to draft, pending, or private.

selecting a post or page to attach a video to is currently broken

So upgrading people know to get API keys

attaching a video from the external videos collection to a post or page is currently broken

Hi Silvia,

Looks like vimeo has deprecated the" Vimeo Advanced API" - check here. This could be the reason for the video feed not updating from Vimeo.

I'm looking into making their API work with the plugin. Let me know if you're already working on it, otherwise I'll proceed.

Forgot to sanitize these and use our pre-formed post_meta embed_url (instead of remaking the embed code with WP_Embed and post_meta video_url) as is currently done in the functions.

During debugging I have noticed that the options seem to be loaded many many times during admin init, and I think there is something fundamentally inefficient about the way I've written it. I just don't understand OOP/Classes enough to know how to fix it.

Since each host formats author options a different way, i suspect it may be a case of creating an abstract class in core, and extending it for each host. anyway Silvia if you get a chance, could you summarize for me the proper way to do this?

I think i can do it if you can explain the principle involved! :-)