more comment typos
This commit is contained in:
@@ -56,7 +56,7 @@ abstract class moodle_recordset implements Iterator {
|
||||
//public abstract function next();
|
||||
|
||||
/**
|
||||
* Revinds are not supported!
|
||||
* Rewinds are not supported!
|
||||
* @return void
|
||||
*/
|
||||
public function rewind() {
|
||||
|
||||
@@ -18,7 +18,7 @@
|
||||
/**
|
||||
* Generic temptables object store
|
||||
*
|
||||
* Provides support to databases lacking some "expeted behaviour" related
|
||||
* Provides support to databases lacking some "expected behaviour" related
|
||||
* with some operations with temporary tables like:
|
||||
*
|
||||
* - databases not retrieving temp tables from information schema tables (mysql)
|
||||
@@ -33,7 +33,7 @@
|
||||
* of doing temporary tables support 100% cross-db and transparent within the DB API.
|
||||
*
|
||||
* Only drivers needing it will use this store. Neither moodle_database (abstract) or
|
||||
* databses like postgres need this, because they don't lack any temp functionality.º:w
|
||||
* databases like postgres need this, because they don't lack any temp functionality.º:w
|
||||
|
||||
*
|
||||
* @package moodlecore
|
||||
@@ -124,7 +124,7 @@ class moodle_temptables {
|
||||
// We shouldn't have any temp table registered at the end of the script.
|
||||
// So we error_log that and, at the same time, drop all the pending temptables
|
||||
if ($temptables = $this->get_temptables()) {
|
||||
error_log('Potential coding error - existing temptables found when dispossing database. Must be dropped!');
|
||||
error_log('Potential coding error - existing temptables found when disposing database. Must be dropped!');
|
||||
foreach ($temptables as $temptable) {
|
||||
$this->mdb->get_manager()->drop_temp_table(new xmldb_table($temptable));
|
||||
}
|
||||
|
||||
@@ -73,7 +73,7 @@ class moodle_transaction {
|
||||
/**
|
||||
* Commit delegated transaction.
|
||||
* The real database commit SQL is executed
|
||||
* only after commiting all delegated transactions.
|
||||
* only after committing all delegated transactions.
|
||||
*
|
||||
* Incorrect order of nested commits or rollback
|
||||
* at any level is resulting in rollback of SQL transaction.
|
||||
|
||||
@@ -167,7 +167,7 @@ class mssql_native_moodle_database extends moodle_database {
|
||||
$this->free_result($result);
|
||||
|
||||
// Force ANSI nulls so the NULL check was done by IS NULL and NOT IS NULL
|
||||
// instead of equal(=) and distinct(<>) simbols
|
||||
// instead of equal(=) and distinct(<>) symbols
|
||||
$sql = "SET ANSI_NULLS ON";
|
||||
$this->query_start($sql, null, SQL_QUERY_AUX);
|
||||
$result = mssql_query($sql, $this->mssql);
|
||||
@@ -200,7 +200,7 @@ class mssql_native_moodle_database extends moodle_database {
|
||||
|
||||
$this->free_result($result);
|
||||
|
||||
// Connection stabilished and configured, going to instantiate the temptables controller
|
||||
// Connection stabilised and configured, going to instantiate the temptables controller
|
||||
$this->temptables = new mssql_native_moodle_temptables($this);
|
||||
|
||||
return true;
|
||||
@@ -212,7 +212,7 @@ class mssql_native_moodle_database extends moodle_database {
|
||||
* Do NOT use connect() again, create a new instance if needed.
|
||||
*/
|
||||
public function dispose() {
|
||||
parent::dispose(); // Call parent dispose to write/close session and other common stuff before clossing conn
|
||||
parent::dispose(); // Call parent dispose to write/close session and other common stuff before closing connection
|
||||
if ($this->mssql) {
|
||||
mssql_close($this->mssql);
|
||||
$this->mssql = null;
|
||||
@@ -676,7 +676,7 @@ class mssql_native_moodle_database extends moodle_database {
|
||||
$limitfrom = ($limitfrom < 0) ? 0 : $limitfrom;
|
||||
$limitnum = ($limitnum < 0) ? 0 : $limitnum;
|
||||
if ($limitfrom or $limitnum) {
|
||||
if ($limitnum >= 1) { // Only apply TOP clause if we have any limitnum (limitfrom offset is hadled later)
|
||||
if ($limitnum >= 1) { // Only apply TOP clause if we have any limitnum (limitfrom offset is handled later)
|
||||
$fetch = $limitfrom + $limitnum;
|
||||
$sql = preg_replace('/^([\s(])*SELECT( DISTINCT)?(?!\s*TOP\s*\()/i',
|
||||
"\\1SELECT\\2 TOP $fetch", $sql);
|
||||
@@ -1120,8 +1120,8 @@ class mssql_native_moodle_database extends moodle_database {
|
||||
* NOTE: this was originally returning only function name
|
||||
*
|
||||
* @param string $expr some string field, no aggregates
|
||||
* @param mixed $start integer or expresion evaluating to int
|
||||
* @param mixed $length optional integer or expresion evaluating to int
|
||||
* @param mixed $start integer or expression evaluating to int
|
||||
* @param mixed $length optional integer or expression evaluating to int
|
||||
* @return string sql fragment
|
||||
*/
|
||||
public function sql_substr($expr, $start, $length=false) {
|
||||
|
||||
Reference in New Issue
Block a user