{"comment":{"created_at":"2011-10-09T21:13:32Z","body":"Nice catch!\r\n\r\nThese case statements should (eventually) be replaced with proper polymorphism (sounds like a Visitor), but they're not going away tomorrow.","updated_at":"2011-10-09T21:13:32Z","url":"https://api.github.com/repos/datamapper/dm-core/issues/comments/2341177","id":2341177,"user":{"avatar_url":"https://secure.gravatar.com/avatar/83611ed7f7ed587603fbdffc691ea989?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","url":"https://api.github.com/users/emmanuel","id":7657,"login":"emmanuel"}},"action":"created","issue":{"number":150,"created_at":"2011-10-09T13:25:40Z","pull_request":{"diff_url":"https://github.com/datamapper/dm-core/pull/150.diff","patch_url":"https://github.com/datamapper/dm-core/pull/150.patch","html_url":"https://github.com/datamapper/dm-core/pull/150"},"body":"`Direction` descends from `Operator`, yet it appears *after* `Operator` in the `case..when` statement here.  As a result, any `Direction` used here is rebuilt from the concrete class, which makes it impossible to subclass (as I need to do for part of an extension).\r\n","comments":1,"title":"Change Direction/Operator precedence (fixes regression at d4377a9ee407b48)","updated_at":"2011-10-09T21:13:32Z","url":"https://api.github.com/repos/datamapper/dm-core/issues/150","id":1856054,"assignee":null,"milestone":null,"closed_at":"2011-10-09T21:13:24Z","user":{"avatar_url":"https://secure.gravatar.com/avatar/f855324f298eb7708e39ea117477cbd5?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","url":"https://api.github.com/users/d11wtq","id":37948,"login":"d11wtq"},"html_url":"https://github.com/datamapper/dm-core/issues/150","labels":[],"state":"closed"},"legacy":{"issue_id":1856054,"comment_id":2341177}}