Bug 35407
| Summary: | Grid columns not sortable in specific circumstances | ||
|---|---|---|---|
| Product: | UCS | Reporter: | Florian Best <best> |
| Component: | UMC (Generic) | Assignee: | Florian Best <best> |
| Status: | CLOSED FIXED | QA Contact: | Jürn Brodersen <brodersen> |
| Severity: | normal | ||
| Priority: | P5 | CC: | gohmann, michelsmidt, petersen |
| Version: | UCS 3.2 | Flags: | best:
Patch_Available+
|
| Target Milestone: | UCS 4.1-1-errata | ||
| Hardware: | Other | ||
| OS: | Linux | ||
| What kind of report is it?: | --- | What type of bug is this?: | --- |
| Who will be affected by this bug?: | --- | How will those affected feel about the bug?: | --- |
| User Pain: | Enterprise Customer affected?: | ||
| School Customer affected?: | ISV affected?: | ||
| Waiting Support: | Flags outvoted (downgraded) after PO Review: | ||
| Ticket number: | Bug group (optional): | ||
| Max CVSS v3 score: | |||
*** Bug 36047 has been marked as a duplicate of this bug. *** Reported by customer on #2016022321000671. *** Bug 40837 has been marked as a duplicate of this bug. *** Fixed. univention-management-console-frontend.yaml: r67952 | YAML Bug #35407 Bug #40613 univention-management-console-frontend (5.0.63-29): r67951 | Bug #35407: fixed a typo in umc.tools.cmpObjects which caused that in some situations e.g. the grid header weren't sortable What I tested: Sorted classes by description in schoolwizards/classes. -> Sorting by description is now possible. Changes: OK. YAML: OK. → VERIFIED |
Error in tools.cmpObjects if one of the objects to compare is null instead of a String. This error causes that in UCS@school e.g. the column 'Class' in the Users wizard is not sortable so that one get the error message "Es ist ein Fehler aufgetreten" in the grid instead of grid entries. Exception: TypeError: Cannot read property 'toLowerCase' of null Fix is simply: Index: umc/tools.js =================================================================== --- umc/tools.js (Revision 51947) +++ umc/tools.js (Arbeitskopie) -969,7 +969,7 @@ // check for lowercase var a = _a[o.attr]; var b = _b[o.attr]; - if (o.ignCase && a.toLowerCase && b.toLowerCase) { + if (o.ignCase && a && a.toLowerCase && b && b.toLowerCase) { a = a.toLowerCase(); b = b.toLowerCase(); }