Hi Neil,

Sorry but when I change it from &dir= to just &dir= it breaks it on windows with IIS, it causes folders to be duplicated in recursion in the ui.

& is the correct way to encode & in urls in html, it is not always needed in javascript representations of urls but in this case it seems to be needed or it breaks the feature.

Best,

Joe

Hi Neil,

Sorry but you are not correct.

Changing that breaks it, if you change &dir= to &dir= it causes folders to be duplicated in recursion so a folder contains itself and its children recursively.

If you remove it altogether like you have suggested in your second post it does not show the sub folders and files.

You have not tested it well enough, if you add folders and sub folders and files you will see how it breaks.

Best,

Joe

sorry but 2.3.4.2 is a very old version, probably there were changes in the jqueryfiletreeservice or other changes that affect this.

If you want to suggest a patch you should work with the latest code.

Best,

Joe

Hi Neil,

We are talking about the same line of code. It works correctly like this:

script.Append(", script: '" + navigationRoot + "/Services/jqueryFileTreeMediaBrowser.ashx?type=" + browserType + "&dir=" + currentDir + "'");

as shown in this screen shot:

if I change the code like this:

script.Append(", script: '" + navigationRoot + "/Services/jqueryFileTreeMediaBrowser.ashx?type=" + browserType + "&dir=" + currentDir + "'");

it results in recursive duplicated folders as shown here:

notice how media folder contains itself and if you keep going deeper it keeps duplicating

if I change the code like this:

script.Append(", script: '" + navigationRoot + "/Services/jqueryFileTreeMediaBrowser.ashx?type=" + browserType);

I get no sub folders at all as shown here:

I am testing in VS 2010 on Win7 x64 using .NET 4.0 and IIS7 using CKeditor and the very latest mojoPortal code

If you say it works on Mono and you verify it does not cause those side effects, then I am willing to change the code like this:

#if MONO
            script.Append(", script: '" + navigationRoot + "/Services/jqueryFileTreeMediaBrowser.ashx?type=" + browserType);
#else
            script.Append(", script: '" + navigationRoot + "/Services/jqueryFileTreeMediaBrowser.ashx?type=" + browserType + "&dir=" + currentDir + "'");
#endif

so it can be compiled differently for Mono

But I am very surprised if it works correctly without passing in the expected parameter for the current directory.

Best,

Joe

Hi Neil,

doh! My bad, I was leaving off the closing single quote, it does work, I will make this change.

Sorry to be so difficult ;-D

I am glad to have someone testing on Mono.

Best,

Joe

This change is now in the source code repository.

I am glad you are obstinate and kept pushing back on this, I was obstinate but incorrect :-D

Thanks!

Joe