Namespaced version issue - code window not recognising wire()->cache in ready.php

[Notanotherdotcom]

Hi Adrian,

Thanks for the namespaced version, much appreciated and really sorry I've not checked it out sooner - but today is the day.

I have some code in ready.php which isn't namespaced yet in a project, so when I use the Console it complains when I have code in ready.php like:

$config->settings = wire('cache')->get('settings', WireCache::expireNever, function () {
...
}

With an error: Exception: Class "WireCache" not found etc.

I will get around to moving this very custom PW install to namespaces soon but wondering what's going on because in non-namespaced Tracy it's fine.

I think the answer is probably that I need to be using namespaces across my files for the namespaced Tracy to work (quick fix) but I don't understand why it's not using the compiled files - if you see what I mean. Because ready.php is getting compiled by FileCompiler so it has a namespace when the site runs, so it's like the Console window in Tracy is somehow running against the non-compiled site files.

Hopefully I've described that clearly - possibly not.

[adrianbj]

Hi @Notanotherdotcom - Tracy includes ready.php and finished.php here:

TracyDebugger/includes/CodeProcessor.php

Lines 32 to 37 in be71dd9

	// ready.php and finished.php weren't being loaded, so include here to monitor any bd() etc calls they might have
	// the other approach to fix this is to call an external CodeProcessor.php file via ajax as per PM with @bernhard
	$readyPath = $this->wire('config')->paths->root . 'site/ready.php';
	$finishedPath = $this->wire('config')->paths->root . 'site/finished.php';
	if(file_exists($readyPath)) include_once($readyPath);
	if(file_exists($finishedPath)) include_once($finishedPath);

so I am guessing that's the source of the problem.

I am not sure the best way to handle this though. I got caught up in a nightmare of knots with my AdminActions module trying to support 3rd party actions that were or were not namespaced and I feel like it might be the same issue here. I would have thought PW would check to see if a compiled version of a file existed when loading from $this->wire('config')->paths but I guess not. I suppose I could do that in Tracy. Elsewhere I do this to get the path to the compiled files. Do I just need to check to see if compiled versions of ready.php and finished.php exist and load them from there, falling back to the main path otherwise?

        $compilerCachePath = isset($this->wire('config')->fileCompilerOptions['cachePath']) && $this->wire('config')->fileCompilerOptions['cachePath'] != '' ? $this->wire('config')->fileCompilerOptions['cachePath'] : $this->wire('config')->paths->cache . 'FileCompiler/';
        $compilerCachePath = str_replace('/', DIRECTORY_SEPARATOR, $compilerCachePath);

[adrianbj]

Hey @Notanotherdotcom - did you get a chance to test if pointed those ready and finished includes work if the $compilerCachePath is added? Also, I wonder if using $files->includeOnce() rather than include_once() would also work?

[adrianbj]

Hi @Notanotherdotcom - I just fully uninstalled Tracy and removed the namespace declaration from my ready.php file and I got the same error so I am not sure that this is actually a Tracy issue - I think it might be related to the file compiler not kicking as it should, although I have no idea why.

This is issue report is holding me up from pushing the namespaced version to master, so it would be great to get your feedback on this please.

[adrianbj]

Hey Pete,

I made the change to $files->compileIncludeOnce() - hopefully it works OK at your end.

I am getting ready to push the namespaced version to master for a new release, so speak up now if you have any issues with this still 😁

The easiest way to test is to Modules > Add New > From URL:
https://github.com/adrianbj/TracyDebugger/archive/refs/heads/namespaced.zip

[Notanotherdotcom]

Hi Adrian, sorry again for not getting back to you - I am doing some testing on various things today so will get on this as it is one of a handful of modules I use that needs filecompiler unless I use this version.

I'll test as soon as possible in fact.