Merge pull request #306 from CfirTsabari/cfirtsabari/doesn-t-work-wit…

…h-dot-101

fix: dot in config name

src/file/source/file.rs

@@ -27,7 +27,7 @@ impl FileSourceFile {
     where
         F: FileStoredFormat + Format + 'static,
     {
-        let mut filename = if self.name.is_absolute() {
+        let filename = if self.name.is_absolute() {
             self.name.clone()
         } else {
             env::current_dir()?.as_path().join(&self.name)
@@ -59,6 +59,9 @@ impl FileSourceFile {
                 )))
             };
         }
+        // Adding a dummy extension will make sure we will not override secondary extensions, i.e. "file.local"
+        // This will make the following set_extension function calls to append the extension.
+        let mut filename = add_dummy_extension(filename);
 
         match format_hint {
             Some(format) => {
@@ -121,3 +124,18 @@ where
         })
     }
 }
+
+fn add_dummy_extension(mut filename: PathBuf) -> PathBuf {
+    match filename.extension() {
+        Some(extension) => {
+            let mut ext = extension.to_os_string();
+            ext.push(".");
+            ext.push("dummy");
+            filename.set_extension(ext);
+        }
+        None => {
+            filename.set_extension("dummy");
+        }
+    }
+    filename
+}

tests/Settings2.default.ini

@@ -0,0 +1,9 @@
+debug = true
+production = false
+[place]
+name = Torre di Pisa
+longitude = 43.7224985
+latitude = 10.3970522
+favorite = false
+reviews = 3866
+rating = 4.5

tests/file.rs

@@ -58,3 +58,13 @@ fn test_file_ext() {
     assert_eq!(c.get("debug").ok(), Some(true));
     assert_eq!(c.get("production").ok(), Some(false));
 }
+#[test]
+fn test_file_second_ext() {
+    let c = Config::builder()
+        .add_source(File::with_name("tests/Settings2.default"))
+        .build()
+        .unwrap();
+
+    assert_eq!(c.get("debug").ok(), Some(true));
+    assert_eq!(c.get("production").ok(), Some(false));
+}