Skip to content

[Cache] Add marshaller #17153

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

Open
wants to merge 1 commit into
base: 5.4
Choose a base branch
from
Open

Conversation

mohamedGasmii
Copy link
Contributor

Add marshaller constructor argument.

Add marshaller constructor argument.
@@ -17,7 +17,7 @@ memory, a component appreciably faster than many others, such as the filesystem.
this adapter.

The ApcuAdapter can optionally be provided a namespace, default cache lifetime,
and cache items version string as constructor arguments::
cache items version string and marshaller as constructor arguments::
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We probably should add reference to marshallers docs: https://symfony.com/doc/current/components/cache.html#marshalling-serializing-data

:ref:`marshaller <marshalling-serializing-data>`

components/cache.rst must also be updated from

Marshalling (Serializing) Data
------------------------------

to

.. _marshalling-serializing-data:

Marshalling (Serializing) Data
------------------------------

$version = null
$version = null,

// marshaller (classe which implement MarshallerInterface) to process the cache items before storing them
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
// marshaller (classe which implement MarshallerInterface) to process the cache items before storing them
// marshaller (object implementing Symfony\Component\Cache\Marshaller\MarshallerInterface) to process the cache items before storing them

@xabbuh xabbuh modified the milestones: 5.4, 6.4 Dec 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants