)]}'
{"designate/sqlalchemy/base.py":[{"author":{"_account_id":741,"name":"Kiall Mac Innes","email":"kiall@macinnes.ie","username":"kiall"},"change_message_id":"7dedb9be5203e4d2530e3bf3ae44250949b29c88","unresolved":false,"context_lines":[{"line_number":133,"context_line":""},{"line_number":134,"context_line":"                elif (isinstance(value, six.string_types) and"},{"line_number":135,"context_line":"                        value.startswith(\u0027BETWEEN\u0027)):"},{"line_number":136,"context_line":"                    elements \u003d value.split(\" \")[1].split(\",\")"},{"line_number":137,"context_line":"                    query \u003d query.where(between("},{"line_number":138,"context_line":"                        column, elements[0], elements[1]))"},{"line_number":139,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"ba3cc151_b0018808","line":136,"updated":"2015-07-01 11:00:44.000000000","message":"If someone sends in this \"BETWEEN  1, 3\", this will fail:\n\n    \u003e\u003e\u003e value \u003d \"BETWEEN  1, 3\"\n    \u003e\u003e\u003e elements \u003d value.split(\" \")[1].split(\",\")\n    \u003e\u003e\u003e elements[0]\n    \u0027\u0027\n    \u003e\u003e\u003e elements[1]\n    Traceback (most recent call last):\n      File \"\u003cstdin\u003e\", line 1, in \u003cmodule\u003e\n    IndexError: list index out of range\n\n\nInstead, this will be more reliable:\n\n    \u003e\u003e\u003e value \u003d \"BETWEEN  1, 3\"\n    \u003e\u003e\u003e elements \u003d value.split(\" \", 1)[1].strip(\" \").split(\",\")\n    \u003e\u003e\u003e elements[0].strip(\" \")\n    \u00271\u0027\n    \u003e\u003e\u003e elements[1].strip(\" \")\n    \u00273\u0027","commit_id":"7d1c612ddb3e03771614c1f74c1cfb8ca1083a79"}]}