"Deploy Cache 0 Paths in database" for GCS add-on

[tpoindessous]

Hi !

I'm using GCS add-on and on "caches" page, the result for Deploy Cache is always 0.

I looked in mysql db and in wp_wp2static_deploy_cache, I have 6300 lines .

A select gives me this :

select * from wp_wp2static_deploy_cache limit 1;
+----+----------------------------------+---------------------+----------------------------------+-----------------------------+
| id | path_hash                        | path                | file_hash                        | namespace                   |
+----+----------------------------------+---------------------+----------------------------------+-----------------------------+
|  1 | de5626a32cd74a40321a843bcfdc50d6 | /contact/index.html | 89088e44a5d25e5d0ee05b981e9aedab | wp2static-addon-gcs/default |
+----+----------------------------------+---------------------+----------------------------------+-----------------------------+

thanks !

[leonstafford]

hmm, not familiar enough with the GCS one yet, sorry!

@john-shaffer any ideas?

[john-shaffer]

This is the case with the S3 addon as well. I think it's because they use namespaces, and WP2Static is only counting the default namespace.

[john-shaffer]

Setting true on https://github.com/leonstafford/wp2static/blob/c6b47808a58412f5865a595072380b451316d186/src/ViewRenderer.php#L317 fixes this. The functionality is already there, it just has to be turned on.

Since the default behavior is broken with namespaces, I'd be fine with removing it and making "by namespace" the only behavior. Thoughts?

[petewilcock]

@john-shaffer Keen to see a fix here, mostly because I'm seeing failures with the S3 addon that emit nothing in logs (even PHP logs) and the lack of a deploy cache is making it v. hard to track down.

[john-shaffer]

@petewilcock Just change false to true in the line that I marked above.

[leonstafford]

I've just defaulted it to true.

@john-shaffer what are the implications of removing all the non-combined namespace deploy cache code?

[john-shaffer]

I think removal would be ideal. I think the two branches are only there because the old way was preserved when namespaces were added, but if someone ever does need the old system then they can just use 'default' namespace.

[leonstafford]

Cool, I'll do the cleanup.

[leonstafford]

@john-shaffer could you please give b6f0424 a test with a cpl of namespaced deploys?

I flipped the method names in DeployCache and made minor adjustments to views/CLI. Left the method to get totals just for one namespace, as that should give flexibility in scripting.