fix logic

Author: harminius

server/mergin/sync/files.py

@@ -220,7 +220,6 @@ def validate(self, data, **kwargs):
                     f"Unsupported file type detected: {file_path}. "
                     f"Please remove the file or try compressing it into a ZIP file before uploading.",
                 )
-
         # new checks must restrict only new files not to block existing projects
         for file in data["added"]:
             file_path = file["path"]

server/mergin/sync/utils.py

@@ -349,7 +349,7 @@ def is_valid_path(filepath: str) -> bool:
 
 def has_trailing_space(filepath: str) -> bool:
     """Check filepath for trailing spaces that makes the project impossible to download on Windows"""
-    return all(part == part.rstrip() for part in Path(filepath).parts)
+    return any(part != part.rstrip() for part in Path(filepath).parts)
 
 
 def is_supported_extension(filepath) -> bool:

server/mergin/tests/test_utils.py

@@ -229,7 +229,7 @@ def test_is_valid_path(client, filepath, allow):
 
 
 def test_has_trailing_space():
-    assert has_trailing_space("photos /lutraHQ.jpg") is False
+    assert has_trailing_space("photos /lutraHQ.jpg") is True
 
 
 def test_get_x_accell_uri(client):