toxblh/podcasts
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

EpisodeWidget: Remove Widget::set/get name calls.

Browse Source 
I don't even remember why this was there.
This commit is contained in:
Jordan Petridis 2018-04-12 00:13:43 +03:00
parent 8d4fdb8ece
commit 67bdd3664a
No known key found for this signature in database
GPG Key ID: CEABAD9F5683B9A6
1 changed files with 3 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
hammond-gtk/src/widgets/episode.rs
View File

@ -87,8 +87,6 @@ impl EpisodeWidget {
} }
fn init(&mut self, episode: EpisodeWidgetQuery, sender: Sender<Action>) { fn init(&mut self, episode: EpisodeWidgetQuery, sender: Sender<Action>) {
WidgetExt::set_name(&self.container, &episode.rowid().to_string());
// Set the date label. // Set the date label.
self.set_date(episode.epoch()); self.set_date(episode.epoch());
@ -99,7 +97,7 @@ impl EpisodeWidget {
self.set_duration(episode.duration()); self.set_duration(episode.duration());
// Determine what the state of the media widgets should be. // Determine what the state of the media widgets should be.
if let Err(err) = determine_media_state(self.media.clone(), &episode, &self.container) { if let Err(err) = determine_media_state(self.media.clone(), &episode) {
error!("Something went wrong determining the Media State."); error!("Something went wrong determining the Media State.");
error!("Error: {}", err); error!("Error: {}", err);
} }
@ -120,7 +118,6 @@ impl EpisodeWidget {
})); }));
let media_machine = self.media.clone(); let media_machine = self.media.clone();
let parent = self.container.clone();
media.download_connect_clicked(clone!(media_machine, episode, sender => move |dl| { media.download_connect_clicked(clone!(media_machine, episode, sender => move |dl| {
dl.set_sensitive(false); dl.set_sensitive(false);
if let Ok(ep) = episode.lock() { if let Ok(ep) = episode.lock() {
@ -129,8 +126,7 @@ impl EpisodeWidget {
error!("Error: {}", err); error!("Error: {}", err);
} else { } else {
info!("Donwload started succesfully."); info!("Donwload started succesfully.");
let line_limit = determine_media_state(media_machine.clone(), &ep, &parent); if let Err(err) = determine_media_state(media_machine.clone(), &ep) {
if let Err(err) = line_limit {
error!("Something went wrong determining the Media State."); error!("Something went wrong determining the Media State.");
error!("Error: {}", err); error!("Error: {}", err);
} }
@ -165,12 +161,8 @@ impl EpisodeWidget {
fn determine_media_state( fn determine_media_state(
media_machine: Arc<Mutex<MediaMachine>>, media_machine: Arc<Mutex<MediaMachine>>,
episode: &EpisodeWidgetQuery, episode: &EpisodeWidgetQuery,
parent: &gtk::Box,
) -> Result<(), Error> { ) -> Result<(), Error> {
let id = WidgetExt::get_name(parent) let id = episode.rowid();
.ok_or_else(|| format_err!("Failed to get widget Name"))?
.parse::<i32>()?;
let active_dl = || -> Result<Option<_>, Error> { let active_dl = || -> Result<Option<_>, Error> {
let m = manager::ACTIVE_DOWNLOADS let m = manager::ACTIVE_DOWNLOADS
.read() .read()