{"comment":{"url":"https://api.github.com/repos/jirikuncar/invenio/comments/10167307","html_url":"https://github.com/jirikuncar/invenio/commit/e14b74836d630e0547b711181fe33baa595fd671#commitcomment-10167307","id":10167307,"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":"e14b74836d630e0547b711181fe33baa595fd671","created_at":"2015-03-12T15:51:38Z","updated_at":"2015-03-12T15:51:38Z","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: I101 Imported names are in the wrong order. Should be create_excel, format_record, format_records, get_output_format_content_type\n103:1: F401 'create_excel' imported but unused\n103:1: F401 'get_output_format_content_type' imported but unused\n103:1: F401 'format_record' imported but unused\n103:1: F401 'format_records' imported but unused\n103:1: E402 module level import not at top of file\n104:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibrank.downloads_grapher should be before from invenio.modules.formatter\n104:1: F401 'create_download_history_graph_and_box' imported but unused\n104:1: E402 module level import not at top of file\n105:1: F401 'DataCacher' imported but unused\n105:1: E402 module level import not at top of file\n106:1: I101 Imported names are in the wrong order. Should be perform_external_collection_search, print_external_results_overview\n106:1: F401 'perform_external_collection_search' imported but unused\n106:1: F401 'print_external_results_overview' imported but unused\n106:1: E402 module level import not at top of file\n107:1: E402 module level import not at top of file\n108:1: I101 Imported names are in the wrong order. Should be CFG_ACC_GRANT_AUTHOR_RIGHTS_TO_EMAILS_IN_TAGS, CFG_ACC_GRANT_VIEWER_RIGHTS_TO_EMAILS_IN_TAGS, VIEWRESTRCOLL\n108:1: F401 'CFG_ACC_GRANT_AUTHOR_RIGHTS_TO_EMAILS_IN_TAGS' imported but unused\n108:1: F401 'CFG_ACC_GRANT_VIEWER_RIGHTS_TO_EMAILS_IN_TAGS' imported but unused\n108:1: E402 module level import not at top of file\n111:1: I100 Imports statements are in the wrong order. from invenio.legacy.websearch.adminlib should be before from invenio.modules.access.local_config\n111:1: F401 'get_detailed_page_tabs' imported but unused\n111:1: F401 'get_detailed_page_tabs_counts' imported but unused\n111:1: E402 module level import not at top of file\n112:1: I100 Imports statements are in the wrong order. from intbitset should be before from invenio.legacy.websearch.adminlib\n112:1: I201 Missing newline before sections or imports.\n112:1: E402 module level import not at top of file\n113:1: I101 Imported names are in the wrong order. Should be DatabaseError, InvenioDbQueryWildcardLimitError, deserialize_via_marshal\n113:1: I201 Missing newline before sections or imports.\n113:1: F401 'deserialize_via_marshal' imported but unused\n113:1: F401 'DatabaseError' imported but unused\n113:1: F401 'InvenioDbQueryWildcardLimitError' imported but unused\n113:1: E402 module level import not at top of file\n114:1: F401 'acc_authorize_action' imported but unused\n114:1: E402 module level import not at top of file\n115:1: I100 Imports statements are in the wrong order. from invenio.ext.logging should be before from invenio.modules.access.engine\n115:1: E402 module level import not at top of file\n116:1: I101 Imported names are in the wrong order. Should be encode_for_xml, strip_accents, wash_for_utf8\n116:1: F401 'encode_for_xml' imported but unused\n116:1: F401 'wash_for_utf8' imported but unused\n116:1: F401 'strip_accents' 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.legacy should be before from invenio.utils.text\n117:1: E402 module level import not at top of file\n119:1: E402 module level import not at top of file\n122: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\n122:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibrank.citation_searcher should be before import invenio.legacy.template\n122:1: F401 'get_citedby_hitset' imported but unused\n122:1: F401 'calculate_cited_by_list' imported but unused\n122:1: F401 'get_refersto_hitset' imported but unused\n122:1: F401 'get_records_with_num_cites' imported but unused\n122:1: F401 'get_cited_by_list' imported but unused\n122:1: F401 'get_citers_log' imported but unused\n122:1: F401 'calculate_co_cited_with_list' imported but unused\n122:1: F401 'get_refers_to_list' imported but unused\n122:1: E402 module level import not at top of file\n127:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibrank.citation_grapher should be before from invenio.legacy.bibrank.citation_searcher\n127:1: F401 'create_citation_history_graph_and_box' imported but unused\n127:1: E402 module level import not at top of file\n128: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\n128:1: F401 'get_self_refers_to_list' imported but unused\n128:1: F401 'get_self_cited_by' imported but unused\n128:1: F401 'get_self_cited_by_list' imported but unused\n128:1: E402 module level import not at top of file\n129:55: E126 continuation line over-indented for hanging indent\n132: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\n132:1: F401 'wash_table_column_name' imported but unused\n132:1: F401 'get_table_update_time' imported but unused\n132:1: F401 'run_sql_with_limit' imported but unused\n132:1: E402 module level import not at top of file\n134:1: I101 Imported names are in the wrong order. Should be collect_user_info, getUid\n134:1: F401 'collect_user_info' imported but unused\n134:1: E402 module level import not at top of file\n135:1: I101 Imported names are in the wrong order. Should be create_error_box, pagefooteronly, pageheaderonly, write_warning\n135:1: I100 Imports statements are in the wrong order. from invenio.legacy.webpage should be before from invenio.legacy.webuser\n135:1: F401 'pagefooteronly' imported but unused\n135:1: F401 'create_error_box' imported but unused\n135:1: F401 'write_warning' imported but unused\n135:1: F401 'pageheaderonly' imported but unused\n135:1: E402 module level import not at top of file\n136:1: I100 Imports statements are in the wrong order. from invenio.base.i18n should be before from invenio.legacy.webpage\n136:1: E402 module level import not at top of file\n138:1: F401 'apache' imported but unused\n138:1: E402 module level import not at top of file\n140:1: I100 Imports statements are in the wrong order. from invenio.legacy.websearch_external_collections should be before from invenio.utils\n140:1: F401 'do_calculate_hosted_collections_results' imported but unused\n140:1: F401 'calculate_hosted_collections_results' imported but unused\n140:1: E402 module level import not at top of file\n141:1: F401 'CFG_EXTERNAL_COLLECTION_MAXRESULTS' imported but unused\n141:1: E402 module level import not at top of file\n146:22: E203 whitespace before ':'\n146:14: E225 missing whitespace around operator\n147:24: E203 whitespace before ':'\n148:24: E203 whitespace before ':'\n149:28: E203 whitespace before ':'\n150:30: E203 whitespace before ':'\n151:24: E203 whitespace before ':'\n152:23: E203 whitespace before ':'\n153:29: E203 whitespace before ':'\n154:26: E203 whitespace before ':'\n155:33: E203 whitespace before ':'\n156:30: E203 whitespace before ':'\n157:30: E203 whitespace before ':'\n158:30: E203 whitespace before ':'\n159:30: E203 whitespace before ':'\n160:30: E203 whitespace before ':'\n161:30: E203 whitespace before ':'\n162:41: E703 statement ends with a semicolon\n164:1: E402 module level import not at top of file\n165:1: F401 'collection_restricted_p' imported but unused\n165:1: E402 module level import not at top of file\n166:1: F401 'restricted_collection_cache' imported but unused\n166:1: E402 module level import not at top of file\n167:1: F401 'get_permitted_restricted_collections' imported but unused\n167:1: E402 module level import not at top of file\n168:1: I100 Imports statements are in the wrong order. from invenio.modules.collections.cache should be before from invenio.modules.search.utils\n168:1: F401 'get_all_restricted_recids' imported but unused\n168:1: E402 module level import not at top of file\n171:1: F401 'check_user_can_view_record' imported but unused\n171:1: E402 module level import not at top of file\n173:1: I100 Imports statements are in the wrong order. from invenio.modules.collections.cache should be before from invenio.modules.records.access\n173:1: E402 module level import not at top of file\n174:1: I100 Imports statements are in the wrong order. from invenio.modules.collections.cache should be before from invenio.modules.collections.cache\n174:1: E402 module level import not at top of file\n175:1: F401 'get_field_i18nname' imported but unused\n175:1: E402 module level import not at top of file\n177:1: I100 Imports statements are in the wrong order. from invenio.modules.indexer.models should be before from invenio.modules.search.cache\n177:1: E402 module level import not at top of file\n183: D401 First line should be imperative: 'Create', not 'Creates'\n183: D202 No blank lines allowed *after* function docstring, found 1\n183: D400 First line should end with '.', not 'n'\n183: D205 Blank line missing between one-line summary and description\n191:33: E261 at least two spaces before inline comment\n200:1: F401 'get_synonym_terms' imported but unused\n200:1: E402 module level import not at top of file\n202:1: F403 'from invenio.modules.search.washers import *' used; unable to detect undefined names\n202:1: E402 module level import not at top of file\n205: D401 First line should be imperative: 'Return', not 'Returns'\n205: D300 Expected \"\"\"-quotes, got '-quotes\n226:1: I100 Imports statements are in the wrong order. from invenio.modules.collections.cache should be before from invenio.modules.search.washers\n226:1: F401 'get_collection_allchildren' imported but unused\n226:1: E402 module level import not at top of file\n229: D401 First line should be imperative: 'Return', not 'Returns'\n229: D202 No blank lines allowed *after* function docstring, found 1\n232:5: E266 too many leading '#' for block comment\n236:5: E266 too many leading '#' for block comment\n237:5: E266 too many leading '#' for block comment\n239:34: E261 at least two spaces before inline comment\n256:13: B901 blind except: statement\n261:9: E266 too many leading '#' for block comment\n273:9: E266 too many leading '#' for block comment\n278:17: E266 too many leading '#' for block comment\n287: D400 First line should end with '.', not 'o'\n287: D205 Blank line missing between one-line summary and description\n287: D208 Docstring is over-indented\n332:1: F403 'from invenio.modules.search.searchext.engines.native import *' used; unable to detect undefined names\n332:1: E402 module level import not at top of file\n335: D209 Put multi-line docstring closing quotes on separate line\n335: D400 First line should end with '.', not 'y'\n335: D205 Blank line missing between one-line summary and description\n335: D208 Docstring is over-indented\n348:38: E203 whitespace before ','\n379:1: E302 expected 2 blank lines, found 0\n379: D209 Put multi-line docstring closing quotes on separate line\n379: D400 First line should end with '.', not 'g'\n379: D205 Blank line missing between one-line summary and description\n379: D208 Docstring is over-indented\n408: D209 Put multi-line docstring closing quotes on separate line\n408: D205 Blank line missing between one-line summary and description\n420:1: I100 Imports statements are in the wrong order. from invenio.modules.search.models should be before from invenio.modules.search.searchext.engines.native\n420:1: E402 module level import not at top of file\n425: D290 White spaces around docstring should be trimmed.\n425: D400 First line should end with '.', not 'h'\n425: D205 Blank line missing between one-line summary and description\n443: D401 First line should be imperative: 'I', not 'Is'\n443: D400 First line should end with '.', not '?'\n454:1: E302 expected 2 blank lines, found 1\n454: D205 Blank line missing between one-line summary and description\n454: D208 Docstring is over-indented\n461:1: I101 Imported names are in the wrong order. Should be CACHE_SORTED_DATA, SORTING_METHODS\n461:1: F401 'CACHE_SORTED_DATA' imported but unused\n461:1: E402 module level import not at top of file\n464: D205 Blank line missing between one-line summary and description\n464: D208 Docstring is over-indented\n478:20: E127 continuation line over-indented for visual indent\n480:25: E126 continuation line over-indented for hanging indent\n531: D103 Docstring missing\n541: D401 First line should be imperative: 'calculate', not 'calculates'\n541: D209 Put multi-line docstring closing quotes on separate line\n541: D202 No blank lines allowed *after* function docstring, found 1\n541: D400 First line should end with '.', not 'e'\n541: D205 Blank line missing between one-line summary and description\n552:20: E261 at least two spaces before inline comment\n556:17: E261 at least two spaces before inline comment\n599: D290 White spaces around docstring should be trimmed.\n599: D400 First line should end with '.', not 'n'\n605:25: E261 at least two spaces before inline comment\n606:13: E222 multiple spaces after operator\n629:10: E261 at least two spaces before inline comment\n646:1: E266 too many leading '#' for block comment\n648: D103 Docstring missing\n649:25: E128 continuation line under-indented for visual indent\n650:25: E128 continuation line under-indented for visual indent\n651:25: E128 continuation line under-indented for visual indent\n652:25: E128 continuation line under-indented for visual indent\n669: D401 First line should be imperative: 'Set', not 'Sets'\n669: D202 No blank lines allowed *after* function docstring, found 1\n669: D400 First line should end with '.', not 'l'\n669: D200 One-line docstring should not occupy 3 lines\n670:23: E128 continuation line under-indented for visual indent\n671:23: E128 continuation line under-indented for visual indent\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\n712:9: B901 blind except: statement"}}