{"action":"opened","issue":{"url":"https://api.github.com/repos/Runalyze/Runalyze/issues/1695","labels_url":"https://api.github.com/repos/Runalyze/Runalyze/issues/1695/labels{/name}","comments_url":"https://api.github.com/repos/Runalyze/Runalyze/issues/1695/comments","events_url":"https://api.github.com/repos/Runalyze/Runalyze/issues/1695/events","html_url":"https://github.com/Runalyze/Runalyze/issues/1695","id":126124513,"number":1695,"title":"Internal notes for refactoring tasks","user":{"login":"laufhannes","id":8313118,"avatar_url":"https://avatars.githubusercontent.com/u/8313118?v=3","gravatar_id":"","url":"https://api.github.com/users/laufhannes","html_url":"https://github.com/laufhannes","followers_url":"https://api.github.com/users/laufhannes/followers","following_url":"https://api.github.com/users/laufhannes/following{/other_user}","gists_url":"https://api.github.com/users/laufhannes/gists{/gist_id}","starred_url":"https://api.github.com/users/laufhannes/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/laufhannes/subscriptions","organizations_url":"https://api.github.com/users/laufhannes/orgs","repos_url":"https://api.github.com/users/laufhannes/repos","events_url":"https://api.github.com/users/laufhannes/events{/privacy}","received_events_url":"https://api.github.com/users/laufhannes/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/Runalyze/Runalyze/labels/refactor","name":"refactor","color":"eb6420"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2016-01-12T08:27:05Z","updated_at":"2016-01-12T08:27:05Z","closed_at":null,"body":"*This is only an internal collection of refactoring tasks.*\r\n\r\n--\r\n\r\n**Remove Garmin Communicator** (as soon as we decide to not support it anymore)\r\n - [ ] remove `GARMIN_IGNORE_IDS`\r\n - [ ] remove `lib/garmin`\r\n - [ ] remove `call/ajax.activityMatcher.php`\r\n - [ ] remove `call/ajax.saveTcx.php`\r\n - [ ] ...\r\n\r\n--\r\n\r\n**Other tasks**\r\n - [ ] rename `runalyze_training` to `runalyze_activity`\r\n - [ ] replace `Data\\Weather\\Temperature` by `Activity\\Temperature`\r\n - [ ] use `core/Model/Activity/Splits` instead of `inc/data/class.Splits.php` everywhere\r\n - [ ] use `core/Model/Sport` instead of `inc/data/class.Sport*.php` everywhere\r\n - [ ] use `core/Model/Type` instead of `inc/data/class.Type*.php` everywhere\r\n\r\n--\r\n\r\n**Create some `View\\Activity\\Box\\ValueBox` which requires `ValueInterface` only**\r\n\r\n```php\r\n// class ValueBox extends AbstractBox\r\n\tpublic function __construct(ValueInterface $value)\r\n\t{\r\n\t\tparent::__construct(\r\n\t\t\t\\Helper::Unknown($value->string(false), '-'),\r\n\t\t\t$value->unit(),\r\n\t\t\t$value->label()\r\n\t\t);\r\n\t}\r\n\r\n// now use\r\n$Box = new ValueBox($Context->dataview()->strideLength());\r\n// instead of\r\n$Box = new StrideLength($Context);\r\n```"}}