diff --git a/libresonic-main/src/main/java/org/libresonic/player/dao/AlbumDao.java b/libresonic-main/src/main/java/org/libresonic/player/dao/AlbumDao.java index 71515156..71cffec2 100644 --- a/libresonic-main/src/main/java/org/libresonic/player/dao/AlbumDao.java +++ b/libresonic-main/src/main/java/org/libresonic/player/dao/AlbumDao.java @@ -29,7 +29,6 @@ import java.util.Map; import org.apache.commons.lang.ObjectUtils; import org.springframework.jdbc.core.RowMapper; -import org.springframework.jdbc.core.simple.ParameterizedRowMapper; import org.libresonic.player.domain.Album; import org.libresonic.player.domain.MediaFile; @@ -346,7 +345,7 @@ public class AlbumDao extends AbstractDao { return queryForDate("select created from starred_album where album_id=? and username=?", null, albumId, username); } - private static class AlbumMapper implements ParameterizedRowMapper { + private static class AlbumMapper implements RowMapper { public Album mapRow(ResultSet rs, int rowNum) throws SQLException { return new Album( rs.getInt(1), diff --git a/libresonic-main/src/main/java/org/libresonic/player/dao/ArtistDao.java b/libresonic-main/src/main/java/org/libresonic/player/dao/ArtistDao.java index ea98c9c3..f7b44367 100644 --- a/libresonic-main/src/main/java/org/libresonic/player/dao/ArtistDao.java +++ b/libresonic-main/src/main/java/org/libresonic/player/dao/ArtistDao.java @@ -24,7 +24,6 @@ import org.libresonic.player.domain.Artist; import org.libresonic.player.domain.MusicFolder; import org.springframework.jdbc.core.RowMapper; -import org.springframework.jdbc.core.simple.ParameterizedRowMapper; import java.sql.ResultSet; import java.sql.SQLException; @@ -199,7 +198,7 @@ public class ArtistDao extends AbstractDao { return queryForDate("select created from starred_artist where artist_id=? and username=?", null, artistId, username); } - private static class ArtistMapper implements ParameterizedRowMapper { + private static class ArtistMapper implements RowMapper { public Artist mapRow(ResultSet rs, int rowNum) throws SQLException { return new Artist( rs.getInt(1), diff --git a/libresonic-main/src/main/java/org/libresonic/player/dao/AvatarDao.java b/libresonic-main/src/main/java/org/libresonic/player/dao/AvatarDao.java index b571ffc2..118816ce 100644 --- a/libresonic-main/src/main/java/org/libresonic/player/dao/AvatarDao.java +++ b/libresonic-main/src/main/java/org/libresonic/player/dao/AvatarDao.java @@ -20,7 +20,7 @@ package org.libresonic.player.dao; import org.libresonic.player.domain.Avatar; -import org.springframework.jdbc.core.simple.ParameterizedRowMapper; +import org.springframework.jdbc.core.RowMapper; import java.sql.ResultSet; import java.sql.SQLException; @@ -85,11 +85,11 @@ public class AvatarDao extends AbstractDao { } } - private static class AvatarRowMapper implements ParameterizedRowMapper { + private static class AvatarRowMapper implements RowMapper { public Avatar mapRow(ResultSet rs, int rowNum) throws SQLException { return new Avatar(rs.getInt(1), rs.getString(2), rs.getTimestamp(3), rs.getString(4), rs.getInt(5), rs.getInt(6), rs.getBytes(7)); } } -} \ No newline at end of file +} diff --git a/libresonic-main/src/main/java/org/libresonic/player/dao/BookmarkDao.java b/libresonic-main/src/main/java/org/libresonic/player/dao/BookmarkDao.java index de7a976f..1ea9ac44 100644 --- a/libresonic-main/src/main/java/org/libresonic/player/dao/BookmarkDao.java +++ b/libresonic-main/src/main/java/org/libresonic/player/dao/BookmarkDao.java @@ -19,12 +19,12 @@ */ package org.libresonic.player.dao; +import org.springframework.jdbc.core.RowMapper; + import java.sql.ResultSet; import java.sql.SQLException; import java.util.List; -import org.springframework.jdbc.core.simple.ParameterizedRowMapper; - import org.libresonic.player.domain.Bookmark; /** @@ -81,7 +81,7 @@ public class BookmarkDao extends AbstractDao { update("delete from bookmark where username=? and media_file_id=?", username, mediaFileId); } - private static class BookmarkRowMapper implements ParameterizedRowMapper { + private static class BookmarkRowMapper implements RowMapper { public Bookmark mapRow(ResultSet rs, int rowNum) throws SQLException { return new Bookmark(rs.getInt(1), rs.getInt(2), rs.getLong(3), rs.getString(4), rs.getString(5), rs.getTimestamp(6), rs.getTimestamp(7)); diff --git a/libresonic-main/src/main/java/org/libresonic/player/dao/InternetRadioDao.java b/libresonic-main/src/main/java/org/libresonic/player/dao/InternetRadioDao.java index b1a1d4d4..385e4a90 100644 --- a/libresonic-main/src/main/java/org/libresonic/player/dao/InternetRadioDao.java +++ b/libresonic-main/src/main/java/org/libresonic/player/dao/InternetRadioDao.java @@ -19,12 +19,12 @@ */ package org.libresonic.player.dao; +import org.springframework.jdbc.core.RowMapper; + import java.sql.ResultSet; import java.sql.SQLException; import java.util.List; -import org.springframework.jdbc.core.simple.ParameterizedRowMapper; - import org.libresonic.player.Logger; import org.libresonic.player.domain.InternetRadio; @@ -81,7 +81,7 @@ public class InternetRadioDao extends AbstractDao { update(sql, radio.getName(), radio.getStreamUrl(), radio.getHomepageUrl(), radio.isEnabled(), radio.getChanged(), radio.getId()); } - private static class InternetRadioRowMapper implements ParameterizedRowMapper { + private static class InternetRadioRowMapper implements RowMapper { public InternetRadio mapRow(ResultSet rs, int rowNum) throws SQLException { return new InternetRadio(rs.getInt(1), rs.getString(2), rs.getString(3), rs.getString(4), rs.getBoolean(5), rs.getTimestamp(6)); } diff --git a/libresonic-main/src/main/java/org/libresonic/player/dao/MediaFileDao.java b/libresonic-main/src/main/java/org/libresonic/player/dao/MediaFileDao.java index 25968340..78dd0b47 100644 --- a/libresonic-main/src/main/java/org/libresonic/player/dao/MediaFileDao.java +++ b/libresonic-main/src/main/java/org/libresonic/player/dao/MediaFileDao.java @@ -19,22 +19,15 @@ */ package org.libresonic.player.dao; -import java.sql.ResultSet; -import java.sql.SQLException; -import java.util.Arrays; -import java.util.Collections; -import java.util.Date; -import java.util.HashMap; -import java.util.List; -import java.util.Map; - import org.apache.commons.lang.StringUtils; -import org.springframework.jdbc.core.RowMapper; -import org.springframework.jdbc.core.simple.ParameterizedRowMapper; - import org.libresonic.player.domain.Genre; import org.libresonic.player.domain.MediaFile; import org.libresonic.player.domain.MusicFolder; +import org.springframework.jdbc.core.RowMapper; + +import java.sql.ResultSet; +import java.sql.SQLException; +import java.util.*; import static org.libresonic.player.domain.MediaFile.MediaType; import static org.libresonic.player.domain.MediaFile.MediaType.*; @@ -568,7 +561,7 @@ public class MediaFileDao extends AbstractDao { update("checkpoint"); } - private static class MediaFileMapper implements ParameterizedRowMapper { + private static class MediaFileMapper implements RowMapper { public MediaFile mapRow(ResultSet rs, int rowNum) throws SQLException { return new MediaFile( rs.getInt(1), @@ -604,7 +597,7 @@ public class MediaFileDao extends AbstractDao { } } - private static class MusicFileInfoMapper implements ParameterizedRowMapper { + private static class MusicFileInfoMapper implements RowMapper { public MediaFile mapRow(ResultSet rs, int rowNum) throws SQLException { MediaFile file = new MediaFile(); file.setPlayCount(rs.getInt(1)); @@ -614,7 +607,7 @@ public class MediaFileDao extends AbstractDao { } } - private static class GenreMapper implements ParameterizedRowMapper { + private static class GenreMapper implements RowMapper { public Genre mapRow(ResultSet rs, int rowNum) throws SQLException { return new Genre(rs.getString(1), rs.getInt(2), rs.getInt(3)); } diff --git a/libresonic-main/src/main/java/org/libresonic/player/dao/MusicFolderDao.java b/libresonic-main/src/main/java/org/libresonic/player/dao/MusicFolderDao.java index ca997b45..49dcdc3e 100644 --- a/libresonic-main/src/main/java/org/libresonic/player/dao/MusicFolderDao.java +++ b/libresonic-main/src/main/java/org/libresonic/player/dao/MusicFolderDao.java @@ -19,16 +19,15 @@ */ package org.libresonic.player.dao; +import org.libresonic.player.Logger; +import org.libresonic.player.domain.MusicFolder; +import org.springframework.jdbc.core.RowMapper; + import java.io.File; import java.sql.ResultSet; import java.sql.SQLException; import java.util.List; -import org.springframework.jdbc.core.simple.ParameterizedRowMapper; - -import org.libresonic.player.Logger; -import org.libresonic.player.domain.MusicFolder; - /** * Provides database services for music folders. * @@ -99,7 +98,7 @@ public class MusicFolderDao extends AbstractDao { } } - private static class MusicFolderRowMapper implements ParameterizedRowMapper { + private static class MusicFolderRowMapper implements RowMapper { public MusicFolder mapRow(ResultSet rs, int rowNum) throws SQLException { return new MusicFolder(rs.getInt(1), new File(rs.getString(2)), rs.getString(3), rs.getBoolean(4), rs.getTimestamp(5)); } diff --git a/libresonic-main/src/main/java/org/libresonic/player/dao/PlayQueueDao.java b/libresonic-main/src/main/java/org/libresonic/player/dao/PlayQueueDao.java index f56bd232..a098c43b 100644 --- a/libresonic-main/src/main/java/org/libresonic/player/dao/PlayQueueDao.java +++ b/libresonic-main/src/main/java/org/libresonic/player/dao/PlayQueueDao.java @@ -23,7 +23,6 @@ import java.sql.SQLException; import java.util.List; import org.springframework.jdbc.core.RowMapper; -import org.springframework.jdbc.core.simple.ParameterizedRowMapper; import org.libresonic.player.domain.SavedPlayQueue; @@ -60,7 +59,7 @@ public class PlayQueueDao extends AbstractDao { } } - private static class PlayQueueMapper implements ParameterizedRowMapper { + private static class PlayQueueMapper implements RowMapper { public SavedPlayQueue mapRow(ResultSet rs, int rowNum) throws SQLException { return new SavedPlayQueue(rs.getInt(1), rs.getString(2), diff --git a/libresonic-main/src/main/java/org/libresonic/player/dao/PlayerDao.java b/libresonic-main/src/main/java/org/libresonic/player/dao/PlayerDao.java index be855e61..4530fd35 100644 --- a/libresonic-main/src/main/java/org/libresonic/player/dao/PlayerDao.java +++ b/libresonic-main/src/main/java/org/libresonic/player/dao/PlayerDao.java @@ -19,22 +19,13 @@ */ package org.libresonic.player.dao; +import org.libresonic.player.Logger; +import org.libresonic.player.domain.*; +import org.springframework.jdbc.core.RowMapper; + import java.sql.ResultSet; import java.sql.SQLException; -import java.util.Calendar; -import java.util.Collections; -import java.util.HashMap; -import java.util.List; -import java.util.Map; - -import org.springframework.jdbc.core.simple.ParameterizedRowMapper; - -import org.libresonic.player.Logger; -import org.libresonic.player.domain.CoverArtScheme; -import org.libresonic.player.domain.Player; -import org.libresonic.player.domain.PlayerTechnology; -import org.libresonic.player.domain.PlayQueue; -import org.libresonic.player.domain.TranscodeScheme; +import java.util.*; /** * Provides player-related database services. @@ -95,7 +86,7 @@ public class PlayerDao extends AbstractDao { * @param player The player to create. */ public synchronized void createPlayer(Player player) { - int id = getJdbcTemplate().queryForInt("select max(id) from player") + 1; + int id = getJdbcTemplate().queryForObject("select max(id) from player", Integer.class) + 1; player.setId(String.valueOf(id)); String sql = "insert into player (" + COLUMNS + ") values (" + questionMarks(COLUMNS) + ")"; update(sql, player.getId(), player.getName(), player.getType(), player.getUsername(), @@ -169,7 +160,7 @@ public class PlayerDao extends AbstractDao { player.setPlayQueue(playQueue); } - private class PlayerRowMapper implements ParameterizedRowMapper { + private class PlayerRowMapper implements RowMapper { public Player mapRow(ResultSet rs, int rowNum) throws SQLException { Player player = new Player(); int col = 1; diff --git a/libresonic-main/src/main/java/org/libresonic/player/dao/PlaylistDao.java b/libresonic-main/src/main/java/org/libresonic/player/dao/PlaylistDao.java index 996d84eb..e41e3781 100644 --- a/libresonic-main/src/main/java/org/libresonic/player/dao/PlaylistDao.java +++ b/libresonic-main/src/main/java/org/libresonic/player/dao/PlaylistDao.java @@ -23,7 +23,6 @@ import org.libresonic.player.Logger; import org.libresonic.player.domain.MediaFile; import org.libresonic.player.domain.Playlist; import org.springframework.jdbc.core.RowMapper; -import org.springframework.jdbc.core.simple.ParameterizedRowMapper; import java.sql.ResultSet; import java.sql.SQLException; @@ -125,7 +124,7 @@ public class PlaylistDao extends AbstractDao { new Date(), playlist.getImportedFrom(), playlist.getId()); } - private static class PlaylistMapper implements ParameterizedRowMapper { + private static class PlaylistMapper implements RowMapper { public Playlist mapRow(ResultSet rs, int rowNum) throws SQLException { return new Playlist( rs.getInt(1), diff --git a/libresonic-main/src/main/java/org/libresonic/player/dao/PodcastDao.java b/libresonic-main/src/main/java/org/libresonic/player/dao/PodcastDao.java index ca84b044..b61d870b 100644 --- a/libresonic-main/src/main/java/org/libresonic/player/dao/PodcastDao.java +++ b/libresonic-main/src/main/java/org/libresonic/player/dao/PodcastDao.java @@ -24,7 +24,6 @@ import java.sql.SQLException; import java.util.List; import org.springframework.jdbc.core.RowMapper; -import org.springframework.jdbc.core.simple.ParameterizedRowMapper; import org.libresonic.player.domain.PodcastChannel; import org.libresonic.player.domain.PodcastEpisode; @@ -55,7 +54,7 @@ public class PodcastDao extends AbstractDao { update(sql, null, channel.getUrl(), channel.getTitle(), channel.getDescription(), channel.getImageUrl(), channel.getStatus().name(), channel.getErrorMessage()); - return getJdbcTemplate().queryForInt("select max(id) from podcast_channel"); + return getJdbcTemplate().queryForObject("select max(id) from podcast_channel", Integer.class); } /** @@ -182,7 +181,7 @@ public class PodcastDao extends AbstractDao { } } - private static class PodcastEpisodeRowMapper implements ParameterizedRowMapper { + private static class PodcastEpisodeRowMapper implements RowMapper { public PodcastEpisode mapRow(ResultSet rs, int rowNum) throws SQLException { return new PodcastEpisode(rs.getInt(1), rs.getInt(2), rs.getString(3), rs.getString(4), rs.getString(5), rs.getString(6), rs.getTimestamp(7), rs.getString(8), (Long) rs.getObject(9), diff --git a/libresonic-main/src/main/java/org/libresonic/player/dao/ShareDao.java b/libresonic-main/src/main/java/org/libresonic/player/dao/ShareDao.java index 15ca4fa3..702e4965 100644 --- a/libresonic-main/src/main/java/org/libresonic/player/dao/ShareDao.java +++ b/libresonic-main/src/main/java/org/libresonic/player/dao/ShareDao.java @@ -26,7 +26,7 @@ import java.util.HashMap; import java.util.List; import java.util.Map; -import org.springframework.jdbc.core.simple.ParameterizedRowMapper; +import org.springframework.jdbc.core.RowMapper; import org.libresonic.player.domain.MusicFolder; import org.libresonic.player.domain.Share; @@ -53,7 +53,7 @@ public class ShareDao extends AbstractDao { update(sql, null, share.getName(), share.getDescription(), share.getUsername(), share.getCreated(), share.getExpires(), share.getLastVisited(), share.getVisitCount()); - int id = getJdbcTemplate().queryForInt("select max(id) from share"); + int id = getJdbcTemplate().queryForObject("select max(id) from share", Integer.class); share.setId(id); } @@ -129,14 +129,14 @@ public class ShareDao extends AbstractDao { update("delete from share where id=?", id); } - private static class ShareRowMapper implements ParameterizedRowMapper { + private static class ShareRowMapper implements RowMapper { public Share mapRow(ResultSet rs, int rowNum) throws SQLException { return new Share(rs.getInt(1), rs.getString(2), rs.getString(3), rs.getString(4), rs.getTimestamp(5), rs.getTimestamp(6), rs.getTimestamp(7), rs.getInt(8)); } } - private static class ShareFileRowMapper implements ParameterizedRowMapper { + private static class ShareFileRowMapper implements RowMapper { public String mapRow(ResultSet rs, int rowNum) throws SQLException { return rs.getString(1); } diff --git a/libresonic-main/src/main/java/org/libresonic/player/dao/TranscodingDao.java b/libresonic-main/src/main/java/org/libresonic/player/dao/TranscodingDao.java index bfd43b1f..5ca8a3e9 100644 --- a/libresonic-main/src/main/java/org/libresonic/player/dao/TranscodingDao.java +++ b/libresonic-main/src/main/java/org/libresonic/player/dao/TranscodingDao.java @@ -23,7 +23,7 @@ import java.sql.ResultSet; import java.sql.SQLException; import java.util.List; -import org.springframework.jdbc.core.simple.ParameterizedRowMapper; +import org.springframework.jdbc.core.RowMapper; import org.libresonic.player.Logger; import org.libresonic.player.domain.Transcoding; @@ -82,7 +82,7 @@ public class TranscodingDao extends AbstractDao { * @param transcoding The transcoding to create. */ public synchronized void createTranscoding(Transcoding transcoding) { - int id = getJdbcTemplate().queryForInt("select max(id) + 1 from transcoding2"); + int id = getJdbcTemplate().queryForObject("select max(id) + 1 from transcoding2", Integer.class); transcoding.setId(id); String sql = "insert into transcoding2 (" + COLUMNS + ") values (" + questionMarks(COLUMNS) + ")"; update(sql, transcoding.getId(), transcoding.getName(), transcoding.getSourceFormats(), @@ -115,7 +115,7 @@ public class TranscodingDao extends AbstractDao { transcoding.getStep3(), transcoding.isDefaultActive(), transcoding.getId()); } - private static class TranscodingRowMapper implements ParameterizedRowMapper { + private static class TranscodingRowMapper implements RowMapper { public Transcoding mapRow(ResultSet rs, int rowNum) throws SQLException { return new Transcoding(rs.getInt(1), rs.getString(2), rs.getString(3), rs.getString(4), rs.getString(5), rs.getString(6), rs.getString(7), rs.getBoolean(8)); diff --git a/libresonic-main/src/main/java/org/libresonic/player/dao/UserDao.java b/libresonic-main/src/main/java/org/libresonic/player/dao/UserDao.java index 8af3216e..abd1ac2e 100644 --- a/libresonic-main/src/main/java/org/libresonic/player/dao/UserDao.java +++ b/libresonic-main/src/main/java/org/libresonic/player/dao/UserDao.java @@ -19,20 +19,15 @@ */ package org.libresonic.player.dao; +import org.libresonic.player.Logger; +import org.libresonic.player.domain.*; +import org.libresonic.player.util.StringUtil; +import org.springframework.jdbc.core.RowMapper; + import java.sql.ResultSet; import java.sql.SQLException; import java.util.List; -import org.springframework.jdbc.core.simple.ParameterizedRowMapper; - -import org.libresonic.player.Logger; -import org.libresonic.player.domain.AlbumListType; -import org.libresonic.player.domain.AvatarScheme; -import org.libresonic.player.domain.TranscodeScheme; -import org.libresonic.player.domain.User; -import org.libresonic.player.domain.UserSettings; -import org.libresonic.player.util.StringUtil; - /** * Provides user-related database services. * @@ -296,7 +291,7 @@ public class UserDao extends AbstractDao { } } - private class UserRowMapper implements ParameterizedRowMapper { + private class UserRowMapper implements RowMapper { public User mapRow(ResultSet rs, int rowNum) throws SQLException { User user = new User(rs.getString(1), decrypt(rs.getString(2)), rs.getString(3), rs.getBoolean(4), rs.getLong(5), rs.getLong(6), rs.getLong(7)); @@ -305,7 +300,7 @@ public class UserDao extends AbstractDao { } } - private static class UserSettingsRowMapper implements ParameterizedRowMapper { + private static class UserSettingsRowMapper implements RowMapper { public UserSettings mapRow(ResultSet rs, int rowNum) throws SQLException { int col = 1; UserSettings settings = new UserSettings(rs.getString(col++));