<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.kde.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugs.kde.org/"
          
          maintainer="sysadmin@kde.org"
>

    <bug>
          <bug_id>454182</bug_id>
          
          <creation_ts>2022-05-21 20:32:54 +0000</creation_ts>
          <short_desc>message (part) size display: please follow BinaryUnitDialect everywhere</short_desc>
          <delta_ts>2022-05-23 11:51:43 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>2</classification_id>
          <classification>Applications</classification>
          <product>kmail2</product>
          <component>general</component>
          <version>5.19.3</version>
          <rep_platform>Gentoo Packages</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          <see_also>https://bugs.kde.org/show_bug.cgi?id=364321</see_also>
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>NOR</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          <dependson>57240</dependson>
          
          <everconfirmed>0</everconfirmed>
          <reporter name="Erik Quaeghebeur">bugs.kde.org</reporter>
          <assigned_to name="kdepim bugs">pim-bugs-null</assigned_to>
          
          
          <cf_commitlink>https://invent.kde.org/pim/akonadi/commit/668e3f0dc451b24dadff554ba5e4cbc8d656481c</cf_commitlink>
          <cf_versionfixedin>5.20.2</cf_versionfixedin>
          <cf_sentryurl></cf_sentryurl>
          <votes>0</votes>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>2125378</commentid>
    <comment_count>0</comment_count>
    <who name="Erik Quaeghebeur">bugs.kde.org</who>
    <bug_when>2022-05-21 20:32:54 +0000</bug_when>
    <thetext>SUMMARY
KMail does not seem to follow the BinaryUnitDialect config option everywhere. Likely it is just a matter of replacing some function with some higher-level ones, as in a few places, KMail does do the right thing.

STEPS TO REPRODUCE
1. Set BinaryUnitDialect=2 in [Locale] section of $KDEHOME/.config/kdeglobals (setting to metric/SI instead of IEC)
2. Start KMail
3. Look at message sizes is message list, part sizes in part list and attachment list

OBSERVED RESULT
IEC formatting is used instead of metric for part sizes in part list and attachment list (metric is used in the message list)

EXPECTED RESULT
Metric formatting is used everywhere

SOFTWARE/OS VERSIONS
KDE Plasma Version: 5.24.4
KDE Frameworks Version: 5.92.0
Qt Version: 5.15.3

ADDITIONAL INFORMATION
The BinaryUnitDialect config option is described in Bug 57240. This option is for now still without a GUI, but Bug 364321 makes it clear this is about to change. So apps like KMail should be made ready to follow BinaryUnitDialect.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2125635</commentid>
    <comment_count>1</comment_count>
    <who name="Laurent Montel">montel</who>
    <bug_when>2022-05-23 06:13:43 +0000</bug_when>
    <thetext>Git commit 7b956ab759445ef9b556ee67b60ba3a6292bed9e by Laurent Montel.
Committed on 23/05/2022 at 06:13.
Pushed by mlaurent into branch &apos;release/22.04&apos;.

Fix bug 454182: message (part) size display: please follow BinaryUnitDialect everywhere
FIXED-IN: 5.20.2

M  +4    -3    messagecomposer/src/attachment/attachmentmodel.cpp
M  +2    -1    messagecomposer/src/composer/composerviewinterface.cpp
M  +1    -1    messagecore/src/attachment/attachmentfromurljob.cpp
M  +3    -2    messagecore/src/attachment/attachmentpropertiesdialog.cpp
M  +2    -2    messageviewer/src/viewer/mimeparttree/mimetreemodel.cpp

https://invent.kde.org/pim/messagelib/commit/7b956ab759445ef9b556ee67b60ba3a6292bed9e</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2125642</commentid>
    <comment_count>2</comment_count>
    <who name="Laurent Montel">montel</who>
    <bug_when>2022-05-23 06:50:03 +0000</bug_when>
    <thetext>Git commit 3150b97b3e33971b01f8c6d71c2c5d11430fe729 by Laurent Montel.
Committed on 23/05/2022 at 06:49.
Pushed by mlaurent into branch &apos;release/22.04&apos;.

Follow BinaryUnitDialect option

M  +2    -2    src/collectionpage/collectionquotawidget.cpp
M  +2    -2    src/editor/attachment/attachmentview.cpp

https://invent.kde.org/pim/kmail/commit/3150b97b3e33971b01f8c6d71c2c5d11430fe729</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2125644</commentid>
    <comment_count>3</comment_count>
    <who name="Laurent Montel">montel</who>
    <bug_when>2022-05-23 06:56:58 +0000</bug_when>
    <thetext>Git commit 668e3f0dc451b24dadff554ba5e4cbc8d656481c by Laurent Montel.
Committed on 23/05/2022 at 06:56.
Pushed by mlaurent into branch &apos;master&apos;.

Follow BinaryUnitDialect option

M  +1    -0    src/core/CMakeLists.txt
M  +4    -6    src/core/models/statisticsproxymodel.cpp
M  +2    -2    src/widgets/collectiongeneralpropertiespage.cpp
M  +3    -2    src/widgets/collectionmaintenancepage.cpp
M  +2    -2    src/widgets/collectionstatisticsdelegate.cpp

https://invent.kde.org/pim/akonadi/commit/668e3f0dc451b24dadff554ba5e4cbc8d656481c</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2125645</commentid>
    <comment_count>4</comment_count>
    <who name="Laurent Montel">montel</who>
    <bug_when>2022-05-23 06:59:24 +0000</bug_when>
    <thetext>Git commit c596517c73748271a113cd5e9be6798eaf1d48ee by Laurent Montel.
Committed on 23/05/2022 at 06:58.
Pushed by mlaurent into branch &apos;master&apos;.

Follow BinaryUnitDialect option

M  +3    -3    src/formatter/articleformatter.cpp

https://invent.kde.org/pim/akregator/commit/c596517c73748271a113cd5e9be6798eaf1d48ee</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2125646</commentid>
    <comment_count>5</comment_count>
    <who name="Laurent Montel">montel</who>
    <bug_when>2022-05-23 07:01:21 +0000</bug_when>
    <thetext>Git commit a9a35fc0ccd33f445c8d67518c827a8b36e829d0 by Laurent Montel.
Committed on 23/05/2022 at 07:00.
Pushed by mlaurent into branch &apos;master&apos;.

Fix follow BinaryUnitDialect

M  +2    -2    src/messagemodel.cpp

https://invent.kde.org/pim/akonadi-mime/commit/a9a35fc0ccd33f445c8d67518c827a8b36e829d0</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2125714</commentid>
    <comment_count>6</comment_count>
    <who name="Laurent Montel">montel</who>
    <bug_when>2022-05-23 11:51:43 +0000</bug_when>
    <thetext>Git commit 5d650a87a4c8aebafc4181f1628f08131741d5a3 by Laurent Montel.
Committed on 23/05/2022 at 11:51.
Pushed by mlaurent into branch &apos;master&apos;.

Fix follow BinaryUnitDialect

M  +2    -2    app/browsemainpage.cpp
M  +2    -2    lib/imagemetainfomodel.cpp
M  +3    -3    lib/resize/resizeimagedialog.cpp

https://invent.kde.org/graphics/gwenview/commit/5d650a87a4c8aebafc4181f1628f08131741d5a3</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>