{"comment":{"url":"https://api.github.com/repos/jirikuncar/invenio/comments/9894011","html_url":"https://github.com/jirikuncar/invenio/commit/fa6e5ac2673232f80c56badcc9dee93616254e37#commitcomment-9894011","id":9894011,"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/webalert/alert_engine.py","commit_id":"fa6e5ac2673232f80c56badcc9dee93616254e37","created_at":"2015-02-24T14:26:28Z","updated_at":"2015-02-24T14:26:28Z","body":"1: D100 Docstring missing\n3:1: E266 too many leading '#' for block comment\n4:1: E266 too many leading '#' for block comment\n4: L102 copyright year is outdated, expected 2015 but got 2011\n6:1: E266 too many leading '#' for block comment\n7:1: E266 too many leading '#' for block comment\n8:1: E266 too many leading '#' for block comment\n9: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\n13:1: E266 too many leading '#' for block comment\n14:1: E266 too many leading '#' for block comment\n16:1: E266 too many leading '#' for block comment\n17:1: E266 too many leading '#' for block comment\n18:1: E266 too many leading '#' for block comment\n24:1: E266 too many leading '#' for block comment\n28:1: E402 module level import not at top of file\n29:1: E402 module level import not at top of file\n30:1: E402 module level import not at top of file\n31:1: I100 Imports statements are in the wrong order. import datetime should be before from time\n31:1: E402 module level import not at top of file\n33:1: I101 Imported names are in the wrong order. Should be CFG_LOGDIR, CFG_SITE_ADMIN_EMAIL, CFG_SITE_NAME, CFG_SITE_URL, CFG_WEBALERT_MAX_NUM_OF_RECORDS_IN_ALERT_EMAIL, CFG_WEBALERT_SEND_EMAIL_NUMBER_OF_TRIES, CFG_WEBALERT_SEND_EMAIL_SLEEPTIME_BETWEEN_TRIES\n33:1: E402 module level import not at top of file\n34:6: E126 continuation line over-indented for hanging indent\n41:1: I101 Imported names are in the wrong order. Should be add_to_basket, get_basket_owner_id\n41:1: E402 module level import not at top of file\n42:1: I100 Imports statements are in the wrong order. from invenio.legacy.webbasket.api should be before from invenio.legacy.webbasket.db_layer\n42:1: E402 module level import not at top of file\n43:1: I101 Imported names are in the wrong order. Should be check_user_can_view_record, perform_request_search\n43:1: I100 Imports statements are in the wrong order. from invenio.legacy.search_engine should be before from invenio.legacy.webbasket.api\n43:1: E402 module level import not at top of file\n44:6: E126 continuation line over-indented for hanging indent\n45:1: I100 Imports statements are in the wrong order. from invenio.ext.legacy.handler should be before from invenio.legacy.search_engine\n45:1: E402 module level import not at top of file\n46:1: E402 module level import not at top of file\n47:1: I101 Imported names are in the wrong order. Should be collect_user_info, get_email\n47:1: E402 module level import not at top of file\n48:1: I100 Imports statements are in the wrong order. from invenio.ext.email should be before from invenio.legacy.webuser\n48:1: E402 module level import not at top of file\n49:1: E402 module level import not at top of file\n50:1: E402 module level import not at top of file\n52:1: I101 Imported names are in the wrong order. Should be CFG_EXTERNAL_COLLECTION_MAXRESULTS_ALERTS, CFG_EXTERNAL_COLLECTION_TIMEOUT\n52:1: E402 module level import not at top of file\n53:1: E122 continuation line missing indentation or outdented\n54:1: E122 continuation line missing indentation or outdented\n55:1: E402 module level import not at top of file\n56:1: E402 module level import not at top of file\n57:1: E402 module level import not at top of file\n59:1: I100 Imports statements are in the wrong order. import invenio.legacy.template should be before from invenio.modules.search.models\n59:1: E402 module level import not at top of file\n63:1: E302 expected 2 blank lines, found 1\n63: D202 No blank lines allowed *after* function docstring, found 1\n68:1: E302 expected 2 blank lines, found 1\n68: D202 No blank lines allowed *after* function docstring, found 1\n73:1: E302 expected 2 blank lines, found 1\n73: D401 First line should be imperative: 'Return', not 'Returns'\n73: D202 No blank lines allowed *after* function docstring, found 1\n78:1: E302 expected 2 blank lines, found 1\n78: D401 First line should be imperative: 'Return', not 'Returns'\n78: D202 No blank lines allowed *after* function docstring, found 1\n78: D400 First line should end with '.', not 'm'\n84:1: E302 expected 2 blank lines, found 1\n84: D202 No blank lines allowed *after* function docstring, found 1\n84: D400 First line should end with '.', not 's'\n94:29: E261 at least two spaces before inline comment\n122:99: E502 the backslash is redundant between brackets\n157:1: E302 expected 2 blank lines, found 1\n157: D401 First line should be imperative: 'Return', not 'Returns'\n157: D202 No blank lines allowed *after* function docstring, found 1\n163:1: E302 expected 2 blank lines, found 1\n172:37: E261 at least two spaces before inline comment\n173:29: E261 at least two spaces before inline comment\n203:99: E502 the backslash is redundant between brackets\n262:1: E302 expected 2 blank lines, found 1\n262: D202 No blank lines allowed *after* function docstring, found 1\n267:1: E302 expected 2 blank lines, found 1\n267: D401 First line should be imperative: 'Return', not 'Returns'\n267: D202 No blank lines allowed *after* function docstring, found 1\n271:6: E221 multiple spaces before operator\n272:6: E221 multiple spaces before operator\n273:7: E221 multiple spaces before operator\n274:8: E221 multiple spaces before operator\n275:6: E221 multiple spaces before operator\n276:7: E221 multiple spaces before operator\n277:7: E221 multiple spaces before operator\n278:7: E221 multiple spaces before operator\n279:7: E221 multiple spaces before operator\n280:7: E221 multiple spaces before operator\n281:7: E221 multiple spaces before operator\n282:8: E221 multiple spaces before operator\n283:7: E221 multiple spaces before operator\n284:7: E221 multiple spaces before operator\n285:7: E221 multiple spaces before operator\n286:8: E221 multiple spaces before operator\n287:7: E221 multiple spaces before operator\n288:7: E221 multiple spaces before operator\n289:7: E221 multiple spaces before operator\n290:7: E221 multiple spaces before operator\n298:25: E261 at least two spaces before inline comment\n298:26: E262 inline comment should start with '# '\n300:10: E261 at least two spaces before inline comment\n305:18: E261 at least two spaces before inline comment\n305:19: E262 inline comment should start with '# '\n318:35: E128 continuation line under-indented for visual indent\n319:35: E128 continuation line under-indented for visual indent\n320:35: E128 continuation line under-indented for visual indent\n324:1: E302 expected 2 blank lines, found 1\n324: D209 Put multi-line docstring closing quotes on separate line\n324: D202 No blank lines allowed *after* function docstring, found 1\n367:1: E302 expected 2 blank lines, found 1\n367: D209 Put multi-line docstring closing quotes on separate line\n367: D202 No blank lines allowed *after* function docstring, found 1\n380:1: E302 expected 2 blank lines, found 1\n380: D209 Put multi-line docstring closing quotes on separate line\n380: D202 No blank lines allowed *after* function docstring, found 1\n393:1: E302 expected 2 blank lines, found 1\n393: D209 Put multi-line docstring closing quotes on separate line\n393: D202 No blank lines allowed *after* function docstring, found 1\n405:1: E302 expected 2 blank lines, found 1\n405: D209 Put multi-line docstring closing quotes on separate line\n405: D202 No blank lines allowed *after* function docstring, found 1\n422:1: E302 expected 2 blank lines, found 1\n422: D401 First line should be imperative: 'Log', not 'Logs'\n422: D202 No blank lines allowed *after* function docstring, found 1\n433:1: E302 expected 2 blank lines, found 1\n433: D209 Put multi-line docstring closing quotes on separate line\n433: D202 No blank lines allowed *after* function docstring, found 1\n433: D400 First line should end with '.', not 's'\n433: D205 Blank line missing between one-line summary and description\n451:82: E502 the backslash is redundant between brackets\n456:106: E502 the backslash is redundant between brackets\n461:1: E302 expected 2 blank lines, found 1\n461: D202 No blank lines allowed *after* function docstring, found 1\n461: D400 First line should end with '.', not '?'\n466:1: E302 expected 2 blank lines, found 1\n466: D202 No blank lines allowed *after* function docstring, found 1\n466: D400 First line should end with '.', not '?'\n471:1: E302 expected 2 blank lines, found 1\n471: D209 Put multi-line docstring closing quotes on separate line\n471: D202 No blank lines allowed *after* function docstring, found 1\n480:31: E261 at least two spaces before inline comment\n486:1: E302 expected 2 blank lines, found 1\n486: D209 Put multi-line docstring closing quotes on separate line\n486: D202 No blank lines allowed *after* function docstring, found 1\n486: D400 First line should end with '.', not 's'\n486: D205 Blank line missing between one-line summary and description\n494:1: E302 expected 2 blank lines, found 1\n494: D209 Put multi-line docstring closing quotes on separate line\n494: D202 No blank lines allowed *after* function docstring, found 1\n494: D205 Blank line missing between one-line summary and description\n509:1: E302 expected 2 blank lines, found 1\n509: D209 Put multi-line docstring closing quotes on separate line\n509: D202 No blank lines allowed *after* function docstring, found 1\n509: D400 First line should end with '.', not 's'\n509: D205 Blank line missing between one-line summary and description\n532: D209 Put multi-line docstring closing quotes on separate line\n532: D205 Blank line missing between one-line summary and description\n543:64: E261 at least two spaces before inline comment\n543:65: E262 inline comment should start with '# '\n555:1: E302 expected 2 blank lines, found 1\n555: D202 No blank lines allowed *after* function docstring, found 1\n555: D400 First line should end with '.', not 'l'\n576:44: E261 at least two spaces before inline comment\n576:45: E262 inline comment should start with '# '"}}