fixing unnecessary casting
This commit is contained in:
parent
af9974600b
commit
eff14e8a8b
1 changed files with 32 additions and 55 deletions
|
@ -295,21 +295,20 @@ pub mod song_model {
|
||||||
match result {
|
match result {
|
||||||
Ok(_i) => {
|
Ok(_i) => {
|
||||||
debug!(_i, index);
|
debug!(_i, index);
|
||||||
for song in self
|
if let Some(song) =
|
||||||
.as_mut()
|
self.as_mut().songs_mut().get_mut(index)
|
||||||
.songs_mut()
|
|
||||||
.iter_mut()
|
|
||||||
.filter(|x| x.id == song_id)
|
|
||||||
{
|
{
|
||||||
debug!(?song);
|
debug!(?song);
|
||||||
song.title = updated_title.to_string();
|
song.title = updated_title.to_string();
|
||||||
}
|
|
||||||
self.as_mut().emit_data_changed(
|
self.as_mut().emit_data_changed(
|
||||||
&model_index,
|
&model_index,
|
||||||
&model_index,
|
&model_index,
|
||||||
&vector_roles,
|
&vector_roles,
|
||||||
);
|
);
|
||||||
true
|
true
|
||||||
|
} else {
|
||||||
|
false
|
||||||
|
}
|
||||||
}
|
}
|
||||||
Err(_e) => false,
|
Err(_e) => false,
|
||||||
}
|
}
|
||||||
|
@ -329,10 +328,8 @@ pub mod song_model {
|
||||||
.execute(db);
|
.execute(db);
|
||||||
match result {
|
match result {
|
||||||
Ok(_i) => {
|
Ok(_i) => {
|
||||||
if let Some(song) = self
|
if let Some(song) =
|
||||||
.as_mut()
|
self.as_mut().songs_mut().get_mut(index)
|
||||||
.songs_mut()
|
|
||||||
.get_mut(index as usize)
|
|
||||||
{
|
{
|
||||||
song.lyrics = updated_lyrics.to_string();
|
song.lyrics = updated_lyrics.to_string();
|
||||||
self.as_mut().emit_data_changed(
|
self.as_mut().emit_data_changed(
|
||||||
|
@ -364,10 +361,8 @@ pub mod song_model {
|
||||||
.execute(db);
|
.execute(db);
|
||||||
match result {
|
match result {
|
||||||
Ok(_i) => {
|
Ok(_i) => {
|
||||||
if let Some(song) = self
|
if let Some(song) =
|
||||||
.as_mut()
|
self.as_mut().songs_mut().get_mut(index)
|
||||||
.songs_mut()
|
|
||||||
.get_mut(index as usize)
|
|
||||||
{
|
{
|
||||||
song.author = updated_author.to_string();
|
song.author = updated_author.to_string();
|
||||||
self.as_mut().emit_data_changed(
|
self.as_mut().emit_data_changed(
|
||||||
|
@ -399,10 +394,8 @@ pub mod song_model {
|
||||||
.execute(db);
|
.execute(db);
|
||||||
match result {
|
match result {
|
||||||
Ok(_i) => {
|
Ok(_i) => {
|
||||||
if let Some(song) = self
|
if let Some(song) =
|
||||||
.as_mut()
|
self.as_mut().songs_mut().get_mut(index)
|
||||||
.songs_mut()
|
|
||||||
.get_mut(index as usize)
|
|
||||||
{
|
{
|
||||||
song.audio = updated_audio.to_string();
|
song.audio = updated_audio.to_string();
|
||||||
self.as_mut().emit_data_changed(
|
self.as_mut().emit_data_changed(
|
||||||
|
@ -434,10 +427,8 @@ pub mod song_model {
|
||||||
.execute(db);
|
.execute(db);
|
||||||
match result {
|
match result {
|
||||||
Ok(_i) => {
|
Ok(_i) => {
|
||||||
if let Some(song) = self
|
if let Some(song) =
|
||||||
.as_mut()
|
self.as_mut().songs_mut().get_mut(index)
|
||||||
.songs_mut()
|
|
||||||
.get_mut(index as usize)
|
|
||||||
{
|
{
|
||||||
song.ccli = updated_ccli.to_string();
|
song.ccli = updated_ccli.to_string();
|
||||||
self.as_mut().emit_data_changed(
|
self.as_mut().emit_data_changed(
|
||||||
|
@ -470,10 +461,8 @@ pub mod song_model {
|
||||||
.execute(db);
|
.execute(db);
|
||||||
match result {
|
match result {
|
||||||
Ok(_i) => {
|
Ok(_i) => {
|
||||||
if let Some(song) = self
|
if let Some(song) =
|
||||||
.as_mut()
|
self.as_mut().songs_mut().get_mut(index)
|
||||||
.songs_mut()
|
|
||||||
.get_mut(index as usize)
|
|
||||||
{
|
{
|
||||||
song.verse_order =
|
song.verse_order =
|
||||||
updated_verse_order.to_string();
|
updated_verse_order.to_string();
|
||||||
|
@ -507,10 +496,8 @@ pub mod song_model {
|
||||||
.execute(db);
|
.execute(db);
|
||||||
match result {
|
match result {
|
||||||
Ok(_i) => {
|
Ok(_i) => {
|
||||||
if let Some(song) = self
|
if let Some(song) =
|
||||||
.as_mut()
|
self.as_mut().songs_mut().get_mut(index)
|
||||||
.songs_mut()
|
|
||||||
.get_mut(index as usize)
|
|
||||||
{
|
{
|
||||||
song.background =
|
song.background =
|
||||||
updated_background.to_string();
|
updated_background.to_string();
|
||||||
|
@ -553,10 +540,8 @@ pub mod song_model {
|
||||||
.execute(db);
|
.execute(db);
|
||||||
match result {
|
match result {
|
||||||
Ok(_i) => {
|
Ok(_i) => {
|
||||||
if let Some(song) = self
|
if let Some(song) =
|
||||||
.as_mut()
|
self.as_mut().songs_mut().get_mut(index)
|
||||||
.songs_mut()
|
|
||||||
.get_mut(index as usize)
|
|
||||||
{
|
{
|
||||||
song.background_type =
|
song.background_type =
|
||||||
updated_background_type.to_string();
|
updated_background_type.to_string();
|
||||||
|
@ -594,10 +579,8 @@ pub mod song_model {
|
||||||
.execute(db);
|
.execute(db);
|
||||||
match result {
|
match result {
|
||||||
Ok(_i) => {
|
Ok(_i) => {
|
||||||
if let Some(song) = self
|
if let Some(song) =
|
||||||
.as_mut()
|
self.as_mut().songs_mut().get_mut(index)
|
||||||
.songs_mut()
|
|
||||||
.get_mut(index as usize)
|
|
||||||
{
|
{
|
||||||
song.horizontal_text_alignment =
|
song.horizontal_text_alignment =
|
||||||
updated_horizontal_text_alignment
|
updated_horizontal_text_alignment
|
||||||
|
@ -637,10 +620,8 @@ pub mod song_model {
|
||||||
.execute(db);
|
.execute(db);
|
||||||
match result {
|
match result {
|
||||||
Ok(_i) => {
|
Ok(_i) => {
|
||||||
if let Some(song) = self
|
if let Some(song) =
|
||||||
.as_mut()
|
self.as_mut().songs_mut().get_mut(index)
|
||||||
.songs_mut()
|
|
||||||
.get_mut(index as usize)
|
|
||||||
{
|
{
|
||||||
song.vertical_text_alignment =
|
song.vertical_text_alignment =
|
||||||
updated_vertical_text_alignment
|
updated_vertical_text_alignment
|
||||||
|
@ -674,10 +655,8 @@ pub mod song_model {
|
||||||
.execute(db);
|
.execute(db);
|
||||||
match result {
|
match result {
|
||||||
Ok(_i) => {
|
Ok(_i) => {
|
||||||
if let Some(song) = self
|
if let Some(song) =
|
||||||
.as_mut()
|
self.as_mut().songs_mut().get_mut(index)
|
||||||
.songs_mut()
|
|
||||||
.get_mut(index as usize)
|
|
||||||
{
|
{
|
||||||
song.font = updated_font.to_string();
|
song.font = updated_font.to_string();
|
||||||
debug!(?updated_font);
|
debug!(?updated_font);
|
||||||
|
@ -711,10 +690,8 @@ pub mod song_model {
|
||||||
.execute(db);
|
.execute(db);
|
||||||
match result {
|
match result {
|
||||||
Ok(_i) => {
|
Ok(_i) => {
|
||||||
if let Some(song) = self
|
if let Some(song) =
|
||||||
.as_mut()
|
self.as_mut().songs_mut().get_mut(index)
|
||||||
.songs_mut()
|
|
||||||
.get_mut(index as usize)
|
|
||||||
{
|
{
|
||||||
song.font_size = updated_font_size;
|
song.font_size = updated_font_size;
|
||||||
self.as_mut().emit_data_changed(
|
self.as_mut().emit_data_changed(
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue