Merge pull request #87 from out-of-cheese-error/feat/nested_tags

feat(hierarchy): nested tags create nested folders
out-of-cheese-error · Apr 14, 2021 · f8865c1 · f8865c1
2 parents ced652d + c0e9156
commit f8865c1
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -5,6 +5,10 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/), and this project adheres
 to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## Unreleased
+### Added
+- Nested tag support - "parent/child" tags used with the "Tag" hierarchy create nested folders.
+
 ## [0.8.1] - 2021-03-14
 ### Changed
 - Use local time instead of UTC for search (Issue [#77](https://github.com/out-of-cheese-error/gooseberry/issues/77))

diff --git a/src/gooseberry/knowledge_base.rs b/src/gooseberry/knowledge_base.rs
@@ -206,6 +206,7 @@ impl Gooseberry {
         let mut order_to_annotations = HashMap::new();
         match order {
             OrderBy::Tag => {
+                let path_separator = &std::path::MAIN_SEPARATOR.to_string();
                 for annotation in annotations {
                     if annotation.annotation.tags.is_empty() {
                         order_to_annotations
@@ -214,8 +215,9 @@ impl Gooseberry {
                             .push(annotation);
                     } else {
                         for tag in &annotation.annotation.tags {
+                            let tag = tag.replace("/", path_separator);
                             order_to_annotations
-                                .entry(tag.to_owned())
+                                .entry(tag)
                                 .or_insert_with(Vec::new)
                                 .push(annotation.clone());
                         }
@@ -367,6 +369,11 @@ impl Gooseberry {
                                 .collect::<Result<Vec<String>, _>>()?,
                             raw_annotations: inner_annotations,
                         };
+                        // TODO: check if nested tags work on Windows
+                        // TODO: add tests for nested tags
+                        if let Some(prefix) = path.parent() {
+                            fs::create_dir_all(prefix)?;
+                        }
                         fs::File::create(&path)?
                             .write_all(hbs.render("page", &page_data)?.as_bytes())?;
                     } else {