Merge pull request #1357 from pasamio/oracle_timestamp

Update Oracle driver to set timestamp format as well as date format
joomla · Jul 12, 2012 · 6a9d3d4 · 6a9d3d4
2 parents ec5b559 + bf9f36e
commit 6a9d3d4
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/libraries/joomla/database/driver/oracle.php b/libraries/joomla/database/driver/oracle.php
@@ -411,24 +411,25 @@ public function select($database)
      * that matches the MySQL one used within most Joomla
      * tables.
      *
-     * @param   string  $dateformat  Oracle Date Format String
+     * @param   string  $dateFormat  Oracle Date Format String
      *
      * @return boolean
      *
      * @since  12.1
      */
-	public function setDateFormat($dateformat = 'DD-MON-RR')
+	public function setDateFormat($dateFormat = 'DD-MON-RR')
 	{
 		$this->connect();
 
-		$this->setQuery("alter session set nls_date_format = '$dateformat'");
+		$this->setQuery("ALTER SESSION SET NLS_DATE_FORMAT = '$dateFormat'");
+		$this->setQuery("ALTER SESSION SET NLS_TIMESTAMP_FORMAT = '$dateFormat'");
 
 		if (!$this->execute())
 		{
 			return false;
 		}
 
-		$this->dateformat = $dateformat;
+		$this->dateformat = $dateFormat;
 
 		return true;
 	}