Got all three items fixed in SVN now. Let me know if it works or doesn't. Aaron
On Sat, Jul 16, 2005, ""Christian G. Warden"" <[EMAIL PROTECTED]> said: > On Sun, Jul 17, 2005 at 12:45:09AM -0000, Aaron Stone wrote: >> On Sat, Jul 16, 2005, ""Christian G. Warden"" <[EMAIL PROTECTED]> said: >> >> > On Sat, Jul 16, 2005 at 05:38:34PM -0400, M. J. [Mike] O'Brien wrote: >> >> Suggestion: >> >> 1) Use DbMail 2.0.4 stable >> > >> > I'm using 2.1 mainly for the table prefix. I'm upgrading an old pre 2.0 >> > installation, before the dbmail_ prefix was added to the tables. I have >> > a lot of code that interacts with the dbmail tables so I don't want to >> > rename the tables right now. >> > >> > Unfortunately, it looks like the table prefix code is somewhat broken. >> >> I'm looking at it right now; this bit of code needs a rewrite. > > Thanks, Aaron. > >> > config.c always sets the table prefix to DEFAULT_DBPFX. Also, the >> > iniparser treats and empty right-hand-side the same as if the key >> > doesn't exist so there doesn't appear to be a way to set the prefix to >> > an empty string. >> >> Have you tried quoting? e.g. >> >> table_prefix = "" > > Yes, iniparser treats it as the string, '""', so you end up with > queries against tables named ""users, for example. > >> > I think I'll probably just have to change >> > DEFAULT_DBPFX. And finally, there are a handful of places in db.c where >> > the dbmail_ prefix is hard-coded into the queries, but these are easy to >> > fix. >> >> Send patches! > > I've attached the patch to db.c. > > Christian > > --