From 76a11854c7dff0d07341dd695b7ecb543c8b8e20 Mon Sep 17 00:00:00 2001
From: Manuel Reinhardt <reinhardt@syslab.com>
Date: Fri, 5 Oct 2018 13:36:12 +0200
Subject: [PATCH] Replaced usages of getObjSize with human_readable_size. #60

---
 Products/CMFEditions/browser/configure.zcml                | 2 ++
 .../CMFEditions/browser/templates/version_file_view.pt     | 2 +-
 .../CMFEditions/browser/templates/version_image_view.pt    | 2 +-
 Products/CMFEditions/browser/views.py                      | 7 +++++++
 news/60.feature                                            | 1 +
 5 files changed, 12 insertions(+), 2 deletions(-)
 create mode 100644 news/60.feature

diff --git a/Products/CMFEditions/browser/configure.zcml b/Products/CMFEditions/browser/configure.zcml
index b8047316..f995fad3 100644
--- a/Products/CMFEditions/browser/configure.zcml
+++ b/Products/CMFEditions/browser/configure.zcml
@@ -52,6 +52,7 @@
     <browser:page
         for="*"
         name="version_file_view"
+        class=".views.VersionView"
         template="templates/version_file_view.pt"
         permission="zope2.View"
         />
@@ -59,6 +60,7 @@
     <browser:page
         for="*"
         name="version_image_view"
+        class=".views.VersionView"
         template="templates/version_image_view.pt"
         permission="zope2.View"
         />
diff --git a/Products/CMFEditions/browser/templates/version_file_view.pt b/Products/CMFEditions/browser/templates/version_file_view.pt
index 81b7531b..42039c95 100644
--- a/Products/CMFEditions/browser/templates/version_file_view.pt
+++ b/Products/CMFEditions/browser/templates/version_file_view.pt
@@ -8,7 +8,7 @@
 <metal:main fill-slot="main">
     <tal:main-macro metal:define-macro="main"
            tal:define="size_value context/get_size;
-                       size_label python:size_value and context.getObjSize(context) or None;
+                       size_label python:size_value and view.human_readable_size(size_value) or None;
                        content_type context/get_content_type|context/Format;
                       ">
 
diff --git a/Products/CMFEditions/browser/templates/version_image_view.pt b/Products/CMFEditions/browser/templates/version_image_view.pt
index 672a3b79..e6295a62 100644
--- a/Products/CMFEditions/browser/templates/version_image_view.pt
+++ b/Products/CMFEditions/browser/templates/version_image_view.pt
@@ -26,7 +26,7 @@
             &mdash;
                 <span i18n:translate="label_size">Size</span>:
 
-                <span tal:define="size python:here.getObjSize(here)"
+                <span tal:define="size python:view.human_readable_size(here.get_size())"
                       tal:content="size">
                     File size
                 </span>
diff --git a/Products/CMFEditions/browser/views.py b/Products/CMFEditions/browser/views.py
index 8db16663..29ab7fb9 100644
--- a/Products/CMFEditions/browser/views.py
+++ b/Products/CMFEditions/browser/views.py
@@ -6,6 +6,7 @@
 from Products.CMFEditions.utilities import isObjectChanged
 from Products.CMFEditions.utilities import isObjectVersioned
 from Products.CMFEditions.utilities import maybeSaveVersion
+from Products.CMFPlone.utils import human_readable_size
 from Products.statusmessages.interfaces import IStatusMessage
 
 class UpdateVersionOnEditView(BrowserView):
@@ -86,3 +87,9 @@ def __call__(self):
         tag = '<img src="%s/file_download_version?version_id=%s" %s' % \
               (here_url, version_id, working_copy_tag[altPos:])
         return tag
+
+
+class VersionView(BrowserView):
+
+    def human_readable_size(self):
+        return human_readable_size
diff --git a/news/60.feature b/news/60.feature
new file mode 100644
index 00000000..a45935e0
--- /dev/null
+++ b/news/60.feature
@@ -0,0 +1 @@
+Replaced usages of getObjSize with human_readable_size.