Skip to content

Commit

Permalink
Merge pull request #5121 from eclipse-ee4j/mojarra_issue_5120
Browse files Browse the repository at this point in the history
Fix #5120: no need to doublecheck if view exists when invoked via ViewHandler#deriveLogicalViewId() instead of deriveViewId()
  • Loading branch information
arjantijms authored Jun 19, 2022
2 parents b4aaf12 + e852770 commit 17999cc
Showing 1 changed file with 1 addition and 8 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -503,14 +503,7 @@ protected String convertViewId(FacesContext context, String viewId) {

appendOrReplaceExtension(viewId, ext, length, extIdx, buffer);

String convertedViewId = buffer.toString();

ViewDeclarationLanguage vdl = getViewDeclarationLanguage(context, convertedViewId);

if (vdl.viewExists(context, convertedViewId)) {
// RELEASE_PENDING (rlubke,driscoll) cache the lookup
return convertedViewId;
}
return buffer.toString();
}

return viewId;
Expand Down

0 comments on commit 17999cc

Please sign in to comment.