Force refresh on content/template change and various fixes

Cargo.lock

@@ -1,6 +1,6 @@
 [root]
 name = "gutenberg"
-version = "0.1.0"
+version = "0.0.1"
 dependencies = [
  "chrono 0.3.0 (registry+https://github.com/rust-lang/crates.io-index)",
  "clap 2.20.5 (registry+https://github.com/rust-lang/crates.io-index)",
@@ -774,7 +774,7 @@ dependencies = [
 [[package]]
 name = "tera"
 version = "0.8.0"
-source = "git+https://github.com/Keats/tera?branch=reload#31f8bd8238e3e6bdef9ecf4468079c535609b5eb"
+source = "git+https://github.com/Keats/tera?branch=reload#ee038a6f3519ac35e1878ca9b29ec739a8f84a15"
 dependencies = [
  "chrono 0.3.0 (registry+https://github.com/rust-lang/crates.io-index)",
  "error-chain 0.10.0 (registry+https://github.com/rust-lang/crates.io-index)",

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "gutenberg"
-version = "0.1.0"
+version = "0.0.1"
 authors = ["Vincent Prouillet <vincent@wearewizards.io>"]
 license = "MIT"
 readme = "README.md"

src/cmd/serve.rs

@@ -119,11 +119,13 @@ pub fn serve(interface: &str, port: &str) -> Result<()> {
                         match detect_change_kind(&pwd, &path) {
                             (ChangeKind::Content, _) => {
                                 println!("-> Content changed {}", path.display());
-                                rebuild_done_handling(&broadcaster, site.rebuild(), "");
+                                // Force refresh
+                                rebuild_done_handling(&broadcaster, site.rebuild_after_content_change(), "/x.js");
                             },
                             (ChangeKind::Templates, _) => {
                                 println!("-> Template changed {}", path.display());
-                                rebuild_done_handling(&broadcaster, site.rebuild_after_template_change(), "");
+                                // Force refresh
+                                rebuild_done_handling(&broadcaster, site.rebuild_after_template_change(), "/x.js");
                             },
                             (ChangeKind::StaticFiles, p) => {
                                 println!("-> Static file changes detected {}", path.display());

src/front_matter.rs

@@ -36,9 +36,9 @@ pub struct FrontMatter {
     pub draft: Option<bool>,
     /// Only one category allowed
     pub category: Option<String>,
-    /// Optional layout, if we want to specify which tpl to render for that page
+    /// Optional template, if we want to specify which template to render for that page
     #[serde(skip_serializing)]
-    pub layout: Option<String>,
+    pub template: Option<String>,
     /// Any extra parameter present in the front matter
     pub extra: Option<HashMap<String, Value>>,
 }
@@ -69,6 +69,7 @@ impl FrontMatter {
         Ok(f)
     }
 
+    /// Converts the date in the front matter, which can be in 2 formats, into a NaiveDateTime
     pub fn parse_date(&self) -> Option<NaiveDateTime> {
         match self.date {
             Some(ref d) => {

src/main.rs

@@ -87,6 +87,9 @@ fn main() {
                 Ok(()) => (),
                 Err(e) => {
                     println!("Error: {}", e);
+                    for e in e.iter().skip(1) {
+                        println!("Reason: {}", e)
+                    }
                     ::std::process::exit(1);
                 },
             };

src/page.rs

@@ -182,22 +182,20 @@ impl Page {
         Page::parse(&path.strip_prefix("content").unwrap().to_string_lossy(), &content, config)
     }
 
-    fn get_layout_name(&self) -> String {
-        match self.meta.layout {
-            Some(ref l) => l.to_string(),
-            None => "page.html".to_string()
-        }
-    }
-
     /// Renders the page using the default layout, unless specified in front-matter
     pub fn render_html(&self, tera: &Tera, config: &Config) -> Result<String> {
-        let tpl = self.get_layout_name();
+        let tpl_name = match self.meta.template {
+            Some(ref l) => l.to_string(),
+            None => "page.html".to_string()
+        };
+        // TODO: create a helper to create context to ensure all contexts
+        // have the same names
         let mut context = Context::new();
-        context.add("site", config);
+        context.add("config", config);
         context.add("page", self);
 
-        tera.render(&tpl, &context)
+        tera.render(&tpl_name, &context)
-            .chain_err(|| "Error while rendering template")
+            .chain_err(|| format!("Failed to render page '{}'", self.filename))
     }
 }
 
@@ -225,12 +223,10 @@ impl ser::Serialize for Page {
 impl PartialOrd for Page {
     fn partial_cmp(&self, other: &Page) -> Option<Ordering> {
         if self.meta.date.is_none() {
-            println!("No self data");
             return Some(Ordering::Less);
         }
 
         if other.meta.date.is_none() {
-            println!("No other date");
             return Some(Ordering::Greater);
         }
 
@@ -337,7 +333,6 @@ Hello world"#;
         assert!(res.is_ok());
         let page = res.unwrap();
         assert_eq!(page.url, "posts/intro/hello-world");
-        println!("{}", page.permalink);
         assert_eq!(page.permalink, format!("{}{}", conf.base_url, "/posts/intro/hello-world"));
     }
 

src/site.rs

@@ -149,16 +149,13 @@ impl Site {
         Ok(())
     }
 
-    /// Re-parse and re-generate the site
+    pub fn rebuild_after_content_change(&mut self) -> Result<()> {
-    /// Very dumb for now, ideally it would only rebuild what changed
-    pub fn rebuild(&mut self) -> Result<()> {
         self.parse_site()?;
         self.build()
     }
 
     pub fn rebuild_after_template_change(&mut self) -> Result<()> {
         self.templates.full_reload()?;
-        println!("Reloaded templates");
         self.build_pages()
     }