Thread: [Lxr-dev] [ lxr-Bugs-1119086 ] Postgres.pm missing a few ${prefix}es
Brought to you by:
ajlittoz
From: SourceForge.net <no...@so...> - 2005-02-09 04:46:05
|
Bugs item #1119086, was opened at 2005-02-08 20:46 Message generated for change (Tracker Item Submitted) made by Item Submitter You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=390117&aid=1119086&group_id=27350 Category: Database interface Group: current cvs Status: Open Resolution: None Priority: 5 Submitted By: Nobody/Anonymous (nobody) Assigned to: Nobody/Anonymous (nobody) Summary: Postgres.pm missing a few ${prefix}es Initial Comment: The sequence names in Postgres.pm need to be prefixed with the variable which holds the naming prefix, ie lxr_ ke...@ap... ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=390117&aid=1119086&group_id=27350 |
From: SourceForge.net <no...@so...> - 2005-04-12 20:24:51
|
Bugs item #1119086, was opened at 2005-02-09 05:46 Message generated for change (Comment added) made by citan You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=390117&aid=1119086&group_id=27350 Category: Database interface Group: current cvs Status: Open Resolution: None Priority: 5 Submitted By: Nobody/Anonymous (nobody) Assigned to: Nobody/Anonymous (nobody) Summary: Postgres.pm missing a few ${prefix}es Initial Comment: The sequence names in Postgres.pm need to be prefixed with the variable which holds the naming prefix, ie lxr_ ke...@ap... ---------------------------------------------------------------------- Comment By: Krystian Lewandowski (citan) Date: 2005-04-12 22:24 Message: Logged In: YES user_id=1258114 Patch for 0.9.3 version (maybe CVS too): --- /home/citan/tmp/lxr-0.9.3/lib/LXR/Index/Postgres.pm Tue Jul 27 10:05:45 2004 +++ /usr/local/lxr/lib/LXR/Index/Postgres.pm Tue Apr 12 16:38:40 2005 @@ -57,12 +57,12 @@ $files_select = $dbh->prepare("select fileid from ${prefix}files where filename = ? and revision = ?"); - $filenum_nextval = $dbh->prepare("select nextval('filenum')"); + $filenum_nextval = $dbh->prepare("select nextval('${prefix}filenum')"); $files_insert = $dbh->prepare("insert into ${prefix}files values (?, ?, ?)"); $symbols_byname = $dbh->prepare("select symid from ${prefix}symbols where symname = ?"); $symbols_byid = $dbh->prepare("select symname from ${prefix}symbols where symid = ?"); - $symnum_nextval = $dbh->prepare("select nextval('symnum')"); + $symnum_nextval = $dbh->prepare("select nextval('${prefix}symnum')"); $symbols_insert = $dbh->prepare("insert into ${prefix}symbols values (?, ?)"); $symbols_remove = $dbh->prepare("delete from ${prefix}symbols where symname = ?"); @@ -99,7 +99,7 @@ . "and f.fileid = r.fileid and " . "s.symname = ? and r.release = ?"); - $declid_nextnum = $dbh->prepare("select nextval('declnum')"); + $declid_nextnum = $dbh->prepare("select nextval('${prefix}declnum')"); $decl_select = $dbh->prepare( ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=390117&aid=1119086&group_id=27350 |
From: SourceForge.net <no...@so...> - 2005-04-29 20:22:57
|
Bugs item #1119086, was opened at 2005-02-09 04:46 Message generated for change (Comment added) made by mbox You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=390117&aid=1119086&group_id=27350 Category: Database interface Group: current cvs >Status: Closed >Resolution: Fixed Priority: 5 Submitted By: Nobody/Anonymous (nobody) >Assigned to: Malcolm Box (mbox) Summary: Postgres.pm missing a few ${prefix}es Initial Comment: The sequence names in Postgres.pm need to be prefixed with the variable which holds the naming prefix, ie lxr_ ke...@ap... ---------------------------------------------------------------------- >Comment By: Malcolm Box (mbox) Date: 2005-04-29 21:22 Message: Logged In: YES user_id=215386 As far as I can see this is fixed in the latest CVS version of Postgres.pm. Please can you confirm? Thanks, Malcolm ---------------------------------------------------------------------- Comment By: Krystian Lewandowski (citan) Date: 2005-04-12 21:24 Message: Logged In: YES user_id=1258114 Patch for 0.9.3 version (maybe CVS too): --- /home/citan/tmp/lxr-0.9.3/lib/LXR/Index/Postgres.pm Tue Jul 27 10:05:45 2004 +++ /usr/local/lxr/lib/LXR/Index/Postgres.pm Tue Apr 12 16:38:40 2005 @@ -57,12 +57,12 @@ $files_select = $dbh->prepare("select fileid from ${prefix}files where filename = ? and revision = ?"); - $filenum_nextval = $dbh->prepare("select nextval('filenum')"); + $filenum_nextval = $dbh->prepare("select nextval('${prefix}filenum')"); $files_insert = $dbh->prepare("insert into ${prefix}files values (?, ?, ?)"); $symbols_byname = $dbh->prepare("select symid from ${prefix}symbols where symname = ?"); $symbols_byid = $dbh->prepare("select symname from ${prefix}symbols where symid = ?"); - $symnum_nextval = $dbh->prepare("select nextval('symnum')"); + $symnum_nextval = $dbh->prepare("select nextval('${prefix}symnum')"); $symbols_insert = $dbh->prepare("insert into ${prefix}symbols values (?, ?)"); $symbols_remove = $dbh->prepare("delete from ${prefix}symbols where symname = ?"); @@ -99,7 +99,7 @@ . "and f.fileid = r.fileid and " . "s.symname = ? and r.release = ?"); - $declid_nextnum = $dbh->prepare("select nextval('declnum')"); + $declid_nextnum = $dbh->prepare("select nextval('${prefix}declnum')"); $decl_select = $dbh->prepare( ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=390117&aid=1119086&group_id=27350 |
From: SourceForge.net <no...@so...> - 2005-04-29 20:37:14
|
Bugs item #1119086, was opened at 2005-02-09 05:46 Message generated for change (Comment added) made by citan You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=390117&aid=1119086&group_id=27350 Category: Database interface Group: current cvs Status: Closed Resolution: Fixed Priority: 5 Submitted By: Nobody/Anonymous (nobody) Assigned to: Malcolm Box (mbox) Summary: Postgres.pm missing a few ${prefix}es Initial Comment: The sequence names in Postgres.pm need to be prefixed with the variable which holds the naming prefix, ie lxr_ ke...@ap... ---------------------------------------------------------------------- Comment By: Krystian Lewandowski (citan) Date: 2005-04-29 22:37 Message: Logged In: YES user_id=1258114 I'm not using current cvs version, but if i can see correctly here http://cvs.sourceforge.net/viewcvs.py/lxr/lxr/lib/LXR/Index/Postgres.pm?rev=1.19&only_with_tag=HEAD&view=auto "select nextval('filenum')" and others (symnum,declnum) don't have ${prefix}es i.e. "select nextval('${prefix}filenum')" Sorry, maybe i missed something. ---------------------------------------------------------------------- Comment By: Malcolm Box (mbox) Date: 2005-04-29 22:22 Message: Logged In: YES user_id=215386 As far as I can see this is fixed in the latest CVS version of Postgres.pm. Please can you confirm? Thanks, Malcolm ---------------------------------------------------------------------- Comment By: Krystian Lewandowski (citan) Date: 2005-04-12 22:24 Message: Logged In: YES user_id=1258114 Patch for 0.9.3 version (maybe CVS too): --- /home/citan/tmp/lxr-0.9.3/lib/LXR/Index/Postgres.pm Tue Jul 27 10:05:45 2004 +++ /usr/local/lxr/lib/LXR/Index/Postgres.pm Tue Apr 12 16:38:40 2005 @@ -57,12 +57,12 @@ $files_select = $dbh->prepare("select fileid from ${prefix}files where filename = ? and revision = ?"); - $filenum_nextval = $dbh->prepare("select nextval('filenum')"); + $filenum_nextval = $dbh->prepare("select nextval('${prefix}filenum')"); $files_insert = $dbh->prepare("insert into ${prefix}files values (?, ?, ?)"); $symbols_byname = $dbh->prepare("select symid from ${prefix}symbols where symname = ?"); $symbols_byid = $dbh->prepare("select symname from ${prefix}symbols where symid = ?"); - $symnum_nextval = $dbh->prepare("select nextval('symnum')"); + $symnum_nextval = $dbh->prepare("select nextval('${prefix}symnum')"); $symbols_insert = $dbh->prepare("insert into ${prefix}symbols values (?, ?)"); $symbols_remove = $dbh->prepare("delete from ${prefix}symbols where symname = ?"); @@ -99,7 +99,7 @@ . "and f.fileid = r.fileid and " . "s.symname = ? and r.release = ?"); - $declid_nextnum = $dbh->prepare("select nextval('declnum')"); + $declid_nextnum = $dbh->prepare("select nextval('${prefix}declnum')"); $decl_select = $dbh->prepare( ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=390117&aid=1119086&group_id=27350 |
From: SourceForge.net <no...@so...> - 2005-04-30 01:37:02
|
Bugs item #1119086, was opened at 2005-02-09 04:46 Message generated for change (Comment added) made by mbox You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=390117&aid=1119086&group_id=27350 Category: Database interface Group: current cvs Status: Closed Resolution: Fixed Priority: 5 Submitted By: Nobody/Anonymous (nobody) Assigned to: Malcolm Box (mbox) Summary: Postgres.pm missing a few ${prefix}es Initial Comment: The sequence names in Postgres.pm need to be prefixed with the variable which holds the naming prefix, ie lxr_ ke...@ap... ---------------------------------------------------------------------- >Comment By: Malcolm Box (mbox) Date: 2005-04-30 02:37 Message: Logged In: YES user_id=215386 Yes, I missed those. Should now be fixed in CVS ---------------------------------------------------------------------- Comment By: Krystian Lewandowski (citan) Date: 2005-04-29 21:37 Message: Logged In: YES user_id=1258114 I'm not using current cvs version, but if i can see correctly here http://cvs.sourceforge.net/viewcvs.py/lxr/lxr/lib/LXR/Index/Postgres.pm?rev=1.19&only_with_tag=HEAD&view=auto "select nextval('filenum')" and others (symnum,declnum) don't have ${prefix}es i.e. "select nextval('${prefix}filenum')" Sorry, maybe i missed something. ---------------------------------------------------------------------- Comment By: Malcolm Box (mbox) Date: 2005-04-29 21:22 Message: Logged In: YES user_id=215386 As far as I can see this is fixed in the latest CVS version of Postgres.pm. Please can you confirm? Thanks, Malcolm ---------------------------------------------------------------------- Comment By: Krystian Lewandowski (citan) Date: 2005-04-12 21:24 Message: Logged In: YES user_id=1258114 Patch for 0.9.3 version (maybe CVS too): --- /home/citan/tmp/lxr-0.9.3/lib/LXR/Index/Postgres.pm Tue Jul 27 10:05:45 2004 +++ /usr/local/lxr/lib/LXR/Index/Postgres.pm Tue Apr 12 16:38:40 2005 @@ -57,12 +57,12 @@ $files_select = $dbh->prepare("select fileid from ${prefix}files where filename = ? and revision = ?"); - $filenum_nextval = $dbh->prepare("select nextval('filenum')"); + $filenum_nextval = $dbh->prepare("select nextval('${prefix}filenum')"); $files_insert = $dbh->prepare("insert into ${prefix}files values (?, ?, ?)"); $symbols_byname = $dbh->prepare("select symid from ${prefix}symbols where symname = ?"); $symbols_byid = $dbh->prepare("select symname from ${prefix}symbols where symid = ?"); - $symnum_nextval = $dbh->prepare("select nextval('symnum')"); + $symnum_nextval = $dbh->prepare("select nextval('${prefix}symnum')"); $symbols_insert = $dbh->prepare("insert into ${prefix}symbols values (?, ?)"); $symbols_remove = $dbh->prepare("delete from ${prefix}symbols where symname = ?"); @@ -99,7 +99,7 @@ . "and f.fileid = r.fileid and " . "s.symname = ? and r.release = ?"); - $declid_nextnum = $dbh->prepare("select nextval('declnum')"); + $declid_nextnum = $dbh->prepare("select nextval('${prefix}declnum')"); $decl_select = $dbh->prepare( ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=390117&aid=1119086&group_id=27350 |