✨ saving the data

Author: igorkulman

LocalizationEditor/Models/Localization.swift

@@ -8,10 +8,16 @@
 
 import Foundation
 
-struct Localization {
+class Localization {
     let language: String
     let translations: [LocalizationString]
     let path: String
+
+    init(language: String, translations: [LocalizationString], path: String) {
+        self.language = language
+        self.translations = translations
+        self.path = path
+    }
 }
 
 extension Localization: CustomStringConvertible {

LocalizationEditor/Models/LocalizationString.swift

@@ -8,9 +8,18 @@
 
 import Foundation
 
-struct LocalizationString {
+class LocalizationString {
     let key: String
-    let value: String
+    private(set) var value: String
+
+    init(key: String, value: String) {
+        self.key = key
+        self.value = value
+    }
+
+    func update(value: String) {
+        self.value = value
+    }
 }
 
 extension LocalizationString: CustomStringConvertible {

LocalizationEditor/Providers/LocalizationProvider.swift

@@ -6,20 +6,20 @@
 //  Copyright © 2018 Teamwire. All rights reserved.
 //
 
+import CleanroomLogger
 import Files
 import Foundation
-import CleanroomLogger
 
 class LocalizationProvider {
     func getLocalizations(url: URL) -> [Localization] {
         Log.debug?.message("Searching \(url) for Localizable.strings")
-        
+
         guard let folder = try? Folder(path: url.path) else {
             return []
         }
 
         let localizationFiles = folder.makeFileSequence(recursive: true).filter({ $0.name == "Localizable.strings" })
-        
+
         Log.debug?.message("Found \(localizationFiles) localization files")
 
         return localizationFiles.map({ file in
@@ -40,15 +40,35 @@ class LocalizationProvider {
             let s = LocalizationString(key: key, value: value)
             strings.append(s)
         }
-        
+
         Log.debug?.message("Found \(strings.count) keys for in \(path)")
-        
+
         return strings.sorted(by: { (lhs, rhs) -> Bool in
             lhs.key < rhs.key
         })
     }
 
-    func updateLocalization(localization: Localization, string: LocalizationString) {
-        Log.debug?.message("Updating \(localization) with \(string)")
+    func updateLocalization(localization: Localization, string: LocalizationString, with value: String) {
+        guard string.value != value else {
+            Log.debug?.message("Same value provided for \(string)")
+            return
+        }
+        
+        Log.debug?.message("Updating \(string) with \(value) in \(localization)")
+        
+        string.update(value: value)
+       
+        let data = localization.translations.map { string in
+            "\"\(string.key)\" = \"\(string.value.replacingOccurrences(of: "\"", with: "\\\""))\";"
+            }.reduce("") { (prev, next) in
+                "\(prev)\n\(next)"
+        }
+        
+        do {
+            try data.write(toFile: localization.path, atomically: false, encoding: .utf8)
+            Log.debug?.message("Localization file for \(localization) updated")
+        } catch {
+            Log.error?.message("Writing localization file for \(localization) failed with \(error)")
+        }
     }
 }

LocalizationEditor/Providers/LocalizationsDataSource.swift

@@ -38,11 +38,11 @@ class LocalizationsDataSource: NSObject, NSTableViewDataSource {
         return localization.translations.first(where: { $0.key == masterLocalization.translations[row].key })
     }
 
-    func updateLocalization(language: String, string: LocalizationString) {
+    func updateLocalization(language: String, string: LocalizationString, with value: String) {
         guard let localization = localizations.first(where: { $0.language == language }) else {
             return
         }
-        localizationProvider.updateLocalization(localization: localization, string: string)
+        localizationProvider.updateLocalization(localization: localization, string: string, with: value)
     }
 
     // MARK: - Delegate

LocalizationEditor/UI/Cells/LocalizationCell.swift

@@ -9,7 +9,7 @@
 import Cocoa
 
 protocol LocalizationCellDelegate: class {
-    func userDidUpdateLocalizationString(language: String, string: LocalizationString)
+    func userDidUpdateLocalizationString(language: String, string: LocalizationString, with value: String)
 }
 
 class LocalizationCell: NSTableCellView {
@@ -32,28 +32,14 @@ class LocalizationCell: NSTableCellView {
             valueTextField.delegate = self
         }
     }
-    
-    override init(frame frameRect: NSRect) {
-        super.init(frame: frameRect)
-        setup()
-    }
-    
-    required init?(coder decoder: NSCoder) {
-        super.init(coder: decoder)
-        setup()
-    }
-    
-    private func setup() {
-        
-    }
 }
 
 extension LocalizationCell: NSTextFieldDelegate {
-    override func controlTextDidEndEditing(_ obj: Notification) {
+    override func controlTextDidEndEditing(_: Notification) {
         guard let language = language, let value = value else {
             return
         }
-        
-        delegate?.userDidUpdateLocalizationString(language: language, string: LocalizationString(key: value.key, value: valueTextField.stringValue))
+
+        delegate?.userDidUpdateLocalizationString(language: language, string: value, with: valueTextField.stringValue)
     }
 }

LocalizationEditor/UI/ViewController.swift

@@ -106,7 +106,7 @@ extension ViewController: NSTableViewDelegate {
 }
 
 extension ViewController: LocalizationCellDelegate {
-    func userDidUpdateLocalizationString(language: String, string: LocalizationString) {
-        dataSource.updateLocalization(language: language, string: string)
+    func userDidUpdateLocalizationString(language: String, string: LocalizationString, with value: String) {
+        dataSource.updateLocalization(language: language, string: string, with: value)
     }
 }