Re: PATCH: Fail to restore table/function, if it has special characters.

From: Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com>
To: Dave Page <dave(dot)page(at)enterprisedb(dot)com>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: PATCH: Fail to restore table/function, if it has special characters.
Date: 2010-03-18 14:19:01
Message-ID: 8cf965d31003180719w1726bda2u4713cbe69b7a4894@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

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.

--

Ashesh Vashi
EnterpriseDB INDIA: Enterprise Postgres Company<http://www.enterprisedb.com>

Attachment Content-Type Size
FB14805_v2.patch application/octet-stream 8.4 KB

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dave Page 2010-03-18 14:32:29 Re: PATCH: Fail to restore table/function, if it has special characters.
Previous Message svn 2010-03-18 12:41:42 SVN Commit by dpage: r8231 - trunk/pgadmin3