Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated PDO_SQLSRV connection to use driverOptions in prepare-function #6577

Open
DAdq26 opened this issue Oct 29, 2024 · 4 comments
Open

Updated PDO_SQLSRV connection to use driverOptions in prepare-function #6577

DAdq26 opened this issue Oct 29, 2024 · 4 comments

Comments

@DAdq26
Copy link

DAdq26 commented Oct 29, 2024

Hi team,

Bug Report

Previously closed issue #2185 on the same subject does not exist anymore in version 3.x
Actually the simple script above

$database = "mybase";
$server = "mssql";
$conn = new PDO( "sqlsrv:server=$server ; MultipleActiveResultSets=false; Database = $database", "", "");

$query = "select * from mytable";
$stmt = $conn->prepare($query);
$stmt->execute();

print "Row count : ";
print $stmt->rowCount();

returns everytime -1 because we cannot use PDO::ATTR_CURSOR attribute when we used native prepare

After debugging inside DBAL with PDO_SqlSrv wrapper we discover that prepare cannot be override like in the first issue with driverOptions
It's essential to use native Pdo::prepare($stmt, $options) to precise scrollable cursor to make it works..

$stmt = $conn->prepare($query, array(PDO::ATTR_CURSOR => PDO::CURSOR_SCROLL));

Perhaps we have not seen any option or configuration, but runtime code does not seems to modify native prepare method in
doctrine/dbal/src/Driver/PDO/Connection.php line 55 on version 3.6.6

image

If you have any advice or fix to get back $driverOptions config as in #2185, it would be perfect

Best

@derrabus
Copy link
Member

Sorry, but I don't understand this bug report. Please explain to me:

  • What are you trying to do?
  • How are you attempting to do that using DBAL APIs only?
  • What did you expect to happen?
  • What happens instead?

@DAdq26
Copy link
Author

DAdq26 commented Dec 13, 2024

Hi,

I just try to count my result here from statement method rowCount()
But as in this previous issue #2185 i can't use main driver option that seems removed before but is essential for us here to use driver options to make rowCount work well.

On my screenshot on my first post, you see your code in prepare method which cannot use prepare options anymore at runtime (previsously setted in main doctrine options with $driverOptions variable)

image

image

This was possible in an old doctrine version here
https://github.com/doctrine/dbal/pull/631/files

Actually the count result is always -1
If i use array(PDO::ATTR_CURSOR => PDO::CURSOR_SCROLL) in prepare option with buffered cursor the count result is right one

Do you need more debug or script ?

Best

@derrabus
Copy link
Member

And you need those options to be always passed to the PDO prepare() method? Or only occasionally?

@DAdq26
Copy link
Author

DAdq26 commented Dec 13, 2024

everytime ;) for all SELECT queries generated from prepare

we have hacked doctrine files to make it works temporary

image

but if we can use common driverOptions it would be better

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants