Feature requests for dOOdads

[Peter Welzien]

Hi!

I have some feature requests:

- ConnectionString is virtual and can be overridden but all the functions uses the internal _raw variable, wouldn't it be better to let them use the virtual property as well? As for the other properties as well.
- Let LoadFromSqlNoExec() return the integer value from ExecuteNonQuery()
- Can you add a LoadFromSqlScalar() function?
- Add a Validate() or modify the Save() function to make it call a virtual \"bool ValidateRow(DataRow, RowState)\" function and abort the save through a exception/return value if it returns false.

Thanks for a great app, it has really saved my butt!


Best regards,

Peter Welzien

[mike.griffin]

Awesome suggestion, I will incorporate as many as these as I can as long as they don't break any backwards compatibility rules.

Thanx !!

[mike.griffin]

Next build of course, but here is the status of your request:

DONE: LoadFromSqlNoExec returns value from ExecuteNonQuery
DONE: Implement LoadFromSqlScalar()

Already available:
modify the Save() function to make it call a virtual \"bool ValidateRow(DataRow, RowState)\"

Look in the OleDbEntity class of the dOOdad DbAdapters folder at a function named HookupRowUpdateEvents, it is the best example for this. It traps the OnRowUpdated to fetch back the identity column for Access databases but shows you how to use those events.

There are two methods you can can trap on the DataAdpater, RowUpdating and RowUpdated.

On your connection string request I'm a little confused? What exactly is it that you're trying to accomplish?

[mthird]

Code: Select all

CREATE PROCEDURE \"PI_AGENCY\"
(
    \"I$AGENCYID\" BIGINT,
    \"I$AGENCYNAME\" VARCHAR(60),
    \"I$AGENCYCODE\" CHAR(10),
    \"I$PHONE1\" CHAR(20),
    \"I$PHONE2\" CHAR(20),
    \"I$FAX\" CHAR(20),
    \"I$WEBSITE\" VARCHAR(120),
    \"I$PRIMARYCONTACT\" VARCHAR(60),
    \"I$ADDRESSID\" BIGINT,
    \"I$CREATEDTIMESTAMP\" TIMESTAMP,
    \"I$CREATEDBYUSERID\" BIGINT,
    \"I$LASTCHANGEDTIMESTAMP\" TIMESTAMP,
    \"I$LASTCHANGEDBYUSERID\" BIGINT,
    \"I$OPTIONS\" BLOB
)
RETURNS
(
    \"AGENCYID\" BIGINT
)

AS
BEGIN

    \"I$AGENCYID\" = GEN_ID(\"KEYGENERATOR\", 1);

    INSERT INTO \"AGENCY\"
    (
        \"AGENCYID\",
        \"AGENCYNAME\",
        \"AGENCYCODE\",
        \"PHONE1\",
        \"PHONE2\",
        \"FAX\",
        \"WEBSITE\",
        \"PRIMARYCONTACT\",
        \"ADDRESSID\",
        \"CREATEDTIMESTAMP\",
        \"CREATEDBYUSERID\",
        \"LASTCHANGEDTIMESTAMP\",
        \"LASTCHANGEDBYUSERID\",
        \"OPTIONS\"
    )
    VALUES
    (
        :\"I$AgencyId\",
        :\"I$AgencyName\",
        :\"I$AgencyCode\",
        :\"I$Phone1\",
        :\"I$Phone2\",
        :\"I$Fax\",
        :\"I$Website\",
        :\"I$PrimaryContact\",
        :\"I$AddressId\",
        :\"I$CreatedTimestamp\",
        :\"I$CreatedByUserId\",
        :\"I$LastChangedTimestamp\",
        :\"I$LastChangedByUserId\",
        :\"I$OPTIONS\"
    );

    \"AGENCYID\" = :\"I$AGENCYID\";

END


[Peter Welzien]

Well Mike,

I'm trying to actually override where dOOdads is retrieving it's connection string. I'm trying to make it read it from a dynamic source so I can't use the config file and I don't really want to pass any parameters to the constructor.

This is what I want to do:

public override string ConnectionString
{
get
{
return MyObject.ConnectionString;
}
}

No real problem as I can set it in the constructor but as far as I see it - making the properties virtual don't make any sense.

/ PW

[Peter Welzien]

There is one more thing I'm missing!

I need to be able to create parameters to call stored procedures, could you add that as well?

Best regards,

Peter Welzien

[mike.griffin]

I need to be able to create parameters to call stored procedures, could you add that as well?

There is a template in the C# dOOdads template area that can do that, do you need to do it a runtime or at compile time, if compile time then we can help there.

[Peter Welzien]

Oops, didn't think of that...

Anyway, a big thanks for the help with the stored procedure handling!!!

/ P W