@kenjis 👍 thanks for reporting this error.

I can confirm this is an error. Inthe original article at Fabien's website, theEXTR_SKIP argument was present:

@javiereguiluz The page that@kenjis changed is actualpart 6, but I agree that it should be fixed here (usingextract() might generally not be the best idea though).

Hm, actually I think it's right to not specify an argument here. The reason this was done in the previos chapters is that the default behaviour ofextract() is to overwrite any existing variables with the same name. However, this is not a good idea in the global context as we might lose our very own framework variables. But this isn't issue anymore in the present code as theextract() call is wrapped in therender_template() function.

@xabbuh theextract() has moved into a function on prev page:http://symfony.com/doc/2.3/create_framework/templating.html
So if we removeEXTR_SKIP, we should do on prev page and add why.

@kenjis Yeah, I agree that it should simply be removed on the previous page.

Hi@kenjis! Thanks for the original fix and the discussion that came with it. As we concluded that the better fix would be to remove the argument from the templating guide, I've taken your commit and changed it so it can be merged:#5568

So I'm closing this PR, but you still get all the credits!

Good news! We decided to add this argument to the other examples again, as it prevents overriding the$request variable. So it makes perfectly sense to merge this PR now. Thank you for creating it.

My pleasure.