Menu

#18 replace character function bug

Version 0.8.1
closed
Matthias
bugfix (1)
5
2017-01-11
2016-03-27
David
No

Hello, Hope you're well,

The replace character function seems to remove spaces from the whole path rather than just the filename. Apologies for not noticing when you sent me the test version.

Cheers :-)

David

Discussion

  • Matthias

    Matthias - 2016-03-27

    Oops! That's true and of course, it doesn't make sense.

    BUt how should that function work then? Should only the file name be replaced?

     
  • David

    David - 2016-03-29

    indeed it should replace just the filename. The directory path would be specified as the location physically on the computer that can be set by the user. With something like google drive, it's located at c:\users\google drive\folders with spaces\in the name\and so on. google drive, for example (and i'm sure the other cloud storage solutions out there), generally just give a random string to the folder in question that don't care about the folder tree preceeding it (or the name of the actual folder itself). you just share the folder in question and generate a link for it that looks like this https://googledrive.com/host/B1gLongR4nd0mNumb3R! regardless of the location in the actual google drive tree. thus the web link doesn't care about the spaces in the actual path on the computer.

    I hope that makes sense. I think i've explained it quite badly but hopefully you can make sense of it.

    Cheers :)

    David

     
  • Matthias

    Matthias - 2016-04-01

    Dear David,

    thanks for your feedback!

    I now moved the replacement and concatenation of the path to the filename so that replacement should only apply to the filename (and the parameter-generated path elements).

    Try and let me know...

     
  • Matthias

    Matthias - 2016-04-01
    • labels: --> bugfix
    • status: open --> accepted
    • assigned_to: Matthias
    • Group: Version 0.8 --> Version 0.8.1
     
  • Matthias

    Matthias - 2017-01-11
    • status: accepted --> closed
     

Log in to post a comment.

MongoDB Logo MongoDB