markdownify.html attempts to read config.toml

[palant]

Generation failed for me because layouts/partials/utils/markdownify.html attempted to read config.toml which doesn't exist in my case (I use the YAML config format). I'd say this file should use .Site.Params.enableEmoji rather than attempt to parse the config file?

[reuixiy]

Hmmm, I did this because I found that there is a Hugo-defined variable enableEmoji, but it cannot access via Site Variables, and I don’t want to repeat it in Params.

Note that this theme requires a configuration file provided by the theme, which is already in TOML format. Did you just convert it to YAML format? If so, well...

Anyway, readFile seems quite rude indeed. I'll make a commit.

[palant]

Oh, I didn't realize that enableEmoji isn't under params, I see...

Yes, I had an existing YAML config, so I added the settings from your config example there.

[palant]

Unfortunately, .RenderString function is unaffected by this setting - emojis seem to be processed during parsing, not rendering.

[reuixiy]

Yes, neither markdownify nor .RenderString respect enableEmoji, and that's why I added this workaround in layouts/partials/utils/markdownify.html.

Also, I took a look at Issues of Hugo and found gohugoio/hugo#7170.

[palant]

If moving the setting around is too much trouble, maybe you could use readDir to make sure the file actually exists so there is at least no error.

[reuixiy]

@palant Just submitted a commit and it should work.

[reuixiy]

wait a minute...

[reuixiy]

Okay, now it should work!

[palant]

Yes, seems to work, thank you!