Added support to set a custom parameter prefix (instead of '@')#152
Added support to set a custom parameter prefix (instead of '@')#152jonataspc wants to merge 7 commits intoDapperLib:mainfrom
Conversation
aravindk777
left a comment
There was a problem hiding this comment.
Looks good to me. A much needed one. If I may also request, can the readme/documentation be updated about this feature/custom support? Let me know if i can be of any help in contributing as well. At this point, I am not sure if I have the authority to approve and the contributor process to adhere in.
Sure, documentation was changed to describe the feature. |
aravindk777
left a comment
There was a problem hiding this comment.
Looks good to me. Already shared my review feedback earlier and providing my approval after review of the file(s) that underwent the change to support this.
|
@jonataspc - This requires a maintainer to give the final approval to merge/approval workflows. |
|
Does it solve ODBC parameters syntax : Or does it need a parameter suffix to be add ? |
I think you'll need a parameter suffix to work with ODBC indeed. |
Added support for defining a custom parameter prefix (instead of the hard-coded
@) for SQL query and parameters collection, separately.Some providers have problems with
@as a default parameter prefix when used with Dapper, such as Devart dotConnect for MySQL. In that specific case, we have to use:on the SQL query instead (e.g:SELECT foo FROM table WHERE bar=:parameter). With this feature we are able to use Dapper.Contrib on those scenarios.The configuration was implemented using delegate pattern (same way of
GetDatabaseTypeandTableNameMapper)Example of use:
This PR may also solve/fix:
#14
#124
#127
Related topics:
DapperLib/Dapper#1296
DapperLib/Dapper#1232