Re: [pgAdmin4][Patch]: RM_ 2658 - Improve help message for "binarypaths" entered in Preferences - Mailing list pgadmin-hackers

From Dave Page
Subject Re: [pgAdmin4][Patch]: RM_ 2658 - Improve help message for "binarypaths" entered in Preferences
Date
Msg-id CA+OCxozo5oLaeZAtzW5isEH01XfwcbNRO+FWSkvqUzFvpwJVmA@mail.gmail.com
Whole thread Raw
In response to [pgAdmin4][Patch]: RM_ 2658 - Improve help message for "binary paths"entered in Preferences  (Surinder Kumar <surinder.kumar@enterprisedb.com>)
Responses Re: [pgAdmin4][Patch]: RM_ 2658 - Improve help message for "binarypaths" entered in Preferences
List pgadmin-hackers


On Tue, Aug 22, 2017 at 1:30 PM, Surinder Kumar <surinder.kumar@enterprisedb.com> wrote:
Hi

Issue:

The user seems to enter sometimes paths with quotes and sometimes add spaces which lead to failure of Backup, Restore or Maintenance utility.

Changes:

 - Added a note in help string `Note: Don't add quote(s) or space(s) before or after the path.`

 - Remove whitespace before/after the value before it gets saved.

Please find attached patch and review.

I wonder if the more appropriate fix here would be to add a new setting type (perhaps two, one for directory and one for file), which is rendered with a browse button in the prefs panel, and includes real validation.

Stripping spaces and warning about quotes seems like it's just kicking the basic problem down the road a few feet. 

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

pgadmin-hackers by date:

Previous
From: Surinder Kumar
Date:
Subject: [pgAdmin4][Patch]: RM_ 2658 - Improve help message for "binary paths"entered in Preferences
Next
From: Surinder Kumar
Date:
Subject: Re: [pgAdmin4][Patch]: RM_ 2658 - Improve help message for "binarypaths" entered in Preferences