{"comment":{"url":"https://api.github.com/repos/jirikuncar/invenio/comments/9868043","html_url":"https://github.com/jirikuncar/invenio/commit/50309d31cb355c94f448ac737f923eaa14c6f7b3#commitcomment-9868043","id":9868043,"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/websubmit/webinterface.py","commit_id":"50309d31cb355c94f448ac737f923eaa14c6f7b3","created_at":"2015-02-23T10:36:26Z","updated_at":"2015-02-23T10:36:26Z","body":"1:1: E266 too many leading '#' for block comment\n1: D100 Docstring missing\n2:1: E266 too many leading '#' for block comment\n2: L102 copyright year is outdated, expected 2015 but got 2014\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\n22:1: E402 module level import not at top of file\n23:1: I100 Imports statements are in the wrong order. import errno should be before import os\n23:1: E402 module level import not at top of file\n24:1: E402 module level import not at top of file\n25:1: I100 Imports statements are in the wrong order. import cgi should be before import time\n25:1: E402 module level import not at top of file\n26:1: E402 module level import not at top of file\n27:1: I100 Imports statements are in the wrong order. import shutil should be before import sys\n27:1: E402 module level import not at top of file\n29:1: E402 module level import not at top of file\n30:1: I100 Imports statements are in the wrong order. from collections should be before from urllib\n30:1: E402 module level import not at top of file\n32:1: E402 module level import not at top of file\n33:1: I101 Imported names are in the wrong order. Should be CFG_ACCESS_CONTROL_LEVEL_SITE, CFG_CERN_SITE, CFG_PREFIX, CFG_SITE_LANG, CFG_SITE_NAME, CFG_SITE_SECURE_URL, CFG_SITE_URL, CFG_WEBSUBMIT_STORAGEDIR\n33:1: I100 Imports statements are in the wrong order. from invenio.config should be before from invenio.legacy.bibrecord\n33:1: E402 module level import not at top of file\n34:6: E126 continuation line over-indented for hanging indent\n42:1: I101 Imported names are in the wrong order. Should be CrossrefError, get_marcxml_for_doi\n42:1: E402 module level import not at top of file\n43:1: I100 Imports statements are in the wrong order. from invenio.utils should be before from invenio.utils.crossref\n43:1: E402 module level import not at top of file\n44:1: I100 Imports statements are in the wrong order. from invenio.legacy.dbquery should be before from invenio.utils\n44:1: E402 module level import not at top of file\n45:1: E402 module level import not at top of file\n46:1: I100 Imports statements are in the wrong order. from invenio.modules.access.control should be before from invenio.modules.access.engine\n46:1: E402 module level import not at top of file\n47:1: I100 Imports statements are in the wrong order. from invenio.legacy.webpage should be before from invenio.modules.access.control\n47:1: E402 module level import not at top of file\n48:1: I101 Imported names are in the wrong order. Should be collect_user_info, getUid, isGuestUser, page_not_authorized\n48:1: E402 module level import not at top of file\n49:29: E126 continuation line over-indented for hanging indent\n50:1: E402 module level import not at top of file\n51:1: I101 Imported names are in the wrong order. Should be WebInterfaceDirectory, wash_urlargd\n51:1: I100 Imports statements are in the wrong order. from invenio.ext.legacy.handler should be before from invenio.modules.records.access\n51:1: E402 module level import not at top of file\n52:1: E402 module level import not at top of file\n53:1: I100 Imports statements are in the wrong order. from invenio.base.i18n should be before from invenio.utils.url\n53:1: E402 module level import not at top of file\n54:1: I101 Imported names are in the wrong order. Should be decompose_file, propose_next_docname, stream_file\n54:1: E402 module level import not at top of file\n56:1: I100 Imports statements are in the wrong order. from invenio.ext.logging should be before from invenio.legacy.bibdocfile.api\n56:1: E402 module level import not at top of file\n57:1: E402 module level import not at top of file\n58:1: I101 Imported names are in the wrong order. Should be InvenioWebSubmitIconCreatorError, create_icon\n58:1: I100 Imports statements are in the wrong order. from invenio.legacy.websubmit.icon_creator should be before from invenio.utils.html\n58:1: E402 module level import not at top of file\n59:1: I100 Imports statements are in the wrong order. from invenio.legacy.ckeditor.connector should be before from invenio.legacy.websubmit.icon_creator\n59:1: E402 module level import not at top of file\n60:1: E402 module level import not at top of file\n62:1: F401 'get_detailed_page_tabs' imported but unused\n62:1: E402 module level import not at top of file\n63:1: I101 Imported names are in the wrong order. Should be CFG_JSON_AVAILABLE, json\n63:1: E402 module level import not at top of file\n64:1: I100 Imports statements are in the wrong order. import invenio.legacy.template should be before from invenio.utils.json\n64:1: E402 module level import not at top of file\n65:1: I100 Imports statements are in the wrong order. from flask should be before import invenio.legacy.template\n65:1: I201 Missing newline before sections or imports.\n65:1: E402 module level import not at top of file\n70:1: I101 Imported names are in the wrong order. Should be action, endaction, home, interface, makeCataloguesTable\n70:1: E402 module level import not at top of file\n72:1: E302 expected 2 blank lines, found 1\n72: D101 Docstring missing\n74:77: E502 the backslash is redundant between brackets\n75:79: E502 the backslash is redundant between brackets\n78: D400 First line should end with '.', not 'r'\n78: D205 Blank line missing between one-line summary and description\n105:16: E127 continuation line over-indented for visual indent\n127:13: B901 blind except: statement\n134:9: B901 blind except: statement\n141:61: E225 missing whitespace around operator\n186:21: E266 too many leading '#' for block comment\n187:21: E266 too many leading '#' for block comment\n192:25: F841 local variable 'n' is assigned to but never used\n211:34: E201 whitespace after '{'\n211:57: E203 whitespace before ':'\n212:57: E203 whitespace before ':'\n212:69: E261 at least two spaces before inline comment\n213:57: E203 whitespace before ':'\n214:57: E203 whitespace before ':'\n215:57: E203 whitespace before ':'\n216:57: E203 whitespace before ':'\n216:67: E261 at least two spaces before inline comment\n217:57: E203 whitespace before ':'\n251:5: E303 too many blank lines (2)\n251: D202 No blank lines allowed *after* method docstring, found 1\n251: D205 Blank line missing between one-line summary and description\n258: D290 White spaces around docstring should be trimmed.\n258: D400 First line should end with '.', not 'm'\n281:16: E127 continuation line over-indented for visual indent\n303:13: B901 blind except: statement\n304:17: F841 local variable 'act' is assigned to but never used\n343:21: E266 too many leading '#' for block comment\n344:21: E266 too many leading '#' for block comment\n359:25: E266 too many leading '#' for block comment\n365:25: E266 too many leading '#' for block comment\n366:25: E266 too many leading '#' for block comment\n373:63: W503 line break before binary operator\n373:63: E127 continuation line over-indented for visual indent\n377:25: E266 too many leading '#' for block comment\n379:29: E266 too many leading '#' for block comment\n380:29: E266 too many leading '#' for block comment\n390:29: E266 too many leading '#' for block comment\n403:25: E266 too many leading '#' for block comment\n406:29: E266 too many leading '#' for block comment\n416:41: E128 continuation line under-indented for visual indent\n417:41: E128 continuation line under-indented for visual indent\n418:41: E128 continuation line under-indented for visual indent\n422:25: B901 blind except: statement\n423:29: E266 too many leading '#' for block comment\n428:25: E266 too many leading '#' for block comment\n429:25: E266 too many leading '#' for block comment\n441:29: E266 too many leading '#' for block comment\n446:25: E266 too many leading '#' for block comment\n452:21: E266 too many leading '#' for block comment\n455:21: E266 too many leading '#' for block comment\n456:26: E703 statement ends with a semicolon\n535: D205 Blank line missing between one-line summary and description\n547:16: E127 continuation line over-indented for visual indent\n548:16: E127 continuation line over-indented for visual indent\n556:9: E303 too many blank lines (2)\n569:9: B901 blind except: statement\n581:9: E266 too many leading '#' for block comment\n582:9: E266 too many leading '#' for block comment\n586:19: E126 continuation line over-indented for hanging indent\n590:35: E231 missing whitespace after ','\n593:30: E201 whitespace after '{'\n593:53: E203 whitespace before ':'\n594:53: E203 whitespace before ':'\n595:53: E203 whitespace before ':'\n596:53: E203 whitespace before ':'\n597:53: E203 whitespace before ':'\n598:53: E203 whitespace before ':'\n598:63: E261 at least two spaces before inline comment\n599:53: E203 whitespace before ':'\n612:64: F841 local variable 'e' is assigned to but never used\n625: D401 First line should be imperative: 'Thi', not 'This'\n625: D290 White spaces around docstring should be trimmed.\n625: D400 First line should end with '.', not 'g'\n625: D205 Blank line missing between one-line summary and description\n633:26: E261 at least two spaces before inline comment\n634:32: E261 at least two spaces before inline comment\n635:33: E116 unexpected indentation (comment)\n638:47: E261 at least two spaces before inline comment\n652: D401 First line should be imperative: 'Return', not 'Returns'\n652: D400 First line should end with '.', not 'e'\n652: D205 Blank line missing between one-line summary and description\n663:9: F841 local variable 'uid' is assigned to but never used\n664:9: F841 local variable 'user_info' is assigned to but never used\n669:74: E502 the backslash is redundant between brackets\n670:62: E502 the backslash is redundant between brackets\n670:39: E221 multiple spaces before operator\n681: D200 One-line docstring should not occupy 3 lines\n709:44: E203 whitespace before ':'\n724:9: F841 local variable 'myQuery' is assigned to but never used\n755:36: E126 continuation line over-indented for hanging indent\n765:22: E211 whitespace before '('\n781: D400 First line should end with '.', not ')'\n789:37: W602 deprecated form of raising exception\n807:32: E203 whitespace before ':'\n815: D102 Docstring missing\n828:9: F841 local variable 't' is assigned to but never used\n829:15: E221 multiple spaces before operator\n840:9: F841 local variable 'nbFields' is assigned to but never used\n846:24: E126 continuation line over-indented for hanging indent\n846:35: E203 whitespace before ':'\n847:31: E203 whitespace before ':'\n848:30: E203 whitespace before ':'\n849:30: E203 whitespace before ':'\n850:23: E126 continuation line over-indented for hanging indent\n852:59: E231 missing whitespace after ','\n855:47: E231 missing whitespace after ','\n856:52: E231 missing whitespace after ','\n863:18: E126 continuation line over-indented for hanging indent\n863:20: E251 unexpected spaces around keyword / parameter equals\n863:22: E251 unexpected spaces around keyword / parameter equals\n864:24: E251 unexpected spaces around keyword / parameter equals\n864:26: E251 unexpected spaces around keyword / parameter equals\n865:16: E126 continuation line over-indented for hanging indent\n867: D401 First line should be imperative: 'Return', not 'Returns'\n867: D200 One-line docstring should not occupy 3 lines\n899: D102 Docstring missing\n916:9: E266 too many leading '#' for block comment\n932:45: E127 continuation line over-indented for visual indent\n935:17: E266 too many leading '#' for block comment\n941:62: E203 whitespace before ':'\n941:109: E203 whitespace before ':'\n941:129: E203 whitespace before ','\n942:48: E126 continuation line over-indented for hanging indent\n945:41: E128 continuation line under-indented for visual indent\n950:62: E203 whitespace before ':'\n950:109: E203 whitespace before ':'\n950:129: E203 whitespace before ','\n951:48: E126 continuation line over-indented for hanging indent\n954:41: E128 continuation line under-indented for visual indent\n955:13: E266 too many leading '#' for block comment\n964:62: E203 whitespace before ':'\n964:109: E203 whitespace before ':'\n964:129: E203 whitespace before ','\n965:45: E126 continuation line over-indented for hanging indent\n974:27: E225 missing whitespace around operator\n985:1: E266 too many leading '#' for block comment\n986:1: E266 too many leading '#' for block comment\n987:1: E266 too many leading '#' for block comment\n988:1: E266 too many leading '#' for block comment\n989:1: E266 too many leading '#' for block comment\n991:1: E266 too many leading '#' for block comment\n992:1: E266 too many leading '#' for block comment\n993:1: E266 too many leading '#' for block comment\n994:1: E266 too many leading '#' for block comment\n995:1: E266 too many leading '#' for block comment\n996:1: E266 too many leading '#' for block comment\n997:1: E266 too many leading '#' for block comment\n998:1: E266 too many leading '#' for block comment\n999:1: E266 too many leading '#' for block comment\n1000:1: E266 too many leading '#' for block comment\n1001:1: E266 too many leading '#' for block comment\n1002:1: E266 too many leading '#' for block comment\n1003:1: E266 too many leading '#' for block comment\n1004:1: E266 too many leading '#' for block comment\n1005:1: E266 too many leading '#' for block comment\n1006:1: E266 too many leading '#' for block comment\n1007:1: E266 too many leading '#' for block comment\n1008:1: E266 too many leading '#' for block comment\n1010:1: E266 too many leading '#' for block comment"}}