{"comment":{"url":"https://api.github.com/repos/b14/p2/comments/15205642","html_url":"https://github.com/b14/p2/commit/e8c04fe8ba23c8af4a09fdb0a3773092a6aa53f7#commitcomment-15205642","id":15205642,"user":{"login":"kasperg","id":73966,"avatar_url":"https://avatars.githubusercontent.com/u/73966?v=3","gravatar_id":"","url":"https://api.github.com/users/kasperg","html_url":"https://github.com/kasperg","followers_url":"https://api.github.com/users/kasperg/followers","following_url":"https://api.github.com/users/kasperg/following{/other_user}","gists_url":"https://api.github.com/users/kasperg/gists{/gist_id}","starred_url":"https://api.github.com/users/kasperg/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kasperg/subscriptions","organizations_url":"https://api.github.com/users/kasperg/orgs","repos_url":"https://api.github.com/users/kasperg/repos","events_url":"https://api.github.com/users/kasperg/events{/privacy}","received_events_url":"https://api.github.com/users/kasperg/received_events","type":"User","site_admin":false},"position":97,"line":103,"path":"ding_entity_rating/ding_entity_rating.module","commit_id":"e8c04fe8ba23c8af4a09fdb0a3773092a6aa53f7","created_at":"2015-12-30T13:54:11Z","updated_at":"2015-12-30T13:54:11Z","body":"If you choose not to follow a TODO please delete it from the codebase. The code comments are not meant to be a discussion between developers.\r\n\r\nThat being said `virtual_field` is used extensively throughout the ding2 system. The p2 modules are not standalone code. Your choice of not using virtual_field does not reduce dependencies for the system. It increases complexity as it introduces another approach for handling fields and limits the flexibility of how ratings can be displayed.\r\n\r\nWe do not use `hook_block_info/view` even though it is a part of Drupal Core. We use CTools content types."}}