Fix - case sensitive file names for razor views

* Failed to generate content for most pipelines on linux * Added exception to Lunr index module ( issue #2 )
alanta · Nov 25, 2020 · 5269404 · 5269404
1 parent 3a23d27
commit 5269404
Show file tree

Hide file tree

Showing 5 changed files with 9 additions and 13 deletions.
diff --git a/.github/workflows/dotnet-core.yml b/.github/workflows/dotnet-core.yml
@@ -9,7 +9,7 @@ on:
 jobs:
   build:
 
-    runs-on: windows-latest
+    runs-on: ubuntu-latest
 
     steps:
     - uses: actions/checkout@v2

diff --git a/Modules/LunrIndexer.cs b/Modules/LunrIndexer.cs
@@ -15,6 +15,11 @@ public class LunrIndexer : Module
 
         protected override async Task<IEnumerable<IDocument>> ExecuteContextAsync(IExecutionContext context)
         {
+            if (context.Inputs.Length == 0)
+            {
+                throw new InvalidOperationException($"No input documents available. Cannot build a search index for pipeline {context.PipelineName}");
+            }
+
             var documents = new Dictionary<string,object>();
 
             Index idx = await Index.Build(async builder =>

diff --git a/Pipelines/Home.cs b/Pipelines/Home.cs
@@ -18,7 +18,7 @@ public Home(SiteSettings site)
                 new ReplaceDocuments(Dependencies.ToArray()),
                 new PaginateDocuments(6),
                 new SetDestination(Config.FromDocument((doc, ctx) => Filename(doc))),
-                new MergeContent(new ReadFiles(patterns: "Index.cshtml")),
+                new MergeContent(new ReadFiles(patterns: "index.cshtml")),
                 new RenderRazor()
                     .WithViewData("Title", "Home")
                     .WithViewData("SiteMetaData", site)

diff --git a/Pipelines/Pages.cs b/Pipelines/Pages.cs
@@ -40,7 +40,7 @@ public Pages(IDeliveryClient deliveryClient, SiteSettings site)
 
             ProcessModules = new ModuleList
             {
-                new MergeContent(new ReadFiles( KontentConfig.Get<Page,string>( ViewForPage))),
+                new MergeContent(new ReadFiles( "page.cshtml")),
                 new RenderRazor()
                     .WithViewData( "SEO", Config.FromDocument((doc, ctx) =>
                     {
@@ -62,15 +62,6 @@ public Pages(IDeliveryClient deliveryClient, SiteSettings site)
                 new WriteFiles(),
             };
         }
-
-        public string ViewForPage(Page page)
-        {
-            return page.System.Codename switch
-            {
-                "contact" => "Contact.cshtml",
-                _ => "Page.cshtml"
-            };
-        }
     }
 }
 

diff --git a/Pipelines/Posts.cs b/Pipelines/Posts.cs
@@ -38,7 +38,7 @@ public Posts(IDeliveryClient deliveryClient, SiteSettings site)
             };
 
             ProcessModules = new ModuleList {
-                new MergeContent(new ReadFiles(patterns: "Post.cshtml") ),
+                new MergeContent(new ReadFiles(patterns: "post.cshtml") ),
                 new RenderRazor()
                     .WithViewData( "SEO", Config.FromDocument((doc, ctx) =>
                     {
-Original file line number
+Diff line change
@@ Expand Up / @@ -9,7 +9,7 @@ on: @@
     jobs:
       build:
-        runs-on: windows-latest
+        runs-on: ubuntu-latest
         steps:
         - uses: actions/checkout@v2
@@ Expand Down @@