{"comment":{"url":"https://api.github.com/repos/jirikuncar/invenio/comments/10622371","html_url":"https://github.com/jirikuncar/invenio/commit/2e438a287250891c2bfec155b9d50820ee11c8e3#commitcomment-10622371","id":10622371,"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/websession/templates.py","commit_id":"2e438a287250891c2bfec155b9d50820ee11c8e3","created_at":"2015-04-08T12:11:50Z","updated_at":"2015-04-08T12:11:50Z","body":"1: D100 Missing docstring in public module\n20:1: E402 module level import not at top of file\n21:1: I100 Imports statements are in the wrong order. import cgi should be before import urllib\n21:1: E402 module level import not at top of file\n23:1: E402 module level import not at top of file\n24:1: I101 Imported names are in the wrong order. Should be CFG_ACCESS_CONTROL_LEVEL_ACCOUNTS, CFG_CERN_SITE, CFG_SITE_LANG, CFG_SITE_NAME, CFG_SITE_NAME_INTL, CFG_SITE_RECORD, CFG_SITE_SECURE_URL, CFG_SITE_SUPPORT_EMAIL, CFG_SITE_URL, CFG_WEBSEARCH_MAX_RECORDS_IN_GROUPS, CFG_WEBSESSION_ADDRESS_ACTIVATION_EXPIRE_IN_DAYS, CFG_WEBSESSION_RESET_PASSWORD_EXPIRE_IN_DAYS\n24:1: F401 'CFG_ACCESS_CONTROL_LEVEL_ACCOUNTS' imported but unused\n24:1: E402 module level import not at top of file\n25:6: E126 continuation line over-indented for hanging indent\n40:1: I101 Imported names are in the wrong order. Should be create_html_link, create_url, make_canonical_urlargd\n40:1: E402 module level import not at top of file\n41:1: I100 Imports statements are in the wrong order. from invenio.utils.html should be before from invenio.utils.url\n41:1: F401 'nmtoken_from_string' imported but unused\n41:1: E402 module level import not at top of file\n42:1: I100 Imports statements are in the wrong order. from invenio.base.i18n should be before from invenio.utils.html\n42:1: E402 module level import not at top of file\n43:1: E402 module level import not at top of file\n44:1: I100 Imports statements are in the wrong order. from invenio.legacy.websession.websession_config should be before from invenio.modules.apikeys.models\n44:1: E402 module level import not at top of file\n47: D101 Missing docstring in public class\n69:17: E225 missing whitespace around operator\n70:16: E126 continuation line over-indented for hanging indent\n70:25: E203 whitespace before ':'\n71:25: E203 whitespace before ':'\n72:25: E203 whitespace before ':'\n73:25: E203 whitespace before ':'\n74:14: E126 continuation line over-indented for hanging indent\n78: D102 Missing docstring in public method\n79:9: F841 local variable '_' is assigned to but never used\n87: D102 Missing docstring in public method\n97:37: E203 whitespace before ':'\n98:28: E203 whitespace before ':'\n99:38: E203 whitespace before ':'\n100:29: E203 whitespace before ':'\n101:30: E203 whitespace before ':'\n103:35: E203 whitespace before ':'\n108:5: E303 too many blank lines (2)\n108: D401 First line should be in imperative mood ('Display', not 'Displays')\n108: D202 No blank lines allowed after function docstring (found 1)\n108: D400 First line should end with a period (not 'r')\n134:22: E126 continuation line over-indented for hanging indent\n134:36: E203 whitespace before ':'\n163:31: E126 continuation line over-indented for hanging indent\n168:43: E203 whitespace before ':'\n196:32: E203 whitespace before ':'\n197:43: E203 whitespace before ':'\n198:27: E203 whitespace before ':'\n199:22: E203 whitespace before ':'\n201:39: E203 whitespace before ':'\n204:16: E126 continuation line over-indented for hanging indent\n208: D401 First line should be in imperative mood ('Display', not 'Displays')\n208: D202 No blank lines allowed after function docstring (found 1)\n263:17: E126 continuation line over-indented for hanging indent\n263:30: E203 whitespace before ':'\n264:30: E203 whitespace before ':'\n265:33: E203 whitespace before ':'\n266:27: E203 whitespace before ':'\n268:34: E203 whitespace before ':'\n268:45: E225 missing whitespace around operator\n269:34: E203 whitespace before ':'\n269:45: E225 missing whitespace around operator\n270:28: E203 whitespace before ':'\n271:28: E203 whitespace before ':'\n272:26: E203 whitespace before ':'\n273:23: E203 whitespace before ':'\n274:29: E203 whitespace before ':'\n275:24: E203 whitespace before ':'\n276:33: E203 whitespace before ':'\n278:38: E203 whitespace before ':'\n279:13: E121 continuation line under-indented for hanging indent\n318:21: E126 continuation line over-indented for hanging indent\n318:34: E203 whitespace before ':'\n319:32: E203 whitespace before ':'\n320:35: E203 whitespace before ':'\n321:35: E203 whitespace before ':'\n323:31: E203 whitespace before ':'\n324:27: E203 whitespace before ':'\n325:36: E203 whitespace before ':'\n326:40: E203 whitespace before ':'\n327:38: E203 whitespace before ':'\n328:33: E203 whitespace before ':'\n329:40: E203 whitespace before ':'\n331:17: E121 continuation line under-indented for hanging indent\n334:21: E128 continuation line under-indented for visual indent\n334:34: E203 whitespace before ':'\n336:139: E203 whitespace before ':'\n337:35: E203 whitespace before ':'\n340:30: E203 whitespace before ':'\n340:96: E203 whitespace before ':'\n344:5: E303 too many blank lines (2)\n344: D400 First line should end with a period (not 'd')\n362:11: E121 continuation line under-indented for hanging indent\n362:26: E203 whitespace before ':'\n363:32: E203 whitespace before ':'\n364:32: E203 whitespace before ':'\n365:37: E203 whitespace before ':'\n366:21: E203 whitespace before ':'\n367:21: E203 whitespace before ':'\n368:28: E203 whitespace before ':'\n374:5: E303 too many blank lines (2)\n374: D102 Missing docstring in public method\n382:11: E121 continuation line under-indented for hanging indent\n382:26: E203 whitespace before ':'\n383:31: E203 whitespace before ':'\n387:27: E203 whitespace before ':'\n388:27: E203 whitespace before ':'\n389:26: E203 whitespace before ':'\n394:26: E203 whitespace before ':'\n395:30: E203 whitespace before ':'\n401:5: E303 too many blank lines (2)\n401: D102 Missing docstring in public method\n409:11: E121 continuation line under-indented for hanging indent\n409:26: E203 whitespace before ':'\n410:26: E203 whitespace before ':'\n413:23: E203 whitespace before ':'\n414:19: E203 whitespace before ':'\n417:23: E203 whitespace before ':'\n418:19: E203 whitespace before ':'\n423:30: E203 whitespace before ':'\n429:5: E303 too many blank lines (2)\n429:51: E251 unexpected spaces around keyword / parameter equals\n429:53: E251 unexpected spaces around keyword / parameter equals\n429:72: E251 unexpected spaces around keyword / parameter equals\n429:74: E251 unexpected spaces around keyword / parameter equals\n429:93: E251 unexpected spaces around keyword / parameter equals\n429:95: E251 unexpected spaces around keyword / parameter equals\n429: D102 Missing docstring in public method\n441:11: E121 continuation line under-indented for hanging indent\n441:26: E203 whitespace before ':'\n442:36: E203 whitespace before ':'\n443:27: E203 whitespace before ':'\n444:30: E203 whitespace before ':'\n445:25: E203 whitespace before ':'\n446:28: E203 whitespace before ':'\n452:25: E126 continuation line over-indented for hanging indent\n452:35: E203 whitespace before ':'\n453:28: E203 whitespace before ':'\n454:21: E121 continuation line under-indented for hanging indent\n460:17: E126 continuation line over-indented for hanging indent\n460:34: E203 whitespace before ':'\n461:39: E203 whitespace before ':'\n463:13: E121 continuation line under-indented for hanging indent\n466: D401 First line should be in imperative mood ('Display', not 'Displays')\n466: D202 No blank lines allowed after function docstring (found 1)\n479:256: E203 whitespace before ':'\n479:280: E203 whitespace before ':'\n479:302: E203 whitespace before ':'\n501:20: E203 whitespace before ':'\n502:19: E203 whitespace before ':'\n503:11: E121 continuation line under-indented for hanging indent\n507:45: E203 whitespace before ':'\n508:46: E203 whitespace before ':'\n509:45: E203 whitespace before ':'\n511:46: E203 whitespace before ':'\n515:15: E128 continuation line under-indented for visual indent\n515:73: E201 whitespace after '{'\n515:81: E203 whitespace before ':'\n515:106: E202 whitespace before '}'\n518:9: E303 too many blank lines (2)\n520: D401 First line should be in imperative mood ('Display', not 'Displays')\n520: D202 No blank lines allowed after function docstring (found 1)\n520: D400 First line should end with a period (not 'n')\n542:17: E126 continuation line over-indented for hanging indent\n542:32: E203 whitespace before ':'\n543:25: E128 continuation line under-indented for visual indent\n544:15: E126 continuation line over-indented for hanging indent\n552:22: E126 continuation line over-indented for hanging indent\n552:26: E203 whitespace before ':'\n553:37: E203 whitespace before ':'\n554:38: E203 whitespace before ':'\n555:20: E126 continuation line over-indented for hanging indent\n560:11: E121 continuation line under-indented for hanging indent\n560:15: E203 whitespace before ':'\n561:26: E203 whitespace before ':'\n562:27: E203 whitespace before ':'\n568:9: E122 continuation line missing indentation or outdented\n568:13: E203 whitespace before ':'\n569:9: E122 continuation line missing indentation or outdented\n569:23: E203 whitespace before ':'\n570:9: E122 continuation line missing indentation or outdented\n570:25: E203 whitespace before ':'\n576:13: E122 continuation line missing indentation or outdented\n576:17: E203 whitespace before ':'\n577:13: E122 continuation line missing indentation or outdented\n577:28: E203 whitespace before ':'\n578:13: E122 continuation line missing indentation or outdented\n578:31: E203 whitespace before ':'\n583:11: E121 continuation line under-indented for hanging indent\n583:15: E203 whitespace before ':'\n584:24: E203 whitespace before ':'\n585:27: E203 whitespace before ':'\n592:15: E121 continuation line under-indented for hanging indent\n592:27: E203 whitespace before ':'\n593:30: E203 whitespace before ':'\n594:13: E122 continuation line missing indentation or outdented\n595:13: E122 continuation line missing indentation or outdented\n604: D401 First line should be in imperative mood ('Display', not 'Displays')\n604: D202 No blank lines allowed after function docstring (found 1)\n604: D400 First line should end with a period (not 'e')\n617:17: E225 missing whitespace around operator\n619:19: E225 missing whitespace around operator\n629: D401 First line should be in imperative mood ('Display', not 'Displays')\n629: D202 No blank lines allowed after function docstring (found 1)\n629: D400 First line should end with a period (not 'r')\n652: D401 First line should be in imperative mood ('Display', not 'Displays')\n652: D202 No blank lines allowed after function docstring (found 1)\n652: D400 First line should end with a period (not 'e')\n667:14: E225 missing whitespace around operator\n678: D401 First line should be in imperative mood ('Display', not 'Displays')\n678: D202 No blank lines allowed after function docstring (found 1)\n678: D400 First line should end with a period (not 'e')\n740:32: E128 continuation line under-indented for visual indent\n746:36: E128 continuation line under-indented for visual indent\n750:35: E222 multiple spaces after operator\n751:36: E222 multiple spaces after operator\n752:35: E128 continuation line under-indented for visual indent\n757:33: E222 multiple spaces after operator\n758:34: E222 multiple spaces after operator\n765: D401 First line should be in imperative mood ('Display', not 'Displays')\n765: D202 No blank lines allowed after function docstring (found 1)\n765: D400 First line should end with a period (not 'd')\n779:14: E225 missing whitespace around operator\n780:15: E225 missing whitespace around operator\n787:17: E203 whitespace before ':'\n788:18: E203 whitespace before ':'\n789:24: E203 whitespace before ':'\n790:11: E121 continuation line under-indented for hanging indent\n793: D202 No blank lines allowed after function docstring (found 1)\n793: D400 First line should end with a period (not 't')\n793: D205 1 blank line required between summary line and description (found 0)\n812:17: E128 continuation line under-indented for visual indent\n813:17: E128 continuation line under-indented for visual indent\n814:33: E203 whitespace before ':'\n814:34: E231 missing whitespace after ':'\n815:30: E203 whitespace before ':'\n816:35: E203 whitespace before ':'\n819:21: E203 whitespace before ':'\n820:19: E203 whitespace before ':'\n821:17: E131 continuation line unaligned for hanging indent\n822:25: E203 whitespace before ':'\n823:24: E203 whitespace before ':'\n825:20: E203 whitespace before ':'\n826:21: E203 whitespace before ':'\n830: D202 No blank lines allowed after function docstring (found 1)\n830: D400 First line should end with a period (not 'e')\n830: D205 1 blank line required between summary line and description (found 0)\n849:17: E128 continuation line under-indented for visual indent\n850:17: E128 continuation line under-indented for visual indent\n851:33: E203 whitespace before ':'\n851:34: E231 missing whitespace after ':'\n852:30: E203 whitespace before ':'\n853:35: E203 whitespace before ':'\n856:21: E203 whitespace before ':'\n857:19: E203 whitespace before ':'\n858:17: E131 continuation line unaligned for hanging indent\n859:25: E203 whitespace before ':'\n860:33: E203 whitespace before ':'\n862:20: E203 whitespace before ':'\n863:21: E203 whitespace before ':'\n867: D401 First line should be in imperative mood ('Display', not 'Displays')\n867: D202 No blank lines allowed after function docstring (found 1)\n867: D400 First line should end with a period (not 't')\n881:14: E225 missing whitespace around operator\n885: D401 First line should be in imperative mood ('Display', not 'Displays')\n885: D202 No blank lines allowed after function docstring (found 1)\n885: D400 First line should end with a period (not 't')\n900: D401 First line should be in imperative mood ('Display', not 'Displays')\n900: D202 No blank lines allowed after function docstring (found 1)\n900: D400 First line should end with a period (not 't')\n900: D205 1 blank line required between summary line and description (found 0)\n905:9: F841 local variable '_' is assigned to but never used\n909:22: E126 continuation line over-indented for hanging indent\n909:26: E203 whitespace before ':'\n914: D401 First line should be in imperative mood ('Display', not 'Displays')\n914: D202 No blank lines allowed after function docstring (found 1)\n914: D400 First line should end with a period (not 'r')\n1006: D401 First line should be in imperative mood ('Display', not 'Displays')\n1006: D202 No blank lines allowed after function docstring (found 1)\n1006: D400 First line should end with a period (not 'k')\n1052:22: E126 continuation line over-indented for hanging indent\n1053:26: E203 whitespace before ':'\n1054:33: E203 whitespace before ':'\n1055:31: E203 whitespace before ':'\n1056:29: E203 whitespace before ':'\n1057:20: E126 continuation line over-indented for hanging indent\n1061:21: E126 continuation line over-indented for hanging indent\n1061:36: E203 whitespace before ':'\n1062:25: E203 whitespace before ':'\n1063:31: E203 whitespace before ':'\n1064:16: E121 continuation line under-indented for hanging indent\n1066:21: E126 continuation line over-indented for hanging indent\n1066:36: E203 whitespace before ':'\n1067:25: E203 whitespace before ':'\n1068:29: E203 whitespace before ':'\n1069:17: E121 continuation line under-indented for hanging indent\n1072: D400 First line should end with a period (not 's')\n1072: D205 1 blank line required between summary line and description (found 0)\n1093: D400 First line should end with a period (not 'r')\n1093: D205 1 blank line required between summary line and description (found 0)\n1100:9: F841 local variable '_' is assigned to but never used\n1109: D401 First line should be in imperative mood ('Display', not 'Displays')\n1109: D202 No blank lines allowed after function docstring (found 1)\n1109: D400 First line should end with a period (not 'l')\n1113:52: E251 unexpected spaces around keyword / parameter equals\n1113:54: E251 unexpected spaces around keyword / parameter equals\n1134:9: F841 local variable '_' is assigned to but never used\n1149:14: E225 missing whitespace around operator\n1159:14: E225 missing whitespace around operator\n1163:5: E303 too many blank lines (2)\n1163: D202 No blank lines allowed after function docstring (found 1)\n1182:9: E303 too many blank lines (2)\n1183:102: E202 whitespace before ')'\n1203:35: E225 missing whitespace around operator\n1210:15: E225 missing whitespace around operator\n1213:46: E203 whitespace before ':'\n1214:44: E203 whitespace before ':'\n1216:42: E231 missing whitespace after ':'\n1217:43: E231 missing whitespace after ':'\n1218:45: E231 missing whitespace after ':'\n1220:49: E203 whitespace before ':'\n1221:47: E203 whitespace before ':'\n1223:45: E231 missing whitespace after ':'\n1224:46: E231 missing whitespace after ':'\n1225:48: E231 missing whitespace after ':'\n1284:15: E225 missing whitespace around operator\n1308:16: E127 continuation line over-indented for visual indent\n1309:29: E231 missing whitespace after ':'\n1310:16: E124 closing bracket does not match visual indentation\n1314:5: E303 too many blank lines (2)\n1347:15: E225 missing whitespace around operator\n1361: D400 First line should end with a period (not ':')\n1361: D205 1 blank line required between summary line and description (found 0)\n1384:20: E225 missing whitespace around operator\n1399:46: E231 missing whitespace after ','\n1468:25: E203 whitespace before ':'\n1471:28: E203 whitespace before ':'\n1479:30: E231 missing whitespace after ':'\n1480:31: E231 missing whitespace after ':'\n1481:28: E231 missing whitespace after ':'\n1483:30: E203 whitespace before ':'\n1483:31: E231 missing whitespace after ':'\n1486:16: E124 closing bracket does not match visual indentation\n1489: D201 No blank lines allowed before function docstring (found 1)\n1489: D205 1 blank line required between summary line and description (found 0)\n1580:25: E203 whitespace before ':'\n1584:25: E231 missing whitespace after ':'\n1585:29: E231 missing whitespace after ':'\n1588:31: E231 missing whitespace after ':'\n1589:29: E203 whitespace before ':'\n1589:30: E231 missing whitespace after ':'\n1589:64: E231 missing whitespace after ','\n1590:33: E203 whitespace before ':'\n1591:16: E124 closing bracket does not match visual indentation\n1594: D202 No blank lines allowed after function docstring (found 1)\n1729:19: E203 whitespace before ':'\n1730:26: E222 multiple spaces after operator\n1735:37: E231 missing whitespace after ','\n1735:23: E222 multiple spaces after operator\n1736:13: E203 whitespace before ':'\n1738:27: E203 whitespace before ':'\n1739:27: E222 multiple spaces after operator\n1747:38: E231 missing whitespace after ','\n1747:23: E222 multiple spaces after operator\n1748:13: E203 whitespace before ':'\n1757:21: E203 whitespace before ':'\n1758:30: E203 whitespace before ':'\n1759:27: E203 whitespace before ':'\n1767:9: E122 continuation line missing indentation or outdented\n1767:73: E201 whitespace after '{'\n1767:81: E203 whitespace before ':'\n1768:65: E128 continuation line under-indented for visual indent\n1768:78: E203 whitespace before ':'\n1769:65: E124 closing bracket does not match visual indentation\n1770:26: E128 continuation line under-indented for visual indent\n1774:13: E128 continuation line under-indented for visual indent\n1776:24: E231 missing whitespace after ':'\n1777:30: E203 whitespace before ':'\n1778:31: E203 whitespace before ':'\n1778:32: E231 missing whitespace after ':'\n1779:25: E231 missing whitespace after ':'\n1780:24: E231 missing whitespace after ':'\n1789:31: E231 missing whitespace after ':'\n1790:21: E231 missing whitespace after ':'\n1858:21: E222 multiple spaces after operator\n1861:13: E203 whitespace before ':'\n1867:25: E203 whitespace before ':'\n1868:29: E203 whitespace before ':'\n1869:25: E231 missing whitespace after ':'\n1871:24: E203 whitespace before ':'\n1872:31: E231 missing whitespace after ':'\n1873:21: E203 whitespace before ':'\n1873:22: E231 missing whitespace after ':'\n1874:25: E203 whitespace before ':'\n1879:5: E303 too many blank lines (2)\n1879: D400 First line should end with a period (not 'p')\n1879: D205 1 blank line required between summary line and description (found 0)\n1910:11: E225 missing whitespace around operator\n1911:19: E231 missing whitespace after ':'\n1914:22: E231 missing whitespace after ':'\n1916:15: E124 closing bracket does not match visual indentation\n1919: D400 First line should end with a period (not 'e')\n1919: D205 1 blank line required between summary line and description (found 0)\n1950:11: E225 missing whitespace around operator\n1951:19: E231 missing whitespace after ':'\n1954:22: E231 missing whitespace after ':'\n1956:15: E124 closing bracket does not match visual indentation\n1959: D400 First line should end with a period (not 'y')\n1959: D205 1 blank line required between summary line and description (found 0)\n1973: D210 No whitespaces allowed surrounding docstring text\n1973: None: None\n1973: D400 First line should end with a period (not 'u')\n1973: D205 1 blank line required between summary line and description (found 0)\n1978:20: E203 whitespace before ':'\n1980:62: E225 missing whitespace around operator\n1981:13: E203 whitespace before ':'\n1988:54: E225 missing whitespace around operator\n1993:5: E303 too many blank lines (2)\n1993: D400 First line should end with a period (not 's')\n1993: D205 1 blank line required between summary line and description (found 0)\n1999:9: F841 local variable '_' is assigned to but never used\n2011: D400 First line should end with a period (not ':')\n2011: D205 1 blank line required between summary line and description (found 0)\n2026:15: E271 multiple spaces after keyword\n2028: D400 First line should end with a period (not ':')\n2028: D205 1 blank line required between summary line and description (found 0)\n2045: D400 First line should end with a period (not 'p')\n2045: D205 1 blank line required between summary line and description (found 0)\n2064: D400 First line should end with a period (not 'd')\n2064: D205 1 blank line required between summary line and description (found 0)\n2089: D400 First line should end with a period (not 'd')\n2089: D205 1 blank line required between summary line and description (found 0)\n2107: D400 First line should end with a period (not ')')\n2107: D205 1 blank line required between summary line and description (found 0)\n2125: D400 First line should end with a period (not 'e')\n2125: D205 1 blank line required between summary line and description (found 0)\n2173:19: E128 continuation line under-indented for visual indent"}}