HandleCatalogNameStoredProcedure Issue

Posts   
 
    
dustyd
User
Posts: 1
Joined: 29-Sep-2005
# Posted on: 29-Sep-2005 23:16:04   

We're running into a problem with the Action Stored procedures using the Adapter scenario. I've traced the issue into the HandleCatalogNameStoredProcedure function. The problem is, our Catalog name contains dots (ie DatabaseName_1.2.0). In the Handle function, it's looking for indexOf("."), which does not return the correct position in our case.

I made a change to the code there, to look for ".[" instead, but this will get written over if we regenerate. I also had to comment out the line:

procName = DynamicQueryEngine.GetNewStoredProcedureName(procName)

in the CallRetrievalStoredProcedure function, as it was always returning a blank string.

Any thoughts for a workaround?

Thanks in advance

Dusty

Drewes
User
Posts: 67
Joined: 18-Aug-2003
# Posted on: 30-Sep-2005 04:03:39   

It may be a hack but to avoid the file from being overwritten you can make it read-only after you have made your changes. Maybe when Frans is back he can think of a more permanent fix.

Otis avatar
Otis
LLBLGen Pro Team
Posts: 39590
Joined: 17-Aug-2003
# Posted on: 03-Oct-2005 11:58:21   

This issue is probably solved in 1.0.2004.2, and the latest templates. There were some issues with the name construction code in earlier versions. I'll see if I can reproduce it here and fix it a.s.a.p. if it's a bug.

Frans Bouma | Lead developer LLBLGen Pro
Otis avatar
Otis
LLBLGen Pro Team
Posts: 39590
Joined: 17-Aug-2003
# Posted on: 03-Oct-2005 16:21:55   

Reproduced. Working on fix.

(edit): I'll fix this in 1.0.2005.1. For 1.0.2004.2, you have to use your updated templates. This is a cornercase, and the fix requires more changes than anticipated, but because it's a cornercase, the issue's impact is really low, as there aren't many people running into this.

The best you can do is to update the template daaAdapter.template for the sqlserver driver, for example by making a copy and a copy of the C# template set, and use that template set for the time being.

(edit2): fixed in next build of 1.0.2005.1 beta

Frans Bouma | Lead developer LLBLGen Pro