Preserve legacy front matter visibility keys

Author: samdark

docs/content.md

@@ -141,11 +141,12 @@ extra:
 - **aliases** — old URLs for this entry. YiiPress writes redirect pages from each alias to the entry permalink, or to `redirect_to` when the entry is itself a redirect. Aliases are site-root paths, must be unique across aliases and generated page permalinks, and are not added to feeds, listings, or sitemap
 - **layout** — template layout name (default: collection-specific or `entry`)
 - **theme** — theme name for this entry; overrides the site-level default (see [Templates](templates.md))
+- **show_title** — set to `false` to suppress the generated entry `<h1>` while retaining the title for metadata and navigation. The legacy `showTitle` key remains supported; `show_title` wins when both are present
 - **weight** — integer for custom sorting in non-blog collections (lower = first)
 - **language** — language code for multilingual content (e.g., `en`, `ru`)
 - **redirect_to** — URL to redirect to; generates a redirect HTML page (with `<meta http-equiv="refresh">`, JS `window.location.replace()`, and `<link rel="canonical">`) instead of rendering content. Redirect entries are excluded from feeds, sitemaps, listings, archives, and taxonomy pages. Root-relative targets such as `/new-url/` are YiiPress site-root paths; when `base_url` includes a deployment path, YiiPress prefixes that path in the generated redirect target. Absolute URLs are emitted unchanged
 - **previous**, **next** — per-entry pager overrides. Omit a direction to inherit its link from sidebar navigation when `navigation_pager` is enabled for the collection, set it to `false` to hide that direction, or provide both `text` and `link` for a custom link. Custom links also work when collection navigation paging is disabled. Relative and root-relative internal links and absolute HTTP(S) links are accepted; unsafe URL schemes fail the build. Root-relative links are rendered relative to the output page, so they work when `base_url` contains a deployment subdirectory
-- **edit_link** — set to `false` to hide the edit-page action for this entry. When omitted or `true`, the entry inherits the site-wide `edit_page` configuration. This does not affect the report-issue action
+- **edit_link** — set to `false` to hide the edit-page action for this entry. When omitted or `true`, the entry inherits the site-wide `edit_page` configuration. The legacy `editLink` key remains supported; `edit_link` wins when both are present. This does not affect the report-issue action
 - **last_updated** — set to `true` or `false` to override last-updated output for this entry. When omitted, the entry inherits the site-wide `last_updated` setting. Disabled entries do not read the source modification time
 - **extra** — arbitrary key-value pairs accessible in templates
 

src/Build/EntryRenderer.php

