{"comment":{"url":"https://api.github.com/repos/jirikuncar/invenio/comments/9915643","html_url":"https://github.com/jirikuncar/invenio/commit/d7f3a4aa21ff6c6ba0488b8e684d6d91acbec9de#commitcomment-9915643","id":9915643,"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/webbasket/api.py","commit_id":"d7f3a4aa21ff6c6ba0488b8e684d6d91acbec9de","created_at":"2015-02-25T14:36:06Z","updated_at":"2015-02-25T14:36:06Z","body":"18: L101 copyright is missing\n22:1: E402 module level import not at top of file\n30:1: E402 module level import not at top of file\n32:1: I100 Imports statements are in the wrong order. import cgi should be before from intbitset\n32:1: E402 module level import not at top of file\n33:1: E402 module level import not at top of file\n34:1: I101 Imported names are in the wrong order. Should be HTTPConnection, urlsplit\n34:1: E402 module level import not at top of file\n36:1: E402 module level import not at top of file\n37:1: I100 Imports statements are in the wrong order. import re should be before from zlib\n37:1: E402 module level import not at top of file\n39:1: E402 module level import not at top of file\n40:6: E126 continuation line over-indented for hanging indent\n41:1: I100 Imports statements are in the wrong order. from invenio.base.i18n should be before from invenio.config\n41:1: E402 module level import not at top of file\n42:1: I100 Imports statements are in the wrong order. from invenio.base.globals should be before from invenio.base.i18n\n42:1: E402 module level import not at top of file\n43:1: I101 Imported names are in the wrong order. Should be convert_datestruct_to_dategui, convert_datetext_to_dategui, convert_datetext_to_datestruct\n43:1: E402 module level import not at top of file\n44:31: E126 continuation line over-indented for hanging indent\n46:1: I100 Imports statements are in the wrong order. from invenio.modules.formatter should be before from invenio.utils.date\n46:1: E402 module level import not at top of file\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, isGuestUser\n48:1: I100 Imports statements are in the wrong order. from invenio.legacy.webuser should be before from invenio.utils.url\n48:1: E402 module level import not at top of file\n49:1: I101 Imported names are in the wrong order. Should be check_user_can_view_record, get_merged_recid, record_exists\n49:1: I100 Imports statements are in the wrong order. from invenio.legacy.search_engine should be before from invenio.legacy.webuser\n49:1: E402 module level import not at top of file\n50:6: E126 continuation line over-indented for hanging indent\n54:1: E402 module level import not at top of file\n60:1: E402 module level import not at top of file\n61:1: I100 Imports statements are in the wrong order. from invenio.legacy.websearch_external_collections should be before from invenio.legacy.websearch_external_collections.utils\n61:1: E402 module level import not at top of file\n62:1: E402 module level import not at top of file\n63:1: E402 module level import not at top of file\n64:1: I100 Imports statements are in the wrong order. from invenio.ext.logging should be before from invenio.legacy.websearch_external_collections.getter\n64:1: E402 module level import not at top of file\n65:1: E402 module level import not at top of file\n66:1: E402 module level import not at top of file\n68:1: E402 module level import not at top of file\n70:1: E266 too many leading '#' for block comment\n73:1: E302 expected 2 blank lines, found 1\n73: D209 Put multi-line docstring closing quotes on separate line\n73: D202 No blank lines allowed *after* function docstring, found 1\n73: D205 Blank line missing between one-line summary and description\n106:13: E116 unexpected indentation (comment)\n124:75: E128 continuation line under-indented for visual indent\n125:75: E128 continuation line under-indented for visual indent\n126:75: E128 continuation line under-indented for visual indent\n127:75: E128 continuation line under-indented for visual indent\n128:75: E128 continuation line under-indented for visual indent\n129:75: E128 continuation line under-indented for visual indent\n130:75: E128 continuation line under-indented for visual indent\n149:55: E231 missing whitespace after ','\n154:59: E128 continuation line under-indented for visual indent\n155:59: E128 continuation line under-indented for visual indent\n156:59: E128 continuation line under-indented for visual indent\n157:59: E128 continuation line under-indented for visual indent\n158:59: E128 continuation line under-indented for visual indent\n159:59: E128 continuation line under-indented for visual indent\n160:59: E128 continuation line under-indented for visual indent\n161:59: E128 continuation line under-indented for visual indent\n182:1: E302 expected 2 blank lines, found 1\n182: D209 Put multi-line docstring closing quotes on separate line\n182: D202 No blank lines allowed *after* function docstring, found 1\n182: D205 Blank line missing between one-line summary and description\n260:1: E302 expected 2 blank lines, found 1\n260: D209 Put multi-line docstring closing quotes on separate line\n260: D202 No blank lines allowed *after* function docstring, found 1\n260: D205 Blank line missing between one-line summary and description\n335:1: E302 expected 2 blank lines, found 1\n335: D209 Put multi-line docstring closing quotes on separate line\n335: D201 No blank lines allowed *before* function docstring, found 1\n335: D202 No blank lines allowed *after* function docstring, found 1\n335: D205 Blank line missing between one-line summary and description\n391:1: E302 expected 2 blank lines, found 1\n391: D400 First line should end with '.', not 'm'\n391: D205 Blank line missing between one-line summary and description\n447:1: E302 expected 2 blank lines, found 1\n447: D290 White spaces around docstring should be trimmed.\n447: D205 Blank line missing between one-line summary and description\n480:44: E261 at least two spaces before inline comment\n517:1: E266 too many leading '#' for block comment\n519:1: E302 expected 2 blank lines, found 1\n519: D209 Put multi-line docstring closing quotes on separate line\n519: D202 No blank lines allowed *after* function docstring, found 1\n519: D205 Blank line missing between one-line summary and description\n552:5: E303 too many blank lines (2)\n637:96: E502 the backslash is redundant between brackets\n820:1: E302 expected 2 blank lines, found 1\n820: D209 Put multi-line docstring closing quotes on separate line\n820: D202 No blank lines allowed *after* function docstring, found 1\n820: D205 Blank line missing between one-line summary and description\n867:49: E261 at least two spaces before inline comment\n879:30: E225 missing whitespace around operator\n894:9: E266 too many leading '#' for block comment\n895:9: E266 too many leading '#' for block comment\n896:9: E266 too many leading '#' for block comment\n938:20: E222 multiple spaces after operator\n941:1: E302 expected 2 blank lines, found 1\n941: D209 Put multi-line docstring closing quotes on separate line\n941: D202 No blank lines allowed *after* function docstring, found 1\n941: D205 Blank line missing between one-line summary and description\n1029:48: E128 continuation line under-indented for visual indent\n1030:48: E128 continuation line under-indented for visual indent\n1031:48: E128 continuation line under-indented for visual indent\n1035:48: E126 continuation line over-indented for hanging indent\n1045:1: E302 expected 2 blank lines, found 1\n1045: D209 Put multi-line docstring closing quotes on separate line\n1045: D202 No blank lines allowed *after* function docstring, found 1\n1045: D205 Blank line missing between one-line summary and description\n1061:5: F841 local variable '_' is assigned to but never used\n1161:9: E266 too many leading '#' for block comment\n1170:22: E221 multiple spaces before operator\n1172:22: E221 multiple spaces before operator\n1173:27: E221 multiple spaces before operator\n1181:22: E221 multiple spaces before operator\n1183:22: E221 multiple spaces before operator\n1184:22: E221 multiple spaces before operator\n1185:22: E221 multiple spaces before operator\n1186:27: E221 multiple spaces before operator\n1203:26: E221 multiple spaces before operator\n1205:26: E221 multiple spaces before operator\n1206:31: E221 multiple spaces before operator\n1212:62: E201 whitespace after '('\n1212:143: E202 whitespace before ')'\n1235:9: E266 too many leading '#' for block comment\n1244:22: E221 multiple spaces before operator\n1246:22: E221 multiple spaces before operator\n1247:27: E221 multiple spaces before operator\n1249:27: E221 multiple spaces before operator\n1254:43: E221 multiple spaces before operator\n1260:22: E221 multiple spaces before operator\n1262:22: E221 multiple spaces before operator\n1263:27: E221 multiple spaces before operator\n1265:22: E221 multiple spaces before operator\n1266:22: E221 multiple spaces before operator\n1267:27: E221 multiple spaces before operator\n1279:47: E221 multiple spaces before operator\n1287:26: E221 multiple spaces before operator\n1289:26: E221 multiple spaces before operator\n1290:31: E221 multiple spaces before operator\n1292:31: E221 multiple spaces before operator\n1298:59: E201 whitespace after '('\n1298:134: E202 whitespace before ')'\n1301:47: E221 multiple spaces before operator\n1324:9: E266 too many leading '#' for block comment\n1333:22: E221 multiple spaces before operator\n1336:27: E221 multiple spaces before operator\n1341:43: E221 multiple spaces before operator\n1347:22: E221 multiple spaces before operator\n1350:22: E221 multiple spaces before operator\n1351:22: E221 multiple spaces before operator\n1352:27: E221 multiple spaces before operator\n1364:47: E221 multiple spaces before operator\n1372:26: E221 multiple spaces before operator\n1375:31: E221 multiple spaces before operator\n1381:60: E201 whitespace after '('\n1381:137: E202 whitespace before ')'\n1384:47: E221 multiple spaces before operator\n1407:9: E266 too many leading '#' for block comment\n1416:22: E221 multiple spaces before operator\n1419:27: E221 multiple spaces before operator\n1424:43: E221 multiple spaces before operator\n1430:22: E221 multiple spaces before operator\n1433:22: E221 multiple spaces before operator\n1434:22: E221 multiple spaces before operator\n1435:27: E221 multiple spaces before operator\n1447:47: E221 multiple spaces before operator\n1455:26: E221 multiple spaces before operator\n1458:31: E221 multiple spaces before operator\n1464:64: E201 whitespace after '('\n1464:149: E202 whitespace before ')'\n1467:47: E221 multiple spaces before operator\n1520:1: E302 expected 2 blank lines, found 1\n1520: D400 First line should end with '.', not 'm'\n1520: D205 Blank line missing between one-line summary and description\n1581:1: E302 expected 2 blank lines, found 1\n1581: D290 White spaces around docstring should be trimmed.\n1581: D205 Blank line missing between one-line summary and description\n1619:62: E502 the backslash is redundant between brackets\n1620:16: E129 visually indented line with same indent as next logical line\n1659:1: E302 expected 2 blank lines, found 1\n1659: D202 No blank lines allowed *after* function docstring, found 1\n1708:1: E302 expected 2 blank lines, found 1\n1708: D209 Put multi-line docstring closing quotes on separate line\n1708: D202 No blank lines allowed *after* function docstring, found 1\n1708: D400 First line should end with '.', not 's'\n1708: D205 Blank line missing between one-line summary and description\n1916:17: E201 whitespace after '('\n1916:53: E202 whitespace before ')'\n1916:60: E201 whitespace after '('\n1916:78: E201 whitespace after '('\n1916:120: E202 whitespace before ')'\n1916:122: E202 whitespace before ')'\n1935:28: E126 continuation line over-indented for hanging indent\n1935:59: E502 the backslash is redundant between brackets\n2021:1: E302 expected 2 blank lines, found 1\n2021: D209 Put multi-line docstring closing quotes on separate line\n2021: D202 No blank lines allowed *after* function docstring, found 1\n2021: D205 Blank line missing between one-line summary and description\n2041:43: F841 local variable 'exc' is assigned to but never used\n2058:1: E302 expected 2 blank lines, found 1\n2058: D205 Blank line missing between one-line summary and description\n2069:1: E302 expected 2 blank lines, found 1\n2069: D205 Blank line missing between one-line summary and description\n2081:1: E302 expected 2 blank lines, found 1\n2081: D400 First line should end with '.', not 's'\n2081: D205 Blank line missing between one-line summary and description\n2082:35: E251 unexpected spaces around keyword / parameter equals\n2082:37: E251 unexpected spaces around keyword / parameter equals\n2110:43: F841 local variable 'exc' is assigned to but never used\n2166:1: E302 expected 2 blank lines, found 1\n2166: D205 Blank line missing between one-line summary and description\n2197:1: E302 expected 2 blank lines, found 1\n2197: D400 First line should end with '.', not 'h'\n2197: D205 Blank line missing between one-line summary and description\n2216:1: E302 expected 2 blank lines, found 1\n2216: D400 First line should end with '.', not ','\n2216: D205 Blank line missing between one-line summary and description\n2225:39: E251 unexpected spaces around keyword / parameter equals\n2225:41: E251 unexpected spaces around keyword / parameter equals\n2227:42: E251 unexpected spaces around keyword / parameter equals\n2227:44: E251 unexpected spaces around keyword / parameter equals\n2254:41: E261 at least two spaces before inline comment\n2292:1: E302 expected 2 blank lines, found 1\n2292: D401 First line should be imperative: 'Subscribe', not 'Subscribes'\n2292: D209 Put multi-line docstring closing quotes on separate line\n2292: D205 Blank line missing between one-line summary and description\n2322:1: E302 expected 2 blank lines, found 1\n2322: D401 First line should be imperative: 'Unsubscribe', not 'Unsubscribes'\n2322: D209 Put multi-line docstring closing quotes on separate line\n2322: D205 Blank line missing between one-line summary and description\n2352:1: E302 expected 2 blank lines, found 1\n2352: D290 White spaces around docstring should be trimmed.\n2352: D400 First line should end with '.', not 't'\n2361:1: E302 expected 2 blank lines, found 1\n2361: D290 White spaces around docstring should be trimmed.\n2361: D400 First line should end with '.', not 's'\n2370:1: E302 expected 2 blank lines, found 1\n2374:15: E126 continuation line over-indented for hanging indent\n2379:1: E302 expected 2 blank lines, found 1\n2379: D290 White spaces around docstring should be trimmed.\n2379: D202 No blank lines allowed *after* function docstring, found 1\n2389:1: E302 expected 2 blank lines, found 1\n2389: D400 First line should end with '.', not 'm'\n2393:1: E302 expected 2 blank lines, found 1\n2393: D205 Blank line missing between one-line summary and description\n2405:41: E126 continuation line over-indented for hanging indent\n2410:1: E302 expected 2 blank lines, found 1\n2410: D209 Put multi-line docstring closing quotes on separate line\n2410: D205 Blank line missing between one-line summary and description\n2434:68: E502 the backslash is redundant between brackets\n2435:37: E127 continuation line over-indented for visual indent\n2444:77: E502 the backslash is redundant between brackets\n2445:46: E127 continuation line over-indented for visual indent\n2462:68: E502 the backslash is redundant between brackets\n2463:37: E127 continuation line over-indented for visual indent\n2472:77: E502 the backslash is redundant between brackets\n2473:46: E127 continuation line over-indented for visual indent\n2488:75: E502 the backslash is redundant between brackets\n2489:42: E127 continuation line over-indented for visual indent\n2495:1: E302 expected 2 blank lines, found 1\n2495: D209 Put multi-line docstring closing quotes on separate line\n2495: D202 No blank lines allowed *after* function docstring, found 1\n2495: D205 Blank line missing between one-line summary and description\n2589:1: E302 expected 2 blank lines, found 1\n2589: D400 First line should end with '.', not 'e'\n2615:1: E302 expected 2 blank lines, found 1\n2626:10: E261 at least two spaces before inline comment\n2630:1: E302 expected 2 blank lines, found 1\n2630: D400 First line should end with '.', not 'r'\n2636:1: E302 expected 2 blank lines, found 1\n2641:1: E266 too many leading '#' for block comment\n2644:1: E302 expected 2 blank lines, found 1\n2644: D209 Put multi-line docstring closing quotes on separate line\n2644: D202 No blank lines allowed *after* function docstring, found 1\n2644: D400 First line should end with '.', not 's'\n2644: D205 Blank line missing between one-line summary and description\n2693:1: E302 expected 2 blank lines, found 1\n2693: D209 Put multi-line docstring closing quotes on separate line\n2693: D202 No blank lines allowed *after* function docstring, found 1\n2693: D400 First line should end with '.', not 'm'\n2693: D205 Blank line missing between one-line summary and description\n2717:9: E116 unexpected indentation (comment)\n2747:1: E266 too many leading '#' for block comment\n2750:1: E302 expected 2 blank lines, found 1\n2750: D401 First line should be imperative: 'Return', not 'Returns'\n2750: D202 No blank lines allowed *after* function docstring, found 1\n2757:20: E222 multiple spaces after operator\n2769:5: B901 blind except: statement\n2782:1: E302 expected 2 blank lines, found 1\n2782: D301 Use r\"\"\" if any backslashes in a docstring\n2782: D209 Put multi-line docstring closing quotes on separate line\n2782: D202 No blank lines allowed *after* function docstring, found 1\n2782: D400 First line should end with '.', not '('\n2782: D205 Blank line missing between one-line summary and description\n2787:1: E302 expected 2 blank lines, found 1\n2811:1: E302 expected 2 blank lines, found 1\n2826:1: E302 expected 2 blank lines, found 1\n2842:1: E302 expected 2 blank lines, found 1\n2855:1: E302 expected 2 blank lines, found 1\n2868:1: E302 expected 2 blank lines, found 1\n2868: D209 Put multi-line docstring closing quotes on separate line\n2868: D400 First line should end with '.', not 'd'\n2868: D205 Blank line missing between one-line summary and description\n2897:1: E302 expected 2 blank lines, found 1\n2897: D400 First line should end with '.', not 't'\n2919:1: E302 expected 2 blank lines, found 1\n2919: D209 Put multi-line docstring closing quotes on separate line\n2919: D202 No blank lines allowed *after* function docstring, found 1\n2919: D205 Blank line missing between one-line summary and description"}}