{"action":"created","issue":{"url":"https://api.github.com/repos/ARTFL-Project/PhiloLogic4/issues/349","labels_url":"https://api.github.com/repos/ARTFL-Project/PhiloLogic4/issues/349/labels{/name}","comments_url":"https://api.github.com/repos/ARTFL-Project/PhiloLogic4/issues/349/comments","events_url":"https://api.github.com/repos/ARTFL-Project/PhiloLogic4/issues/349/events","html_url":"https://github.com/ARTFL-Project/PhiloLogic4/issues/349","id":54585981,"number":349,"title":"Crash in MetadataQuery when search field that doesn't exist","user":{"login":"clovis","id":357350,"avatar_url":"https://avatars.githubusercontent.com/u/357350?v=3","gravatar_id":"","url":"https://api.github.com/users/clovis","html_url":"https://github.com/clovis","followers_url":"https://api.github.com/users/clovis/followers","following_url":"https://api.github.com/users/clovis/following{/other_user}","gists_url":"https://api.github.com/users/clovis/gists{/gist_id}","starred_url":"https://api.github.com/users/clovis/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/clovis/subscriptions","organizations_url":"https://api.github.com/users/clovis/orgs","repos_url":"https://api.github.com/users/clovis/repos","events_url":"https://api.github.com/users/clovis/events{/privacy}","received_events_url":"https://api.github.com/users/clovis/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/ARTFL-Project/PhiloLogic4/labels/bug","name":"bug","color":"fc2929"}],"state":"open","locked":false,"assignee":{"login":"rwhaling","id":253435,"avatar_url":"https://avatars.githubusercontent.com/u/253435?v=3","gravatar_id":"","url":"https://api.github.com/users/rwhaling","html_url":"https://github.com/rwhaling","followers_url":"https://api.github.com/users/rwhaling/followers","following_url":"https://api.github.com/users/rwhaling/following{/other_user}","gists_url":"https://api.github.com/users/rwhaling/gists{/gist_id}","starred_url":"https://api.github.com/users/rwhaling/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rwhaling/subscriptions","organizations_url":"https://api.github.com/users/rwhaling/orgs","repos_url":"https://api.github.com/users/rwhaling/repos","events_url":"https://api.github.com/users/rwhaling/events{/privacy}","received_events_url":"https://api.github.com/users/rwhaling/received_events","type":"User","site_admin":false},"milestone":{"url":"https://api.github.com/repos/ARTFL-Project/PhiloLogic4/milestones/1","html_url":"https://github.com/ARTFL-Project/PhiloLogic4/milestones/PhiloLogic%204.0","labels_url":"https://api.github.com/repos/ARTFL-Project/PhiloLogic4/milestones/1/labels","id":909517,"number":1,"title":"PhiloLogic 4.0","description":"","creator":{"login":"clovis","id":357350,"avatar_url":"https://avatars.githubusercontent.com/u/357350?v=3","gravatar_id":"","url":"https://api.github.com/users/clovis","html_url":"https://github.com/clovis","followers_url":"https://api.github.com/users/clovis/followers","following_url":"https://api.github.com/users/clovis/following{/other_user}","gists_url":"https://api.github.com/users/clovis/gists{/gist_id}","starred_url":"https://api.github.com/users/clovis/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/clovis/subscriptions","organizations_url":"https://api.github.com/users/clovis/orgs","repos_url":"https://api.github.com/users/clovis/repos","events_url":"https://api.github.com/users/clovis/events{/privacy}","received_events_url":"https://api.github.com/users/clovis/received_events","type":"User","site_admin":false},"open_issues":3,"closed_issues":8,"state":"open","created_at":"2014-12-18T23:07:04Z","updated_at":"2015-04-07T19:40:28Z","due_on":null,"closed_at":null},"comments":1,"created_at":"2015-01-16T15:16:15Z","updated_at":"2015-04-07T19:41:04Z","closed_at":null,"body":"If we try to search for something in a field that doesn't exist, we get a crash. Replicating is easy. Just search for anything in a non-existent field in the search form. Here's the traceback:\r\n\r\n<pre><code>Traceback (most recent call last):\r\nFile \"/var/www/html/philologic4/test/dispatcher.py\", line 23, in philo_dispatcher\r\nyield getattr(reports, report or \"concordance\")(environ,start_response)\r\nFile \"/var/www/html/philologic4/test/reports/concordance.py\", line 31, in concordance\r\nconcordance_object, hits = concordance_results(db, request, config)\r\nFile \"/var/www/html/philologic4/test/reports/concordance.py\", line 41, in concordance_results\r\nhits = db.query(q[\"q\"],q[\"method\"],q[\"arg\"],**q.metadata)\r\nFile \"/usr/lib/python2.6/site-packages/philologic/DB.py\", line 148, in query\r\ncorpus = MetadataQuery.metadata_query(self,corpus_file,metadata_dicts)\r\nFile \"/usr/lib/python2.6/site-packages/philologic/MetadataQuery.py\", line 19, in metadata_query\r\nfor corpus_obj in query:\r\nFile \"/usr/lib/python2.6/site-packages/philologic/MetadataQuery.py\", line 31, in query_recursive\r\nr = query_lowlevel(db,param_dict)\r\nFile \"/usr/lib/python2.6/site-packages/philologic/MetadataQuery.py\", line 81, in query_lowlevel\r\nresults = db.dbh.execute(query,vars)\r\nOperationalError: no such column: who</code></pre>\r\n"},"comment":{"url":"https://api.github.com/repos/ARTFL-Project/PhiloLogic4/issues/comments/90709535","html_url":"https://github.com/ARTFL-Project/PhiloLogic4/issues/349#issuecomment-90709535","issue_url":"https://api.github.com/repos/ARTFL-Project/PhiloLogic4/issues/349","id":90709535,"user":{"login":"clovis","id":357350,"avatar_url":"https://avatars.githubusercontent.com/u/357350?v=3","gravatar_id":"","url":"https://api.github.com/users/clovis","html_url":"https://github.com/clovis","followers_url":"https://api.github.com/users/clovis/followers","following_url":"https://api.github.com/users/clovis/following{/other_user}","gists_url":"https://api.github.com/users/clovis/gists{/gist_id}","starred_url":"https://api.github.com/users/clovis/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/clovis/subscriptions","organizations_url":"https://api.github.com/users/clovis/orgs","repos_url":"https://api.github.com/users/clovis/repos","events_url":"https://api.github.com/users/clovis/events{/privacy}","received_events_url":"https://api.github.com/users/clovis/received_events","type":"User","site_admin":false},"created_at":"2015-04-07T19:41:04Z","updated_at":"2015-04-07T19:41:04Z","body":"I believe this is now fixed with the way the HitList works now, right?"}}