fix: LCFile creaet_at and updated_at

close #533

Author: weakish

leancloud/file_.py

@@ -35,6 +35,8 @@ def __init__(self, name="", data=None, mime_type=None):
         self._name = name
         self.key = None
         self.id = None
+        self.created_at = None
+        self.updated_at = None
         self._url = None
         self._successful_url = None
         self._acl = None
@@ -226,9 +228,18 @@ def _save_external(self):
         }
         response = client.post("/files".format(self._name), data)
         content = response.json()
+
         self.id = content["objectId"]
+
         self._successful_url = self._url
 
+        _created_at = utils.decode_date_string(content.get("createdAt"))
+        _updated_at = utils.decode_updated_at(content.get("updatedAt"), _created_at)
+        if _created_at is not None:
+            self.created_at = _created_at
+        if _updated_at is not None:
+            self.updated_at = _updated_at
+
     def _save_to_qcloud(self, token, upload_url):
         headers = {
             "Authorization": token,
@@ -295,6 +306,13 @@ def _update_data(self, server_data):
             self._mime_type = server_data["mime_type"]
         if "metaData" in server_data:
             self._metadata = server_data.get("metaData")
+        
+        _created_at = utils.decode_date_string(server_data.get("createdAt"))
+        _updated_at = utils.decode_updated_at(server_data.get("updatedAt"), _created_at)
+        if _created_at is not None:
+            self.created_at = _created_at
+        if _updated_at is not None:
+            self.updated_at = _updated_at
 
     def _get_file_token(self):
         data = {

leancloud/utils.py

@@ -137,12 +137,16 @@ def decode(key, value):
     if _type == "File":
         f = leancloud.File(value["name"])
         meta_data = value.get("metaData")
-        key = value.get("key")
-        if meta_data:
-            f._metadata = meta_data  
+        file_key = value.get("key")
+        if file_key is not None:
+            f.key = file_key
+        if meta_data is not None:
+            f._metadata = meta_data
         f._url = value["url"]
         f._successful_url = value["url"]
         f.id = value["objectId"]
+        f.created_at = decode_date_string(value.get("createdAt"))
+        f.updated_at = decode_date_string(value.get("updatedAt"))
         return f
 
 

tests/test_file.py

@@ -108,6 +108,7 @@ def test_save():  # type: () -> None
     assert f.name == "Blah.txt"
     assert f.mime_type == "text/plain"
     assert not f.url.endswith(".")
+    assert f.created_at == f.updated_at
 
 
 @with_setup(setup_func)
@@ -118,6 +119,7 @@ def test_save_with_specified_key():  # type: () -> None
     f.save()
 
     assert f.id
+    assert f.created_at == f.updated_at
     assert f.name == "Blah.txt"
     assert f.mime_type == "text/plain"
     path = urlparse(f.url).path
@@ -146,9 +148,11 @@ def test_query():  # type: () -> None
     files = leancloud.Query("File").find()
     for f in files:
         assert isinstance(f, File)
+        assert f.id
         assert f.url
         assert f.name
         assert f.metadata
+        assert f.created_at == f.updated_at
         if f.metadata.get("__source") == 'external':
             assert f.url
         else:
@@ -161,6 +165,7 @@ def test_scan():  # type: () -> None
     files = leancloud.Query("File").scan()
     for f in files:
         assert isinstance(f, File)
+        assert f.created_at == f.updated_at
         assert f.name
         assert f.metadata
         if f.metadata.get("__source") == 'external':
@@ -176,6 +181,7 @@ def test_save_external():  # type: () -> None
     f = File.create_with_url(file_name, file_url)
     f.save()
     assert f.id
+    assert f.created_at == f.updated_at
     file_on_cloud = File.create_without_data(f.id)
     file_on_cloud.fetch()
     assert file_on_cloud.name == file_name