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

Bug with volt #13459

Closed
TimurFlush opened this Issue Aug 4, 2018 · 4 comments

Comments

Projects
3 participants
@TimurFlush
Copy link

TimurFlush commented Aug 4, 2018

Hello.
I create empty volt template and text it:

{{ select_static(['name': 'kek'], ['Y': 'Yes', 'N': 'No']) }}

Volt interpretated it as:

$this->tag->selectStatic([['name' => 'kek'], ['Y' => 'Yes', 'N' => 'No']]);

And i catch next error:

Phalcon\Tag\Exception: Value at index: 'id' type: 'array' cannot be rendered

Screenshot: https://prnt.sc/keq6it

Details

  • Phalcon version: 3.4.0
  • PHP Version: PHP 7.2.8
  • Operating System: Ubuntu 16.04 TLS
  • Installation type: installing via package manager
  • Zephir version (if any): Version 0.10.10-d1b4cc68d9
  • Server: Apache
@TimurFlush

This comment has been minimized.

Copy link
Author

TimurFlush commented Aug 4, 2018

Please https://travis-ci.org/TimurFlush/phalcon-bug/builds/412177947

https://github.com/TimurFlush/phalcon-bug
@sergeyklay accept my request for contributing my package if you want to check this bug in this repository

@TimurFlush TimurFlush changed the title Bug with phalcon Bug with volt Aug 4, 2018

@sergeyklay sergeyklay self-assigned this Aug 4, 2018

@sergeyklay

This comment has been minimized.

Copy link
Member

sergeyklay commented Aug 4, 2018

I'll take a look as soon as I can

@TimurFlush

This comment has been minimized.

Copy link
Author

TimurFlush commented Aug 5, 2018

This is due to the fact that Phalcon looks for the id key in the first argument-array and does not find it, so it assigns a zero key as an ID and it is an array, on this basis it cannot render it and throws an error. So the error is related to Volt.

Excuse me for my english.

@stale stale bot added the stale label Nov 3, 2018

@sergeyklay sergeyklay removed the stale label Nov 3, 2018

@sergeyklay sergeyklay removed their assignment Nov 3, 2018

@phalcon phalcon deleted a comment from stale bot Dec 23, 2018

@niden niden added this to To do in 4.0 Release via automation Dec 23, 2018

@niden niden added the Bug - Low label Dec 23, 2018

@niden niden self-assigned this Dec 23, 2018

CameronHall added a commit to CameronHall/cphalcon that referenced this issue Dec 24, 2018

Fixes phalcon#13459: Removed tags that accept multiple args from arra…
…y helper

Removed `link_to`, `image`, `select`, and `select_static` from array helper.

CameronHall added a commit to CameronHall/cphalcon that referenced this issue Dec 24, 2018

Fixes phalcon#13459: Removed tags that accept multiple args from arra…
…y helper

Removed `select`, and `select_static` from array helper.

CameronHall added a commit to CameronHall/cphalcon that referenced this issue Dec 27, 2018

Fixes phalcon#13459: Removed tags that accept multiple args from arra…
…y helper

Removed `select`, and `select_static` from array helper.

CameronHall added a commit to CameronHall/cphalcon that referenced this issue Dec 27, 2018

CameronHall added a commit to CameronHall/cphalcon that referenced this issue Dec 27, 2018

CameronHall added a commit to CameronHall/cphalcon that referenced this issue Dec 30, 2018

CameronHall added a commit to CameronHall/cphalcon that referenced this issue Jan 5, 2019

niden added a commit to niden/cphalcon that referenced this issue Jan 5, 2019

[4.0.x] - Merge remote-tracking branch 'upstream/4.0.x' into 4.0.x
* upstream/4.0.x:
  Updated/Refactored Libmemcached tests
  phalcon#13459: Created test for issue and updated result checking
  Fixed phalcon#13497 returning "empty" values as `null`

CameronHall added a commit to CameronHall/cphalcon that referenced this issue Jan 6, 2019

Fixes phalcon#13459: Removed tags that accept multiple args from arra…
…y helper

Removed `select`, and `select_static` from array helper.

CameronHall added a commit to CameronHall/cphalcon that referenced this issue Jan 12, 2019

Fixes phalcon#13459: Removed tags that accept multiple args from arra…
…y helper

Removed `select`, and `select_static` from array helper.

niden added a commit that referenced this issue Jan 12, 2019

Fixes #13459: Removed tags that accept multiple args from array helper (
#13695)

Removed `select`, and `select_static` from array helper.
@niden

This comment has been minimized.

Copy link
Member

niden commented Jan 12, 2019

This has been resolved

@niden niden closed this Jan 12, 2019

4.0 Release automation moved this from To do to Done Jan 12, 2019

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