Ticket #10497: storage_times.diff

django/core/files/storage.py

@@ -2 +2 @@
 import errno
 import urlparse
 import itertools
+import datetime
 
 from django.conf import settings
 from django.core.exceptions import ImproperlyConfigured, SuspiciousOperation
@@ -120 +121 @@
         """
         raise NotImplementedError()
 
+    def accessed_time(self, name):
+        """
+        Returns the last accessed time (as Datetime object) for a given name.
+        If the subclass don't support accessed times, a NotImplementedError 
+        should be raised. (just dont overwrite this method in the subclass)
+        """
+        raise NotImplementedError("This backend doesn't support accessed time.")
+
+    def created_time(self, name):
+        """
+        Returns the creation time (as Datetime object) for a given name.
+        If the subclass don't support creation times, a NotImplementedError
+        should be raised. (just dont overwrite this method in the subclass)
+        """
+        raise NotImplementedError("This backend doesn't support created time.")
+
+    def modified_time(self, name):
+        """
+        Returns the last modified time (as Datetime object) for a given name.
+        If the subclass don't support modified times, a NotImplementedError
+        should be raised. (just dont overwrite this method in the subclass)
+        """
+        raise NotImplementedError("This backend doesn't support modified time.")
+
 class FileSystemStorage(Storage):
     """
     Standard filesystem storage
@@ -220 +245 @@
             raise ValueError("This file is not accessible via a URL.")
         return urlparse.urljoin(self.base_url, name).replace('\\', '/')
 
+    def accessed_time(self, name):
+        return datetime.datetime.fromtimestamp(
+            os.path.getatime(self.path(name)))
+
+    def created_time(self, name):
+        return datetime.datetime.fromtimestamp(
+            os.path.getctime(self.path(name)))
+
+    def modified_time(self, name):
+        return datetime.datetime.fromtimestamp(
+            os.path.getmtime(self.path(name)))
+
 def get_storage_class(import_path=None):
     if import_path is None:
         import_path = settings.DEFAULT_FILE_STORAGE

docs/ref/files/storage.txt

@@ -13 +13 @@
 ``open()``. For storage systems that aren't accessible from the local
 filesystem, this will raise ``NotImplementedError`` instead.
 
+``Storage.accessed_time(name)``
+~~~~~~~~~~~~~~~~~~~~~~
+
+Returns a ``Datetime`` object containing the last accessed time of the file.
+For storage systems that aren't able to return the last accessed time, this
+will raise ``NotImplementedError`` instead.
+
+``Storage.created_time(name)``
+~~~~~~~~~~~~~~~~~~~~~~
+
+Returns a ``Datetime`` object containing the creation time of the file.
+For storage systems that aren't able to return the creation time, this
+will raise ``NotImplementedError`` instead.
+
+``Storage.modified_time(name)``
+~~~~~~~~~~~~~~~~~~~~~~
+
+Returns a ``Datetime`` object containing the last modified time. For storage
+systems that aren't able to return the last modified time, this will raise
+``NotImplementedError`` instead.
+
 ``Storage.size(name)``
 ~~~~~~~~~~~~~~~~~~~~~~
 

tests/regressiontests/file_storage/tests.py

@@ -6 +6 @@
 import time
 import unittest
 from cStringIO import StringIO
+from datetime import datetime
 from django.conf import settings
 from django.core.exceptions import SuspiciousOperation
-from django.core.files.base import ContentFile
+from django.core.files.base import ContentFile, File
 from django.core.files.images import get_image_dimensions
 from django.core.files.storage import FileSystemStorage
 from django.core.files.uploadedfile import UploadedFile
@@ -57 +58 @@
         self.storage.delete('storage_test')
         self.failIf(self.storage.exists('storage_test'))
 
+    def test_file_accessed_time(self):
+        """
+        File storage returns a Datetime object for the last accessed time of
+        a file.
+        """
+        self.failIf(self.storage.exists('test.file'))
+
+        f = File(open(__file__, 'rb'))
+        f_name = self.storage.save('test.file', f)
+
+        self.assertEqual(
+            self.storage.accessed_time(f_name),
+            datetime.fromtimestamp(os.path.getatime(self.storage.path(f_name)))
+        )
+
+        self.storage.delete(f_name)
+
+    def test_file_created_time(self):
+        """
+        File storage returns a Datetime object for the creation time of
+        a file.
+        """
+        self.failIf(self.storage.exists('test.file'))
+
+        f = File(open(__file__, 'rb'))
+        f_name = self.storage.save('test.file', f)
+
+        self.assertEqual(
+            self.storage.created_time(f_name),
+            datetime.fromtimestamp(os.path.getctime(self.storage.path(f_name)))
+        )
+
+        self.storage.delete(f_name)
+
+    def test_file_modified_time(self):
+        """
+        File storage returns a Datetime object for the last modified time of
+        a file.
+        """
+        self.failIf(self.storage.exists('test.file'))
+
+        f = File(open(__file__, 'rb'))
+        f_name = self.storage.save('test.file', f)
+
+        self.assertEqual(
+            self.storage.modified_time(f_name),
+            datetime.fromtimestamp(os.path.getmtime(self.storage.path(f_name)))
+        )
+
+        self.storage.delete(f_name)
+
     def test_file_storage_prevents_directory_traversal(self):
         """
         File storage prevents directory traversal (files can only be accessed if