Package Generator Wizard

Discussion of open issues, suggestions and bugs regarding ODAC (Oracle Data Access Components) for Delphi, C++Builder, Lazarus (and FPC)
Post Reply
jfudickar
Posts: 202
Joined: Fri 10 Mar 2006 13:03
Location: Oberursel / Germany

Package Generator Wizard

Post by jfudickar » Sun 18 Nov 2007 07:51

Hi,

i have problems with the Package Generator Wizard.

Take a look for example at the dbms_metadata on oracle 10g.

The datatypes of the generated procedures are often wrong. For example the open procedure. The OBJECT_TYPE parameter should be a ftString but is generated as ftFloat.

Or, when the variant parameter option is activated, there is no way to create overloads, that's not good.

Greetings
Jens

Plash
Devart Team
Posts: 2844
Joined: Wed 10 May 2006 07:09

Post by Plash » Mon 19 Nov 2007 10:09

We have fixed the problem with incorrect parameters datatypes. The fix will be included in the next build of ODAC.

Delphi distinguishes overloads by types and count of parameters. If all parameters have Variant type, Package Wizard cannot create valid overloads for any overloaded stored procedure because overloads can differ only by type of parameters.

jfudickar
Posts: 202
Joined: Fri 10 Mar 2006 13:03
Location: Oberursel / Germany

Post by jfudickar » Mon 19 Nov 2007 10:30

If you have only variant parameters, you are correct. But if you have for exampe TOraDataset parameters in combination with variant parameters, an overload is possible.

So i would suggest, to enable it in this case also.

Greetings
Jens

P.S. Is it possible to get the fix before the next release?

Plash
Devart Team
Posts: 2844
Joined: Wed 10 May 2006 07:09

Post by Plash » Tue 20 Nov 2007 11:32

We have enabled overloads for variant parameters. This change will be included in the next ODAC build. If a package contains overloads with the same parameters count, code generated by Wizard can be not compilable. In this case you should manually rename generated methods.
We have sent the Package Wizard with the fixes to your e-mail.

Post Reply