Problem/Motivation

Problems with the sort widgets configuration

Steps to reproduce

When trying to manage sort widget.

Proposed resolution

Need to check for empty array or objects.

CommentFileSizeAuthor
#2 usort_solr_sort_wigdet.patch697 bytessundeep88
Screenshot from 2023-10-04 18-38-22.png299.96 KBsundeep88

Comments

sundeep88 created an issue. See original summary.

sundeep88’s picture

sundeep88 commented
StatusFileSize
new usort_solr_sort_wigdet.patch697 bytes
peri22’s picture

peri22 commented
Related issues: +#3488348: Sort weight is not respected

I confirmed this on 1.x HEAD. The same line causes this and #3488348: Sort weight is not respected.

  • The bug: for a display whose widget config has just been created, the sorts setting is NULL, and uasort() expects an array — so opening the manage form the first time is a fatal error.
  • The fix: MR !6 in #3488348: Sort weight is not respected replaces that call with a null-safe sort on the fields.

I verified both states: the form crashes before the MR and builds fine after it.

peri22’s picture

peri22 commented
Status: Active » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

peri22’s picture

peri22 commented

Fixed by the same change, committed in #3488348: Sort weight is not respected.