Skip to content

WebUI showing NaN for folder size #6237

Closed

Description

@Pro

Expected behaviour

If the folder size is unknown, the total size is indicated as NaN undefined:
nan

Why is the folder size even unknown? The server should know how big the folder is?

Server configuration

Operating system: Ubuntu 10.04 Server

Web server: Apache

Database: MySQL

ownCloud version: 6.0RC4

Updated from an older ownCloud or fresh install: fresh install

List of activated app: Encryption

Client configuration

Browser: Firefox

Activity

  1. karlitschek commented on Dec 8, 2013

    @karlitschek
    Contributor

    @schiesbn Can you have a look?

  2. MorrisJobke commented on Dec 8, 2013

    @MorrisJobke
    Contributor

    @icewind1991 Could it be, that the cache scan isnt finished yet?

    @Pro Is it now displayed correct? Or is it still a "?"

  3. Pro commented on Dec 8, 2013

    @Pro
    Author

    Yes, it was due to the cache scan. Now it shows the correct folder size in both places.
    Anyways instead of the "NaN undefined" there should also be a "?".

    And maybe it would be a good idea to show a tooltip when the "?" is shown indicating what "?" means (e.g. "Calculating, please wait")

  4. MorrisJobke commented on Dec 8, 2013

    @MorrisJobke
    Contributor

    @Pro Yes. Now I see it. You mean the summary (I just not read it as accurate as I should :() Thanks.

  5. PVince81 commented on Dec 8, 2013

    @PVince81
    Contributor

    I guess we can make it a "?" if it is completely unknown.

    If we have a few unknown values, we could show value + '+'.
    Example:

    • Folder A - 12 MB
    • Folder B - 15 MB
    • Folder C - ?

    Total size: 27+ MB

    Not sure whether that makes sense ?
    CC @owncloud/designers

  6. Pro commented on Dec 8, 2013

    @Pro
    Author

    You could also consider showing a spinner (animated rotating circle) during cache scan instead of the "?"

  7. schiessle commented on Dec 9, 2013

    @schiessle
    Contributor

    I like the idea from @PVince81
    Regarding the idea with the spinner, it is probably not possible for the background scanner to hook into the UI

  8. MorrisJobke commented on Dec 9, 2013

    @MorrisJobke
    Contributor

    Concerning the spinner: I don't like it, because it attracts to much in relation to its importance. I'm fine with @PVince81 's approach

  9. jancborchardt commented on Dec 9, 2013

    @jancborchardt
    Contributor

    Yeah, seconding @PVince81, and I’m also against a spinner because it’s not that important.

    @Pro nice username!

  10. PVince81 commented on Jan 9, 2014

    @PVince81
    Contributor

    Just found out that if you select a mountpoint (external storage) using the checkbox, which itself doesn't have a size, it will also show "NaN" in the header. This could be fixed here as well.

  11. Pro commented on Jan 29, 2014

    @Pro
    Author

    It seems there's another problem with the size calculation (on 6.0.1):
    Here a sceenshot of the parent folder:
    Parent

    The value of 374,5MB is totally wrong, the folder is approx. 77GB.
    Here a few subfolders:
    unbenannt

  12. PVince81 commented on May 6, 2014

    @PVince81
    Contributor

    Setting as junior job.
    Should be a matter of adding a check in the code to display "Pending" instead of NaN in case where a size is not known.

  13. karlitschek commented on Dec 5, 2014

    @karlitschek
    Contributor

    @PVince81 Is this still open?

  14. PVince81 commented on Dec 5, 2014

    @PVince81
    Contributor

    Yes, likely

  15. added this to the milestone on Mar 20, 2015
  16. modified the milestones: , on Apr 4, 2015
  17. PVince81 commented on Aug 3, 2015

    @PVince81
    Contributor

    @Xenopathic didn't you fix something similar a while ago ?

  18. RobinMcCorkell commented on Aug 3, 2015

    @RobinMcCorkell
    Contributor

    The only thing I could find was #12639, but that was for the modified date not the file size.

  19. PVince81 commented on Aug 17, 2015

    @PVince81
    Contributor

    Works properly on OC 8.0.6 and later. No more "NaN" but "Pending" appears.

  20. locked as resolved and limited conversation to collaborators on Aug 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions