XYplorer Beta Club

Some difference exists in how functions parse paths.

writefile(".\subfolder\data.txt", "content"); // writes to <curpath>\subfolder\data.txt
listfolder(".\subfolder"); // takes <xypath> as base, and lists <xypath>\subfolder.

(I think only goto states this <xypath>-based parsing explicitly.)

But since we can browse to paths with .\ in addressbar, it's only intuitive to expect such relative paths should resolve to <curpath> as base with all functions (except for obvious exceptions, such as load, catalogload, columnlayout, ctbicons, loadlayout, loadsettings, paperfolder, tabset, tagload, toolbar etc which take feature-specific basepaths.)

[ed+] Moreover, most places in the reference say that paths "defaults to the current folder". So it's expected that relative paths will also follow current folder as well.

Or go the other way and make writefile, readfile's parsing of dotted relative paths <xypath>-based as well.

If this is done, use <curpath> please...

1. Yes, SC listfolder had to be fixed. I just did not work as advertised.


2. SC goto is tricky. The behavior of the Address Bar actually depends on this setting (factory default is OFF):
Configuration | General | Controls and More | Miscellaneous | Address Bar relative to application path

Scripts should not depend on this setting, of course.

All in all I tend to change goto to make it work like all (? ... I hope so) other commands: make it relative to <curpath>.
However, this might break old code. So I cannot do that.

ok. listfolder now works okay. I take it other functions will follow suit?
for example: folderreport(,'r', '..\'); // list <xypath>\..\
(I can make a list of commands&functions needing update in a while.)

goto: bummer. burn all old scripts! :pitchfork:

Also

admin wrote: ↑ I just did not work as advertised.

haven't seen any advert, but from what we've seen, you are perfectly fine!

LOL

SammaySarkar wrote: ↑27 Jan 2019 15:45 (I can make a list of commands&functions needing update in a while.)

That would be nice!

I fixed folderreport() in the meantime. Don't see any others...

EDIT: ok, I see these: inputfile, inputfolder, resolvepath

OK, these commands process relative paths as <xypath>-based. broken/incomplete/clarification wanted: So far.

Thanks, that's a long list! But, are you sure that this is not advertised?

Eg for attrstamp the help says:

itemlist: |-separated list of items (full path) to attrstamp;

So you are not supposed to pass a relative path.

Judge it yourself.
But the fact of the matter is attrstamp() does not fail if given a relative path, so that's already in violation of the docs, so why not go all the way?

Good point.

Did them all. Apart from:

wow, great! (I'll add more suggestion here when/if found)

admin wrote: ↑28 Jan 2019 13:28 Did them all. Apart from:

Code: Select all

      - exists: needs to react on unaltered inputs to make sense.

Rubbish. SC exists() will be added in the next beta.

Some more notes (sorry if I'm being pedantic or to pressing ):

• savethumb() does not support relative paths. Plans to support?
• columnlayout(), loadlayout(), paperfolder() partially accepts relative paths in that they can prepend <xydata>\(columns|layouts|paper), but breaks with .\ and ..\. Do you think these should also correctly support dotted relative paths, being relative to their function-specific base locations?
  so for example, columnlayout("..\..\xydata.bak\columns\defaultcolumns.txt") will work, (load columns from a backup profile directory by pathfinding to parent folders of <xydata>\columns)
• catalogload also does not support dotted relative paths. This is more confusing because for example, catalogload("..\..\catalog.dat") fails, but the error message has the correct path!

  Code: Select all

  Not Found
  -----------------------------------
  Could not find: 
  
  D:\tools\XYplorer\catalog.dat      (note: this file exists in this exact path.)

  so the error supports relative paths better than the command itself!

SammaySarkar wrote: ↑29 Jan 2019 11:10 catalogload also does not support dotted relative paths. This is more confusing because for example, catalogload("..\..\catalog.dat") fails, but the error message has the correct path!

Code: Select all

Not Found
-----------------------------------
Could not find: 

D:\tools\XYplorer\catalog.dat      (note: this file exists in this exact path.)

so the error supports relative paths better than the command itself!

1) Cannot confirm.

2) The help says: "Can be relative to the application data folder." So I cannot change anything here.