{"comment":{"url":"https://api.github.com/repos/jirikuncar/invenio/comments/10853000","html_url":"https://github.com/jirikuncar/invenio/commit/26fd85429e55151f59c390b1c454c4e95a03b671#commitcomment-10853000","id":10853000,"user":{"login":"jirikuncar","id":189798,"avatar_url":"https://avatars.githubusercontent.com/u/189798?v=3","gravatar_id":"","url":"https://api.github.com/users/jirikuncar","html_url":"https://github.com/jirikuncar","followers_url":"https://api.github.com/users/jirikuncar/followers","following_url":"https://api.github.com/users/jirikuncar/following{/other_user}","gists_url":"https://api.github.com/users/jirikuncar/gists{/gist_id}","starred_url":"https://api.github.com/users/jirikuncar/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jirikuncar/subscriptions","organizations_url":"https://api.github.com/users/jirikuncar/orgs","repos_url":"https://api.github.com/users/jirikuncar/repos","events_url":"https://api.github.com/users/jirikuncar/events{/privacy}","received_events_url":"https://api.github.com/users/jirikuncar/received_events","type":"User","site_admin":false},"position":0,"line":null,"path":"invenio/legacy/search_engine/__init__.py","commit_id":"26fd85429e55151f59c390b1c454c4e95a03b671","created_at":"2015-04-22T18:12:33Z","updated_at":"2015-04-22T18:12:33Z","body":"29:1: E402 module level import not at top of file\n30:1: E402 module level import not at top of file\n31:1: E402 module level import not at top of file\n32:1: E402 module level import not at top of file\n33:1: E402 module level import not at top of file\n36:1: I101 Imported names are in the wrong order. Should be CFG_ACCESS_CONTROL_LEVEL_ACCOUNTS, CFG_BASE_URL, CFG_BIBFORMAT_HIDDEN_TAGS, CFG_BIBINDEX_CHARS_PUNCTUATION, CFG_BIBRANK_SHOW_DOWNLOAD_GRAPHS, CFG_BIBRANK_SHOW_DOWNLOAD_GRAPHS, CFG_BIBSORT_BUCKETS, CFG_BIBSORT_ENABLED, CFG_BIBUPLOAD_EXTERNAL_SYSNO_TAG, CFG_BIBUPLOAD_EXTERNAL_SYSNO_TAG, CFG_BIBUPLOAD_SERIALIZE_RECORD_STRUCTURE, CFG_BIBUPLOAD_SERIALIZE_RECORD_STRUCTURE, CFG_CERN_SITE, CFG_INSPIRE_SITE, CFG_LOGDIR, CFG_LOGDIR, CFG_OAI_ID_FIELD, CFG_SITE_LANG, CFG_SITE_NAME, CFG_SITE_RECORD, CFG_SITE_URL, CFG_SOLR_URL, CFG_WEBSEARCH_DEF_RECORDS_IN_GROUPS, CFG_WEBSEARCH_DETAILED_META_FORMAT, CFG_WEBSEARCH_DISPLAY_NEAREST_TERMS, CFG_WEBSEARCH_FIELDS_CONVERT, CFG_WEBSEARCH_FULLTEXT_SNIPPETS, CFG_WEBSEARCH_IDXPAIRS_EXACT_SEARCH, CFG_WEBSEARCH_IDXPAIRS_FIELDS, CFG_WEBSEARCH_PREV_NEXT_HIT_LIMIT, CFG_WEBSEARCH_SYNONYM_KBRS, CFG_WEBSEARCH_VIEWRESTRCOLL_POLICY, CFG_WEBSEARCH_WILDCARD_LIMIT, CFG_WEBSEARCH_WILDCARD_LIMIT, CFG_XAPIAN_ENABLED\n36:1: F999 redefinition of unused 'CFG_BIBUPLOAD_SERIALIZE_RECORD_STRUCTURE' from line 36\n36:1: F999 redefinition of unused 'CFG_BIBUPLOAD_EXTERNAL_SYSNO_TAG' from line 36\n36:1: F999 redefinition of unused 'CFG_BIBRANK_SHOW_DOWNLOAD_GRAPHS' from line 36\n36:1: F999 redefinition of unused 'CFG_LOGDIR' from line 36\n36:1: F999 redefinition of unused 'CFG_WEBSEARCH_WILDCARD_LIMIT' from line 36\n36:1: F401 'CFG_XAPIAN_ENABLED' imported but unused\n36:1: F401 'CFG_BIBINDEX_CHARS_PUNCTUATION' imported but unused\n36:1: F401 'CFG_OAI_ID_FIELD' imported but unused\n36:1: F401 'CFG_WEBSEARCH_VIEWRESTRCOLL_POLICY' imported but unused\n36:1: F401 'CFG_WEBSEARCH_SYNONYM_KBRS' imported but unused\n36:1: F401 'CFG_BIBUPLOAD_SERIALIZE_RECORD_STRUCTURE' imported but unused\n36:1: F401 'CFG_WEBSEARCH_DISPLAY_NEAREST_TERMS' imported but unused\n36:1: F401 'CFG_WEBSEARCH_FIELDS_CONVERT' imported but unused\n36:1: F401 'CFG_ACCESS_CONTROL_LEVEL_ACCOUNTS' imported but unused\n36:1: F401 'CFG_SITE_RECORD' imported but unused\n36:1: F401 'CFG_BIBRANK_SHOW_DOWNLOAD_GRAPHS' imported but unused\n36:1: F401 'CFG_INSPIRE_SITE' imported but unused\n36:1: F401 'CFG_LOGDIR' imported but unused\n36:1: F401 'CFG_BIBUPLOAD_EXTERNAL_SYSNO_TAG' imported but unused\n36:1: F401 'CFG_WEBSEARCH_PREV_NEXT_HIT_LIMIT' imported but unused\n36:1: F401 'CFG_BASE_URL' imported but unused\n36:1: F401 'CFG_BIBFORMAT_HIDDEN_TAGS' imported but unused\n36:1: F401 'CFG_WEBSEARCH_WILDCARD_LIMIT' imported but unused\n36:1: F401 'CFG_WEBSEARCH_DETAILED_META_FORMAT' imported but unused\n36:1: F401 'CFG_BIBSORT_BUCKETS' imported but unused\n36:1: F401 'CFG_WEBSEARCH_FULLTEXT_SNIPPETS' imported but unused\n36:1: F401 'CFG_SOLR_URL' imported but unused\n36:1: F401 'CFG_WEBSEARCH_IDXPAIRS_EXACT_SEARCH' imported but unused\n36:1: F401 'CFG_WEBSEARCH_IDXPAIRS_FIELDS' imported but unused\n36:1: E402 module level import not at top of file\n37:6: E126 continuation line over-indented for hanging indent\n80:1: E402 module level import not at top of file\n81:1: F401 'get_most_popular_field_values' imported but unused\n81:1: E402 module level import not at top of file\n82:1: I101 Imported names are in the wrong order. Should be InvenioWebSearchCitedbyLimitError, InvenioWebSearchReferstoLimitError, InvenioWebSearchUnknownCollectionError, InvenioWebSearchWildcardLimitError\n82:1: I100 Imports statements are in the wrong order. from invenio.modules.search.errors should be before from invenio.modules.search.utils\n82:1: F401 'InvenioWebSearchUnknownCollectionError' imported but unused\n82:1: F401 'InvenioWebSearchReferstoLimitError' imported but unused\n82:1: F401 'InvenioWebSearchWildcardLimitError' imported but unused\n82:1: F401 'InvenioWebSearchCitedbyLimitError' imported but unused\n82:1: E402 module level import not at top of file\n83:6: E126 continuation line over-indented for hanging indent\n87:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibrecord should be before from invenio.modules.search.errors\n87:1: F401 'get_fieldvalues_alephseq_like' imported but unused\n87:1: E402 module level import not at top of file\n89:1: E402 module level import not at top of file\n90:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibrecord should be before from .utils\n90:1: I201 Missing newline before sections or imports.\n90:1: F401 'create_record' imported but unused\n90:1: F401 'record_xml_output' imported but unused\n90:1: E402 module level import not at top of file\n91:1: I101 Imported names are in the wrong order. Should be get_bibrank_methods, is_method_valid, rank_by_citations, rank_records\n91:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibrank.record_sorter should be before from invenio.legacy.bibrecord\n91:1: F401 'rank_by_citations' imported but unused\n91:1: F401 'get_bibrank_methods' imported but unused\n91:1: E402 module level import not at top of file\n96:1: I101 Imported names are in the wrong order. Should be calculate_reading_similarity_list, register_page_view_event\n96:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibrank.downloads_similarity should be before from invenio.legacy.bibrank.record_sorter\n96:1: F401 'register_page_view_event' imported but unused\n96:1: F401 'calculate_reading_similarity_list' imported but unused\n96:1: E402 module level import not at top of file\n97:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibindex.engine_stemmer should be before from invenio.legacy.bibrank.downloads_similarity\n97:1: F401 'stem' imported but unused\n97:1: E402 module level import not at top of file\n98:1: F401 'BibIndexDefaultTokenizer' imported but unused\n98:1: E402 module level import not at top of file\n99:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibindex.engine_utils should be before from invenio.modules.indexer.tokenizers.BibIndexDefaultTokenizer\n99:1: F401 'author_name_requires_phrase_search' imported but unused\n99:1: F401 'get_field_tags' imported but unused\n99:1: E402 module level import not at top of file\n101:1: I101 Imported names are in the wrong order. Should be lower_index_term, wash_author_name, wash_index_term\n101:1: F401 'wash_index_term' imported but unused\n101:1: F401 'wash_author_name' imported but unused\n101:1: F401 'lower_index_term' imported but unused\n101:1: E402 module level import not at top of file\n102:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibindex.engine_config should be before from invenio.legacy.bibindex.engine_washer\n102:1: F401 'CFG_BIBINDEX_SYNONYM_MATCH_TYPE' imported but unused\n102:1: E402 module level import not at top of file\n103:1: F401 'create_download_history_graph_and_box' imported but unused\n103:1: E402 module level import not at top of file\n104:1: F401 'DataCacher' imported but unused\n104:1: E402 module level import not at top of file\n105:1: I101 Imported names are in the wrong order. Should be perform_external_collection_search, print_external_results_overview\n105:1: F401 'perform_external_collection_search' imported but unused\n105:1: F401 'print_external_results_overview' imported but unused\n105:1: E402 module level import not at top of file\n106:1: E402 module level import not at top of file\n107:1: I101 Imported names are in the wrong order. Should be CFG_ACC_GRANT_AUTHOR_RIGHTS_TO_EMAILS_IN_TAGS, CFG_ACC_GRANT_AUTHOR_RIGHTS_TO_USERIDS_IN_TAGS, CFG_ACC_GRANT_VIEWER_RIGHTS_TO_EMAILS_IN_TAGS, CFG_ACC_GRANT_VIEWER_RIGHTS_TO_USERIDS_IN_TAGS, VIEWRESTRCOLL\n107:1: F401 'CFG_ACC_GRANT_AUTHOR_RIGHTS_TO_EMAILS_IN_TAGS' imported but unused\n107:1: F401 'CFG_ACC_GRANT_AUTHOR_RIGHTS_TO_USERIDS_IN_TAGS' imported but unused\n107:1: F401 'CFG_ACC_GRANT_VIEWER_RIGHTS_TO_USERIDS_IN_TAGS' imported but unused\n107:1: F401 'CFG_ACC_GRANT_VIEWER_RIGHTS_TO_EMAILS_IN_TAGS' imported but unused\n107:1: E402 module level import not at top of file\n113:1: I100 Imports statements are in the wrong order. from invenio.legacy.websearch.adminlib should be before from invenio.modules.access.local_config\n113:1: F401 'get_detailed_page_tabs' imported but unused\n113:1: F401 'get_detailed_page_tabs_counts' imported but unused\n113:1: E402 module level import not at top of file\n114:1: I100 Imports statements are in the wrong order. from intbitset should be before from invenio.legacy.websearch.adminlib\n114:1: I201 Missing newline before sections or imports.\n114:1: E402 module level import not at top of file\n115:1: I101 Imported names are in the wrong order. Should be DatabaseError, InvenioDbQueryWildcardLimitError, deserialize_via_marshal\n115:1: I201 Missing newline before sections or imports.\n115:1: F401 'deserialize_via_marshal' imported but unused\n115:1: F401 'DatabaseError' imported but unused\n115:1: F401 'InvenioDbQueryWildcardLimitError' imported but unused\n115:1: E402 module level import not at top of file\n116:1: F401 'acc_authorize_action' imported but unused\n116:1: E402 module level import not at top of file\n117:1: I100 Imports statements are in the wrong order. from invenio.ext.logging should be before from invenio.modules.access.engine\n117:1: E402 module level import not at top of file\n118:1: I101 Imported names are in the wrong order. Should be encode_for_xml, strip_accents, wash_for_utf8\n118:1: F401 'encode_for_xml' imported but unused\n118:1: F401 'wash_for_utf8' imported but unused\n118:1: F401 'strip_accents' imported but unused\n118:1: E402 module level import not at top of file\n119:1: I100 Imports statements are in the wrong order. from invenio.legacy should be before from invenio.utils.text\n119:1: E402 module level import not at top of file\n121:1: E402 module level import not at top of file\n124:1: I101 Imported names are in the wrong order. Should be calculate_cited_by_list, calculate_co_cited_with_list, get_cited_by_list, get_citedby_hitset, get_citers_log, get_records_with_num_cites, get_refers_to_list, get_refersto_hitset\n124:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibrank.citation_searcher should be before import invenio.legacy.template\n124:1: F401 'get_citedby_hitset' imported but unused\n124:1: F401 'get_records_with_num_cites' imported but unused\n124:1: F401 'calculate_cited_by_list' imported but unused\n124:1: F401 'get_refersto_hitset' imported but unused\n124:1: F401 'get_cited_by_list' imported but unused\n124:1: F401 'get_citers_log' imported but unused\n124:1: F401 'calculate_co_cited_with_list' imported but unused\n124:1: F401 'get_refers_to_list' imported but unused\n124:1: E402 module level import not at top of file\n129:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibrank.citation_grapher should be before from invenio.legacy.bibrank.citation_searcher\n129:1: F401 'create_citation_history_graph_and_box' imported but unused\n129:1: E402 module level import not at top of file\n130:1: I101 Imported names are in the wrong order. Should be get_self_cited_by, get_self_cited_by_list, get_self_refers_to_list\n130:1: F401 'get_self_refers_to_list' imported but unused\n130:1: F401 'get_self_cited_by' imported but unused\n130:1: F401 'get_self_cited_by_list' imported but unused\n130:1: E402 module level import not at top of file\n131:55: E126 continuation line over-indented for hanging indent\n134:1: I101 Imported names are in the wrong order. Should be get_table_update_time, run_sql, run_sql_with_limit, wash_table_column_name\n134:1: F401 'wash_table_column_name' imported but unused\n134:1: F401 'get_table_update_time' imported but unused\n134:1: F401 'run_sql_with_limit' imported but unused\n134:1: E402 module level import not at top of file\n136:1: I101 Imported names are in the wrong order. Should be collect_user_info, getUid\n136:1: F401 'collect_user_info' imported but unused\n136:1: E402 module level import not at top of file\n137:1: I101 Imported names are in the wrong order. Should be create_error_box, pagefooteronly, pageheaderonly, write_warning\n137:1: I100 Imports statements are in the wrong order. from invenio.legacy.webpage should be before from invenio.legacy.webuser\n137:1: F401 'pagefooteronly' imported but unused\n137:1: F401 'create_error_box' imported but unused\n137:1: F401 'write_warning' imported but unused\n137:1: F401 'pageheaderonly' imported but unused\n137:1: E402 module level import not at top of file\n138:1: I100 Imports statements are in the wrong order. from invenio.base.i18n should be before from invenio.legacy.webpage\n138:1: E402 module level import not at top of file\n140:1: F401 'apache' imported but unused\n140:1: E402 module level import not at top of file\n142:1: I100 Imports statements are in the wrong order. from invenio.legacy.websearch_external_collections should be before from invenio.utils\n142:1: F401 'do_calculate_hosted_collections_results' imported but unused\n142:1: F401 'calculate_hosted_collections_results' imported but unused\n142:1: E402 module level import not at top of file\n143:1: F401 'CFG_EXTERNAL_COLLECTION_MAXRESULTS' imported but unused\n143:1: E402 module level import not at top of file\n148:22: E203 whitespace before ':'\n148:14: E225 missing whitespace around operator\n149:24: E203 whitespace before ':'\n150:24: E203 whitespace before ':'\n151:28: E203 whitespace before ':'\n152:30: E203 whitespace before ':'\n153:24: E203 whitespace before ':'\n154:23: E203 whitespace before ':'\n155:29: E203 whitespace before ':'\n156:26: E203 whitespace before ':'\n157:33: E203 whitespace before ':'\n158:30: E203 whitespace before ':'\n159:30: E203 whitespace before ':'\n160:30: E203 whitespace before ':'\n161:30: E203 whitespace before ':'\n162:30: E203 whitespace before ':'\n163:30: E203 whitespace before ':'\n164:41: E703 statement ends with a semicolon\n166:1: E402 module level import not at top of file\n167:1: F401 'collection_restricted_p' imported but unused\n167:1: E402 module level import not at top of file\n168:1: F401 'restricted_collection_cache' imported but unused\n168:1: E402 module level import not at top of file\n169:1: F401 'get_permitted_restricted_collections' imported but unused\n169:1: E402 module level import not at top of file\n170:1: I100 Imports statements are in the wrong order. from invenio.modules.collections.cache should be before from invenio.modules.search.utils\n170:1: F401 'get_all_restricted_recids' imported but unused\n170:1: E402 module level import not at top of file\n173:1: F401 'check_user_can_view_record' imported but unused\n173:1: E402 module level import not at top of file\n175:1: I100 Imports statements are in the wrong order. from invenio.modules.collections.cache should be before from invenio.modules.records.access\n175:1: E402 module level import not at top of file\n176:1: I100 Imports statements are in the wrong order. from invenio.modules.collections.cache should be before from invenio.modules.collections.cache\n176:1: E402 module level import not at top of file\n177:1: F401 'get_field_i18nname' imported but unused\n177:1: E402 module level import not at top of file\n179:1: I100 Imports statements are in the wrong order. from invenio.modules.indexer.models should be before from invenio.modules.search.cache\n179:1: E402 module level import not at top of file\n185: D401 First line should be in imperative mood ('Create', not 'Creates')\n185: D202 No blank lines allowed after function docstring (found 1)\n185: D400 First line should end with a period (not 'n')\n185: D205 1 blank line required between summary line and description (found 0)\n193:33: E261 at least two spaces before inline comment\n202:1: F401 'get_synonym_terms' imported but unused\n202:1: E402 module level import not at top of file\n204:1: F403 'from invenio.modules.search.washers import *' used; unable to detect undefined names\n204:1: E402 module level import not at top of file\n207: D401 First line should be in imperative mood ('Return', not 'Returns')\n207: D300 Use \"\"\"triple double quotes\"\"\" (found '-quotes)\n228:1: I100 Imports statements are in the wrong order. from invenio.modules.collections.cache should be before from invenio.modules.search.washers\n228:1: F401 'get_collection_allchildren' imported but unused\n228:1: E402 module level import not at top of file\n231: D401 First line should be in imperative mood ('Return', not 'Returns')\n231: D202 No blank lines allowed after function docstring (found 1)\n234:5: E266 too many leading '#' for block comment\n238:5: E266 too many leading '#' for block comment\n239:5: E266 too many leading '#' for block comment\n241:34: E261 at least two spaces before inline comment\n258:13: B901 blind except: statement\n263:9: E266 too many leading '#' for block comment\n275:9: E266 too many leading '#' for block comment\n280:17: E266 too many leading '#' for block comment\n289: D400 First line should end with a period (not 'o')\n289: D205 1 blank line required between summary line and description (found 0)\n289: D208 Docstring is over-indented\n334:1: F403 'from invenio.modules.search.searchext.engines.native import *' used; unable to detect undefined names\n334:1: E402 module level import not at top of file\n337: D209 Multi-line docstring closing quotes should be on a separate line\n337: D400 First line should end with a period (not 'y')\n337: D205 1 blank line required between summary line and description (found 0)\n337: D208 Docstring is over-indented\n350:38: E203 whitespace before ','\n381:1: E302 expected 2 blank lines, found 0\n381: D209 Multi-line docstring closing quotes should be on a separate line\n381: D400 First line should end with a period (not 'g')\n381: D205 1 blank line required between summary line and description (found 0)\n381: D208 Docstring is over-indented\n410: D209 Multi-line docstring closing quotes should be on a separate line\n410: D205 1 blank line required between summary line and description (found 0)\n422:1: I100 Imports statements are in the wrong order. from invenio.modules.search.models should be before from invenio.modules.search.searchext.engines.native\n422:1: E402 module level import not at top of file\n427: D210 No whitespaces allowed surrounding docstring text\n427: None: None\n427: D400 First line should end with a period (not 'h')\n427: D205 1 blank line required between summary line and description (found 0)\n445: D401 First line should be in imperative mood ('I', not 'Is')\n445: D400 First line should end with a period (not '?')\n456:1: E302 expected 2 blank lines, found 1\n456: D205 1 blank line required between summary line and description (found 0)\n456: D208 Docstring is over-indented\n463:1: I101 Imported names are in the wrong order. Should be CACHE_SORTED_DATA, SORTING_METHODS\n463:1: F401 'CACHE_SORTED_DATA' imported but unused\n463:1: E402 module level import not at top of file\n466: D205 1 blank line required between summary line and description (found 0)\n466: D208 Docstring is over-indented\n480:20: E127 continuation line over-indented for visual indent\n482:25: E126 continuation line over-indented for hanging indent\n533: D103 Missing docstring in public function\n543: D401 First line should be in imperative mood ('calculate', not 'calculates')\n543: D209 Multi-line docstring closing quotes should be on a separate line\n543: D202 No blank lines allowed after function docstring (found 1)\n543: D400 First line should end with a period (not 'e')\n543: D205 1 blank line required between summary line and description (found 0)\n554:20: E261 at least two spaces before inline comment\n558:17: E261 at least two spaces before inline comment\n601: D210 No whitespaces allowed surrounding docstring text\n601: None: None\n601: D400 First line should end with a period (not 'n')\n607:25: E261 at least two spaces before inline comment\n608:13: E222 multiple spaces after operator\n631:10: E261 at least two spaces before inline comment\n648:1: E266 too many leading '#' for block comment\n650: D103 Missing docstring in public function\n651:25: E128 continuation line under-indented for visual indent\n652:25: E128 continuation line under-indented for visual indent\n653:25: E128 continuation line under-indented for visual indent\n654:25: E128 continuation line under-indented for visual indent\n671: D401 First line should be in imperative mood ('Set', not 'Sets')\n671: D202 No blank lines allowed after function docstring (found 1)\n671: D400 First line should end with a period (not 'l')\n671: D200 One-line docstring should fit on one line with quotes (found 3)\n672:23: E128 continuation line under-indented for visual indent\n673:23: E128 continuation line under-indented for visual indent\n674:23: E128 continuation line under-indented for visual indent\n675:23: E128 continuation line under-indented for visual indent\n676:23: E128 continuation line under-indented for visual indent\n714:9: B901 blind except: statement"}}