Forum Replies Created
-
-
Hi all. We started getting “html5: Video file not found” errors again something like 10 minutes ago today.
Are you guys seeing something on your end? Is the chain broken again somewhere?
Even more importantly: what are the % chances of a regression with this issue?
Tyvm, seems to work.
How did you manage to fix it without plugin updates? Are all video plays somehow going through a FV server?
Thanks. We’re seeing the other threads about this issue popping up, so we know it’s a global problem.
How is this going. Anything available in the latest betas?
Here’s an example data structure for `! is_countable( $video_data )`
==> www-institute-wp-content-debug.log <== [19-Jun-2019 11:40:03 UTC] stdClass Object ( [id] => 299 [src] => https://vimeo.com/302873057 [src1] => [src2] => [splash] => [splash_text] => [caption] => Online Course: Account based marketing [end] => [mobile] => [rtmp] => [rtmp_path] => [start] => )
I agree, browsers have reached a high enough stability level that this WP AJAX performance issue is the bigger problem.
Btw Delicious Brains also face the same issue https://deliciousbrains.com/wordpress-rest-api-vs-custom-request-handlers/ with their famous WP Migrate DB Pro plugin. Maybe some useful insights in their thoughts?
I have the same concern, happy to see someone else writing it up.
Martin, have you guys considered SHORTINIT approach?
https://www.google.com/search?client=firefox-b-d&q=admin+ajax+shortinit
I’m not 100% sure it satisfies all requirements, but maybe?
6 years ago in reply to: Cross-origin Request Blocking – FV PlayerI like
SHORTINIT
idea. If you’re needing to get WP env infos in a sane fashion, and maybe WP API access, it sounds like the next best available option. Possibly allows sanitizing other parts offv-vimeo-ajax.php
as a positive side effect?PS I’m also hoping FVP code quality improves over time. I see formatting inconsistencies and whitespace errors all over the place, which usually are signs of.. let’s say “early” professionals.
Recommendations:
* https://github.com/foliovision/fv-wordpress-flowplayer/blob/7.3.9.727/composer.json should become stronger, I’d recommend integrating
dealerdirect/phpcodesniffer-composer-installer
similar to https://github.com/justintadlock/mythic/blob/1.1.0/composer.json#L27 and get everyone on the same page.* centralize on PhpStorm, if you haven’t already (+ have whole team upvote https://youtrack.jetbrains.com/issue/IDEABKL-708 lol)
Well, depends on the amount of metadata you need to store. Post meta can perform fine even with millions of rows of data, as long as you don’t self-join it multiple times with itself. Custom tables are always doable, regardless of interface decisions. But your team is probably aware of all this.
It’d just be a more consistent experience across screens, if people would forego innovating on islands and joined more forces on high quality existing efforts like https://nestedpages.com or on some more experimental things like https://github.com/WP-API/new-list-tables
React is now real w/ Gutenberg, that’s bound to bring new innovation to the crappy post list tables soon enough.
Unless you plan to contribute your custom screen to core, and somehow get it accepted, it’ll look and work weird and different forever. I try to avoid such busywork in my own solutions, even when it brings temporary pain, while we work with upstream. Something to consider, perhaps.
How come FV Player admin screen isn’t implemented as a standard post list? With some custom AJAX magic, which usually ends up being worse quality and inconsistent UX, we’re also losing out on valuable standard mechanics like http://admincolumns.com compatibility (everyone could easily self-service add almost any column they need).
Your thoughts?
Kyle, should we run a
wp cli
query to look for shortcodes and get a report of the shortcode variations going?Thanks. We haven’t been able to figure out whether MPEG-DASH should default to “enabled” or “disabled”.
With current “disabled” state, we have not heard any video breakage complaints for a long time, which is great.
But are we missing out on something useful?
> The plugin activation hook in WordPress doesn’t run on plugin updates though, so we will have to find a better way.
Indeed, but many many plugins are doing db migrations during updates. There has to be a best practice mechanism? I just haven’t built a plugin myself with that requirement (yet), so I can’t offer experience-based advice right away.
I put a simple error_log REQUEST_URI statement, and for example the video on https://conversionxl.com/institute/lesson/psychological-design-toolkit/ was just shown to trigger.
It seems to be a systemic problem, many different URLs are shown.
I hope you still have access to the Institute, please e-mail me if not.
Ah, indeed, forgot the traceback.
[05-Feb-2019 16:35:06 UTC] WordPress database error Table 'wordpress.wp_fv_player_videos' doesn't exist for query SELECT id, src FROM `wp_fv_player_videos` as v left join ( select id_video from wp_fv_player_videomet a WHERE meta_key = 'duration' ) as m ON v.id = m.id_video where m.id_video IS NULL ORDER BY id DESC made by include('phar:///usr/local/bin/wp/php/boot-phar.php'), include('phar:///usr/local/bin/wp/vendor/wp-cli/wp-cli/php/wp-cli.php'), WP _CLI\bootstrap, WP_CLI\Bootstrap\LaunchRunner->process, WP_CLI\Runner->start, WP_CLI\Runner->_run_command_and_exit, WP_CLI\Runner->run_command, WP_CLI\Dispatcher\Subcommand->invoke, call_user_func, WP_CLI\Dispatcher\CommandFactory::WP_CLI \Dispatcher\{closure}, call_user_func, Cron_Event_Command->run, Cron_Event_Command::run_event, do_action_ref_array('fv_flowplayer_checker_event'), WP_Hook->do_action, WP_Hook->apply_filters, FV_Player_Checker->checker_cron
Admin has been visited by users all the time, but tables were never generated.
Also, you shouldn’t be doing table checking on every request, only on plugin activation.
6 years ago in reply to: FV Player settings page causes js SyntaxErrorVersion 7.2.4.727
Thanks. “wp_options” can handle a ton of data (I’m not worried about the table size at all), but yeah must avoid autoloading the rows. You should really find a way to retroactively run the SQL query for other customers, who may not be savvy enough to do it on their own, or even know to search for it as a bug.
Looking forward to the data model migration, but this change alone should certainly work well enough for a while.
Would really be nice if your comment formatter would manager to understand standard
</code> and <code>
something` markup better.6 years ago in reply to: Vimeo reliability issuesOur videos are paid content, public links won’t help.
We could provide one-off access to your support staff when incidents arise. Do you think we could do a simple cross-company Slack channel for posting the links? They have that built in now.
Np. Thanks for your attention here.
7 years ago in reply to: PHP Notice on the line 748Code still hasn’t been fixed in 0.9.15.
Just curious – do you guys feel like it’s totally OK to keep spamming your customers production site logs with this?
Compared to the in-depth business blog posts at https://foliovision.com/weblog I find the effort here quite disappointing.
7 years ago in reply to: PHP Notice on the line 748Oops, maybe you can re-format the previous message to use
<code>
tags.Would be nice to support backticks `
7 years ago in reply to: PHP Notice on the line 748Hey guys. This is super simple to fix, what’s the hold up? Every update thus far starts spamming logs again. Here’s some help:
$bShouldBreak = ( $time_now - $time_last > 5 ) ? true : false; // insert paragraph break if entries are more than 5 seconds apart if( $bShouldBreak ) { $time_last_break = $time_now; } if( $bSentenceEnd && $time_now - $time_last_break > 30 ) { // is sentenced has ended previously and it's 30 seconds since last break also insert paragraph break $bShouldBreak = true; $time_last_break = $time_now; }
`$time_last_break` is first initialized inside `if`.
Maybe the simplest thing would be to add `$time_last_break = 0` initialization earlier.
- This reply was modified 7 years ago by Eduard.
7 years ago in reply to: PHP Notice on the line 748Thanks for the forum invite. I looked at the code, and you’re initializing this variable inside a conditional, that may not always execute.