Script directory is renamed to 'TempScriptDirectory' when saving XML

[d.healey]

@Christoph-Hart said in Script directory is renamed to 'TempScriptDirectory' when saving XML:

there's not a single occurrence of "TempScriptDirectory" in the entire codebase

What you drinking today?

HISE/hi_backend/backend/BackendApplicationCommands.cpp at develop · christophhart/HISE

The open source framework for sample based instruments - HISE/hi_backend/backend/BackendApplicationCommands.cpp at develop · christophhart/HISE

GitHub (github.com)

[Christoph Hart]

@d-healey said in Script directory is renamed to 'TempScriptDirectory' when saving XML:

TempScriptDirectory

ah fail, I looked in the wrong project (a compiled one). makes sense now.

@Orvillain have you opened the script files in another editor? It tries to replace the files but if it fails (most likely because they are opened somewhere), it stops at the TempScriptDirectory folder.

[d.healey]

@Christoph-Hart As I put in my other thread, I was getting the same error even though the files weren't open anywhere else. Could it be because I had them open in other script tabs? I haven't seen this error for a while.

[Orvillain]

@Christoph-Hart

Unfortunately not, no. Nothing so simple!

Only HISE is open.

I'm on Windows 11 if that helps, running HISE:

[Dan Korneff]

I've been experiencing this as well. Scripts don't update when I hit compile... and then the main script gets moved to a temp folder. I haven't had time to investigate yet

[Orvillain]

@Christoph-Hart

Getting this a lot. Sometimes files even disappear outright. I've just lost an Interface.js altogether for example.

Could it have anything to do with there being hypens in the folder directory structure?

[d.healey]

@Orvillain I think Christoph fixed this today, have you built the latest commits?

[Orvillain]

@d-healey ooooo, no I haven't! I will do so!

[Christoph Hart]

@Orvillain Not sure if it fixes this issue, but I fixed the reload bug when using an external editor.

[Orvillain]

@Christoph-Hart

I'll check it out. The only oddity I can think of is, my repository has hypens in the name:
drew-hise-experiments

And some of my projects also have hypens in the name:
pad-sampler-test

etc etc.

Don't know if there is maybe a parsing issue going on when you do a save operation?

[Orvillain]

@Christoph-Hart

Unfortunately, this is still an issue. I rebuilt the latest HISE on the 17th at least, and this still happens. Nothing to do with hypens being in the name of folders or anything either.

[Dan Korneff]

@Orvillain are you working on a network drive?

[Orvillain]

@Dan-Korneff
I'm not no, but all my dev stuff is on my H: drive which is an internal SATA drive.

[Orvillain]

Still getting this even with a commit from the 28th of May.

[d.healey]

This has reappeared for me again. If I have a project and I add a sampler to it and load a sample map, when I go to save the xml I get this error and I can't save a .hip at all. It just says in the console that it's saved to the project's Presets folder, but nothing is in there.

This doesn't happen with a new project, but once the project is closed and reopened it happens.

Edit: Ahhhh if I remove the setAllowDuplicateSamples line then I don't get this either... everything's connected :( https://forum.hise.audio/topic/10719/allowduplicatesamples-causing-crash

[d.healey]

@Christoph-Hart This is still happening and it's super annoying because there's no way to save your work. You have to copy your script to a text file and close/reopen HISE then paste in your script and hope that you copied everything you needed.

I took a brief look at the source where this is happening - you love your nests :) - I see a couple of calls to deleteRecursively(); which spook me, why are things beings deleted when I try to save a file??? Ok, after stepping through the code I see what's happening here.

There are no external scripts in the test project I'm working on, just the interface script, nothing fancy.

Something I've noticed while trying to debug:

• When the issue occurs the script file (Interface.js in this case) is not created in the Temporary directory.

[d.healey]

I spent a few hours on this last night but couldn't solve it, I can't figure out why the temp file isn't being created. But I did unravel your nest to make it easier to follow the flow - https://github.com/christophhart/HISE/pull/651