{"comment":{"url":"https://api.github.com/repos/jirikuncar/invenio/comments/11364006","html_url":"https://github.com/jirikuncar/invenio/commit/21c2c20e9318552f0a2865e08a7bbe347bc0e342#commitcomment-11364006","id":11364006,"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/webcomment/webinterface.py","commit_id":"21c2c20e9318552f0a2865e08a7bbe347bc0e342","created_at":"2015-05-26T13:32:27Z","updated_at":"2015-05-26T13:32:27Z","body":"1: D210 No whitespaces allowed surrounding docstring text\n1: D400 First line should end with a period (not 'e')\n27:1: E402 module level import not at top of file\n29:1: E402 module level import not at top of file\n31:1: I101 Imported names are in the wrong order. Should be check_comment_belongs_to_record, check_recID_is_in_range, check_user_can_attach_file_to_comments, check_user_can_send_comments, check_user_can_view_comment, check_user_can_view_comments, get_user_subscription_to_discussion, is_comment_deleted, perform_display_your_comments, perform_request_add_comment_or_remark, perform_request_display_comments_or_remarks, perform_request_report, perform_request_vote, query_get_comment, subscribe_user_to_discussion, toggle_comment_visibility, unsubscribe_user_from_discussion\n31:1: I100 Imports statements are in the wrong order. from invenio.modules.comments.api should be before from six\n31:1: E402 module level import not at top of file\n49:1: I101 Imported names are in the wrong order. Should be CFG_ACCESS_CONTROL_LEVEL_SITE, CFG_COMMENTSDIR, CFG_SITE_LANG, CFG_SITE_NAME, CFG_SITE_NAME_INTL, CFG_SITE_RECORD, CFG_SITE_SECURE_URL, CFG_SITE_URL, CFG_TMPSHAREDDIR, CFG_WEBCOMMENT_ALLOW_COMMENTS, CFG_WEBCOMMENT_ALLOW_REVIEWS, CFG_WEBCOMMENT_MAX_ATTACHED_FILES, CFG_WEBCOMMENT_MAX_ATTACHMENT_SIZE, CFG_WEBCOMMENT_USE_MATHJAX_IN_COMMENTS\n49:1: I100 Imports statements are in the wrong order. from invenio.config should be before from invenio.modules.comments.api\n49:1: E402 module level import not at top of file\n50:6: E126 continuation line over-indented for hanging indent\n64:1: I101 Imported names are in the wrong order. Should be collect_user_info, getUid, isGuestUser, page_not_authorized\n64:1: E402 module level import not at top of file\n65:1: I101 Imported names are in the wrong order. Should be page, pagefooteronly, pageheaderonly\n65:1: I100 Imports statements are in the wrong order. from invenio.legacy.webpage should be before from invenio.legacy.webuser\n65:1: E402 module level import not at top of file\n66:1: I100 Imports statements are in the wrong order. from invenio.legacy.search_engine should be before from invenio.legacy.webpage\n66:1: E402 module level import not at top of file\n68:1: I101 Imported names are in the wrong order. Should be make_canonical_urlargd, redirect_to_url\n68:1: E402 module level import not at top of file\n70:1: I100 Imports statements are in the wrong order. from invenio.utils.html should be before from invenio.utils.url\n70:1: E402 module level import not at top of file\n71:1: I100 Imports statements are in the wrong order. from invenio.ext.logging should be before from invenio.utils.html\n71:1: E402 module level import not at top of file\n72:1: I100 Imports statements are in the wrong order. from invenio.base.i18n should be before from invenio.ext.logging\n72:1: E402 module level import not at top of file\n73:1: I101 Imported names are in the wrong order. Should be WebInterfaceDirectory, wash_urlargd\n73:1: E402 module level import not at top of file\n74:1: E402 module level import not at top of file\n76:1: E402 module level import not at top of file\n77:1: I101 Imported names are in the wrong order. Should be mail_cookie_check_common, mail_cookie_create_authorize_action, mail_cookie_create_common\n77:1: E402 module level import not at top of file\n81:1: I100 Imports statements are in the wrong order. from invenio.modules.access.errors should be before from invenio.modules.access.mailcookie\n81:1: E402 module level import not at top of file\n84:1: E402 module level import not at top of file\n86:1: I100 Imports statements are in the wrong order. import invenio.legacy.template should be before from invenio.modules.comments.config\n86:1: E402 module level import not at top of file\n89:1: I100 Imports statements are in the wrong order. import os should be before import invenio.legacy.template\n89:1: E402 module level import not at top of file\n90:1: I201 Missing newline before sections or imports.\n90:1: E402 module level import not at top of file\n91:1: I101 Imported names are in the wrong order. Should be decompose_file, propose_next_docname, stream_file\n91:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibdocfile.api should be before from invenio.utils\n91:1: E402 module level import not at top of file\n95:1: E402 module level import not at top of file\n105: D102 Missing docstring in public method\n107:34: E261 at least two spaces before inline comment\n110: D401 First line should be in imperative mood ('Redirect', not 'Redirects')\n110: D400 First line should end with a period (not 'n')\n110: D200 One-line docstring should fit on one line with quotes (found 3)\n116: D205 1 blank line required between summary line and description (found 0)\n149:64: E261 at least two spaces before inline comment\n158:86: E203 whitespace before ':'\n160:90: E502 the backslash is redundant between brackets\n160:63: E203 whitespace before ':'\n160:87: E203 whitespace before ':'\n161:17: E128 continuation line under-indented for visual indent\n164:52: E502 the backslash is redundant between brackets\n165:17: E128 continuation line under-indented for visual indent\n165:21: E251 unexpected spaces around keyword / parameter equals\n165:23: E251 unexpected spaces around keyword / parameter equals\n198:51: E502 the backslash is redundant between brackets\n199:85: E502 the backslash is redundant between brackets\n201:53: E502 the backslash is redundant between brackets\n228:17: E128 continuation line under-indented for visual indent\n249:69: E225 missing whitespace around operator\n252:78: E225 missing whitespace around operator\n262:13: E303 too many blank lines (2)\n263:25: E128 continuation line under-indented for visual indent\n264:25: E128 continuation line under-indented for visual indent\n265:25: E128 continuation line under-indented for visual indent\n266:25: E128 continuation line under-indented for visual indent\n266:38: E251 unexpected spaces around keyword / parameter equals\n266:40: E251 unexpected spaces around keyword / parameter equals\n267:25: E128 continuation line under-indented for visual indent\n268:25: E128 continuation line under-indented for visual indent\n269:25: E128 continuation line under-indented for visual indent\n270:25: E128 continuation line under-indented for visual indent\n271:21: E127 continuation line over-indented for visual indent\n272:21: E127 continuation line over-indented for visual indent\n273:21: E127 continuation line over-indented for visual indent\n274:21: E127 continuation line over-indented for visual indent\n287: D400 First line should end with a period (not '0')\n287: D205 1 blank line required between summary line and description (found 0)\n331:86: E203 whitespace before ':'\n335:53: E127 continuation line over-indented for visual indent\n340:53: E127 continuation line over-indented for visual indent\n342:90: E502 the backslash is redundant between brackets\n342:63: E203 whitespace before ':'\n342:87: E203 whitespace before ':'\n343:17: E128 continuation line under-indented for visual indent\n346:52: E502 the backslash is redundant between brackets\n347:17: E128 continuation line under-indented for visual indent\n347:21: E251 unexpected spaces around keyword / parameter equals\n347:23: E251 unexpected spaces around keyword / parameter equals\n353:56: E502 the backslash is redundant between brackets\n354:48: E251 unexpected spaces around keyword / parameter equals\n354:50: E251 unexpected spaces around keyword / parameter equals\n357:13: E303 too many blank lines (2)\n362:56: E502 the backslash is redundant between brackets\n363:48: E251 unexpected spaces around keyword / parameter equals\n363:50: E251 unexpected spaces around keyword / parameter equals\n380:26: E261 at least two spaces before inline comment\n386:55: E261 at least two spaces before inline comment\n396:109: E502 the backslash is redundant between brackets\n402:25: B901 blind except: statement\n405:21: E266 too many leading '#' for block comment\n406:21: E266 too many leading '#' for block comment\n407:21: E266 too many leading '#' for block comment\n429:32: E127 continuation line over-indented for visual indent\n437:41: E128 continuation line under-indented for visual indent\n450:21: B901 blind except: statement\n464:69: E225 missing whitespace around operator\n468:92: E127 continuation line over-indented for visual indent\n469:92: E127 continuation line over-indented for visual indent\n470:92: E127 continuation line over-indented for visual indent\n470:107: E225 missing whitespace around operator\n471:92: E127 continuation line over-indented for visual indent\n472:92: E127 continuation line over-indented for visual indent\n472:107: E225 missing whitespace around operator\n482:69: E127 continuation line over-indented for visual indent\n486:99: E502 the backslash is redundant between brackets\n486:74: E225 missing whitespace around operator\n486:93: E225 missing whitespace around operator\n487:86: E502 the backslash is redundant between brackets\n487:78: E225 missing whitespace around operator\n543: D202 No blank lines allowed after function docstring (found 1)\n585:86: E203 whitespace before ':'\n587:90: E502 the backslash is redundant between brackets\n587:63: E203 whitespace before ':'\n587:87: E203 whitespace before ':'\n588:17: E128 continuation line under-indented for visual indent\n591:52: E502 the backslash is redundant between brackets\n592:17: E128 continuation line under-indented for visual indent\n592:21: E251 unexpected spaces around keyword / parameter equals\n592:23: E251 unexpected spaces around keyword / parameter equals\n596:52: E502 the backslash is redundant between brackets\n597:44: E251 unexpected spaces around keyword / parameter equals\n597:46: E251 unexpected spaces around keyword / parameter equals\n602:52: E502 the backslash is redundant between brackets\n603:44: E251 unexpected spaces around keyword / parameter equals\n603:46: E251 unexpected spaces around keyword / parameter equals\n607:52: E502 the backslash is redundant between brackets\n608:44: E251 unexpected spaces around keyword / parameter equals\n608:46: E251 unexpected spaces around keyword / parameter equals\n622: D202 No blank lines allowed after function docstring (found 1)\n622: D400 First line should end with a period (not 't')\n622: D205 1 blank line required between summary line and description (found 0)\n662:86: E203 whitespace before ':'\n664:90: E502 the backslash is redundant between brackets\n664:63: E203 whitespace before ':'\n664:87: E203 whitespace before ':'\n665:17: E128 continuation line under-indented for visual indent\n668:52: E502 the backslash is redundant between brackets\n669:17: E128 continuation line under-indented for visual indent\n669:21: E251 unexpected spaces around keyword / parameter equals\n669:23: E251 unexpected spaces around keyword / parameter equals\n673:52: E502 the backslash is redundant between brackets\n674:44: E251 unexpected spaces around keyword / parameter equals\n674:46: E251 unexpected spaces around keyword / parameter equals\n679:52: E502 the backslash is redundant between brackets\n680:44: E251 unexpected spaces around keyword / parameter equals\n680:46: E251 unexpected spaces around keyword / parameter equals\n684:52: E502 the backslash is redundant between brackets\n685:44: E251 unexpected spaces around keyword / parameter equals\n685:46: E251 unexpected spaces around keyword / parameter equals\n696:90: E225 missing whitespace around operator\n699: D400 First line should end with a period (not 'w')\n699: D205 1 blank line required between summary line and description (found 0)\n711:86: E203 whitespace before ':'\n713:90: E502 the backslash is redundant between brackets\n713:63: E203 whitespace before ':'\n713:87: E203 whitespace before ':'\n714:17: E128 continuation line under-indented for visual indent\n717:52: E502 the backslash is redundant between brackets\n718:17: E128 continuation line under-indented for visual indent\n718:21: E251 unexpected spaces around keyword / parameter equals\n718:23: E251 unexpected spaces around keyword / parameter equals\n725: D200 One-line docstring should fit on one line with quotes (found 3)\n735:86: E203 whitespace before ':'\n737:90: E502 the backslash is redundant between brackets\n737:63: E203 whitespace before ':'\n737:87: E203 whitespace before ':'\n738:17: E128 continuation line under-indented for visual indent\n746: D400 First line should end with a period (not 'r')\n746: D200 One-line docstring should fit on one line with quotes (found 3)\n762:63: E502 the backslash is redundant between brackets\n763:90: E502 the backslash is redundant between brackets\n766:1: E302 expected 2 blank lines, found 1\n766: D203 1 blank line required before class docstring (found 0)\n766: D208 Docstring is over-indented\n774: D102 Missing docstring in public method\n776:34: E261 at least two spaces before inline comment\n778: D210 No whitespaces allowed surrounding docstring text\n778: D401 First line should be in imperative mood ('Thi', not 'This')\n778: D400 First line should end with a period (not 'g')\n778: D205 1 blank line required between summary line and description (found 0)\n785:28: E261 at least two spaces before inline comment\n787:43: E261 at least two spaces before inline comment\n800: D401 First line should be in imperative mood ('Return', not 'Returns')\n814:9: F841 local variable 'uid' is assigned to but never used\n820:86: E203 whitespace before ':'\n822:90: E502 the backslash is redundant between brackets\n822:63: E203 whitespace before ':'\n822:87: E203 whitespace before ':'\n823:17: E128 continuation line under-indented for visual indent\n826:52: E502 the backslash is redundant between brackets\n827:44: E251 unexpected spaces around keyword / parameter equals\n827:46: E251 unexpected spaces around keyword / parameter equals\n840:86: E203 whitespace before ':'\n842:90: E502 the backslash is redundant between brackets\n842:63: E203 whitespace before ':'\n842:87: E203 whitespace before ':'\n843:17: E128 continuation line under-indented for visual indent\n846:52: E502 the backslash is redundant between brackets\n847:44: E251 unexpected spaces around keyword / parameter equals\n847:46: E251 unexpected spaces around keyword / parameter equals\n852:52: E502 the backslash is redundant between brackets\n853:44: E251 unexpected spaces around keyword / parameter equals\n853:46: E251 unexpected spaces around keyword / parameter equals\n878:1: E302 expected 2 blank lines, found 1\n878: D203 1 blank line required before class docstring (found 0)\n883: D202 No blank lines allowed after function docstring (found 1)\n899:65: E502 the backslash is redundant between brackets\n900:46: E127 continuation line over-indented for visual indent\n907:30: E203 whitespace before ':'\n910:25: E203 whitespace before ':'\n916:52: E502 the backslash is redundant between brackets\n917:44: E251 unexpected spaces around keyword / parameter equals\n917:46: E251 unexpected spaces around keyword / parameter equals\n926:30: E251 unexpected spaces around keyword / parameter equals\n927:34: E126 continuation line over-indented for hanging indent\n927:49: E203 whitespace before ':'\n929:43: E203 whitespace before ':'\n930:31: E126 continuation line over-indented for hanging indent"}}