[KULRICE-6587] Document Search Results open in a new window regardless of the system parameter (DOCUMENT_SEARCH_POPUP_IND) value Created: 27/Jan/12  Updated: 23/Feb/12  Resolved: 06/Feb/12

Status: Closed
Project: Kuali Rice Development
Component/s: Development
Affects Version/s: None
Fix Version/s: 2.0.0-rc2, 2.0
Security Level: Public (Public: Anyone can view)

Type: Bug Fix Priority: Critical
Reporter: Chitra Chandran Assignee: Sona Sona (Inactive)
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Rice Module:
KAI Review Status: Not Required
KTI Review Status: Not Required


ActionList and DocumentSearch both use System parameter values to decide if a link should open in a new window or not. While ActionList behaves as expected (System parameter ACTION_LIST_DOCUMENT_POPUP_IND value drives this behavior), document search does not.

DocumentSearchCriteriaBoLookupableHelperService.generateDocumentHandlerUrl method generates the link url and it uses the following piece of code to decide the target window behavior.

if (isDocumentHandlerPopup()) {

The target is set only when the system parameter maps to a true value. However, KualiLookup.jsp overrides this behavior with this if/else logic.

<c:when test="${empty column.columnAnchor.target}"><c:set var="anchorTarget" value="_blank" /></c:when>
<c:otherwise><c:set var="anchorTarget" value="${column.columnAnchor.target}" /></c:otherwise>

So, in cases where apps have overridden the DOCUMENT_SEARCH_POPUP_IND parameter to have a false value, DocumentSearchCriteriaBoLookupableHelperService won't set the anchor target and the KualiLookup.jsp will default that to open in a new window.

Since KualiLookup.jsp is common across all lookups and this maybe the default behavior we want for BO lookups, it would help to replace DocumentSearchCriteriaBoLookupableHelperService code to

if (isDocumentHandlerPopup()) {
} else {

Comment by Jessica Coltrin (Inactive) [ 23/Feb/12 ]

Closing since these items are now in the release notes.

Generated at Sat Oct 31 04:47:18 CDT 2020 using JIRA 7.0.11#70121-sha1:19d24976997c1d95f06f3e327e087be0b71f28d4.