diff --git a/resources/views/metatags.antlers.html b/resources/views/metatags.antlers.html index d9a04cf..7d54485 100644 --- a/resources/views/metatags.antlers.html +++ b/resources/views/metatags.antlers.html @@ -16,8 +16,8 @@ {{ if advanced_generator }}{{ /if }} {{ if advanced_image }}{{ /if }} {{ if advanced_rights }}{{ /if }} - {{ if advanced_rating }}{{ /if }} - {{ if advanced_referrer }}{{ /if }} + {{ if advanced_rating }}{{ /if }} + {{ if advanced_referrer }}{{ /if }} {{ if advanced_short_link }}{{ /if }} {{ if advanced_original_source }}{{ /if }} {{ if advanced_geo_position }}{{ /if }} @@ -52,7 +52,7 @@ {{ if dublin_core_publisher }}{{ /if }} {{ if dublin_core_contributor }}{{ /if }} {{ if dublin_core_date }}{{ /if }} - {{ if dublin_core_type }}{{ /if }} + {{ if dublin_core_type }}{{ /if }} {{ if dublin_core_format }}{{ /if }} {{ if dublin_core_identifier }}{{ /if }} {{ if dublin_core_source }}{{ /if }} @@ -112,9 +112,9 @@ {{ fields }} - {{ if og_type }}{{ /if }} + {{ if og_type }}{{ /if }} - {{ if og_determiner }}{{ /if }} + {{ if og_determiner }}{{ /if }} {{ if og_description }}{{ /if }} {{ if og_image }} {{ foreach:og_image as="ogimage" }} @@ -140,7 +140,7 @@ {{ /foreach:og_video }} {{ /if }} - {{ if og_type == 'article' }} + {{ if og_type.value == 'article' }} {{ if og_article_author }} {{ og_article_author }} @@ -157,7 +157,7 @@ {{ if og_article_expiration_time }}{{ /if }} {{ /if }} - {{ if og_type == 'book' }} + {{ if og_type.value == 'book' }} {{ if og_book_author }} {{ og_book_author }} @@ -172,14 +172,14 @@ {{ /if }} {{ /if }} - {{ if og_type == 'profile' }} + {{ if og_type.value == 'profile' }} {{ if og_profile_first_name }}{{ /if }} {{ if og_profile_last_name }}{{ /if }} {{ if og_profile_username }}{{ /if }} - {{ if og_profile_gender }}{{ /if }} + {{ if og_profile_gender }}{{ /if }} {{ /if }} - {{ if og_type | contains:video }} + {{ if og_type.value | contains:video }} {{ if og_video_actor }} {{ og_video_actor }} @@ -207,12 +207,12 @@ {{ /og_video_tag }} {{ /if }} - {{ if og_type == 'video.episode' }} + {{ if og_type.value == 'video.episode' }} {{ if og_video_series }}{{ /if }} {{ /if }} {{ /if }} - {{ if og_type | contains:music }} + {{ if og_type.value | contains:music }} {{ if og_type == 'music.song' }} {{ if og_music_duration }}{{ /if }} {{ if og_music_album }} @@ -229,7 +229,7 @@ {{ /if }} {{ /if }} - {{ if og_type == 'music.album' }} + {{ if og_type.value == 'music.album' }} {{ if og_music_song }}{{ /if }} {{ if og_music_song_disc }}{{ /if }} {{ if og_music_song_track }}{{ /if }} @@ -241,19 +241,19 @@ {{ if og_music_release_date }}{{ /if }} {{ /if }} - {{ if og_type == 'music.playlist' }} + {{ if og_type.value == 'music.playlist' }} {{ if og_music_song }}{{ /if }} {{ if og_music_song_disc }}{{ /if }} {{ if og_music_song_track }}{{ /if }} {{ if og_music_creator }}{{ /if }} {{ /if }} - {{ if og_type == 'music.radio_station' }} + {{ if og_type.value == 'music.radio_station' }} {{ if og_music_creator }}{{ /if }} {{ /if }} {{ /if }} - {{ if og_type == 'product' }} + {{ if og_type.value == 'product' }} {{ if og_product_plural_title }}{{ /if }} {{ if og_product_price_amount }} {{ og_product_price_amount }} @@ -299,13 +299,13 @@ {{ if pinterest_rating_scale }}{{ /if }} {{ if pinterest_rating_count }}{{ /if }} - {{ if pinterest_type == 'article' }} + {{ if pinterest_type.value == 'article' }} {{ if pinterest_modified_time }}{{ /if }} {{ if pinterest_section }}{{ /if }} {{ if pinterest_tag }}{{ /if }} {{ /if }} - {{ if pinterest_type == 'product' }} + {{ if pinterest_type.value == 'product' }} {{ if pinterest_start_date }}{{ /if }} {{ if pinterest_end_date }}{{ /if }} {{ if pinterest_code_type && pinterest_code_id }}{{ /if }} @@ -336,8 +336,8 @@ {{ if settings | in_array:twitter }} {{ fields }} - {{ if twitter_type }}{{ /if }} - {{ if twitter_type == 'summary' }} + {{ if twitter_type }}{{ /if }} + {{ if twitter_type.value == 'summary' }} {{ if twitter_site }}{{ /if }} {{ if twitter_site_id }}{{ /if }} {{ if twitter_creator }}{{ /if }} @@ -351,13 +351,13 @@ {{ /twitter_image }} {{ /if }} {{ /if }} - {{ if twitter_type == 'player' }} + {{ if twitter_type.value == 'player' }} {{ if twitter_player }}{{ /if }} {{ if twitter_player_width }}{{ /if }} {{ if twitter_player_height }}{{ /if }} {{ if twitter_player_stream }}{{ /if }} {{ /if }} - {{ if twitter_type == 'app' }} + {{ if twitter_type.value == 'app' }} {{ if twitter_app_name_iphone }}{{ /if }} {{ if twitter_app_id_iphone }}{{ /if }} {{ if twitter_app_url_iphone }}{{ /if }} @@ -415,7 +415,7 @@ {{ fields }} {{ if apple_format_detection }}{{ /if }} {{ if apple_apple_itunes_app }}{{ /if }} - {{ if apple_apple_mobile_web_app_capable }}{{ /if }} + {{ if apple_apple_mobile_web_app_capable }}{{ /if }} {{ if apple_apple_mobile_web_app_status_bar_style }}{{ /if }} {{ /fields }} {{ /if }} @@ -431,9 +431,11 @@ {{ if settings | in_array:favicons }} {{ fields }} - {{ if favicons_mask_icon }}{{ /if }} - {{ if favicons_apple_touch_icon }}{{ /if }} - {{ if favicons_icon_16 }}{{ /if }} - {{ if favicons_icon_32 }}{{ /if }} + {{ if favicons_mask_icon }} + + {{ /if }} + {{ if favicons_apple_touch_icon }}{{ /if }} + {{ if favicons_icon_16 }}{{ /if }} + {{ if favicons_icon_32 }}{{ /if }} {{ /fields }} {{ /if }} diff --git a/src/Tags/MetatagsTags.php b/src/Tags/MetatagsTags.php index 79088eb..ce4665d 100644 --- a/src/Tags/MetatagsTags.php +++ b/src/Tags/MetatagsTags.php @@ -7,7 +7,6 @@ use Gioppy\StatamicMetatags\DefaultMetatags; use Gioppy\StatamicMetatags\Settings; use Illuminate\Support\Str; -use Statamic\Facades\Asset; use Statamic\Fields\Value; use Statamic\Support\Arr; use Statamic\Tags\Tags; @@ -22,7 +21,7 @@ public function index() { $settingsMeta = $settings->onlyMeta(); $settingsDefault = $settings->excludedMeta(); - $defaultValues = Arr::removeNullValues(DefaultMetatags::make()->values()); + $defaultValues = Arr::removeNullValues(DefaultMetatags::make()->augmented()); $page = collect($this->context->get('page')); @@ -43,39 +42,7 @@ public function index() { // Merge default values with page metatags $fields = collect($defaultValues) ->filter() - ->merge($pageFields) - ->map(function ($field, $key) { - // Filed with media - if (Str::endsWith($key, ['image', 'video', 'audio'])) { - // Default value, it seems that media value from default settings will be a simple string array... - if (is_array($field) && array_key_exists(0, $field)) { - return collect($field)->map(function ($item) { - return is_array($item) ? - new Value($item) : - new Value(Asset::findByUrl($this->assetContainerPath($item))); - })->toArray(); - } - - // ...otherwise, value from field inside a collection is an Asset! - if (is_array($field) && array_key_exists('path', $field)) { - return new Value($field); - } - - return new Value(Asset::findByUrl($this->assetContainerPath($field))); - } - - // Field with single option - if (is_array($field) && array_key_exists('value', $field) && !is_null($field['value'])) { - return new Value($field['value']); - } - - // Field with multiple options - if (is_array($field) && Arr::isList($field)) { - return new Value($field); - } - - return new Value($field); - }); + ->merge($pageFields); // Check if fields have title, otherwise load default title if (!array_key_exists('basic_title', $fields->all())) { @@ -93,10 +60,4 @@ public function index() { ] ]); } - - private function assetContainerPath($asset) { - $settings = Settings::make(); - $containerPath = $settings->excludedMeta()['image_asset_container']; - return "$containerPath/$asset"; - } }