RE: retrieve_all method to support order_by

[prev] [thread] [next] [Date index for 2005/02/23]

Is there any reason why retrieve_all method doesn't support 'ORDER BY'
clause? Will it be in future release?

I have patched our system with:

__PACKAGE__->set_sql(RetrieveAll =3D> <<'');
SELECT __ESSENTIAL__
FROM   __TABLE__
%s

sub retrieve_all {
    my ($class, $arg) =3D @_;

    if (ref $arg eq "HASH" && $arg->{order_by}) {
        return $class->sth_to_objects($class->sql_RetrieveAll(" ORDER BY
$arg->{order_by}"));
    } else {
        return $class->sth_to_objects('RetrieveAll');
    }
}




john