Riclep\Storyblok\Traits\SchemaOrg

Complexity

Total Complexity

9

Size/Duplication

Total Lines	51
Duplicated Lines	0 %

Importance

Changes	3
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	schemaOrgScript()	0	9	2
A	initSchemaOrg()	0	11	5
A	addschemaOrg()	0	9	2

<?php


namespace Riclep\Storyblok\Traits;


use Riclep\Storyblok\Page;

trait SchemaOrg
{
	/**
	 * Automatically called to add a schema to the Page
	 */
	protected function initSchemaOrg(): void
	{
		if (method_exists($this, 'schemaOrg')) {
			if ($this instanceof Page) {
				$page = $this;
			} else {
				$page = $this->page();

It seems like page() must be provided by classes using this trait. How about adding it as abstract method to this trait?

			}

			if ($page && count($this->_componentPath) <= config('storyblok.schema_org_depth')) {
				$this->addschemaOrg($page);
			}
		}

	}

	/**
	 * Returns the JavaScript JSON-LD string
	 *
	 * @return string
	 */
	public function schemaOrgScript(): string
	{
		$schemaJson = '';

		foreach ($this->meta()['schema_org'] as $schema) {

It seems like meta() must be provided by classes using this trait. How about adding it as abstract method to this trait?

			$schemaJson .= $schema->toScript();
		}

		return $schemaJson;
	}

	/**
	 * Adds the schema to the meta of the current page
	 *
	 * @param $page
	 */
	protected function addschemaOrg($page): void
	{
		$currentSchemaOrg = $page->meta('schema_org');

        if ($schema = $this->schemaOrg()) {

The method schemaOrg() does not exist on Riclep\Storyblok\Traits\SchemaOrg. Did you maybe mean schemaOrgScript()?

            $currentSchemaOrg[] = $schema;
        }

		$page->replaceMeta('schema_org', $currentSchemaOrg ?? []);
	}
}