{"issue":{"title":"model example (2 minor thing)","user":{"id":354596,"login":"butu5","url":"https://api.github.com/users/butu5","gravatar_id":"87d3c795e45f6f80c5eb96ba5ee952fc","avatar_url":"https://secure.gravatar.com/avatar/87d3c795e45f6f80c5eb96ba5ee952fc?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png"},"milestone":null,"created_at":"2011-12-22T17:44:23Z","labels":[],"state":"open","updated_at":"2011-12-22T18:00:31Z","id":2639704,"html_url":"https://github.com/addyosmani/backbone-fundamentals/issues/8","closed_at":null,"pull_request":{"patch_url":null,"html_url":null,"diff_url":null},"body":"I created following example using the backbone-fundamentals readme file\r\n\r\n[Models example][1]\r\n\r\nWhile running the code found one minor issue (most probably my mistake :) )\r\nThere is some syntax error at [L16 changeSrc][2]\r\n\r\nAlso, initially got confused how to access the direct attributes. \r\nI thought the way to access is direct attribute is myPhoto.src [L44 direct access][3]\r\n\r\n[1]: https://github.com/butu5/backbone-fundamentals/blob/master/examples/models.js \"Models example\"\r\n[2]: https://github.com/butu5/backbone-fundamentals/blob/master/examples/models.js#L16-22 \"L16 changeSrc\"\r\n[3]: https://github.com/butu5/backbone-fundamentals/blob/master/examples/models.js#L44 \"L44 direct access\"","number":8,"url":"https://api.github.com/repos/addyosmani/backbone-fundamentals/issues/8","comments":1,"assignee":null},"comment":{"user":{"id":110953,"login":"addyosmani","url":"https://api.github.com/users/addyosmani","gravatar_id":"96270e4c3e5e9806cf7245475c00b275","avatar_url":"https://secure.gravatar.com/avatar/96270e4c3e5e9806cf7245475c00b275?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png"},"created_at":"2011-12-22T18:00:31Z","updated_at":"2011-12-22T18:00:31Z","id":3252178,"body":"Hey @butu5 :)\r\n\r\nI've just pushed a minor change to the models example you're referring to which will hopefully clarify how changeSrc is used (see the second line of the example). I think there *may* be a stray semi-colon missing in yours. \r\n\r\nI should definitely point out that although the examples in fundamentals (for the basics section) are tied together, they haven't been written in a way in which they can be dropped into a new Backbone app and just used straight-away. This may however be something I should change in the near-future.\r\n\r\nIn terms of direct access, are there any particular lines in the models section which you think could be improved to clarify how attributes should be read?","url":"https://api.github.com/repos/addyosmani/backbone-fundamentals/issues/comments/3252178"},"action":"created"}