Merge pull request #5744 from sharadrsoni/patch-1

Adding support for proper CRLF

2 files changed, 3 insertions, 3 deletions

diff --git a/system/core/Log.php b/system/core/Log.php
index d651e8abe..fde0bebf2 100644
--- a/system/core/Log.php
+++ b/system/core/Log.php
@@ -249,11 +249,11 @@ class CI_Log {
 	 * @param	string	$level 	The error level
 	 * @param	string	$date 	Formatted date string
 	 * @param	string	$message 	The log message
-	 * @return	string	Formatted log line with a new line character '\n' at the end
+	 * @return	string	Formatted log line with a new line character at the end
 	 */
 	protected function _format_line($level, $date, $message)
 	{
-		return $level.' - '.$date.' --> '.$message."\n";
+		return $level.' - '.$date.' --> '.$message.PHP_EOL;
 	}
 
 	// --------------------------------------------------------------------
diff --git a/tests/codeigniter/core/Log_test.php b/tests/codeigniter/core/Log_test.php
index 103af342b..927984385 100644
--- a/tests/codeigniter/core/Log_test.php
+++ b/tests/codeigniter/core/Log_test.php
@@ -57,7 +57,7 @@ class Log_test extends CI_TestCase {
 		$format_line->setAccessible(TRUE);
 		$this->assertEquals(
 			$format_line->invoke($instance, 'LEVEL', 'Timestamp', 'Message'),
-			"LEVEL - Timestamp --> Message\n"
+			"LEVEL - Timestamp --> Message".PHP_EOL
 		);
 	}
 }