)]}'
{"cinder/volume/drivers/dell_emc/sc/storagecenter_api.py":[{"author":{"_account_id":23630,"name":"Tovin Seven","email":"tovin07@gmail.com","username":"tovin07"},"change_message_id":"7a0c9a3f8fdded3ba87ef52999f78a33df5155c0","unresolved":false,"context_lines":[{"line_number":815,"context_line":"        basename \u003d os.path.basename(foldername)"},{"line_number":816,"context_line":"        pf.append(\u0027Name\u0027, basename)"},{"line_number":817,"context_line":"        # save the user from themselves."},{"line_number":818,"context_line":"        folderpath \u003d foldername.lstrip(\u0027/\u0027).rstrip(\u0027/\u0027)"},{"line_number":819,"context_line":"        folderpath \u003d os.path.dirname(folderpath)"},{"line_number":820,"context_line":"        # If we have any kind of path we throw it into the filters."},{"line_number":821,"context_line":"        if folderpath !\u003d \u0027\u0027:"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f515b1d_161c6cbf","line":818,"range":{"start_line":818,"start_character":32,"end_line":818,"end_character":55},"updated":"2017-09-27 01:51:55.000000000","message":"this can be .strip(\u0027/\u0027) for short","commit_id":"f520644ad98cf8e6f388ef55741eb8bb93b8c6fe"},{"author":{"_account_id":12112,"name":"Tom Swanson","email":"tom_swanson@dell.com","username":"tomswanson"},"change_message_id":"93c650ebf88c30dd5f767e3f7b024c2f13259e97","unresolved":false,"context_lines":[{"line_number":815,"context_line":"        basename \u003d os.path.basename(foldername)"},{"line_number":816,"context_line":"        pf.append(\u0027Name\u0027, basename)"},{"line_number":817,"context_line":"        # save the user from themselves."},{"line_number":818,"context_line":"        folderpath \u003d foldername.lstrip(\u0027/\u0027).rstrip(\u0027/\u0027)"},{"line_number":819,"context_line":"        folderpath \u003d os.path.dirname(folderpath)"},{"line_number":820,"context_line":"        # If we have any kind of path we throw it into the filters."},{"line_number":821,"context_line":"        if folderpath !\u003d \u0027\u0027:"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f515b1d_8d044280","line":818,"range":{"start_line":818,"start_character":32,"end_line":818,"end_character":55},"in_reply_to":"7f515b1d_161c6cbf","updated":"2017-10-03 19:17:11.000000000","message":"Oh, nice. Will be fixed in the next version.","commit_id":"f520644ad98cf8e6f388ef55741eb8bb93b8c6fe"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"0a3fac48062b0bcb7ec370b4175b40b40185983a","unresolved":false,"context_lines":[{"line_number":662,"context_line":"                elif splitver[1] \u003d\u003d \u00271\u0027:"},{"line_number":663,"context_line":"                    self.legacypayloadfilters \u003d True"},{"line_number":664,"context_line":"            self.legacyfoldernames \u003d (splitver[0] \u003c \u00274\u0027)"},{"line_number":665,"context_line":"            return"},{"line_number":666,"context_line":""},{"line_number":667,"context_line":"        except Exception:"},{"line_number":668,"context_line":"            # Good return but not the login response we were expecting."}],"source_content_type":"text/x-python","patch_set":2,"id":"3f4b6375_7d04a24a","line":665,"updated":"2017-10-23 16:32:12.000000000","message":"nit: we don\u0027t need \u0027return\u0027 here","commit_id":"be8d4b10cbf28bb25b565ec8ac7ec5bd78735350"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"0a3fac48062b0bcb7ec370b4175b40b40185983a","unresolved":false,"context_lines":[{"line_number":819,"context_line":"        folderpath \u003d os.path.dirname(folderpath)"},{"line_number":820,"context_line":"        # If we have any kind of path we throw it into the filters."},{"line_number":821,"context_line":"        if folderpath !\u003d \u0027\u0027:"},{"line_number":822,"context_line":"            # Legacy didn\u0027t begin with a slash."},{"line_number":823,"context_line":"            if not self.legacyfoldernames:"},{"line_number":824,"context_line":"                folderpath \u003d \u0027/\u0027 + folderpath"},{"line_number":825,"context_line":"            # SC convention is to end with a \u0027/\u0027 so make sure we do."},{"line_number":826,"context_line":"            folderpath +\u003d \u0027/\u0027"},{"line_number":827,"context_line":"            pf.append(\u0027folderPath\u0027, folderpath)"}],"source_content_type":"text/x-python","patch_set":2,"id":"3f4b6375_7db26296","line":824,"range":{"start_line":822,"start_character":17,"end_line":824,"end_character":21},"updated":"2017-10-23 16:32:12.000000000","message":"Please, add unit tests to cover new logic","commit_id":"be8d4b10cbf28bb25b565ec8ac7ec5bd78735350"}]}