[xoops-trackers] [ xoops-Feature Requests-3561030 ] Additional ability in criteria setOrder

[SourceForge.net <no...@so...>]

Feature Requests item #3561030, was opened at 2012-08-23 08:28
Message generated for change (Tracker Item Submitted) made by zyspec
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430843&aid=3561030&group_id=41586

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Core
Group: XOOPS 2.6
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: zyspec (zyspec)
Assigned to: Nobody/Anonymous (nobody)
Summary: Additional ability in criteria setOrder

Initial Comment:
Currently the only options for the setOrder method in the Criteria element is to set it to DESC. The object is created with the order set to 'ASC' but if setOrder is called with 'DESC' then it can't be changed back without destroying the object.  SetOrder should allow both ASC & DESCas possible order as values passed to the method.  Recommend changing method to:

    /**
     * @param string $order
     */
    public function setOrder($order)
    {
        if ('DESC' == strtoupper($order)) {
            $this->order = 'DESC';
        } else {
            $this->order = 'ASC';
        }
    }

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430843&aid=3561030&group_id=41586