matthiasbeyer
/
imag
1
0
Fork 0
Code Releases Activity

Refactor: tmp-content creation, file opening 

tmp-content creation is now done by a helper function, as is the
markdown parsing, the tmp-file creation and the xdg-open call.
This commit is contained in:
Matthias Beyer 2016-01-05 15:48:00 +01:00
parent 34a2b75b89
commit 8abb740c13
1 changed files with 49 additions and 69 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

118
src/module/notes/mod.rs
View File

@ -125,17 +125,6 @@ impl<'a> Notes<'a> {
}
fn command_open(&self, matches: &ArgMatches) -> bool {
use std::io::Write;
use std::process::exit;
use open;
use self::header::get_name_from_header;
use self::header::get_tags_from_header;
use ui::external::get_tempfile;
use module::helpers::content::markdown::MarkdownParser;
let parser = Parser::new(JsonHeaderParser::new(None));
let filter = {
@ -158,67 +147,13 @@ impl<'a> Notes<'a> {
if matches.is_present("onepage") {
let tmpcontent = files.fold(String::new(), |acc, file| {
let heading = {
let name = get_name_from_header(file.deref().borrow().header());
let tagsstr = {
let tags = get_tags_from_header(file.deref().borrow().header());
if tags.len() != 0 {
format!(" <small>(<i>{}</i>)</small>", tags.join(", "))
} else {
format!(" <small>(No Tags)</small>")
}
};
if name.len() == 0 {
format!("<h1>{}</h1>{}", file.deref().borrow().id, tagsstr)
} else {
format!("<h1>{}</h1> <small>({})</small>{}",
name, file.deref().borrow().id(), tagsstr)
}
};
format!("{}\n\n{}\n\n{}", acc, heading, file.deref().borrow().data())
let content = self.preprocess_file_for_markdown(file);
format!("{}\n\n{}", acc, content)
});
let (temppath, mut tempfile) = match get_tempfile("html") {
Some(tpl) => tpl,
None => {
error!("Could not create tempfile");
exit(1);
}
};
let html = MarkdownParser::new(&tmpcontent).to_html_page();
tempfile.write_all(html.as_ref());
open::that(&temppath[..]).is_ok()
self.open_tmpcontent(tmpcontent)
} else {
let result = files.map(|file| {
let (temppath, mut tempfile) = match get_tempfile("html") {
Some(tpl) => tpl,
None => {
error!("Could not create tempfile");
exit(1);
}
};
let tagsstr = {
let tags = get_tags_from_header(file.deref().borrow().header());
if tags.len() != 0 {
format!(" <small>(<i>{}</i>)</small>", tags.join(", "))
} else {
format!(" <small>(No Tags)</small>")
}
};
let content = format!("<h1>{}</h1>{}\n\n{}",
get_name_from_header(file.deref().borrow().header()),
tagsstr,
file.deref().borrow().data());
let html = MarkdownParser::new(&content).to_html_page();
tempfile.write_all(html.as_ref());
open::that(&temppath[..]).is_ok()
self.open_tmpcontent(self.preprocess_file_for_markdown(file))
})
.fold((0, 0), |acc, succeeded| {
let (worked, failed) = acc;
@ -239,6 +174,51 @@ impl<'a> Notes<'a> {
}
fn preprocess_file_for_markdown(&self, file: Rc<RefCell<File>>) -> String {
use self::header::get_name_from_header;
use self::header::get_tags_from_header;
let tagsstr = {
let tags = get_tags_from_header(file.deref().borrow().header());
if tags.len() != 0 {
format!(" <small>(<i>{}</i>)</small>", tags.join(", "))
} else {
format!(" <small>(No Tags)</small>")
}
};
let (name, id) = {
let notename = get_name_from_header(file.deref().borrow().header());
if notename.len() == 0 {
(format!("{}", file.deref().borrow().id()), String::new())
} else {
(notename, format!("{}", file.deref().borrow().id()))
}
};
format!("<h1>{}</h1><small>{}</small>{}\n\n{}", name, id, tagsstr,
file.deref().borrow().data())
}
fn open_tmpcontent(&self, s: String) -> bool {
use std::process::exit;
use std::io::Write;
use open;
use ui::external::get_tempfile;
use module::helpers::content::markdown::MarkdownParser;
let (temppath, mut tempfile) = match get_tempfile("html") {
Some(tpl) => tpl,
None => {
error!("Could not create tempfile");
exit(1);
}
};
tempfile.write_all(MarkdownParser::new(&s).to_html_page().as_ref());
open::that(&temppath[..]).is_ok()
}
fn command_list(&self, matches: &ArgMatches) -> bool {
use ansi_term::Colour::{Red, Green};
use ui::file::{FilePrinter, TablePrinter};