{"action":"created","issue":{"url":"https://api.github.com/repos/renatoathaydes/Automaton/issues/33","labels_url":"https://api.github.com/repos/renatoathaydes/Automaton/issues/33/labels{/name}","comments_url":"https://api.github.com/repos/renatoathaydes/Automaton/issues/33/comments","events_url":"https://api.github.com/repos/renatoathaydes/Automaton/issues/33/events","html_url":"https://github.com/renatoathaydes/Automaton/pull/33","id":107903562,"number":33,"title":"Extract table cell value from cell renderer rather than model (Fixes #32)","user":{"login":"meyertee","id":270718,"avatar_url":"https://avatars.githubusercontent.com/u/270718?v=3","gravatar_id":"","url":"https://api.github.com/users/meyertee","html_url":"https://github.com/meyertee","followers_url":"https://api.github.com/users/meyertee/followers","following_url":"https://api.github.com/users/meyertee/following{/other_user}","gists_url":"https://api.github.com/users/meyertee/gists{/gist_id}","starred_url":"https://api.github.com/users/meyertee/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/meyertee/subscriptions","organizations_url":"https://api.github.com/users/meyertee/orgs","repos_url":"https://api.github.com/users/meyertee/repos","events_url":"https://api.github.com/users/meyertee/events{/privacy}","received_events_url":"https://api.github.com/users/meyertee/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-09-23T11:51:13Z","updated_at":"2015-09-24T19:00:08Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/renatoathaydes/Automaton/pulls/33","html_url":"https://github.com/renatoathaydes/Automaton/pull/33","diff_url":"https://github.com/renatoathaydes/Automaton/pull/33.diff","patch_url":"https://github.com/renatoathaydes/Automaton/pull/33.patch"},"body":"Suggested fix incl. test for https://github.com/renatoathaydes/Automaton/issues/32\r\n\r\nIt reads the cell's value from the renderer component, if it is a JLabel (as is the case in the DefaultTableCellRenderer). If the renderer component isn't a JLabel it returns the model value itself as previously.\r\n\r\nThis may break existing tests if they rely on the difference between model value and rendered value. But I can't think of a reason why you'd want that other than to work around the issue itself."},"comment":{"url":"https://api.github.com/repos/renatoathaydes/Automaton/issues/comments/143019873","html_url":"https://github.com/renatoathaydes/Automaton/pull/33#issuecomment-143019873","issue_url":"https://api.github.com/repos/renatoathaydes/Automaton/issues/33","id":143019873,"user":{"login":"renatoathaydes","id":1901277,"avatar_url":"https://avatars.githubusercontent.com/u/1901277?v=3","gravatar_id":"","url":"https://api.github.com/users/renatoathaydes","html_url":"https://github.com/renatoathaydes","followers_url":"https://api.github.com/users/renatoathaydes/followers","following_url":"https://api.github.com/users/renatoathaydes/following{/other_user}","gists_url":"https://api.github.com/users/renatoathaydes/gists{/gist_id}","starred_url":"https://api.github.com/users/renatoathaydes/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/renatoathaydes/subscriptions","organizations_url":"https://api.github.com/users/renatoathaydes/orgs","repos_url":"https://api.github.com/users/renatoathaydes/repos","events_url":"https://api.github.com/users/renatoathaydes/events{/privacy}","received_events_url":"https://api.github.com/users/renatoathaydes/received_events","type":"User","site_admin":false},"created_at":"2015-09-24T19:00:08Z","updated_at":"2015-09-24T19:00:08Z","body":"That's a good point... but Automaton is meant to be used by testers who need to test what they can actually see, not what the underlying Table model or whatever looks like... so this change is a big improvement in any case."}}