{"comment":{"url":"https://api.github.com/repos/jirikuncar/invenio/comments/9866805","html_url":"https://github.com/jirikuncar/invenio/commit/95bf68d152f2b3741f612c6862b588d6fefd2a19#commitcomment-9866805","id":9866805,"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/bibedit/utils.py","commit_id":"95bf68d152f2b3741f612c6862b588d6fefd2a19","created_at":"2015-02-23T09:22:01Z","updated_at":"2015-02-23T09:22:01Z","body":"1:1: E266 too many leading '#' for block comment\n2:1: E266 too many leading '#' for block comment\n2: L102 copyright year is outdated, expected 2015 but got 2013\n4:1: E266 too many leading '#' for block comment\n5:1: E266 too many leading '#' for block comment\n6:1: E266 too many leading '#' for block comment\n7:1: E266 too many leading '#' for block comment\n9:1: E266 too many leading '#' for block comment\n10:1: E266 too many leading '#' for block comment\n11:1: E266 too many leading '#' for block comment\n12:1: E266 too many leading '#' for block comment\n14:1: E266 too many leading '#' for block comment\n15:1: E266 too many leading '#' for block comment\n16:1: E266 too many leading '#' for block comment\n29: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\n34:1: E402 module level import not at top of file\n35:1: E402 module level import not at top of file\n36:1: I100 Imports statements are in the wrong order. import tempfile should be before import zlib\n36:1: E402 module level import not at top of file\n37:1: I100 Imports statements are in the wrong order. import sys should be before import tempfile\n37:1: E402 module level import not at top of file\n38:1: F401 'traceback' imported but unused\n38:1: E402 module level import not at top of file\n39:1: E402 module level import not at top of file\n40:1: I201 Missing newline before sections or imports.\n40:1: E402 module level import not at top of file\n47:1: I101 Imported names are in the wrong order. Should be cache_exists, delete_cache, delete_hp_change, get_cache, get_cache_post_date, get_record_last_modification_date, get_record_revision_author, uids_with_active_caches, update_cache, update_cache_post_date\n47:1: E402 module level import not at top of file\n48:5: E128 continuation line under-indented for visual indent\n49:5: E128 continuation line under-indented for visual indent\n50:5: E128 continuation line under-indented for visual indent\n51:1: I101 Imported names are in the wrong order. Should be create_record, create_records, field_add_subfield, field_get_subfield_codes, field_get_subfield_values, print_rec, record_add_field, record_add_fields, record_delete_fields, record_get_field_instances, record_get_field_value, record_get_field_values, record_has_field, record_make_all_subfields_volatile, record_modify_controlfield, record_modify_subfield, record_order_subfields, record_strip_empty_fields, record_strip_empty_volatile_subfields, record_xml_output\n51:1: E402 module level import not at top of file\n59:1: E402 module level import not at top of file\n60:1: I101 Imported names are in the wrong order. Should be CFG_BIBEDIT_EXTEND_RECORD_WITH_COLLECTION_TEMPLATE, CFG_BIBEDIT_LOCKLEVEL, CFG_BIBEDIT_QUEUE_CHECK_METHOD, CFG_BIBEDIT_TIMEOUT, CFG_BIBUPLOAD_EXTERNAL_OAIID_TAG, CFG_BIBUPLOAD_EXTERNAL_SYSNO_TAG, CFG_PYLIBDIR\n60:1: I100 Imports statements are in the wrong order. from invenio.config should be before from invenio.legacy.bibsched.bibtask\n60:1: F401 'CFG_PYLIBDIR' imported but unused\n60:1: E402 module level import not at top of file\n66:1: E402 module level import not at top of file\n67:1: E402 module level import not at top of file\n68:1: I101 Imported names are in the wrong order. Should be get_bibupload_task_opts, get_info_of_record_revision, get_marcxml_of_record_revision, get_record_revisions\n68:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibedit.db_layer should be before from invenio.utils.text\n68:1: E402 module level import not at top of file\n71:1: I101 Imported names are in the wrong order. Should be get_all_collections_of_a_record, get_record, guess_primary_collection_of_a_record, record_exists\n71:1: E402 module level import not at top of file\n72:6: E126 continuation line over-indented for hanging indent\n74:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibrecord should be before from invenio.legacy.search_engine\n74:1: E402 module level import not at top of file\n75:1: I101 Imported names are in the wrong order. Should be collect_user_info, get_email, get_user_info, get_user_preferences, list_registered_users\n75:1: E402 module level import not at top of file\n76:6: E126 continuation line over-indented for hanging indent\n77:1: I100 Imports statements are in the wrong order. from invenio.legacy.dbquery should be before from invenio.legacy.webuser\n77:1: E402 module level import not at top of file\n78:1: E402 module level import not at top of file\n79:1: E402 module level import not at top of file\n80:1: I100 Imports statements are in the wrong order. from invenio.legacy.refextract.api should be before from invenio.modules.access.engine\n80:1: E402 module level import not at top of file\n81:36: E126 continuation line over-indented for hanging indent\n83:1: I101 Imported names are in the wrong order. Should be ParseError, transform_file\n83:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibrecord.textmarc2xmlmarc should be before from invenio.legacy.refextract.api\n83:1: E402 module level import not at top of file\n84:1: E402 module level import not at top of file\n85:1: I100 Imports statements are in the wrong order. from invenio.modules.editor.registry should be before from invenio.modules.knowledge.api\n85:1: E402 module level import not at top of file\n88:1: I100 Imports statements are in the wrong order. from invenio.base.globals should be before from invenio.modules.editor.registry\n88:1: E402 module level import not at top of file\n89:1: E402 module level import not at top of file\n90:1: E402 module level import not at top of file\n95:5: F401 'loads' imported but unused\n96:1: E402 module level import not at top of file\n116: D101 Docstring missing\n119:1: E302 expected 2 blank lines, found 1\n119: D203 Expected 1 blank line *before* class docstring, found 0\n119: D204 Expected 1 blank line *after* class docstring, found 0\n119: D400 First line should end with '.', not 's'\n125:1: E302 expected 2 blank lines, found 1\n125: D290 White spaces around docstring should be trimmed.\n125: D400 First line should end with '.', not 'n'\n125: D205 Blank line missing between one-line summary and description\n129: D290 White spaces around docstring should be trimmed.\n129: D400 First line should end with '.', not 'e'\n172:1: E302 expected 2 blank lines, found 1\n172: D103 Docstring missing\n178:1: E302 expected 2 blank lines, found 1\n178: D401 First line should be imperative: 'Thi', not 'This'\n178: D400 First line should end with '.', not 'l'\n178: D205 Blank line missing between one-line summary and description\n229:1: E302 expected 2 blank lines, found 1\n229: D400 First line should end with '.', not 'e'\n229: D205 Blank line missing between one-line summary and description\n244:1: E302 expected 2 blank lines, found 1\n244: D401 First line should be imperative: 'Ha', not 'Has'\n244: D400 First line should end with '.', not 'y'\n244: D205 Blank line missing between one-line summary and description\n252:1: E302 expected 2 blank lines, found 1\n252: D400 First line should end with '.', not 'l'\n252: D205 Blank line missing between one-line summary and description\n278:1: E302 expected 2 blank lines, found 1\n278: D400 First line should end with '.', not 'e'\n278: D205 Blank line missing between one-line summary and description\n285:1: E302 expected 2 blank lines, found 1\n294:1: E302 expected 2 blank lines, found 1\n305:1: E302 expected 2 blank lines, found 1\n305: D400 First line should end with '.', not 'n'\n305: D205 Blank line missing between one-line summary and description\n316:1: E302 expected 2 blank lines, found 1\n320:1: E302 expected 2 blank lines, found 1\n320: D205 Blank line missing between one-line summary and description\n331:1: E302 expected 2 blank lines, found 1\n331: D103 Docstring missing\n335:1: E302 expected 2 blank lines, found 1\n335: D400 First line should end with '.', not 'm'\n335: D205 Blank line missing between one-line summary and description\n403: D400 First line should end with '.', not 'd'\n403: D205 Blank line missing between one-line summary and description\n416: D290 White spaces around docstring should be trimmed.\n416: D400 First line should end with '.', not 'd'\n416: D200 One-line docstring should not occupy 2 lines\n428: D290 White spaces around docstring should be trimmed.\n428: D400 First line should end with '.', not 'e'\n428: D205 Blank line missing between one-line summary and description\n451: D400 First line should end with '.', not 'e'\n451: D205 Blank line missing between one-line summary and description\n481:24: E713 test for membership should be 'not in'\n490:1: E302 expected 2 blank lines, found 1\n490: D401 First line should be imperative: 'Convert', not 'Converts'\n490: D400 First line should end with '.', not 'p'\n490: D200 One-line docstring should not occupy 3 lines\n497:1: E302 expected 2 blank lines, found 1\n497: D401 First line should be imperative: 'Convert', not 'Converts'\n497: D400 First line should end with '.', not 'e'\n497: D200 One-line docstring should not occupy 3 lines\n509:1: E302 expected 2 blank lines, found 1\n509: D400 First line should end with '.', not 'd'\n517:1: E302 expected 2 blank lines, found 1\n517: D209 Put multi-line docstring closing quotes on separate line\n517: D400 First line should end with '.', not 's'\n517: D205 Blank line missing between one-line summary and description\n517: D208 Docstring is over-indented\n530:1: E302 expected 2 blank lines, found 1\n530: D205 Blank line missing between one-line summary and description\n540:1: E302 expected 2 blank lines, found 1\n540: D205 Blank line missing between one-line summary and description\n551:1: E302 expected 2 blank lines, found 1\n551: D205 Blank line missing between one-line summary and description\n558:1: E302 expected 2 blank lines, found 1\n558: D205 Blank line missing between one-line summary and description\n578:1: E302 expected 2 blank lines, found 1\n584:1: E302 expected 2 blank lines, found 1\n584: D103 Docstring missing\n591:1: E302 expected 2 blank lines, found 1\n591: D205 Blank line missing between one-line summary and description\n607: D290 White spaces around docstring should be trimmed.\n607: D205 Blank line missing between one-line summary and description\n628:9: E128 continuation line under-indented for visual indent\n631:1: E302 expected 2 blank lines, found 1\n631: D209 Put multi-line docstring closing quotes on separate line\n631: D400 First line should end with '.', not ']'\n631: D205 Blank line missing between one-line summary and description\n631: D208 Docstring is over-indented\n670:1: E302 expected 2 blank lines, found 1\n670: D401 First line should be imperative: 'Return', not 'Returns'\n670: D400 First line should end with '.', not ']'\n675:1: E302 expected 2 blank lines, found 1\n693: D205 Blank line missing between one-line summary and description\n701:1: E302 expected 2 blank lines, found 1\n714:1: E302 expected 2 blank lines, found 1\n754:1: E302 expected 2 blank lines, found 1\n772: D400 First line should end with '.', not 's'\n772: D205 Blank line missing between one-line summary and description\n793: D401 First line should be imperative: 'Return', not 'Returns'\n793: D290 White spaces around docstring should be trimmed.\n793: D400 First line should end with '.', not '0'\n793: D205 Blank line missing between one-line summary and description\n804:61: E128 continuation line under-indented for visual indent\n805:61: E128 continuation line under-indented for visual indent\n806:61: E128 continuation line under-indented for visual indent\n807:61: E128 continuation line under-indented for visual indent\n810: D290 White spaces around docstring should be trimmed.\n810: D400 First line should end with '.', not 't'\n810: D205 Blank line missing between one-line summary and description\n825: D290 White spaces around docstring should be trimmed.\n825: D400 First line should end with '.', not 't'\n848:48: E128 continuation line under-indented for visual indent\n852:1: E266 too many leading '#' for block comment\n854:1: E302 expected 2 blank lines, found 1\n854: D400 First line should end with '.', not 'd'\n896:1: E266 too many leading '#' for block comment\n898:1: E302 expected 2 blank lines, found 1\n898: D400 First line should end with '.', not 't'\n898: D205 Blank line missing between one-line summary and description\n916: D400 First line should end with '.', not 'e'\n916: D205 Blank line missing between one-line summary and description\n934:5: E122 continuation line missing indentation or outdented\n950:36: E127 continuation line over-indented for visual indent\n951:36: E127 continuation line over-indented for visual indent\n952:36: E127 continuation line over-indented for visual indent\n953:36: E127 continuation line over-indented for visual indent\n957: D400 First line should end with '.', not 'n'\n1013:1: E266 too many leading '#' for block comment\n1015: D401 First line should be imperative: 'Create', not 'Creates'\n1015: D400 First line should end with '.', not 'e'\n1015: D205 Blank line missing between one-line summary and description\n1030: D400 First line should end with '.', not 'n'\n1030: D205 Blank line missing between one-line summary and description\n1050: D401 First line should be imperative: 'Change', not 'Changes'\n1050: D400 First line should end with '.', not ','\n1050: D205 Blank line missing between one-line summary and description\n1068:9: E128 continuation line under-indented for visual indent\n1078:13: E128 continuation line under-indented for visual indent\n1081: D401 First line should be imperative: 'Return', not 'Returns'\n1081: D290 White spaces around docstring should be trimmed.\n1081: D400 First line should end with '.', not 'e'\n1106:1: E266 too many leading '#' for block comment\n1109: D103 Docstring missing\n1123:25: E128 continuation line under-indented for visual indent\n1162: D401 First line should be imperative: 'Load', not 'Loads'\n1162: D205 Blank line missing between one-line summary and description\n1178:52: E201 whitespace after '{'\n1178:62: E203 whitespace before ':'\n1178:89: E203 whitespace before ':'\n1178:105: E202 whitespace before '}'\n1191: D401 First line should be imperative: 'Split', not 'Splits'\n1191: D400 First line should end with '.', not ':'\n1191: D205 Blank line missing between one-line summary and description\n1191: D300 Expected \"\"\"-quotes, got '''-quotes\n1271: D401 First line should be imperative: 'Create', not 'Creates'\n1271: D400 First line should end with '.', not 'e'\n1271: D205 Blank line missing between one-line summary and description\n1271: D300 Expected \"\"\"-quotes, got '''-quotes\n1306: D205 Blank line missing between one-line summary and description\n1306: D300 Expected \"\"\"-quotes, got '''-quotes"}}