Skip to content
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

Using dynamic function for computed annotations results in warning #4575

Closed
2 tasks done
nazar-pc opened this issue Apr 29, 2017 · 1 comment
Closed
2 tasks done

Using dynamic function for computed annotations results in warning #4575

nazar-pc opened this issue Apr 29, 2017 · 1 comment

Comments

@nazar-pc
Copy link
Contributor

nazar-pc commented Apr 29, 2017

Description

The feature requested in #3299 and implemented in #3358 has minor issue in 2.x, it results in warning:

method `xyz` not defined

Even though it works fine eventually.

Live Demo

http://jsbin.com/feberifezo/1/edit?html,console,output

Steps to Reproduce

  1. Define computed property with type Function and another simple property with type String (Number should work as well)
  2. Use it in element's template with another element's property as argument
  3. Add element to the document with another property as it's attribute

Expected Results

No warning

Actual Results

Warning:

method `xyz` not defined

Browsers Affected

Only tested in these:

  • Chromium nightly
  • Firefox

Versions

Polymer: 2.0-preview

@dfreedm
Copy link
Member

dfreedm commented May 1, 2017

Seems like this line should block printing that warning: https://github.com/Polymer/polymer/blob/master/lib/mixins/property-effects.html#L769

@kevinpschaaf is dynamicFn set later?

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

No branches or pull requests

3 participants