Clear TODO and FIXME that are no longer releavant.

This commit is contained in:
Jordan Petridis 2017-12-30 20:54:23 +02:00
parent 4894683924
commit 5730e71e2a
No known key found for this signature in database
GPG Key ID: CEABAD9F5683B9A6
7 changed files with 5 additions and 12 deletions

View File

@ -99,7 +99,7 @@ $ tree -d
│   ├── resources # GResources folder │   ├── resources # GResources folder
│   │   └── gtk # Contains the glade.ui files. │   │   └── gtk # Contains the glade.ui files.
│   └── src │   └── src
│   ├── views # Currently only contains the Podcasts_view. │   ├── views # Currently only contains the Empty and Episodes views.
│   └── widgets # Contains custom widgets such as Podcast and Episode. │   └── widgets # Contains custom widgets such as Podcast and Episode.
``` ```

View File

@ -612,7 +612,6 @@ impl<'a> Source {
let etag = headers.get::<ETag>(); let etag = headers.get::<ETag>();
let lmod = headers.get::<LastModified>(); let lmod = headers.get::<LastModified>();
// FIXME: This dsnt work most of the time apparently
if self.http_etag() != etag.map(|x| x.tag()) || self.last_modified != lmod.map(|x| { if self.http_etag() != etag.map(|x| x.tag()) || self.last_modified != lmod.map(|x| {
format!("{}", x) format!("{}", x)
}) { }) {
@ -659,9 +658,6 @@ impl<'a> Source {
} }
} }
// FIXME: I have fucked up somewhere here.
// Getting back 200 codes even though I supposedly sent etags.
// info!("Headers: {:?}", headers);
let client = reqwest::Client::builder().referer(false).build()?; let client = reqwest::Client::builder().referer(false).build()?;
let mut req = client.get(self.uri()).headers(headers).send()?; let mut req = client.get(self.uri()).headers(headers).send()?;

View File

@ -8,7 +8,6 @@ use utils::replace_extra_spaces;
use errors::*; use errors::*;
// TODO: Extend the support for parsing itunes extensions
/// Parses a `rss::Channel` into a `NewPodcast` Struct. /// Parses a `rss::Channel` into a `NewPodcast` Struct.
pub(crate) fn new_podcast(chan: &Channel, source_id: i32) -> NewPodcast { pub(crate) fn new_podcast(chan: &Channel, source_id: i32) -> NewPodcast {
let title = chan.title().trim(); let title = chan.title().trim();

View File

@ -64,6 +64,7 @@ impl App {
} }
pub fn setup_actions(&self) { pub fn setup_actions(&self) {
// Updates the database and refreshes every view.
let update = gio::SimpleAction::new("update", None); let update = gio::SimpleAction::new("update", None);
let content = self.content.clone(); let content = self.content.clone();
let header = self.header.clone(); let header = self.header.clone();
@ -72,6 +73,7 @@ impl App {
}); });
self.app_instance.add_action(&update); self.app_instance.add_action(&update);
// Refreshes the `Content`
let refresh = gio::SimpleAction::new("refresh", None); let refresh = gio::SimpleAction::new("refresh", None);
let content = self.content.clone(); let content = self.content.clone();
refresh.connect_activate(move |_, _| { refresh.connect_activate(move |_, _| {
@ -79,6 +81,7 @@ impl App {
}); });
self.app_instance.add_action(&refresh); self.app_instance.add_action(&refresh);
// Refreshes the `EpisodesStack`
let refresh_episodes = gio::SimpleAction::new("refresh_episodes", None); let refresh_episodes = gio::SimpleAction::new("refresh_episodes", None);
let content = self.content.clone(); let content = self.content.clone();
refresh_episodes.connect_activate(move |_, _| { refresh_episodes.connect_activate(move |_, _| {
@ -88,6 +91,7 @@ impl App {
}); });
self.app_instance.add_action(&refresh_episodes); self.app_instance.add_action(&refresh_episodes);
// Refreshes the `ShowStack`
let refresh_shows = gio::SimpleAction::new("refresh_shows", None); let refresh_shows = gio::SimpleAction::new("refresh_shows", None);
let content = self.content.clone(); let content = self.content.clone();
refresh_shows.connect_activate(move |_, _| { refresh_shows.connect_activate(move |_, _| {
@ -143,7 +147,6 @@ impl App {
fn build_ui(window: &gtk::Window, app: &gtk::Application) { fn build_ui(window: &gtk::Window, app: &gtk::Application) {
window.set_application(app); window.set_application(app);
window.show_all(); window.show_all();
window.activate(); window.activate();
app.connect_activate(move |_| ()); app.connect_activate(move |_| ());

View File

@ -73,9 +73,6 @@ impl Header {
let header = Rc::new(self.clone()); let header = Rc::new(self.clone());
add_button.connect_clicked(clone!(content, header, add_popover, new_url => move |_| { add_button.connect_clicked(clone!(content, header, add_popover, new_url => move |_| {
on_add_bttn_clicked(content.clone(), header.clone(), &new_url); on_add_bttn_clicked(content.clone(), header.clone(), &new_url);
// TODO: lock the button instead of hiding and add notification of feed added.
// TODO: map the spinner
add_popover.hide(); add_popover.hide();
})); }));

View File

@ -193,7 +193,6 @@ impl EpisodesViewWidget {
let container: gtk::Box = builder.get_object("container").unwrap(); let container: gtk::Box = builder.get_object("container").unwrap();
let image: gtk::Image = builder.get_object("cover").unwrap(); let image: gtk::Image = builder.get_object("cover").unwrap();
// FIXME:
if let Ok(pd) = dbqueries::get_podcast_cover_from_id(episode.podcast_id()) { if let Ok(pd) = dbqueries::get_podcast_cover_from_id(episode.podcast_id()) {
let img = get_pixbuf_from_path(&pd, 64); let img = get_pixbuf_from_path(&pd, 64);
if let Some(i) = img { if let Some(i) = img {

View File

@ -44,7 +44,6 @@ impl ShowsPopulated {
use gtk::WidgetExt; use gtk::WidgetExt;
// TODO: handle unwraps. // TODO: handle unwraps.
// TODO: implement back button.
self.flowbox self.flowbox
.connect_child_activated(clone!(show => move |_, child| { .connect_child_activated(clone!(show => move |_, child| {
// This is such an ugly hack... // This is such an ugly hack...