{"action":"created","issue":{"url":"https://api.github.com/repos/OCA/maintainer-tools/issues/82","labels_url":"https://api.github.com/repos/OCA/maintainer-tools/issues/82/labels{/name}","comments_url":"https://api.github.com/repos/OCA/maintainer-tools/issues/82/comments","events_url":"https://api.github.com/repos/OCA/maintainer-tools/issues/82/events","html_url":"https://github.com/OCA/maintainer-tools/issues/82","id":95483538,"number":82,"title":"Variable name CamelCase potential problem of `redefined` variable.","user":{"login":"moylop260","id":6644187,"avatar_url":"https://avatars.githubusercontent.com/u/6644187?v=3","gravatar_id":"","url":"https://api.github.com/users/moylop260","html_url":"https://github.com/moylop260","followers_url":"https://api.github.com/users/moylop260/followers","following_url":"https://api.github.com/users/moylop260/following{/other_user}","gists_url":"https://api.github.com/users/moylop260/gists{/gist_id}","starred_url":"https://api.github.com/users/moylop260/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/moylop260/subscriptions","organizations_url":"https://api.github.com/users/moylop260/orgs","repos_url":"https://api.github.com/users/moylop260/repos","events_url":"https://api.github.com/users/moylop260/events{/privacy}","received_events_url":"https://api.github.com/users/moylop260/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":8,"created_at":"2015-07-16T17:22:52Z","updated_at":"2015-07-20T15:03:08Z","closed_at":null,"body":"I saw in oca guidelines this matter:\r\n******************\r\n#### Variable name :\r\n*  **use camelcase for model variable**\r\n* use underscore lowercase notation for common variable.\r\n* since new API works with record or recordset instead of id list, don't suffix\r\n  variable name with `_id` or `_ids` if they do not contain an id or a list of\r\n  ids.\r\n\r\n```python\r\nResPartner = self.env['res.partner']\r\npartners = ResPartner.browse(ids)\r\npartner_id = partners[0].id\r\n```\r\n******************\r\n\r\nI want comment a problem with this:\r\n 1. Use CamelCase in a variable don't is a pep8 standard.\r\n 1. We will have potential problem of redefined variable.\r\n   1. By example:\r\n```python\r\n        class ResPartner(model.Models):\r\n        ...\r\n        ResPartner = self.env['res.partner']\r\n```\r\n\r\nWhy don't use as pep8 told us: snake_case to variable name?"},"comment":{"url":"https://api.github.com/repos/OCA/maintainer-tools/issues/comments/122912187","html_url":"https://github.com/OCA/maintainer-tools/issues/82#issuecomment-122912187","issue_url":"https://api.github.com/repos/OCA/maintainer-tools/issues/82","id":122912187,"user":{"login":"JonathanNEMRY","id":7601731,"avatar_url":"https://avatars.githubusercontent.com/u/7601731?v=3","gravatar_id":"","url":"https://api.github.com/users/JonathanNEMRY","html_url":"https://github.com/JonathanNEMRY","followers_url":"https://api.github.com/users/JonathanNEMRY/followers","following_url":"https://api.github.com/users/JonathanNEMRY/following{/other_user}","gists_url":"https://api.github.com/users/JonathanNEMRY/gists{/gist_id}","starred_url":"https://api.github.com/users/JonathanNEMRY/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/JonathanNEMRY/subscriptions","organizations_url":"https://api.github.com/users/JonathanNEMRY/orgs","repos_url":"https://api.github.com/users/JonathanNEMRY/repos","events_url":"https://api.github.com/users/JonathanNEMRY/events{/privacy}","received_events_url":"https://api.github.com/users/JonathanNEMRY/received_events","type":"User","site_admin":false},"created_at":"2015-07-20T15:03:08Z","updated_at":"2015-07-20T15:03:08Z","body":"Then what is your suggestion knowing that the current one is not PEP8....I just try to find a solution by proposing this one but others ideas are welcome\r\n@nhomar  @bwrsandman @dreispt "}}