On Fri 09 Jul 2004 19:16, Tim Bunce <[EMAIL PROTECTED]> wrote:
> > Why not create a new method call, as you (IMHO) should have done in 1.38
> > The change of behaviour in the tables () method is a straightforward nightmare
> > for me. Adding those quotations broke about all my applications.
> 
> Some of mine too. But quotes are technically required whenever a
> schema name and table name are combined. Without them you can't reliably
> separate the names as they may (in theory) contain '.'.

If it would have been up to me to set the standards and defenintions, I would
*_NEVER_* ever have allowed blanks^wnon-word characters in table(space)- and
field- names.

I have just had the luck^wawful experience of having to convert a M$Access DB
to Oracle. Table names as "Tabel leerlingen & verantw." and field names as "LW
34_18.w" were the M$ DB designer's regular choice.

> What I probably should have done is "redefine tables() to default
> to tables accessible from current schema without further qualification"
> _and_ made tables not use quote_identifier in that case if the name
> matched /^\w+$/.

Should have done is dead. We will have to live with what he have, and the fact
that I also maintain a DBD somehow forced me to upgrade DBI to be sure it
still worked. If I would have been a `regular' customer, I would never have
gone beyond 1.37. No hard feelings. what's done is done.

> But let's not get into that now. They'll be plenty of time down the road
> and I'll be open to suggestions as we go.

That's what BOF's are for. And these lists :) And - let's not forget - the
past and our experience. We learn from both good and less good decisions.

> > >   Always taint check the $sql for do() and prepare()
> > >   if perl is in taint mode (can't be disabled).
> > 
> > These two lines belong together, right?
> 
> Yes. But like most items in the ToDo it needs more thought.
> It's more a statement of direction than a specification at this stage.

Which I support ...

-- 
H.Merijn Brand        Amsterdam Perl Mongers (http://amsterdam.pm.org/)
using perl-5.6.1, 5.8.0 & 633 on HP-UX 10.20 & 11.00, AIX 4.2, AIX 4.3,
     WinNT 4, Win2K pro & WinCE 2.11 often with Tk800.024 &/| DBD-Unify
ftp://ftp.funet.fi/pub/languages/perl/CPAN/authors/id/H/HM/HMBRAND/


Reply via email to