Relationship Tab 'sort by' option is running on dd/mm/yy so fails

[petednz]

Tested on Demo - seems to be sorting by Day then Month then Year

Do we get patch-welcomed?

[Dave Greenberg]

Pete - The problem is that the jscript 'sorting widget' is sorting on the actual string values (e.g. 'September' will sort below 'November'). Not sure how to solve that??

[xavier]

Hi,

Pretty sure there is an option to define the type of each column (or to add a custom sort function).

The problem is that even if you can avoid searching per string, the date format can be different (if the user changes the date format)

The usual trick is to add the date in a hidden computer readable format  (ISO 8601) and sort on that  instead of the date as displayed.

Code: [Select]

<td data-iso="2012-11-27">November 27 2012</td>

and sort on the data-iso if the type is date

rem: if you were to patch it, you can in jquery access any html5 data-xxx through $(selector).data("xxx")

[petednz]

so this needs to be addressed in a more global fashion i expect

[xavier]

Think it can be changed only at the template level I think (assuming the row data is assigned to the template and not pre-html digested  in php)

[Eileen]

I thought I had this but it slipped away on me

This looks like it should work

Index: templates/CRM/Contact/Page/View/Relationship.tpl
===================================================================
--- templates/CRM/Contact/Page/View/Relationship.tpl   (revision 44121)
+++ templates/CRM/Contact/Page/View/Relationship.tpl   (working copy)
@@ -52,12 +52,13 @@
             <th>{ts}Relationship{/ts}</th>
             <th></th>
             <th>{ts}Start{/ts}</th>
-            <th>{ts}End{/ts}</th>
+            <th id="end_date">{ts}End{/ts}</th>
             <th>{ts}City{/ts}</th>
             <th>{ts}State/Prov{/ts}</th>
             <th>{ts}Email{/ts}</th>
             <th>{ts}Phone{/ts}</th>
             <th></th>
+            <th class="hiddenElement"></th>
         </tr>
         </thead>
         {foreach from=$currentRelationships item=rel}
@@ -96,6 +97,7 @@
                 <td>{$rel.email}</td>
                 <td>{$rel.phone}</td>
                 <td class="nowrap">{$rel.action|replace:'xx':$rel.id}</td>
+                <td class="end_date hiddenElement">{$rel.end_date|crmDate}</td>
             </tr>
         {/foreach}
         </table>

(based on reading jsortable & FullText.tpl - but it's not quite there)

[Eileen]

http://issues.civicrm.org/jira/browse/CRM-11436