CakePHP 2.x - Syntax errors when baking views. Parse error: syntax error, unexpected '$this' (T_VARIABLE), expecting identifier (T_STRING)

Question

When attempting to bake views, I consistently get stopped with syntax errors as follows:

Parse error: syntax error, unexpected '$this' (T_VARIABLE), expecting identifier (T_STRING) in /vagrant/lib/Cake/Console/Templates/default/views/index.ctp on line 27

The context for the code causing the error here doesn't seem problematic to me.

<?
foreach ($fields as $field):
  if (!in_array($field, array('created', 'body', 'description', 'position', 'slug'))) {
    echo "\t\t<th><?php echo $this->Paginator->sort('{$field}'); ?></th>\n";
  }
endforeach;
?>

It seems to have problems because it is trying to interpret the code it should generate in the string as code it should run, and thus thinks the string '$this->Paginator->sort(...)' is an object variable calling a function, and so on.

I am running PHP 5.3.37-1 on Ubuntu Trusty x64 on a Vagrant box.

`echo` inside `echo`. Do you really think it works this way? — u_mulder, Nov 01 '16 at 20:27
@u_mulder Yes, it works that way, CakePHP bake templates do generate PHP/template files. — ndm, Nov 01 '16 at 21:12

score 1 · Answer 1 · answered Nov 01 '16 at 21:23

You really shouldn't modify the core, actually you souldn't modify any vendor files at all! If you need custom bake output, then do it the proper way as described in the docs:

Cookbook > Shells, Tasks & Console Tools > Code Generation with Bake > Modify default HTML produced by “baked” templates

That being said, you are using double quotes, hence $ has a special meaning, to actually echo a $ you have to escape it using \, ie like this:

echo "\t\t<th><?php echo \$this->Paginator->sort('{$field}'); ?></th>\n";

Which is also what is used in the original bake template. Also, as mentioned in another answer, do not use short open tags!

score 0 · Answer 2 · answered Nov 01 '16 at 20:38

0

Don't use short tags: <?

Use <?php to open a PHP script and use <?= to echo when out of PHP.

As others are saying it is a simple syntax error. If you have trouble seeing syntax errors, or understanding what your logs are telling you, you may want to consider using an IDE that would help highlight these mistakes.

<?php 
foreach ($fields as $field) {
    if (!in_array($field, array('created', 'body', 'description', 'position', 'slug'))) {
        echo "\t\t<th>" .  $this->Paginator->sort($field) . "</th>\n";
    }
}
?>

answered Nov 01 '16 at 20:38

ficuscr

6,975
2
32
52

Short tags indeed shouldn't be used, they aren't the problem in this case though. Also the code is supposed to echo PHP code, as that is what CakePHP bake tempates do, they generate PHP/templates. The actual problem is the unescaped `$` in `$this`. – ndm Nov 01 '16 at 21:15
Never liked cake... – ficuscr Nov 02 '16 at 18:51

CakePHP 2.x - Syntax errors when baking views. Parse error: syntax error, unexpected '$this' (T_VARIABLE), expecting identifier (T_STRING)

2 Answers2