@@ -126,6 +126,8 @@ private function cacheContext(
         ?CrossReferenceResolver $crossRefResolver,
         ?array $navigationPager,
     ): string {
+        $lastUpdated = $this->isLastUpdatedEnabled($siteConfig, $entry);
+
         return hash('xxh128', serialize([
             'siteConfig' => $siteConfig,
             'permalink' => $permalink,
@@ -135,7 +137,7 @@ private function cacheContext(
             'crossReferences' => $crossRefResolver?->signature() ?? '',
             'related' => $this->relatedIndex?->signature() ?? '',
             'translations' => $this->translationIndex?->signature() ?? '',
-            'lastUpdated' => $lastUpdated = $this->isLastUpdatedEnabled($siteConfig, $entry),
+            'lastUpdated' => $lastUpdated,
             'lastUpdatedMtime' => $lastUpdated ? filemtime($entry->sourceFilePath()) : null,
         ]));
     }

src/Content/Parser/EntryParser.php

@@ -104,7 +104,7 @@ public function parse(string $filePath, string $collectionName, string $language
             bodyLength: $result['bodyLength'],
             image: (string) ($fields['image'] ?? ''),
             translationKey: (string) ($fields['translation_key'] ?? ''),
-            showTitle: (bool) ($fields['show_title'] ?? true),
+            showTitle: (bool) ($fields['show_title'] ?? $fields['showTitle'] ?? true),
             aliases: isset($fields['aliases']) && is_array($fields['aliases'])
                 ? array_values(array_map(strval(...), $fields['aliases']))
                 : [],
@@ -120,22 +120,29 @@ public function parse(string $filePath, string $collectionName, string $language
      */
     private function parseEditLink(array $fields, string $filePath): ?bool
     {
-        return $this->parseBooleanOverride($fields, 'edit_link', $filePath);
+        return $this->parseBooleanOverride($fields, 'edit_link', $filePath, 'editLink');
     }
 
     /**
      * @param array<string, mixed> $fields
      */
-    private function parseBooleanOverride(array $fields, string $name, string $filePath): ?bool
-    {
+    private function parseBooleanOverride(
+        array $fields,
+        string $name,
+        string $filePath,
+        ?string $legacyName = null,
+    ): ?bool {
+        if (!array_key_exists($name, $fields) && $legacyName !== null && array_key_exists($legacyName, $fields)) {
+            $fields[$name] = $fields[$legacyName];
+        }
         if (!array_key_exists($name, $fields) || $fields[$name] === null) {
             return null;
         }
         if (!is_bool($fields[$name])) {
             throw new InvalidContentConfigException(
                 "Invalid \"$name\" visibility override in front matter: $filePath",
                 $filePath,
-                "Omit \"$name\" to inherit it, or set it to true or false.",
+                "Omit \"$name\" or set it to null to inherit it, or set it to true or false.",
             );
         }
 

tests/Unit/Content/Parser/EntryParserTest.php

@@ -122,6 +122,34 @@ public function testEditLinkVisibilityIsInheritedWhenOmitted(): void
         }
     }
 
+    public function testLegacyEditLinkVisibilityRemainsSupported(): void
+    {
+        $file = tempnam(sys_get_temp_dir(), 'yiipress-entry-edit-link-legacy-');
+        file_put_contents($file, "---\ntitle: Generated\neditLink: false\n---\n\nBody.\n");
+
+        try {
+            $entry = $this->parser->parse($file, 'docs');
+
+            assertFalse($entry->editLink);
+        } finally {
+            unlink($file);
+        }
+    }
+
+    public function testSnakeCaseEditLinkTakesPrecedenceOverLegacyKey(): void
+    {
+        $file = tempnam(sys_get_temp_dir(), 'yiipress-entry-edit-link-precedence-');
+        file_put_contents($file, "---\ntitle: Generated\neditLink: false\nedit_link: true\n---\n\nBody.\n");
+
+        try {
+            $entry = $this->parser->parse($file, 'docs');
+
+            assertTrue($entry->editLink);
+        } finally {
+            unlink($file);
+        }
+    }
+
     public function testParsesLastUpdatedVisibilityOverride(): void
     {
         $file = tempnam(sys_get_temp_dir(), 'yiipress-entry-last-updated-');
@@ -345,4 +373,23 @@ public function testParsesTopLevelShowTitle(): void
             unlink($filePath);
         }
     }
+
+    public function testLegacyShowTitleRemainsSupportedWithSnakeCasePrecedence(): void
+    {
+        $legacyFile = tempnam(sys_get_temp_dir(), 'yiipress-entry-show-title-legacy-');
+        $precedenceFile = tempnam(sys_get_temp_dir(), 'yiipress-entry-show-title-precedence-');
+        file_put_contents($legacyFile, "---\ntitle: Legacy\nshowTitle: false\n---\n\nBody.\n");
+        file_put_contents(
+            $precedenceFile,
+            "---\ntitle: Precedence\nshowTitle: false\nshow_title: true\n---\n\nBody.\n",
+        );
+
+        try {
+            assertFalse($this->parser->parse($legacyFile, 'docs')->showTitle);
+            assertTrue($this->parser->parse($precedenceFile, 'docs')->showTitle);
+        } finally {
+            unlink($legacyFile);
+            unlink($precedenceFile);
+        }
+    }
 }