diff options
author | Colin Seymour <lildood@gmail.com> | 2018-01-11 16:42:49 +0000 |
---|---|---|
committer | Bjørn Erik Pedersen <bjorn.erik.pedersen@gmail.com> | 2018-01-28 11:24:31 +0100 |
commit | 3752348ef13ced8f6f528b42ee7d76a12a97ae5c (patch) | |
tree | a7227c1a3af89f2516095ea96c35f86c3acc7504 /commands/import_jekyll.go | |
parent | 4eb2fec67c3a72a3ac98aa834dc56fd4504626d8 (diff) |
Only set 'url' if permalink in metadata and remove duplicate confirm msg
The current behaviour addes a `url` attribute to the frontmatter of all posts imported from Jeklly and assumes the desired permalink structure is /:year/:month/:day/:title/. This may be the case for most peeps, but poses a problem for those that don't use this permalink structure as the `url` attribute takes precedence over the `permalink` attribute in the site-wide configuration meaning it can't be overruled.
This changes the behaviour to only set the `url` attribute if the `permalink` attribute is set in the Jekyll frontmatter.
The duplication of the confirmation message is also removed.
Tests have been updated to reflect this change in behaviour.
Fixes #1887
Diffstat (limited to 'commands/import_jekyll.go')
-rw-r--r-- | commands/import_jekyll.go | 10 |
1 files changed, 1 insertions, 9 deletions
diff --git a/commands/import_jekyll.go b/commands/import_jekyll.go index 327bf6095..8c9c4d093 100644 --- a/commands/import_jekyll.go +++ b/commands/import_jekyll.go @@ -144,11 +144,6 @@ func importFromJekyll(cmd *cobra.Command, args []string) error { "$ git clone https://github.com/spf13/herring-cove.git " + args[1] + "/themes/herring-cove") jww.FEEDBACK.Println("$ cd " + args[1] + "\n$ hugo server --theme=herring-cove") - jww.FEEDBACK.Println("Congratulations!", fileCount, "post(s) imported!") - jww.FEEDBACK.Println("Now, start Hugo by yourself:\n" + - "$ git clone https://github.com/spf13/herring-cove.git " + args[1] + "/themes/herring-cove") - jww.FEEDBACK.Println("$ cd " + args[1] + "\n$ hugo server --theme=herring-cove") - return nil } @@ -478,8 +473,6 @@ func convertJekyllPost(s *hugolib.Site, path, relPath, targetDir string, draft b } func convertJekyllMetaData(m interface{}, postName string, postDate time.Time, draft bool) (interface{}, error) { - url := postDate.Format("/2006/01/02/") + postName + "/" - metadata, err := cast.ToStringMapE(m) if err != nil { return nil, err @@ -497,7 +490,7 @@ func convertJekyllMetaData(m interface{}, postName string, postDate time.Time, d delete(metadata, key) case "permalink": if str, ok := value.(string); ok { - url = str + metadata["url"] = str } delete(metadata, key) case "category": @@ -526,7 +519,6 @@ func convertJekyllMetaData(m interface{}, postName string, postDate time.Time, d } - metadata["url"] = url metadata["date"] = postDate.Format(time.RFC3339) return metadata, nil |