status_report

Since drupal 8 video.js is using Video.js CDN. Is it possible to remove this or add a setting for those who using local video.js from library and give warning if there no video.js in their library

The above is using nginx 1.8.1 and php 7 php-fpm and i got the above and error got a php error report.

User error: Invalid placeholder (!url) in string: Missing the Video.js library. Please download Video.js and extract it into the %directory directory. in Drupal\Component\Render\FormattableMarkup::placeholderFormat() (line 240 of /srv/http/mywebsite/core/lib/Drupal/Component/Render/FormattableMarkup.php).

==============================
I test with https://simplytest.me/ drupal 8.0.5 with video.js modules install only. There no error.
Only local images are allowed.

Suspect php 7 or php-fpm that cause it. Can one confirm this.

Comments

spacereactor created an issue. See original summary.

spacereactor’s picture

Issue summary: View changes
StatusFileSize
new9.79 KB
arnoldski’s picture

I'm having the same issue.
Using:
Drupal 8.1.8
Database MySQL
Database 5.7.16-10
PHP 5.6.26

zalak.addweb’s picture

Issue tags: +status report, +video.js
dobrzyns’s picture

I am experiencing this as well and am also using PHP 7.

PHP: 7.0.17
Drupal: 8.2.7
MySQL 5.7
Apache 2.4.10 (Debian)

mqanneh’s picture

+1

mqanneh’s picture

Status: Active » Needs review
StatusFileSize
new1.23 KB

Added a new patch.

zanvidmar’s picture

The patch from #7 fix the url issue.

  • _shY committed 4505b92d on 8.x-1.x authored by mqanneh
    Issue #2698585 by mqanneh: status report missing video.js
    

  • _shY committed 49922dc9 on 3.x authored by mqanneh
    Issue #2698585 by mqanneh: status report missing video.js
    
_shy’s picture

Status: Needs review » Fixed

Merged to the 8.x-1.x and 3.x branches.
Thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

abhijeet.kumar2107’s picture

I tried patch from #7 but still the error is coming on drupal status reports. Its seems to be path are coming Null which are the for this.

abhijeet.kumar2107’s picture