)]}'
{"openstack_dashboard/api/rest/swift.py":[{"author":{"_account_id":12071,"name":"Richard Jones","email":"r1chardj0n3s@gmail.com","username":"richard"},"change_message_id":"524f7a058ed269263a44f663e213770559083c34","unresolved":false,"context_lines":[{"line_number":120,"context_line":"        :param container:"},{"line_number":121,"context_line":"        :return:"},{"line_number":122,"context_line":"        \"\"\""},{"line_number":123,"context_line":""},{"line_number":124,"context_line":"        path \u003d request.GET.get(\u0027path\u0027)"},{"line_number":125,"context_line":"        if path is not None:"},{"line_number":126,"context_line":"            path \u003d urlunquote(path)"}],"source_content_type":"text/x-python","patch_set":6,"id":"1a95cdbc_425e58ea","line":123,"updated":"2016-10-05 04:35:42.000000000","message":"Why the added blank line?","commit_id":"254775d87efe251c4eeb5b883f6cc463a7caaa63"},{"author":{"_account_id":20509,"name":"Daniel Castellanos","email":"luis.daniel.castellanos@intel.com","username":"lcastell"},"change_message_id":"c1277800ec961cd839f7b194d49af075ed8e4b6c","unresolved":false,"context_lines":[{"line_number":120,"context_line":"        :param container:"},{"line_number":121,"context_line":"        :return:"},{"line_number":122,"context_line":"        \"\"\""},{"line_number":123,"context_line":""},{"line_number":124,"context_line":"        path \u003d request.GET.get(\u0027path\u0027)"},{"line_number":125,"context_line":"        if path is not None:"},{"line_number":126,"context_line":"            path \u003d urlunquote(path)"}],"source_content_type":"text/x-python","patch_set":6,"id":"1a95cdbc_a9e7e6e2","line":123,"in_reply_to":"1a95cdbc_425e58ea","updated":"2016-10-05 15:14:57.000000000","message":"Done","commit_id":"254775d87efe251c4eeb5b883f6cc463a7caaa63"}],"openstack_dashboard/dashboards/project/static/dashboard/project/containers/containers-model.service.js":[{"author":{"_account_id":17172,"name":"Kenji Ishii","email":"ken-ishii@sx.jp.nec.com","username":"kenji-ishii"},"change_message_id":"d28c93cda1dbe9adccef3fc2979bd0b723086fe9","unresolved":false,"context_lines":[{"line_number":279,"context_line":"          state.counted.folders++;"},{"line_number":280,"context_line":"          var spec \u003d {"},{"line_number":281,"context_line":"            delimiter: model.DELIMETER,"},{"line_number":282,"context_line":"            path: encodeURIComponent(item.path).replace(/%2F/g, \u0027/\u0027)"},{"line_number":283,"context_line":"          };"},{"line_number":284,"context_line":"          return swiftAPI.getObjects(model.container.name, spec)"},{"line_number":285,"context_line":"            .then(function objects(response) {"}],"source_content_type":"text/javascript","patch_set":5,"id":"3a98d1f4_7987bcb4","line":282,"updated":"2016-10-04 01:09:31.000000000","message":"It is better to use model.DELIMETER instead of \u0027/\u0027.\nIMHO, also %2F is valid only when we use \u0027/\u0027 as a delimiter, so it also should be a variouble.","commit_id":"b619a98644696a6a574021825f6e047376817ad6"},{"author":{"_account_id":20509,"name":"Daniel Castellanos","email":"luis.daniel.castellanos@intel.com","username":"lcastell"},"change_message_id":"9e01b4e3c5a355ea38a362a175963dc927329f8b","unresolved":false,"context_lines":[{"line_number":279,"context_line":"          state.counted.folders++;"},{"line_number":280,"context_line":"          var spec \u003d {"},{"line_number":281,"context_line":"            delimiter: model.DELIMETER,"},{"line_number":282,"context_line":"            path: encodeURIComponent(item.path).replace(/%2F/g, \u0027/\u0027)"},{"line_number":283,"context_line":"          };"},{"line_number":284,"context_line":"          return swiftAPI.getObjects(model.container.name, spec)"},{"line_number":285,"context_line":"            .then(function objects(response) {"}],"source_content_type":"text/javascript","patch_set":5,"id":"1a95cdbc_8f03450f","line":282,"in_reply_to":"3a98d1f4_7987bcb4","updated":"2016-10-04 15:07:56.000000000","message":"Done","commit_id":"b619a98644696a6a574021825f6e047376817ad6"},{"author":{"_account_id":12071,"name":"Richard Jones","email":"r1chardj0n3s@gmail.com","username":"richard"},"change_message_id":"524f7a058ed269263a44f663e213770559083c34","unresolved":false,"context_lines":[{"line_number":279,"context_line":"          state.counted.folders++;"},{"line_number":280,"context_line":"          var spec \u003d {"},{"line_number":281,"context_line":"            delimiter: model.DELIMETER,"},{"line_number":282,"context_line":"            path: encodeURIComponent(item.path).replace(/%2F/g, model.DELIMETER)"},{"line_number":283,"context_line":"          };"},{"line_number":284,"context_line":"          return swiftAPI.getObjects(model.container.name, spec)"},{"line_number":285,"context_line":"            .then(function objects(response) {"}],"source_content_type":"text/javascript","patch_set":6,"id":"1a95cdbc_22322cf2","line":282,"updated":"2016-10-05 04:35:42.000000000","message":"I\u0027m pretty sure this isn\u0027t correct - it\u0027ll replace encoded \"/\" in the path with DELIMETER, regardless of what DELIMETER is. This substitution should only be done if the DELIMITER is \"/\".","commit_id":"254775d87efe251c4eeb5b883f6cc463a7caaa63"},{"author":{"_account_id":20509,"name":"Daniel Castellanos","email":"luis.daniel.castellanos@intel.com","username":"lcastell"},"change_message_id":"c1277800ec961cd839f7b194d49af075ed8e4b6c","unresolved":false,"context_lines":[{"line_number":279,"context_line":"          state.counted.folders++;"},{"line_number":280,"context_line":"          var spec \u003d {"},{"line_number":281,"context_line":"            delimiter: model.DELIMETER,"},{"line_number":282,"context_line":"            path: encodeURIComponent(item.path).replace(/%2F/g, model.DELIMETER)"},{"line_number":283,"context_line":"          };"},{"line_number":284,"context_line":"          return swiftAPI.getObjects(model.container.name, spec)"},{"line_number":285,"context_line":"            .then(function objects(response) {"}],"source_content_type":"text/javascript","patch_set":6,"id":"1a95cdbc_e9264e15","line":282,"in_reply_to":"1a95cdbc_22322cf2","updated":"2016-10-05 15:14:57.000000000","message":"Done. I didn\u0027t see that earlier. Thanks Richard, I\u0027ve just undone that change to add \u0027/\u0027, let me know if this needs more work like wrap this around a condition or so","commit_id":"254775d87efe251c4eeb5b883f6cc463a7caaa63"}]}