volosied commented on code in PR #338:
URL: https://github.com/apache/myfaces/pull/338#discussion_r992929555
##########
impl/src/main/java/org/apache/myfaces/application/ViewIdSupport.java:
##########
@@ -482,14 +482,7 @@ protected String handleSuffixMapping(FacesContext context,
String requestViewId)
}
}
- // Otherwise, if a physical resource exists with the name
requestViewId let that value be viewId.
- if (isViewExistent(context,requestViewId))
- {
- return requestViewId;
- }
-
- //Otherwise return null.
- return null;
+ return requestViewId;
Review Comment:
Physical page doesn't exist, so it doesn't make sense to check if the view
exists. Hope this doesn't cause any errors, but the build was successful. I'll
see what the TCK says.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscr...@myfaces.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
