Fix variable name

[makmaoui]

No description provided.

[OskarStark]

Hi 👋

First of all: your change looks correct, also the target branch 👍

There were a lot of changes around this variable in the last time:

• edit normalizer deprecated method on 4.2 #11394
• Update serializer.rst #11781
• Update serializer.rst #11887

However I don't now why we should name it $dateCallback, I prefer $callback (in the whole example of course).

@wouterj is there any reason I can get, to name it $dateCallback?

[makmaoui]

Hi @OskarStark ,

Thanks for your reply, i propose this change especially to make example working. Actually the variable name isn't the same at creation ($callback) and at use ($dateCallback).

I choose to keep the "use" version ($dateCallback) because i consider it more descriptive but that's not the main purpose of my PR.

[wouterj]

I agree with the $dateCallback naming: It's a serializer callback specifically for the Date object. I'm not sure why it's not directly provided as argument (instead of using a variable), but I guess that has to do with the large amount of parameters it has.

So let's merge :) Thanks @makmaoui for fixing the code examples!