Diffrent volt partial() in php 5.6/7.0

[FlexIDK]

Expected and Actual Behavior

In PHP 5.6

Volt:
partial("email/tag_open", {'type': 'p'}) - $type is local var
{{ type }} - not set
partial("email/tag_open", {}) - $type inside - not set

In PHP 7

Volt:
partial("email/tag_open", {'type': 'p'}) - $type is global var
{{ type }} - last state set
partial("email/tag_open", {}) - $type inside - last state set

Details

• Phalcon version: 3.0.1
• PHP Version: 5.6.24-0+deb8u1 / 7.0.10-1~dotdeb+8.1
• Operating System: Debian 8.5
• Installation type: Compiling from source

Refs

• require statement breaks local variable scope zephir-lang/zephir#1621

[andresgutierrez]

PHP 5 allowed us to create intermediate symbol tables per render, in php7 that is not possible, we have to research more about it.

[janisbiz]

I would like to suggest this fixing ASAP, as this is braking all the partial system in the views. (quite important thing)

This happens only on php7, as @andresgutierrez described, thought - php5 (with phalcon 3) is all good.

Also it is happening with .phtml, not only with Volt.

[challet]

It's also driving the loop context crazy :

{% for d in data %}
    {{ dump(loop.first) }} // true 
    // there is a {% for t in tags %} inside it
    {% include "header" with ['active': loop.first, 'tags': d.tags] %} 
    {{ dump(active) }} // true 
    {{ dump(loop.first) }} // false 
{% endfor %}

[jl91]

some news about it? someone knows when it will be fixed? what branch ? or date? i've identified this problem in phalcon 3.0.0, but now i'm migrating to phalcon 3.0.3!

[Jurigag]

Im guessing if someone would know then he would do PR with fix. We have to wait until @andresgutierrez will do something about this.

[jellisii]

Can confirm this is still a problem, particularly the loop context that @challet referenced. I can put together a test case if needed, as this is causing me headaches.

[janisbiz]

Can confirm as now. (23.11.2017). Due to this issue cannot upgrade two of my projects to php 7+ :( Same behavior is with .phtml, so @sergeyklay Volt label can be removed, as it's affecting all kind of views.