Browse Source

修改测试发现的问题

ying.rong 9 years ago
parent
commit
e70e8ff95a

+ 6 - 9
src/main/java/com/cloudcross/ssp/model/mapper/dspBanner.sql.xml

@@ -26,13 +26,16 @@
 	
 	<update id="update" parameterType="java.util.List">
 	 	<foreach collection="dspBannerList" item="item" index="index" open="" close="" separator=";">
-	        update t_dsp_banner set path = #{item.path},height = #{item.height},width = #{item.width},
-	        rheight = #{item.rheight}, rwidth = #{item.rwidth}, state = #{item.state},
+	        update t_dsp_banner set advertiser_id = #{item.advertiserId}, path = #{item.path}, height = #{item.height}, 
+	        width = #{item.width},rheight = #{item.rheight}, rwidth = #{item.rwidth},
+	        <if test="item.state != null and item.state != '' ">
+	        	state = #{item.state},
+	       	</if>
 	        <if test="item.memo != null and item.memo != '' ">
 	        	memo = #{item.memo},
 	       	</if>
 	        updated = NOW()
-           	where id=#{item.id} and dsp_id = #{item.dspId} and advertiser_id = #{item.advertiserId}
+           	where id=#{item.id} and dsp_id = #{item.dspId}
     	</foreach>
 	</update>
 	
@@ -55,9 +58,6 @@
 		<if test="dspId != null and dspId != '' ">
 	    	and dsp_id = #{dspId}
 	    </if>
-	    <if test="advertiserId != null and advertiserId != '' ">
-	    	and advertiser_id = #{advertiserId}
-	    </if>
 	</select>
 	
 	<select id="selectMaxId" parameterType="java.util.Map" resultType="java.lang.Long">
@@ -76,9 +76,6 @@
 		<if test="dspId != null and dspId != '' ">
 	    	and dsp_id = #{dspId}
 	    </if>
-	    <if test="advertiserId != null and advertiserId != '' ">
-	    	and advertiser_id = #{advertiserId}
-	    </if>
 	</select>
 	
 	<select id="countByParams" parameterType="java.util.Map" resultType="int">

+ 1 - 1
src/main/java/com/cloudcross/ssp/web/dsp/DspAdvertiserController.java

@@ -162,7 +162,7 @@ public class DspAdvertiserController extends SimpleController{
 						if(obj.containsKey("advertiserIds") && obj.get("advertiserIds") != null 
 							&& !"".equals(obj.get("advertiserIds"))){
 							List<Long> list = new ArrayList<Long>();
-							String[] ids = paramMap.get("advertiserIds").toString().split(",");
+							String[] ids = obj.get("advertiserIds").toString().split(",");
 							for (int j = 0, m = ids.length; j < m; j++) {
 								list.add(Long.valueOf(ids[j]));
 							}

+ 3 - 3
src/main/java/com/cloudcross/ssp/web/dsp/DspBannerController.java

@@ -143,7 +143,7 @@ public class DspBannerController extends SimpleController{
 						if(obj.containsKey("advertiserIds") && obj.get("advertiserIds") != null 
 							&& !"".equals(obj.get("advertiserIds"))){
 							List<Long> list = new ArrayList<Long>();
-							String[] ids = paramMap.get("advertiserIds").toString().split(",");
+							String[] ids = obj.get("advertiserIds").toString().split(",");
 							for (int j = 0, m = ids.length; j < m; j++) {
 								list.add(Long.valueOf(ids[j]));
 							}
@@ -188,7 +188,7 @@ public class DspBannerController extends SimpleController{
 						if(obj.containsKey("bannerIds") && obj.get("bannerIds") != null 
 							&& !"".equals(obj.get("bannerIds"))){
 							List<Long> list = new ArrayList<Long>();
-							String[] ids = paramMap.get("bannerIds").toString().split(",");
+							String[] ids = obj.get("bannerIds").toString().split(",");
 							for (int j = 0, m = ids.length; j < m; j++) {
 								list.add(Long.valueOf(ids[j]));
 							}
@@ -233,7 +233,7 @@ public class DspBannerController extends SimpleController{
 						if(obj.containsKey("bannerIds") && obj.get("bannerIds") != null 
 							&& !"".equals(obj.get("bannerIds"))){
 							List<Long> list = new ArrayList<Long>();
-							String[] ids = paramMap.get("bannerIds").toString().split(",");
+							String[] ids = obj.get("bannerIds").toString().split(",");
 							for (int j = 0, m = ids.length; j < m; j++) {
 								list.add(Long.valueOf(ids[j]));
 							}