{"action":"opened","issue":{"url":"https://api.github.com/repos/mweastwood/CasaCore.jl/issues/62","repository_url":"https://api.github.com/repos/mweastwood/CasaCore.jl","labels_url":"https://api.github.com/repos/mweastwood/CasaCore.jl/issues/62/labels{/name}","comments_url":"https://api.github.com/repos/mweastwood/CasaCore.jl/issues/62/comments","events_url":"https://api.github.com/repos/mweastwood/CasaCore.jl/issues/62/events","html_url":"https://github.com/mweastwood/CasaCore.jl/issues/62","id":186021927,"number":62,"title":"subtable keywords are broken","user":{"login":"mweastwood","id":4071225,"avatar_url":"https://avatars.githubusercontent.com/u/4071225?v=3","gravatar_id":"","url":"https://api.github.com/users/mweastwood","html_url":"https://github.com/mweastwood","followers_url":"https://api.github.com/users/mweastwood/followers","following_url":"https://api.github.com/users/mweastwood/following{/other_user}","gists_url":"https://api.github.com/users/mweastwood/gists{/gist_id}","starred_url":"https://api.github.com/users/mweastwood/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mweastwood/subscriptions","organizations_url":"https://api.github.com/users/mweastwood/orgs","repos_url":"https://api.github.com/users/mweastwood/repos","events_url":"https://api.github.com/users/mweastwood/events{/privacy}","received_events_url":"https://api.github.com/users/mweastwood/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2016-10-28T21:05:03Z","updated_at":"2016-10-28T21:05:03Z","closed_at":null,"body":"When I switched from wrapping `TableProxy` to just regular `Table`, I missed a conversion that `TableProxy` was doing automatically. I mistakenly assumed `Table` would do the same thing.\r\n\r\nWhen reading a keyword, `TableProxy` does the following if the keyword points to another table.\r\n```\r\n  ...\r\n  case TpTable:\r\n    return ValueHolder (\"Table: \"+keySet.tableAttributes(fieldId).name());\r\n  ...\r\n```\r\n\r\nBecause I'm now wrapping `Table`, I could probably just return the actual `Table` object itself, which seems like the sensible thing to do. However at the moment it is just broken because I haven't defined a wrapper function for the case where the data type is `TpTable`."}}
{"actor":{"display_login":"mweastwood"}}