Hi Sanket,
In your patch, I see couple of issues with your patch:
1. In pgadmin/frm/frmQuery.cpp:
You've used "settings->Read(wxT("frmQuery/AutoCommit"), &bVal, true)"
directly.
You should be using the function, you've defined in sysSettings.
i.e.
sysSettings::GetAutoCommit()
and, sysSettings::SetAutoCommit()
Please follow the correct naming convention for the
frmQuery::CommandNoBegin(...) function.
Function must not start with a capital letter for a regular function (we do
declare/define only the event functions with capital letters).
2. In pgadmin/ui/frmOptions.xrc:
A lot of changes are in this patch, which not required for this feature.
--
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>
On Tue, Mar 24, 2015 at 6:01 PM, Sanket Mehta <sanket.me...@enterprisedb.com
> wrote:
> Hi,
>
> Below is the description of the autocommit feature implementation in
> pgadmin:
>
> This feature is only applicable in query editor.
>
> Autocommit can be set on/off from 2 places.
>
> 1. pgadmin main browser -> File menu -> Options -> Query tool -> Query
> editor -> Enable Auto commit
> 2. In Query editor -> Query menu -> Auto-commmit
>
> By default auto commit will be enabled in pgadmin.
>
>
> In any query editor session, once user uncheck this autocommit checkbox,
> depending upon postgreSQL standard for executing the query it will execute
> "BEGIN;" statement before executing the actual query and thus setting the
> autocommit off for that session.
>
> User can set the autocommit on at any time by selecting the checkbox from
> above mentioned places.
> But it will be active only after user has completed the transaction
> block(started by BEGIN as mentioned above) by END or ROLLBACK or COMMIT;
>
> Let me know in case of any queries.
>
> Regards,
> Sanket Mehta
> Sr Software engineer
> Enterprisedb
>
