Thanks. I thought i checked for the var init. Guess i missed a code path. On Saturday, August 6, 2011, Guillaume Lelarge <guilla...@lelarge.info> wrote: > On Fri, 2011-07-29 at 10:33 +0100, Dave Page wrote: >> On Fri, Jul 29, 2011 at 9:14 AM, Erwin Brandstetter >> <brandstet...@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 > >
-- Dave Page Blog: http://pgsnake.blogspot.com Twitter: @pgsnake EnterpriseDB UK: http://www.enterprisedb.com The Enterprise PostgreSQL Company
