Re: Changed default ordering in tables - Mailing list pgadmin-hackers

From Guillaume Lelarge
Subject Re: Changed default ordering in tables
Date
Msg-id 1312634982.2039.19.camel@laptop
Whole thread Raw
In response to Re: Changed default ordering in tables  (Dave Page <dpage@pgadmin.org>)
Responses Re: Changed default ordering in tables
Re: Changed default ordering in tables
List pgadmin-hackers
On Fri, 2011-07-29 at 10:33 +0100, Dave Page wrote:
> On Fri, Jul 29, 2011 at 9:14 AM, Erwin Brandstetter
> <brandstetter@falter.at> wrote:
> > So, that's two distinct issue:
> > - "Last 100 rows" broken for multi-column pkeys.
> > - Default sort order DESC for edit grid, where it should be ASC.
>
> That one I can reproduce, and fix. Patch attached.
>
> Ashesh, can you review and test please? We're getting close to release
> so I want to be extra careful.
>

I checked your patch. Seems good to me. So I applied it. I also applied
another patch to fix Erwin's bug (missing initialization of the
pkAscending variable).


--
Guillaume
  http://blog.guillaume.lelarge.info
  http://www.dalibo.com


pgadmin-hackers by date:

Previous
From: Guillaume Lelarge
Date:
Subject: pgAdmin III commit: Fix the ordering on View Data
Next
From: Dave Page
Date:
Subject: Re: Changed default ordering in tables