1. Hugh is now back at work, more or less full time, after a slower than expected recovery from his neck surgery early this month. Obviously we have a backlog of support to clear, and are working as hard as we can to take care of it. If you have an unanswered thread more than 3 days old, please post one (and only one) "bump" on it, to move it in to our "last 3 days" list. If you have an unanswered issue in a forum you no longer have access to due to subscriptions timing out, please use the Contact Us form, and provide us with the URL to the thread.

very limited bug: on table view, the columns with sort ability don't active...

Discussion in 'Fabrik 2.0 Beta Testing' started by astroplan, May 1, 2009.

  1. astroplan New Member

    Level: Community
    very limited bug: on table view, the columns with sort ability don't active table resort render

    testing SVN at 1593

    CODE SEEK: /model/table.php CODE ROW: 1072-1078

    PROPOSAL JUST A FAST-FIX:
    moving the code row 1072 just before code row $strOrder .= " $field $dir"... like that:

    ...

    //build the order by statement from the session
    $strOrder = '';
    foreach ($orderByFields as $field) {
    // from here (row 1072)
    $context = 'com_fabrik.table'. $this->_id.'.order.'.str_replace('.', '___', $field);
    $dir = $session->get( $context );
    if ($dir != '' && $dir != '-' && trim( $dir ) != 'Array' ) {
    $strOrder == '' ? $strOrder = "\n ORDER BY " : $strOrder .= ',';
    // to here
    $field = FabrikWorker::getDbSafeName( $field );
    $strOrder .= " $field $dir";
    }
    }

    ...

    $orderByFields array mismatch anytime because $orderByFields array is composed without ` char (safemode).
  2. cheesegrits Support Gopher

    Level: Community
    Fix has been applied in SVN. Resolves ticket number #391.

    Thanks for the debugging work, much appreciated.

    -- hugh

Share This Page