Skip to content

Commit bbfef58

Browse files
committed
fix var name, fix embedded assets check
1 parent 9e447e0 commit bbfef58

File tree

2 files changed

+10
-4
lines changed

2 files changed

+10
-4
lines changed

cmd/embedded.go

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -174,6 +174,12 @@ func runViewDo(c *cli.Context) error {
174174
return err
175175
}
176176

177+
if len(assets) == 0 {
178+
return fmt.Errorf("no files matched the given pattern")
179+
} else if len(assets) > 1 {
180+
return fmt.Errorf("too many files matched the given pattern, try to be more specific")
181+
}
182+
177183
data, err := assets[0].Section.AssetFS.ReadFile(assets[0].Name)
178184
if err != nil {
179185
return fmt.Errorf("%s: %w", assets[0].Path, err)
@@ -288,7 +294,7 @@ func buildAssetList(sec *section, globs []glob.Glob, c *cli.Context) []asset {
288294
files, err := sec.AssetFS.ListFiles(".", true)
289295
if err != nil {
290296
log.Error("Error listing files in %q: %v", sec.Path, err)
291-
return results
297+
return nil
292298
}
293299
for _, name := range files {
294300
if sec.Path == "public" &&

modules/templates/mailer.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -68,13 +68,13 @@ func Mailer(ctx context.Context) (*texttmpl.Template, *template.Template) {
6868
}
6969

7070
for _, assetPath := range assetPaths {
71-
content, providerName, err := assetFS.ReadLayeredFile(assetPath)
71+
content, layerName, err := assetFS.ReadLayeredFile(assetPath)
7272
if err != nil {
73-
log.Warn("Failed to read mail template %s: %v (by %s)", assetPath, err, providerName)
73+
log.Warn("Failed to read mail template %s: %v (by %s)", assetPath, err, layerName)
7474
continue
7575
}
7676
tmplName := strings.TrimPrefix(strings.TrimSuffix(assetPath, ".tmpl"), "mail/")
77-
log.Debug("Adding mail template %s: %s (by %s)", tmplName, assetPath, providerName)
77+
log.Debug("Adding mail template %s: %s (by %s)", tmplName, assetPath, layerName)
7878
buildSubjectBodyTemplate(subjectTemplates, bodyTemplates, tmplName, content)
7979
}
8080
}

0 commit comments

Comments
 (0)