Merge pull request #733 from michalv8/feature/schema.org

Added schema.org tags
pradoslaw · Sep 1, 2023 · 969fb1a · 969fb1a
2 parents 6701dd6 + 9b1f247
commit 969fb1a
Show file tree

Hide file tree

Showing 8 changed files with 129 additions and 4 deletions.
diff --git a/app/Http/Controllers/Forum/TopicController.php b/app/Http/Controllers/Forum/TopicController.php
@@ -22,6 +22,7 @@
 use Coyote\Services\Forum\TreeBuilder\ListDecorator;
 use Coyote\Topic;
 use Illuminate\Http\Request;
+use Spatie\SchemaOrg\Schema;
 
 class TopicController extends BaseController
 {
@@ -130,6 +131,18 @@ public function index(Request $request, $forum, $topic)
 
         TopicResource::withoutWrapping();
 
+        $author = $topic->firstPost()->get()->first()->user->name;
+        $schema = Schema::discussionForumPosting()
+            ->identifier($request->getUri())
+            ->headline($topic->title)
+            ->author(
+                Schema::person()
+                ->name($author)
+            )->interactionStatistic(
+                Schema::interactionCounter()
+                    ->userInteractionCount($topic->replies)
+            );
+
         return $this->view('forum.topic', compact('posts', 'forum', 'paginate', 'reasons'))->with([
             'mlt'           => $this->moreLikeThis($topic),
             'model'         => $topic, // we need eloquent model in twig to show information about locked/moved topic
@@ -139,7 +152,8 @@ public function index(Request $request, $forum, $topic)
             'all_forums'    => $allForums,
             'user_forums'   => $userForums,
             'description'   => excerpt(array_first($posts['data'])['text'], 100),
-            'flags'         => $this->flags($forum)
+            'flags'         => $this->flags($forum),
+            'schema'        => $schema,
         ]);
     }
 

diff --git a/app/Services/Breadcrumb.php b/app/Services/Breadcrumb.php
@@ -3,6 +3,7 @@
 namespace Coyote\Services;
 
 use Illuminate\Contracts\Support\Arrayable;
+use Spatie\SchemaOrg\Schema;
 
 /**
  * Prosta klasa sluzaca do budowania elementu obecnego na kazdej podstronie, czyli breadcrumb
@@ -48,7 +49,24 @@ public function push($name, $url = null)
      */
     public function render()
     {
-        return view('components/breadcrumb', ['breadcrumbs' => $this->breadcrumbs]);
+        return view(
+            'components/breadcrumb',
+            [
+                'breadcrumbs' => $this->breadcrumbs,
+                'schema' => Schema::breadcrumbList()
+                    ->itemListElement(array_map(
+                        function ($breadcrumb, int $index) {
+                            return Schema::listItem()
+                                ->position($index + 1)
+                                ->identifier($breadcrumb['url'])
+                                ->name($breadcrumb['name']);
+                        },
+                        $this->breadcrumbs,
+                        array_keys($this->breadcrumbs),
+                    )
+                ),
+            ]
+        );
     }
 
     /**

diff --git a/composer.json b/composer.json
@@ -50,6 +50,7 @@
         "rawr/t-regx": "^v0.17.0",
         "rcrowe/twigbridge": "^0.12",
         "sentry/sentry-laravel": "^2.3",
+        "spatie/schema-org": "^3.14",
         "stripe/stripe-php": "^7.66",
         "twig/extensions": "^1.5"
     },

diff --git a/composer.lock b/composer.lock
diff --git a/resources/views/components/breadcrumb.twig b/resources/views/components/breadcrumb.twig
@@ -20,3 +20,5 @@
         </ul>
     </div>
 </div>
+
+{{ schema.toScript()|raw }}
diff --git a/resources/views/components/schema.twig b/resources/views/components/schema.twig
@@ -0,0 +1,14 @@
+{% macro organization() %}
+<script type="application/ld+json">
+{
+  "@context": "http://schema.org",
+  "@type": "Organization",
+  "logo": "https://www.4programmers.net/img/logo-mobile.png",
+  "email": "[email protected]",
+  "name": "4programmers.net",
+  "sameAs": [
+    "https://www.facebook.com/4programmers.net"
+  ]
+}
+</script>
+{% endmacro %}
diff --git a/resources/views/forum/topic.twig b/resources/views/forum/topic.twig
@@ -7,6 +7,7 @@
 {% import 'components.modals' as modals %}
 
 {% block content %}
+  {{ schema.to_script|raw }}
 
   {% if model.locked_at and model.locker.id %}
     <p class="alert alert-warning">
@@ -36,7 +37,6 @@
           </div>
         {% endif %}
       </div>
-
       {% for post in posts.data %}
         <div class="card card-post">
           <div class="card-header d-none d-lg-block">

diff --git a/resources/views/layout.twig b/resources/views/layout.twig
@@ -47,6 +47,9 @@
     {% block body %}
       {% import 'components.forms' as forms %}
       {% import 'components.widgets' as widgets %}
+      {% import 'components.schema' as schema %}
+
+      {{ schema.organization() }}
 
       <header class="{{ auth_guest() or user('allow_sticky_header') ? ' fixed-top' }}">
         <div class="navbar-border-top"></div>