As the commit message says, this might fix a bug in CSV import.
However, this should be tested in all OSs to be sure.

miika
From 9f9125b918c3524be0bd3b6da4dc0ba776590929 Mon Sep 17 00:00:00 2001
From: Miika Turkia <[email protected]>
Date: Tue, 26 Apr 2016 21:58:38 +0300
Subject: [PATCH] Try different way of stating LF on CSV import
To: [email protected]

This works on my Linux box with proper new lines and with dos style line
endings. However, I have no idea how universal this is on our other OSs.
Bug #976 states that excel exported CSV file is not parsed properly due
to incorrect line termination. I assume this patch would fix reported
case. But we still need to test how it works in Windows and I suppose a
bit more testing in OSX would be in order as well...

Signed-off-by: Miika Turkia <[email protected]>
---
 xslt/csv2xml.xslt | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/xslt/csv2xml.xslt b/xslt/csv2xml.xslt
index 0f83629..10931e7 100644
--- a/xslt/csv2xml.xslt
+++ b/xslt/csv2xml.xslt
@@ -24,8 +24,7 @@
   <xsl:param name="hw" select="hw"/>
   <xsl:output method="xml" indent="yes"/>
 
-  <xsl:variable name="lf"><xsl:text>
-</xsl:text></xsl:variable>
+  <xsl:variable name="lf" select="'&#10;'"/>
   <xsl:variable name="fs">
     <xsl:choose>
       <xsl:when test="$separatorIndex = 0"><xsl:text>	</xsl:text></xsl:when>
-- 
2.5.0

_______________________________________________
subsurface mailing list
[email protected]
http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface

Reply via email to