Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update default cache name based on CakePHP 5.1.0 Application Skeleton #148

Merged
merged 1 commit into from
Sep 16, 2024

Conversation

@dereuromark dereuromark merged commit 2cd7d77 into dereuromark:master Sep 16, 2024
1 of 8 checks passed
@dereuromark
Copy link
Owner

For some reason the tests fail

@dereuromark
Copy link
Owner

dereuromark commented Sep 16, 2024

Maybe we shouldn't use translations key
cake_model seems more appropriate

@MarksSoftwareGmbH
Copy link
Contributor Author

For some reason the tests fail

Correct, after creating a new cakephp/app based on the latest skeleton and installing tinyauth the app runs into the non existent "cake_translations" cache configuration.

[Cake\Cache\Exception\InvalidArgumentException] The cake_core cache configuration does not exist.

You are right the test failed.

@MarksSoftwareGmbH
Copy link
Contributor Author

Maybe we shouldn't use translations key cake_model seems more appropriate

I will solve this issue by updating the latest cakephp/app app.php Cache Config array with 'cake_core' and 'cake_translations'

`'cake_translations' => [
'className' => FileEngine::class,
'prefix' => 'myapp_cake_translations_',
'path' => CACHE . 'persistent' . DS,
'serialize' => true,
'duration' => '+1 years',
'url' => env('CACHE_CAKECORE_URL', null),
],

'cake_core' => [
'className' => FileEngine::class,
'prefix' => 'myapp_cake_core_',
'path' => CACHE . 'persistent' . DS,
'serialize' => true,
'duration' => '+1 years',
'url' => env('CACHE_CAKECORE_URL', null),
]`

@dereuromark
Copy link
Owner

What do you think about this one?
9780aa0

@MarksSoftwareGmbH
Copy link
Contributor Author

What do you think about this one? 9780aa0

You wrote, what I thought about a few min ago, to capsule / check for the old config name. thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants