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

Single Property Schema results in NaN for value #2059

Closed
JGL opened this Issue Nov 7, 2016 · 0 comments

Comments

Projects
None yet
2 participants
@JGL

JGL commented Nov 7, 2016

Description:

Using the Component docs, I created a single property schema as follows:

AFRAME.registerComponent('single-property-schema-bug', {
// This component can be used multiple times
multiple: true,
// Allow material-side-modifier component a single property schema, of type int, defaulting to 2
schema: { default: 2 },
init: function () {
console.log("Inside of init() of single-property-schema-bug");
var side = this.data; //should be 2, the default value
console.log("The value of this.data is ", this.data); //why isn't this reporting properly?
console.log("The value of var side is ", side); //why isn't this reporting properly?
}
});

This resulted in the following output in my JavaScript console:

Inside of init() of single-property-schema-bug
build.js:87663 The value of this.data is NaN
build.js:87664 The value of var side is NaN

  • A-Frame Version: 0.3.2
  • Platform/Device: Mac Book Pro, running Chrome, macOS 10.12.1 (16B2555)
  • Reproducible Code Snippet or Demo URL [highly encouraged]:

Demo url:
https://jgl.github.io/APieceOfArtAsBigAsIndia/singlePropertySchemaBugComponentDemo.html
Source (html):
https://github.com/JGL/APieceOfArtAsBigAsIndia/blob/master/docs/singlePropertySchemaBugComponentDemo.html
Source (js);
https://github.com/JGL/APieceOfArtAsBigAsIndia/blob/master/src/js/single-property-schema-bug.js

@ngokevin ngokevin added the regression label Nov 7, 2016

@ngokevin ngokevin added this to the 0.4.0 milestone Nov 7, 2016

@ngokevin ngokevin self-assigned this Nov 8, 2016

ngokevin added a commit to ngokevin/aframe that referenced this issue Nov 8, 2016

@dmarcos dmarcos closed this in c1d55ec Nov 8, 2016

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