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

FileSystem Storage Adapter does not take correct path #89

Closed
modir opened this issue May 31, 2024 · 4 comments
Closed

FileSystem Storage Adapter does not take correct path #89

modir opened this issue May 31, 2024 · 4 comments
Labels
Bug Something isn't working
Milestone

Comments

@modir
Copy link
Contributor

modir commented May 31, 2024

Bug Report

I followed roughly this page https://docs.laminas.dev/laminas-cache/v3/application-integration/usage-in-a-laminas-mvc-application/

Even tough I have the cache_dir parameter like here

return [
    'caches' => [
        'default-cache' => [
            'adapter' => Laminas\Cache\Storage\Adapter\Filesystem::class,
            'options' => [
                'cache_dir' => __DIR__ . '/../../data/cache',
            ],
        ],
    ],
];

it is not used. It seems as if under Windows always the default directory from php.ini is taken and this parameter does nothing. My error message is:

PHP Fatal error: Uncaught Laminas\Cache\Exception\InvalidArgumentException: Cache directory 'C:\Windows\TEMP' not writable in D:\webdirectory\dispotool\vendor\laminas\laminas-cache-storage-adapter-filesystem\src\FilesystemOptions.php:444

Q A
Version(s) laminas-cache (3.10.1), laminas-cache-storage-adapter-filesystem (2.3.0)

Expected behavior

That the directory defined with cache_dir would be taken as the cache directory.

@modir modir added the Bug Something isn't working label May 31, 2024
@froschdesign
Copy link
Member

@modir
Please check the following issue report: #65 and the related pull request because it looks like the same problem. Please try the suggested solution and give feedback.
Thanks in advance! 👍🏻

@modir
Copy link
Contributor Author

modir commented Jun 13, 2024

@modir Please check the following issue report: laminas/laminas-cache-storage-adapter-filesystem#65 and the related pull request because it looks like the same problem. Please try the suggested solution and give feedback. Thanks in advance! 👍🏻

It is indeed the same problem. I will test it today and then give feedback. Should I give the feedback here or directly in the other issue? Or even in the pull request?

@froschdesign
Copy link
Member

The related pull request is a good place for this.

@boesing boesing transferred this issue from laminas/laminas-cache Jun 16, 2024
@boesing boesing added this to the 2.4.1 milestone Jul 10, 2024
@boesing
Copy link
Member

boesing commented Jul 10, 2024

Fixed with #84

@boesing boesing closed this as completed Jul 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants