JdbcCollectionDao.add(): use DTO instead of entity as argument.

Addressed to #120

No functional changes.

Author: php-coder

src/main/java/ru/mystamps/web/dao/JdbcCollectionDao.java

@@ -17,11 +17,11 @@
  */
 package ru.mystamps.web.dao;
 
-import ru.mystamps.web.entity.Collection;
+import ru.mystamps.web.dao.dto.AddCollectionDbDto;
 import ru.mystamps.web.service.dto.LinkEntityDto;
 
 public interface JdbcCollectionDao {
 	Iterable<LinkEntityDto> findLastCreated(int quantity);
 	long countCollectionsOfUsers();
-	Integer add(Collection collection);
+	Integer add(AddCollectionDbDto collection);
 }

src/main/java/ru/mystamps/web/dao/dto/AddCollectionDbDto.java

@@ -0,0 +1,30 @@
+/*
+ * Copyright (C) 2009-2015 Slava Semushin <[email protected]>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
+ */
+package ru.mystamps.web.dao.dto;
+
+import lombok.Getter;
+import lombok.Setter;
+import lombok.ToString;
+
+@Getter
+@Setter
+@ToString
+public class AddCollectionDbDto {
+	private Integer ownerId;
+	private String slug;
+}

src/main/java/ru/mystamps/web/dao/impl/JdbcCollectionDaoImpl.java

@@ -32,7 +32,7 @@
 import lombok.RequiredArgsConstructor;
 
 import ru.mystamps.web.dao.JdbcCollectionDao;
-import ru.mystamps.web.entity.Collection;
+import ru.mystamps.web.dao.dto.AddCollectionDbDto;
 import ru.mystamps.web.service.dto.LinkEntityDto;
 
 @RequiredArgsConstructor
@@ -68,9 +68,9 @@ public long countCollectionsOfUsers() {
 	}
 
 	@Override
-	public Integer add(Collection collection) {
+	public Integer add(AddCollectionDbDto collection) {
 		Map<String, Object> params = new HashMap<>();
-		params.put("user_id", collection.getOwner().getId());
+		params.put("user_id", collection.getOwnerId());
 		params.put("slug", collection.getSlug());
 
 		KeyHolder holder = new GeneratedKeyHolder();

src/main/java/ru/mystamps/web/service/CollectionServiceImpl.java

@@ -29,6 +29,7 @@
 
 import ru.mystamps.web.dao.CollectionDao;
 import ru.mystamps.web.dao.JdbcCollectionDao;
+import ru.mystamps.web.dao.dto.AddCollectionDbDto;
 import ru.mystamps.web.entity.Collection;
 import ru.mystamps.web.entity.Series;
 import ru.mystamps.web.entity.User;
@@ -47,17 +48,16 @@ public class CollectionServiceImpl implements CollectionService {
 	public void createCollection(User user) {
 		Validate.isTrue(user != null, "User must be non null");
 
-		Collection collection = new Collection();
-		collection.setOwner(user);
+		AddCollectionDbDto collection = new AddCollectionDbDto();
+		collection.setOwnerId(user.getId());
 
 		String slug = SlugUtils.slugify(user.getLogin());
 		Validate.isTrue(slug != null, "Slug for string '%s' is null", user.getLogin());
 		collection.setSlug(slug);
 
-		Integer collectionId = jdbcCollectionDao.add(collection);
-		collection.setId(collectionId);
+		Integer id = jdbcCollectionDao.add(collection);
 
-		LOG.info("Collection has been created ({})", collection);
+		LOG.info("Collection #{} has been created ({})", id, collection);
 	}
 
 	@Override