Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug: feed error if all entries created by web #39

Open
Eliot00 opened this issue Dec 24, 2021 · 1 comment
Open

Bug: feed error if all entries created by web #39

Eliot00 opened this issue Dec 24, 2021 · 1 comment

Comments

@Eliot00
Copy link
Contributor

Eliot00 commented Dec 24, 2021

Describe the bug

If all entries are created by webapp, it wouldn't create entries.csv file,feed will pass it:

let table = entry.path().join(EntryPaths::entries_csv());
if !table.exists() {
continue;
}

this will cause a runtime error:

fs::write(temp_file, map)?;
meili_exec::feed_documents(&conf.search_url, &entry_type)?;
meili_exec::feed_settings(&conf.search_url, &define)?;
info!("done '{:}' feed", &entry_type);
}
fs::remove_file(temp_file)?;

remove temp file after loop, but it hasn't the tmp file beacuse all entry_type has not entries.csv

Steps to reproduce (please include code)

Environment

  • quake version:
  • Rust version:
  • OS: [e.g. OSX 10.13.4, Windows 10]
@phodal
Copy link
Owner

phodal commented Dec 24, 2021

哦,当时这么设计的原因是,担心找的是非 entry 目录。

应该遍历 defines 生成新的路径

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants