New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Problem with switch and filters in php7 #13242

Closed
taketi32 opened this Issue Dec 30, 2017 · 5 comments

Comments

Projects
None yet
3 participants
@taketi32

taketi32 commented Dec 30, 2017

Hello,

In the new version (3.3) of Phalcon, the switch was brought in php7. My project is in the process of migrating from php5 to php7, so I have just encountered a problem between the switch and the default filter.

screenshot_20171230_112339

{{ myvar|default("abc") }}
@sergeyklay

This comment has been minimized.

Show comment
Hide comment
@sergeyklay

sergeyklay Dec 30, 2017

Member

Could you please provide the code to reproduce?

Member

sergeyklay commented Dec 30, 2017

Could you please provide the code to reproduce?

@Jurigag

This comment has been minimized.

Show comment
Hide comment
@Jurigag

Jurigag Dec 30, 2017

Member

He provided, just use this code in volt.

Member

Jurigag commented Dec 30, 2017

He provided, just use this code in volt.

@taketi32

This comment has been minimized.

Show comment
Hide comment
@taketi32

taketi32 Dec 31, 2017

Parts of volt file :

{% for c in columns %}  
   <th data-column-id="{{c['name']}}" data-type="{{c['type']}}" data-sortable="{{c['sortable']}}" data-visible={{ c['visible']|default('true') }}>{{c['label']}}</th>
{% endfor %}

taketi32 commented Dec 31, 2017

Parts of volt file :

{% for c in columns %}  
   <th data-column-id="{{c['name']}}" data-type="{{c['type']}}" data-sortable="{{c['sortable']}}" data-visible={{ c['visible']|default('true') }}>{{c['label']}}</th>
{% endfor %}
@sergeyklay

This comment has been minimized.

Show comment
Hide comment
@sergeyklay

sergeyklay Dec 31, 2017

Member

I'll try to sort out as soon as I can (most likely after 3rd January)

Member

sergeyklay commented Dec 31, 2017

I'll try to sort out as soon as I can (most likely after 3rd January)

@sergeyklay sergeyklay added the Volt label Jan 3, 2018

@sergeyklay sergeyklay self-assigned this Jan 3, 2018

@sergeyklay sergeyklay referenced this issue Jan 3, 2018

Merged

Fixed `default` Volt filter #13246

3 of 3 tasks complete
@sergeyklay

This comment has been minimized.

Show comment
Hide comment
@sergeyklay

sergeyklay Jan 3, 2018

Member

Fixed in the 3.3.x branch. Feel free to open a new issue if the problem appears again. Thank you for contributing.

Member

sergeyklay commented Jan 3, 2018

Fixed in the 3.3.x branch. Feel free to open a new issue if the problem appears again. Thank you for contributing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment