{"action":"created","issue":{"url":"https://api.github.com/repos/riot/riot/issues/2056","repository_url":"https://api.github.com/repos/riot/riot","labels_url":"https://api.github.com/repos/riot/riot/issues/2056/labels{/name}","comments_url":"https://api.github.com/repos/riot/riot/issues/2056/comments","events_url":"https://api.github.com/repos/riot/riot/issues/2056/events","html_url":"https://github.com/riot/riot/issues/2056","id":187100973,"number":2056,"title":"'each' loops are triggering mount events when a global mixin is used","user":{"login":"jpoppe","id":65505,"avatar_url":"https://avatars.githubusercontent.com/u/65505?v=3","gravatar_id":"","url":"https://api.github.com/users/jpoppe","html_url":"https://github.com/jpoppe","followers_url":"https://api.github.com/users/jpoppe/followers","following_url":"https://api.github.com/users/jpoppe/following{/other_user}","gists_url":"https://api.github.com/users/jpoppe/gists{/gist_id}","starred_url":"https://api.github.com/users/jpoppe/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jpoppe/subscriptions","organizations_url":"https://api.github.com/users/jpoppe/orgs","repos_url":"https://api.github.com/users/jpoppe/repos","events_url":"https://api.github.com/users/jpoppe/events{/privacy}","received_events_url":"https://api.github.com/users/jpoppe/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-11-03T15:47:37Z","updated_at":"2016-11-03T21:10:39Z","closed_at":null,"body":"## Help us to manage our issues by answering the following:\r\n\r\n1. Describe your issue:\r\nWhen a global mixin is used 'each' items are triggering mount events. (I really hope I am not doing something stupid here).\r\n\r\n2. Can you reproduce the issue?\r\n\r\n  Post the link using one of our bug report templates:\r\n  \r\n  - http://plnkr.co/edit/1sHvhp5ssfbIUWz6aLjI  (Please watch the console output for clarification)\r\n\r\n**Console output**:\r\n```\r\nfrom global mixin: mounted - my-tag mixins.js (line 6)\r\nfrom baseMixin: mounted - my-tag mixins.js (line 16)\r\n6 * from global mixin: mounted - li\r\n```\r\n\r\n**Expected console output**:\r\n```\r\nfrom global mixin: mounted - my-tag mixins.js (line 6)\r\nfrom baseMixin: mounted - my-tag mixins.js (line 16)\r\n```\r\n\r\n3. On which browser/OS does the issue appear?\r\nTested on latest Firefox/Chrome and Chromium (using Arch Linux).\r\n\r\n4. Which version of Riot does it affect?\r\nNext and master.\r\n\r\n5. How would you tag this issue?\r\n\r\n  - [ ] Question\r\n  - [x] Bug\r\n  - [ ] Discussion\r\n  - [ ] Feature request\r\n  - [ ] Tip\r\n  - [ ] Enhancement\r\n  - [ ] Performance\r\n"},"comment":{"url":"https://api.github.com/repos/riot/riot/issues/comments/258275058","html_url":"https://github.com/riot/riot/issues/2056#issuecomment-258275058","issue_url":"https://api.github.com/repos/riot/riot/issues/2056","id":258275058,"user":{"login":"papas-source","id":3933935,"avatar_url":"https://avatars.githubusercontent.com/u/3933935?v=3","gravatar_id":"","url":"https://api.github.com/users/papas-source","html_url":"https://github.com/papas-source","followers_url":"https://api.github.com/users/papas-source/followers","following_url":"https://api.github.com/users/papas-source/following{/other_user}","gists_url":"https://api.github.com/users/papas-source/gists{/gist_id}","starred_url":"https://api.github.com/users/papas-source/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/papas-source/subscriptions","organizations_url":"https://api.github.com/users/papas-source/orgs","repos_url":"https://api.github.com/users/papas-source/repos","events_url":"https://api.github.com/users/papas-source/events{/privacy}","received_events_url":"https://api.github.com/users/papas-source/received_events","type":"User","site_admin":false},"created_at":"2016-11-03T21:10:39Z","updated_at":"2016-11-03T21:10:39Z","body":"I am in no way javascript experienced programmer\r\nI am completely new to riot\r\nnever looked at the code of riot.\r\n\r\nAnd I am completely new to github / not even sure what a pull reqest is\r\nI dont know how to clone the repo or do the tests\r\nbut I think that in line 1671\r\n\r\n```\r\n      if (!isInStub(self.root)) {\r\n        self.parent.isMounted = self.isMounted = true\r\n        self.trigger('mount') /* this line */\r\n      }\r\n```\r\n\r\nit should be \r\n```\r\n      if (!isInStub(self.root)) {\r\n        self.parent.isMounted = self.isMounted = true\r\n        !isLoop && self.trigger('mount') /* this line */\r\n      }\r\n```\r\n\r\ncould be wrong though...\r\n\r\nfeel free to delete the comment if in any way is wrong/comfusing or something not suitable.\r\n\r\n"}}
{"actor":{"display_login":"papas-source"}}