[
https://issues.apache.org/jira/browse/OFBIZ-10768?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16736956#comment-16736956 ]
Gil Portenseigne commented on OFBIZ-10768:
------------------------------------------
Hello [~taher],
Reviewing the patch it seems that your fix solve your issue, but it appears that the ‘formElement.getAttribute("sort-field-parameter-name")’ is not used, so i would suggest something like :
{code:java}
if (sortFieldParameterName.isEmpty() && parentModel != null) {
this.sortFieldParameterName = parentModel.getSortFieldParameterName();
} else if (sortFieldParameterName.isEmpty()){
this.sortFieldParameterName = "sortField";
} else {
this.sortFieldParameterName = sortFieldParameterName;
}{code}
It seems strange that wasn't used since... This feature might not be used in OFBiz codebase
> Cannot sort fields of a form if it has a parent
> -----------------------------------------------
>
> Key: OFBIZ-10768
> URL:
https://issues.apache.org/jira/browse/OFBIZ-10768> Project: OFBiz
> Issue Type: Bug
> Components: framework
> Reporter: Taher Alkhateeb
> Priority: Minor
> Attachments: OFBIZ-10768.patch
>
>
> To repeat the bug:
> * Create a form with a few fields and do not set the sort-field-parameter-name attribute
> * Create another form that extends from this form and again do not set the sort-field-parameter-name attribute
> * Now make any field sortable by setting sort-field="true"
> * Observe how the link is broken. instead of having sortField=whatever in the URL, instead you would get intra-app=whatever. Thus sort field functionality becomes broken.
> After investigating this issue, I realized the reason was an error in the ModelForm.java file. I will provide a patch for a fix.
>
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
Locked
