Re: PATCH: Fail to restore table/function, if it has special characters. - Mailing list pgadmin-hackers

From Ashesh Vashi
Subject Re: PATCH: Fail to restore table/function, if it has special characters.
Date
Msg-id 8cf965d31003182310r254ba7d4r3d00ecb1b6f5173f@mail.gmail.com
Whole thread Raw
In response to Re: PATCH: Fail to restore table/function, if it has special characters.  (Dave Page <dave.page@enterprisedb.com>)
Responses Re: PATCH: Fail to restore table/function, if it has special characters.
List pgadmin-hackers

Please find the attached patch for REL_1_10_PATCHES branches and updated patch for the trunk (with few tab characters to 4 spaces.)

--

Ashesh Vashi
EnterpriseDB INDIA: Enterprise Postgres Company

On Thu, Mar 18, 2010 at 8:02 PM, Dave Page <dave.page@enterprisedb.com> wrote:
On Thu, Mar 18, 2010 at 2:19 PM, Ashesh Vashi
> Hi,
> The bug was also there for the windows. :-(
> I had to modify the commandLineCleanOption function to resolve the issue and
> to make it platform-independent.
>
> Please find the attached patch for the same.
> I have made changes in the following files as described:
> * include/utils/misc.h
> - Changed the definition of the "commandLineCleanOption" function
> * utils/misc.cpp
> - Changed the "commandLineCleanOption" function to make it work on any
> platform.
> /**
>  * FUNCTION: commandLineCleanOption
>  * INPUTS:
>  *       option       - input string needs to be reformatted
>  *       schemaObject - Is this an object related to schema?
>  * PURPOSE:
>  *  - Fixup a (double-quoted) string for use on the command line
>  *
>  *    1. Schema objects needs three (slash & double-quote) combination
>  *       instead of a double-quote to be used as a command-line option
>  *    2. Options other than the schema objects needs (slash & double-quote)
>  *       combination instead of a double-quote to be used as a command-line
>  *       option.
>  *
>  *    i.e. CASE: Dump a table named em"p and database name tes"t having
> username
>  *         xy\z. The command-line arguments for these values will be as
> below:
>  *
>  *         pg_dump --username "xy\\z" --table public."em\"\"\"p" "tes\"t"
>  */
> The above comments says everything about its implementation.
> * frm/frmBackup.cpp
> * frm/frmBackupGlobals.cpp
> * frm/frmBackupServer.cpp
> * frm/frmRestore.cpp
> Dave,
> In order the solve this issue on 1_10_PATCHES branch, we may need to do the
> similar refactoring.
> Is it ok?
> If yes, I can share the patch for the same.

Please do.

Attachment

pgadmin-hackers by date:

Previous
From: Quan Zongliang
Date:
Subject: Re: Donation of 3 FEITIAN PKI smart cards to the pgAdmin community
Next
From: svn@pgadmin.org
Date:
Subject: SVN Commit by dpage: r8232 - in branches/REL-1_10_0_PATCHES/pgadmin3: . pgadmin/include/utils pgadmin/utils