[bug] Full expansion audio files not being pooled.

[d.healey]

@Dan-Korneff Is the Icon.png image in the images folder?

[Dan Korneff]

@d-healey nope... I just downloaded the zip file for the 3rd time. Finally has it
Checking

[d.healey]

@Dan-Korneff Probably some kind of caching issue.

[Dan Korneff]

@d-healey I'm getting 2 error in PresetHandler.cpp

Is this normal?

[d.healey]

That might be because there isn't a Samples folder in appData. What is the value of dir?

[Dan Korneff]

@d-healey Samples (5)

[d.healey]

@Dan-Korneff Adding a Samples folder to appData should solve that.

[d.healey]

I'm guessing the problem can be traced from ScriptingApi::Engine::loadAudioFilesIntoPool()

It's like it looks in the binary (Rhapsody) but not in the expansion hxi.

[Christoph Hart]

@d-healey Spot on!

https://github.com/christophhart/HISE/commit/01c1780c17a51e5608b2837c0ceb2e315ab70a55

[d.healey]

@Christoph-Hart Thank you!

[d.healey]

@Christoph-Hart I think I've hit another similar problem. This time I'm using the loop player. I save a preset with a file (from the expansion's samples folder) loaded but when I open the preset in the player the file isn't there.

I've been poking around in ExternalFilePool.cpp and think I'm in the right location but do you have any pointers?

[d.healey]

@Christoph-Hart I've narrowed it right down now.

The problem is here - https://github.com/christophhart/HISE/blob/develop/hi_core/hi_core/ExternalFilePool.cpp#L538

There isn't any handling for the sampleFolderWildcard.

So as a test I added this:

		else if (input.startsWith(sampleFolderWildcard))
		{
			if (auto e = mc->getExpansionHandler().getCurrentExpansion())
			{
				input = input.replace(sampleFolderWildcard, "/media/dave/Projects/projectName/Samples/");
			}
		}

And it works. However when I try to use the expansion wildcard instead of the absolute path it still doesn't resolve. Can you suggest the last bit of the solution?

[d.healey]

Aha!

input = input.replace(sampleFolderWildcard, e->getSubDirectory(FileHandlerBase::Samples).getFullPathName() + "/");

[Dan Korneff]

@d-healey