Merge pull request #25 from bummzack/pulls/fix-images

Fix issue where `DBFile` wasn't recognized as file.
tractorcow · Mar 8, 2018 · b971bff · b971bff
2 parents 8f0c9e0 + 393d2a6
commit b971bff
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/travis.yml → .travis.yml b/travis.yml → .travis.yml
diff --git a/composer.json b/composer.json
@@ -1,7 +1,7 @@
 {
 	"name": "tractorcow/silverstripe-opengraph",
 	"description": "Implementation of the Facebook Open Graph protocol for SilverStripe",
-	"type": "silverstripe-module",
+	"type": "silverstripe-vendormodule",
 	"keywords": [
 		"silverstripe",
 		"facebook",

diff --git a/src/ObjectBuilders/OpenGraphBuilder.php b/src/ObjectBuilders/OpenGraphBuilder.php
@@ -2,6 +2,7 @@
 
 namespace TractorCow\OpenGraph\ObjectBuilders;
 
+use SilverStripe\Assets\Storage\DBFile;
 use SilverStripe\Control\HTTP;
 use SilverStripe\Core\Extensible;
 use SilverStripe\Core\Injector\Injectable;
@@ -153,7 +154,7 @@ protected function appendMediaMetaTags(&$tags, $namespace, $value, $https = null
         }
 
         // Handle File objects
-        if ($value instanceof File) {
+        if ($value instanceof File || $value instanceof DBFile) {
             if (!$value->exists()) {
                 return;
             }