#332: SQL for TRIGGER with WHEN clause broken
----------------------+-----------------------------------------------------
 Reporter:  brsa      |       Owner:  dpage
     Type:  patch     |      Status:  new  
 Priority:  minor     |   Milestone:       
Component:  pgadmin   |     Version:  trunk
 Keywords:  SQL pane  |    Platform:  all  
----------------------+-----------------------------------------------------
Changes (by brsa):

  * type:  bug => patch


Comment:

 The fix is pretty obvious in this case. The code mistakenly trims all
 outer enclosing brackets, which leads to unmatched brackets in test 2.) -
 both left brackets were removed.

 pgTrigger.cpp, line 393:

 - wxT("  trim(substring(pg_get_triggerdef(t.oid), 'WHEN (.*) EXECUTE
 PROCEDURE'), '()') AS whenclause\n")
 + wxT("  substring(pg_get_triggerdef(t.oid), 'WHEN (.*) EXECUTE
 PROCEDURE') AS whenclause\n")

-- 
Ticket URL: <http://code.pgadmin.org/trac/ticket/332#comment:1>
pgAdmin III <http://code.pgadmin.org/trac/>
pgAdmin III

-- 
Sent via pgadmin-hackers mailing list ([email protected])
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers

Reply via email to