refactored everything

Author: AlexVanderbist

src/OpenGraph.php

@@ -0,0 +1,107 @@
+<?php
+
+namespace Spatie\OpenGraph\Types;
+
+
+use Spatie\OpenGraph\OpenGraphImage;
+use Spatie\OpenGraph\OpenGraphObject;
+
+abstract class OpenGraphBaseType extends OpenGraphObject
+{
+    /** @var string */
+    protected $type;
+
+    public function __construct(?string $title = '', ?string $url, $image = null)
+    {
+        $this->type($this->type);
+        $this->title($title);
+        $this->url($url);
+        $this->image($image);
+    }
+
+    /**
+     * Creates a new Open Graph object with the correct Open Graph object type.
+     *
+     * This method's `title`, `url` and `image` parameters are required
+     * properties for an Open Graph object.
+     * You can set them here or use the methods on the returned OpenGraphBaseType
+     * instance.
+     *
+     * @param null|string $title
+     * @param null|string $url
+     * @param null|string|\Spatie\OpenGraph\OpenGraphImage $image
+     *
+     * @return OpenGraphBaseType
+     */
+    public static function create(?string $title = '', ?string $url, $image = null)
+    {
+        return new static(...func_get_args());
+    }
+
+    public function title($title)
+    {
+        $this->addTag('title', $title, 'og');
+
+        return $this;
+    }
+
+    public function url($title)
+    {
+        $this->addTag('url', $title, 'og');
+
+        return $this;
+    }
+
+    public function image($image)
+    {
+        if (is_string($image)) {
+            $image = new OpenGraphImage($image);
+        }
+
+        $this->objects[] = $image;
+
+        return $this;
+    }
+
+    public function description(string $description)
+    {
+        $this->addTag('description', $description, 'og');
+
+        return $this;
+    }
+
+    public function determiner(string $determiner)
+    {
+        $this->addTag('determiner', $determiner, 'og');
+
+        return $this;
+    }
+
+    public function locale(string $locale = 'en_US')
+    {
+        $this->addTag('locale', $locale, 'og');
+
+        return $this;
+    }
+
+    public function alternateLocale(string $locale = 'en_US')
+    {
+        $this->addTag('locale:alternate', $locale, 'og');
+
+        return $this;
+    }
+
+    public function siteName(string $siteName)
+    {
+        $this->addTag('site_name', $siteName, 'og');
+
+        return $this;
+    }
+
+    protected function type(string $type)
+    {
+        $this->addTag('type', $type, 'og');
+
+        return $this;
+    }
+}

src/Types/OpenGraphBaseType.php

@@ -1,8 +1,9 @@
 <?php
 
-namespace Spatie\OpenGraph;
+namespace Spatie\OpenGraph\Types;
 
-class OpenGraphBook extends OpenGraph
+
+class OpenGraphBook extends OpenGraphBaseType
 {
     /** @var string */
     protected $prefix = 'book';

src/OpenGraphBook.php src/Types/OpenGraphBook.phpsrc/OpenGraphBook.php renamed to src/Types/OpenGraphBook.php

@@ -1,8 +1,9 @@
 <?php
 
-namespace Spatie\OpenGraph;
+namespace Spatie\OpenGraph\Types;
 
-class OpenGraphMusic extends OpenGraph
+
+class OpenGraphMusic extends OpenGraphBaseType
 {
     /** @var string */
     protected $prefix = 'music';

src/OpenGraphMusicRadioStation.php src/Types/OpenGraphMusicRadioStation.phpsrc/OpenGraphMusicRadioStation.php renamed to src/Types/OpenGraphMusicRadioStation.php

@@ -0,0 +1,10 @@
+<?php
+
+namespace Spatie\OpenGraph\Types;
+
+
+class OpenGraphWebsite extends OpenGraphBaseType
+{
+    /** @var string */
+    protected $type = 'website';
+}

src/Types/OpenGraphWebsite.php

@@ -3,8 +3,8 @@
 namespace Spatie\Skeleton\Test;
 
 use PHPUnit\Framework\TestCase;
-use Spatie\OpenGraph\OpenGraph;
-use Spatie\OpenGraph\OpenGraphBook;
+use Spatie\OpenGraph\Types\OpenGraphWebsite;
+use Spatie\OpenGraph\Types\OpenGraphBook;
 use Spatie\OpenGraph\OpenGraphImage;
 use Spatie\Snapshots\MatchesSnapshots;
 
@@ -15,7 +15,7 @@ class GenerateOpenGraphTest extends TestCase
     /** @test */
     public function it_can_generate_basic_metadata()
     {
-        $metaTags = OpenGraph::create('Title', 'http://www.example.com', 'http://example.com/image.jpg')
+        $metaTags = OpenGraphWebsite::create('Title', 'http://www.example.com', 'http://example.com/image.jpg')
             ->getMetaTags();
 
         $this->assertMatchesSnapshot($metaTags);
@@ -24,7 +24,7 @@ public function it_can_generate_basic_metadata()
     /** @test */
     public function it_can_generate_optional_metadata()
     {
-        $metaTags = OpenGraph::create('Title', 'http://www.example.com', 'http://example.com/image.jpg')
+        $metaTags = OpenGraphWebsite::create('Title', 'http://www.example.com', 'http://example.com/image.jpg')
             ->description('This is the page description')
             ->siteName('Test Name')
             ->locale('en_US')
@@ -38,7 +38,7 @@ public function it_can_generate_optional_metadata()
     /** @test */
     public function it_can_generate_metadata_for_multiple_images()
     {
-        $metaTags = OpenGraph::create('Title', 'http://www.example.com', 'http://example.com/image.jpg')
+        $metaTags = OpenGraphWebsite::create('Title', 'http://www.example.com', 'http://example.com/image.jpg')
             ->image(OpenGraphImage::create('http://example.com/image2.jpg', 'https://example.com/image2.jpg'))
             ->image(OpenGraphImage::create('http://example.com/image3.jpg', 'https://example.com/image3.jpg', 'image/jpeg', 800, 600))
             ->getMetaTags();