fixed some sonar code smells

Issue-ID: PORTAL-865
Change-Id: I1714739f137a7f5ce5fe1f518217cc3f7ac2a8aa
Signed-off-by: Rupinder <rupinsi1@in.ibm.com>

1 files changed, 6 insertions, 5 deletions

diff --git a/ecomp-sdk/epsdk-app-common/src/main/java/org/onap/portalapp/controller/sample/BroadcastListController.java b/ecomp-sdk/epsdk-app-common/src/main/java/org/onap/portalapp/controller/sample/BroadcastListController.java
index c9c666fb..4e66743e 100644
--- a/ecomp-sdk/epsdk-app-common/src/main/java/org/onap/portalapp/controller/sample/BroadcastListController.java
+++ b/ecomp-sdk/epsdk-app-common/src/main/java/org/onap/portalapp/controller/sample/BroadcastListController.java
@@ -54,7 +54,8 @@ import org.onap.portalsdk.core.web.support.JsonMessage;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.stereotype.Controller;
 import org.springframework.web.bind.annotation.RequestMapping;
-import org.springframework.web.bind.annotation.RequestMethod;
+import org.springframework.web.bind.annotation.GetMapping;
+import org.springframework.web.bind.annotation.PostMapping;
 import org.springframework.web.servlet.ModelAndView;
 
 import com.fasterxml.jackson.databind.DeserializationFeature;
@@ -71,7 +72,7 @@ public class BroadcastListController extends RestrictedBaseController {
 	@Autowired
 	private BroadcastService broadcastService;
 
-	@RequestMapping(value = { "/broadcast_list" }, method = RequestMethod.GET)
+	@GetMapping(value = { "/broadcast_list" })
 	public ModelAndView broadcastList(HttpServletRequest request) {
 		Map<String, Object> model = new HashMap<>();
 
@@ -79,7 +80,7 @@ public class BroadcastListController extends RestrictedBaseController {
 		return new ModelAndView(getViewName(), model);
 	}
 
-	@RequestMapping(value = { "/get_broadcast_list" }, method = RequestMethod.GET)
+	@GetMapping(value = { "/get_broadcast_list" })
 	public void getBroadcast(HttpServletRequest request, HttpServletResponse response) {
 		Map<String, Object> model = new HashMap<>();
 		ObjectMapper mapper = new ObjectMapper();
@@ -96,7 +97,7 @@ public class BroadcastListController extends RestrictedBaseController {
 
 	}
 
-	@RequestMapping(value = { "/broadcast_list/remove" }, method = RequestMethod.POST)
+	@PostMapping(value = { "/broadcast_list/remove" })
 	public ModelAndView remove(HttpServletRequest request, HttpServletResponse response) throws IOException {
 
 		try {
@@ -131,7 +132,7 @@ public class BroadcastListController extends RestrictedBaseController {
 
 	}
 
-	@RequestMapping(value = { "/broadcast_list/toggleActive" }, method = RequestMethod.POST)
+	@PostMapping(value = { "/broadcast_list/toggleActive" })
 	public ModelAndView toggleActive(HttpServletRequest request, HttpServletResponse response) throws IOException {
 
 		try {