# HG changeset patch
# User Goffi <goffi@goffi.org>
# Date 1447262389 -3600
# Node ID 833bdb227b16e55df991feb2c0313cb9ae719237
# Parent  babd97d80049317757e2de5e6a12a7df3aad136c
plugins XEP-0234, file: moved human file size conversion to file plugi

diff -r babd97d80049 -r 833bdb227b16 src/plugins/plugin_misc_file.py
--- a/src/plugins/plugin_misc_file.py	Wed Nov 11 18:19:47 2015 +0100
+++ b/src/plugins/plugin_misc_file.py	Wed Nov 11 18:19:49 2015 +0100
@@ -170,11 +170,14 @@
             It may content the key used in CONFIRM constant
             It *MUST NOT* contain the "peer" key
             "file_path" will be added to this dict once destination selected
+            "size_human" will also be added with human readable file size
         @param profile: %(doc_profile)s
         return (defer.Deferred): True if transfer is accepted
         """
         filename = file_data['name']
         assert filename and not '/' in filename
+        # human readable size
+        file_data['size_human'] = u'{:.6n} Mio'.format(float(file_data['size'])/(1024**2))
         d = xml_tools.deferDialog(self.host,
             _(CONFIRM).format(peer=peer_jid.full(), **file_data),
             _(CONFIRM_TITLE),
diff -r babd97d80049 -r 833bdb227b16 src/plugins/plugin_xep_0234.py
--- a/src/plugins/plugin_xep_0234.py	Wed Nov 11 18:19:47 2015 +0100
+++ b/src/plugins/plugin_xep_0234.py	Wed Nov 11 18:19:49 2015 +0100
@@ -114,12 +114,9 @@
                 file_data[name] = ''
 
         try:
-            size = file_data['size'] = int(file_data['size'])
+            file_data['size'] = int(file_data['size'])
         except ValueError:
             raise failure.Failure(exceptions.DataError)
-        else:
-            # human readable size
-            file_data['size_human'] = u'{:.6n} Mio'.format(float(size)/(1024**2))
 
         name = file_data['name']
         if '/' in name or '\\' in name: