Dave Page a écrit :
> Guillaume Lelarge wrote:
>> Guillaume Lelarge a écrit :
>>> PS : so, I've no objection for beta 2 on wednesday.
>> In fact, I have one. I just fixed a bug on the creation of a function
>> without cost parameter but there's still one unfixed. It's in
>> dlgFunction.cpp source file, method CheckChange. I think the problem is
>> on these lines :
>> if (!isProcedure || !connection->EdbMinimumVersion(8, 0))
>> CheckValid(enable, cbLanguage->GetCurrentSelection() >= 0,
>> _("Please select language."));
>> There's something wrong in this test with a vanilla PostgreSQL. When I
>> comment these lines, I can apply my changes on a function's definition.
>> Otherwise, the OK button remains disabled whatever changes I could have
>> done. But I have absolutely no idea what's going wrong here. Dave, can
>> you take a look at this before building beta 2 ? Thanks.
> I couldn't reproduce the problem myself (against PG 8.2.4), but I agree
> that the logic in the check is in need of attention. I've committed a
> slightly more sane version - please check it's OK for you.
Sorry, I still reproduce it under PostgreSQL 8.2.4+ and PostgreSQL HEAD.
Here are steps to reproduce problem :
- right click on a function
- choose Properties
- change function's name
OK button will not be enabled, SQL string will not show.
In response to
pgadmin-hackers by date
|Next:||From: svn||Date: 2007-07-25 09:22:40|
|Subject: SVN Commit by dpage: r6473 - trunk/www/development|
|Previous:||From: Erwin Brandstetter||Date: 2007-07-24 21:20:15|
|Subject: Re: Testing 1.7: Errors with function pldbg_get_t