Merge pull request #12 from teppokoivula/master

Fix page URL trailing slashes when urlSegmentsStr is set but current page does not use URL segments
Fokke- · Sep 21, 2023 · 552f74e · 552f74e
2 parents f5e172e + 8da1318
commit 552f74e
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/MarkupMetadata.module.php b/MarkupMetadata.module.php
@@ -31,7 +31,7 @@ class MarkupMetadata extends WireData implements Module, ConfigurableModule {
   public static function getModuleInfo() : array {
     return [
       'title' => 'Markup Metadata',
-      'version' => 121,
+      'version' => 122,
       'summary' => 'Set and render meta tags for head section.',
       'author' => 'Ville Fokke Saarivaara',
       'singular' => true,
@@ -125,7 +125,8 @@ private function getPageUrl (?\ProcessWire\Language $language = null) : ?string
     // Add trailing slash if required
     if (
       ($this->page->template->urlSegments === 1 && $this->page->template->slashUrlSegments === 1 && $this->input->urlSegmentStr) ||
-      ($this->page->template->slashUrls === 1 && !$this->input->urlSegmentStr)
+      ($this->page->template->slashUrls === 1 && !$this->input->urlSegmentStr) ||
+      ($this->page->template->slashUrls === 1 && $this->page->template->urlSegments === 0 && $this->input->urlSegmentStr !== '')
     ) {
       $url .= '/';
     }