comma-separated CSV, rather than semicolon-sep.

tjanson · Mar 10, 2015 · 260d3b5 · 260d3b5
1 parent 1223496
commit 260d3b5
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 10 deletions.
diff --git a/app/src/main/assets/logback.xml b/app/src/main/assets/logback.xml
@@ -15,7 +15,7 @@
     <appender name="FILE" class="ch.qos.logback.core.rolling.RollingFileAppender">
         <file>${LOG_DIR}/${LOG_FILE}.csv</file>
         <encoder>
-            <pattern>%date{ISO8601};%msg%n</pattern>
+            <pattern>%date{yyyy-mm-dd HH:mm:ss.SSS},%msg%n</pattern>
         </encoder>
 
         <rollingPolicy class="ch.qos.logback.core.rolling.TimeBasedRollingPolicy">

diff --git a/app/src/main/java/com/tomjanson/wifilocationlogger/WifiBroadcastReceiver.java b/app/src/main/java/com/tomjanson/wifilocationlogger/WifiBroadcastReceiver.java
@@ -72,14 +72,14 @@ public void run() {
     private void log(ScanResult wifi) {
         if (m.logToFile) {
             m.diskLog.info(m.currentLocation.getLatitude()
-                 + ";" + m.currentLocation.getLongitude()
-                 + ";" + m.currentLocation.getAltitude()
-                 + ";" + m.currentLocation.getAccuracy()
-                 + ";" + m.lastLocationUpdateTime.getTime()
-                 + ";" + wifi.SSID
-                 + ";" + wifi.BSSID
-                 + ";" + wifi.level
-                 + ";" + m.lastWifiScanTime.getTime());
+                 + "," + m.currentLocation.getLongitude()
+                 + "," + m.currentLocation.getAltitude()
+                 + "," + m.currentLocation.getAccuracy()
+                 + "," + m.lastLocationUpdateTime.getTime()
+                 + "," + wifi.SSID // TODO: escape commas
+                 + "," + wifi.BSSID
+                 + "," + wifi.level
+                 + "," + m.lastWifiScanTime.getTime());
         }
     }
 

diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
@@ -1,5 +1,5 @@
 <resources>
-    <string name="app_name">WifiLocationLogger</string>
+    <string name="app_name">Wifi Location Logger</string>
 
     <string name="unknown">—</string>
     <string name="logging_start">Start logging</string>