Re: PATCH for CREATE DATABASE with COLLATE, CTYPE & CONNECTION LIMIT
От | Ashesh D Vashi |
---|---|
Тема | Re: PATCH for CREATE DATABASE with COLLATE, CTYPE & CONNECTION LIMIT |
Дата | |
Msg-id | 49229FA7.8060704@enterprisedb.com обсуждение исходный текст |
Ответ на | Re: PATCH for CREATE DATABASE with COLLATE, CTYPE & CONNECTION LIMIT ("Dave Page" <dpage@pgadmin.org>) |
Ответы |
Re: PATCH for CREATE DATABASE with COLLATE, CTYPE & CONNECTION LIMIT
|
Список | pgadmin-hackers |
Hi,
Please find the updated patch.
Datatype for the datconnlimit is integer.
Regards,
Ashesh
Please find the updated patch.
Done.Some thoughts (all minor), in no particular order: - Let's use 'Collation' and 'Character type' in both the properties list view and on the dialog for the labels - it'll look much nicer than COLLATE and CTYPE I think.
Changed <growablerows> to 11.- dlgDatabase needs to have it's sizing fixed - the cell holding the CTYPE controls is stretching, but it should be the comment cell that stretches (<growablerows> needs to change to 11 perhaps?).
Maximum number allowed will be 2147483647.- Entering a new connection limit, and clicking the SQL tab doesn't show me an SQL statement if the value entered is too large, but does enable the OK button The easiest fix is probably to limit the number of characters the control will access. That should still allow a huge number to entered.
Datatype for the datconnlimit is integer.
Done.- Entering a connection limit of -<anything but 1> is silently converted to 1. We should just ignore anything that isn't 1, rather than trying to correct it for the user.
Done.- 'Connection Limit' in the properties list should not have a capital L.
Done.- This code in dlgDatabase may be uneeded now? + // As some of the controls has been made hidden, + // Update() will help to rearrange all the other controls properly. + this->Update();
Done.- The COLLATE and CTYPE options should be omitted from the CREATE DATBASE SQL generated by dlgDatabase, if the user hasn't specified values.
Done.- The CONNECTION LIMIT option seems to have an additional space in the indent: CREATE DATABASE foobar WITH ENCODING='UTF8' COLLATE='' CTYPE='' CONNECTION LIMIT=99;
Regards,
Ashesh
В списке pgadmin-hackers по дате отправления: