1. Hugh (cheesegrits) here. Thank you for all the good wishes. My back surgery was a success, post-op recovery is going well, and I'm getting stronger and feeling better every day. I hope to be getting back in the saddle and working support again over the next few days.
  2. IMPORTANT NOTE - we have made some changes in our github branch naming. The 'master' branch no longer exists, and is now called 'joomla25' (which is Fabrik 3.0). The 'joomla3' branch (which is Fabrik 3.2) remains unchanged. We have also removed all the other testing branches which had accuulated over the last few years. I have posted a thread in the Community forum for 3.0 with some more information, feel free to as questions there.

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