> not being able to insert any new entries gave me the incentive to fix a
> few bugs:
>
>  DBIx::Compat now knows about Pg joins
>

ok

>  s/\*Table/*MainTable/ in several places that look sensible
>

that's also ok

>  stray debugging print now protected by "if debug > 2"
>

ok

>  s/OUTER/LEFT OUTER/ in BuildFields
>

that's wrong. The single OUTER (with out LEFT) is the informix case
('SupportSQLJoin' => 4,) you asked about in your previous mail. The standard
SQL case is 'SupportSQLJoin' => 1, which newer versions of Postgres also
support (and which you have set by your patch)

>
> I'm still fuzzy on SQL JOIN semantics, so someone who knows should
probably
> check that (4) is what was intended.
>

see above

> (2) is the controversial one - but "make test" still works, so there's no
> way I could have broken any existing code ;)
>

I guess you haven't tested it with all supported databases (often one works,
but another doesn't :-(, but your patchs seems to be ok and I will test it
before the next release with all dbms I have access to.

>
> Note: you still can't insert/delete/update multiple tables at once,
> this just makes it work in the presence of !LinkName, by using the
> !Table you gave, not the list of tables that would be used in a SELECT.
>

Yes, insert etc. with multiple tables is still on the todo list, but because
I don't need it at the moment and I am full of other work, I guess it will
not implemented in the next time.

Gerald


-------------------------------------------------------------
Gerald Richter    ecos electronic communication services gmbh
Internetconnect * Webserver/-design/-datenbanken * Consulting

Post:       Tulpenstrasse 5         D-55276 Dienheim b. Mainz
E-Mail:     [EMAIL PROTECTED]         Voice:    +49 6133 925131
WWW:        http://www.ecos.de      Fax:      +49 6133 925152
-------------------------------------------------------------




---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to