tonyair/silverstripe-mssql
1
0
Fork 0
mirror of https://github.com/silverstripe/silverstripe-mssql synced 2024-10-22 08:05:53 +02:00
Code Issues Projects Releases Wiki Activity

MINOR Tidied up comments, added is_resource() check on MSSQLQuery::seek() to be consistent with the other methods

Browse Source
This commit is contained in:
Sean Harvey 2010-10-05 00:55:16 +00:00
parent d2082107b5
commit 35d04ac387
1 changed files with 6 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
code/MSSQLDatabase.php
View File

@ -1604,29 +1604,31 @@ class MSSQLQuery extends SS_Query {
}
public function seek($row) {
if(!is_resource($this->handle)) return false;
if($this->mssql) {
return mssql_data_seek($this->handle, $row);
} else {
user_error("MSSQLQuery::seek() sqlsrv doesn't support seek.", E_USER_WARNING);
user_error('MSSQLQuery::seek() not supported in sqlsrv', E_USER_WARNING);
}
}
public function numRecords() {
if(!is_resource($this->handle)) return false;
if($this->mssql) {
return mssql_num_rows($this->handle);
} else {
// WARNING: This will only work if the cursor type is NOT forward only!
// WARNING: This will only work if the cursor type is scrollable!
if(function_exists('sqlsrv_num_rows')) {
return sqlsrv_num_rows($this->handle);
} else {
user_error("MSSQLQuery::numRecords() not supported on this version of sqlsrv.", E_USER_WARNING);
user_error('MSSQLQuery::numRecords() not supported in this version of sqlsrv', E_USER_WARNING);
}
}
}
public function nextRecord() {
if(!is_resource($this->handle)) return false;
// Coalesce rather than replace common fields.