Hi,
PFA updated patch which will validate user input & throws an error if
empty, ask for input again.
Regards,
Murtuza
--
Regards,
Murtuza Zabuawala
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
On Thu, May 12, 2016 at 10:57 AM, Ashesh Vashi <
ashesh.va...@enterprisedb.com> wrote:
>
> On Thu, May 12, 2016 at 10:47 AM, Murtuza Zabuawala <
> murtuza.zabuaw...@enterprisedb.com> wrote:
>
>> Hi Dave,
>>
>> If user provides empty name then we are already throwing an error.
>> Do you still want me to disable button?
>>
>
> Yes - we should most possible of the validation at client, and server too.
> We should leave least validation for the database server, which is not
> possible in pgAdmin 4.
>
> --
>
> Thanks & Regards,
>
> Ashesh Vashi
> EnterpriseDB INDIA: Enterprise PostgreSQL Company
> <http://www.enterprisedb.com/>
>
>
> *http://www.linkedin.com/in/asheshvashi*
> <http://www.linkedin.com/in/asheshvashi>
>
>>
>> Regards,
>> Murtuza
>>
>> > On 11-May-2016, at 8:45 pm, Dave Page <dp...@pgadmin.org> wrote:
>> >
>> > Committed as is, but please submit a patch to disable the OK button,
>> > until a name has been entered, to avoid accepting an empty name.
>> >
>> > Thanks!
>> >
>> > On Wed, May 11, 2016 at 11:59 AM, Murtuza Zabuawala
>> > <murtuza.zabuaw...@enterprisedb.com> wrote:
>> >> Hi,
>> >>
>> >> PFA updated patch with i18n support added in message.
>> >>
>> >> Regards,
>> >> Murtuza
>> >>
>> >> --
>> >> Regards,
>> >> Murtuza Zabuawala
>> >> EnterpriseDB: http://www.enterprisedb.com
>> >> The Enterprise PostgreSQL Company
>> >>
>> >> On Wed, May 11, 2016 at 3:50 PM, Murtuza Zabuawala
>> >> <murtuza.zabuaw...@enterprisedb.com> wrote:
>> >>>
>> >>> Hi,
>> >>>
>> >>> PFA updated patch for named restore point and I have also updated
>> reload
>> >>> configuration menu enable/disable condition as mentioned.
>> >>>
>> >>> Regards,
>> >>> Murtuza
>> >>>
>> >>> --
>> >>> Regards,
>> >>> Murtuza Zabuawala
>> >>> EnterpriseDB: http://www.enterprisedb.com
>> >>> The Enterprise PostgreSQL Company
>> >>>
>> >>> On Wed, May 11, 2016 at 2:03 PM, Ashesh Vashi
>> >>> <ashesh.va...@enterprisedb.com> wrote:
>> >>>>
>> >>>> On Wed, May 11, 2016 at 1:11 PM, Murtuza Zabuawala
>> >>>> <murtuza.zabuaw...@enterprisedb.com> wrote:
>> >>>>>
>> >>>>> Hi All,
>> >>>>>
>> >>>>>
>> >>>>> PFA patch which will add "Adding named restore point" functionality
>> on
>> >>>>> server.
>> >>>>
>> >>>> Create named restore point is action restricted to the superuser
>> only.
>> >>>> Please check that in the menu enable/disable functionality.
>> >>>>
>> >>>> The check also applicable to 'relaod configuration'.
>> >>>>>
>> >>>>>
>> >>>>> --
>> >>>>> Regards,
>> >>>>> Murtuza Zabuawala
>> >>>>> EnterpriseDB: http://www.enterprisedb.com
>> >>>>> The Enterprise PostgreSQL Company
>> >>>>>
>> >>>>>
>> >>>>> --
>> >>>>> Sent via pgadmin-hackers mailing list (
>> pgadmin-hackers@postgresql.org)
>> >>>>> To make changes to your subscription:
>> >>>>> http://www.postgresql.org/mailpref/pgadmin-hackers
>> >>>>>
>> >>>>
>> >>>
>> >>
>> >>
>> >>
>> >> --
>> >> Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
>> >> To make changes to your subscription:
>> >> http://www.postgresql.org/mailpref/pgadmin-hackers
>> >>
>> >
>> >
>> >
>> > --
>> > Dave Page
>> > Blog: http://pgsnake.blogspot.com
>> > Twitter: @pgsnake
>> >
>> > EnterpriseDB UK: http://www.enterprisedb.com
>> > The Enterprise PostgreSQL Company
>>
>>
>
diff --git a/web/pgadmin/browser/server_groups/servers/templates/servers/servers.js b/web/pgadmin/browser/server_groups/servers/templates/servers/servers.js
index 5300055..ff2e0c3 100644
--- a/web/pgadmin/browser/server_groups/servers/templates/servers/servers.js
+++ b/web/pgadmin/browser/server_groups/servers/templates/servers/servers.js
@@ -229,7 +229,8 @@ function($, _, S, pgAdmin, pgBrowser, alertify) {
// We will execute this function when user clicks on the OK button
function(evt, value) {
// If user has provided a value, send it to the server
- if(!_.isUndefined(value) && !_.isNull(value) && value !== '') {
+ if(!_.isUndefined(value) && !_.isNull(value) && value !== ''
+ && String(value).replace(/^\s+|\s+$/g, '') !== '') {
$.ajax({
url: obj.generate_url(i, 'restore_point', d, true),
method:'POST',
@@ -248,6 +249,7 @@ function($, _, S, pgAdmin, pgBrowser, alertify) {
}
});
} else {
+ evt.cancel = true;
alertify.error('{{ _('Please enter a valid name.') }}', 10);
}
}
--
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers
