I'm not sure if this change is really needed. The code that renders templates uses theextract() PHP function to make the variables available in the template, right?

ob_start();extract($request->query->all(),EXTR_SKIP);// <-- this is where vars are made availableincludesprintf(__DIR__.'/../src/pages/%s.php',$map[$path]);$response =newResponse(ob_get_clean());

I thought the same but when running the code (on windows 10 with PHP7.0.2, not sure if this matters), I do get theNotice: Undefined variable: name Notice.

Do you want me to share the code provided up to this point in the article (To make it easier to get an overview) ?

It looks like you did forget to include thename URL parameter when accessing the page in your browser. However, we may want to think about adding a check that displays a different sentence when no name was given. What do you think?

In the rest of the article there is always a backup, (world is used as alternative). I guess it's a matter of how much you would like to cover in the article.

If you want I can open another pull request with a catch in it.

👍 for adding a default value.

I just noticed that we already have#6419 which does that. So I think we sadly have to close this as a duplicate and merge#6419 instead.

Ok, great.
Thanks for the update!

Thanks for your responses@vanbrabantf. Unfortunately, I'm going to close this one in favor of#6419. I hope to see you back with a PR that we can merge in the future!