Change User input error logging to warn from error

yeastrc · Nov 17, 2019 · f88235d · f88235d
1 parent fcde7c7
commit f88235d
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 9 deletions.
diff --git a/...l_web_app/src/main/java/org/yeastrc/xlink/www/actions/ViewMergedSearchPeptidesAction.java b/...l_web_app/src/main/java/org/yeastrc/xlink/www/actions/ViewMergedSearchPeptidesAction.java
@@ -90,12 +90,12 @@ public ActionForward execute( ActionMapping mapping,
 			Collections.sort( projectSearchIdsListDeduppedSorted );
 			List<Integer> projectIdsFromSearchIds = ProjectIdsForProjectSearchIdsSearcher.getInstance().getProjectIdsForProjectSearchIds( projectSearchIdsSet );
 			if ( projectIdsFromSearchIds.isEmpty() ) {
-				// should never happen
-				String msg = "No project ids for search ids: ";
+				// 
+				String msg = "No project ids for projectSearchIdsFromForm: ";
 				for ( int projectSearchId : projectSearchIdsFromForm ) {
 					msg += projectSearchId + ", ";
 				}
-				log.error( msg );
+				log.warn( msg );
 				return mapping.findForward( StrutsGlobalForwardNames.INVALID_REQUEST_DATA );
 			}
 			if ( projectIdsFromSearchIds.size() > 1 ) {

diff --git a/proxl_web_app/src/main/java/org/yeastrc/xlink/www/actions/ViewSearchProteinsAction.java b/proxl_web_app/src/main/java/org/yeastrc/xlink/www/actions/ViewSearchProteinsAction.java
@@ -76,9 +76,9 @@ public ActionForward execute( ActionMapping mapping,
 			projectSearchIdsSet.add( projectSearchId );
 			List<Integer> projectIdsFromSearchIds = ProjectIdsForProjectSearchIdsSearcher.getInstance().getProjectIdsForProjectSearchIds( projectSearchIdsSet );
 			if ( projectIdsFromSearchIds.isEmpty() ) {
-				// should never happen
-				String msg = "No project ids for search id: " + projectSearchId;
-				log.error( msg );
+				// 
+				String msg = "No project ids for projectSearchId: " + projectSearchId;
+				log.warn( msg );
 				return mapping.findForward( StrutsGlobalForwardNames.INVALID_REQUEST_DATA );
 			}
 			if ( projectIdsFromSearchIds.size() > 1 ) {

diff --git a/proxl_web_app/src/main/java/org/yeastrc/xlink/www/actions/ViewSearchProteinsAllAction.java b/proxl_web_app/src/main/java/org/yeastrc/xlink/www/actions/ViewSearchProteinsAllAction.java
@@ -67,9 +67,9 @@ public ActionForward execute( ActionMapping mapping,
 			projectSearchIdsSet.add( projectSearchId );
 			List<Integer> projectIdsFromSearchIds = ProjectIdsForProjectSearchIdsSearcher.getInstance().getProjectIdsForProjectSearchIds( projectSearchIdsSet );
 			if ( projectIdsFromSearchIds.isEmpty() ) {
-				// should never happen
-				String msg = "No project ids for search id: " + projectSearchId;
-				log.error( msg );
+				// 
+				String msg = "No project ids for projectSearchId: " + projectSearchId;
+				log.warn( msg );
 				return mapping.findForward( StrutsGlobalForwardNames.INVALID_REQUEST_DATA );
 			}
 			if ( projectIdsFromSearchIds.size() > 1 ) {