)]}'
{"horizon/static/horizon/js/horizon.quota.js":[{"author":{"_account_id":6966,"name":"Jeffrey J. Walls","email":"jeff.walls@hpe.com","username":"ftcjeff"},"change_message_id":"a8535b53b628fb0e6702b78fbd4200af90256d97","unresolved":false,"context_lines":[{"line_number":291,"context_line":""},{"line_number":292,"context_line":"    //var update_indicator \u003d progress_element.find(\u0027.progress_bar_selected\u0027);"},{"line_number":293,"context_line":"    var quota_limit \u003d parseInt(progress_element.attr(\u0027data-quota-limit\u0027), 10);"},{"line_number":294,"context_line":"    //var quota_used \u003d parseInt(progress_element.attr(\u0027data-quota-used\u0027), 10);"},{"line_number":295,"context_line":"    var percentage_to_update \u003d ((increment_by / quota_limit) * 100);"},{"line_number":296,"context_line":"    //var percentage_used \u003d ((quota_used / quota_limit) * 100);"},{"line_number":297,"context_line":""}],"source_content_type":"text/javascript","patch_set":3,"id":"AAAAUH%2F9WOM%3D","line":294,"updated":"2014-02-07 15:30:39.000000000","message":"more commented-out code; I see there already was some, but since this patch is primarily to clean up the code, might as well remove this and make it even cleaner, IMO.","commit_id":"4c6f666de792bc0bc914c2c79a796e91bf7c76b9"}],"horizon/static/horizon/js/horizon.tables_inline_edit.js":[{"author":{"_account_id":5623,"name":"David Lyle","email":"dklyle0@gmail.com","username":"david-lyle"},"change_message_id":"393aae13bfe790772f33fdf43f3104eb2c33fc3b","unresolved":false,"context_lines":[{"line_number":63,"context_line":"            var success \u003d $(\u0027\u003cdiv class\u003d\"success\"\u003e\u003c/div\u003e\u0027);"},{"line_number":64,"context_line":"            self.td_element.find(\u0027.inline-edit-status\u0027).append(success);"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"            // var background_color \u003d self.td_element.css(\u0027background-color\u0027);"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"            // edit pencil will disappear and appear again once the check sign has faded"},{"line_number":69,"context_line":"            // also green background will disappear"}],"source_content_type":"text/javascript","patch_set":3,"id":"AAAAUH%2F9jUU%3D","line":66,"updated":"2014-02-06 04:18:39.000000000","message":"why are we just commenting this out?  let\u0027s just remove it","commit_id":"4c6f666de792bc0bc914c2c79a796e91bf7c76b9"},{"author":{"_account_id":6966,"name":"Jeffrey J. Walls","email":"jeff.walls@hpe.com","username":"ftcjeff"},"change_message_id":"a8535b53b628fb0e6702b78fbd4200af90256d97","unresolved":false,"context_lines":[{"line_number":63,"context_line":"            var success \u003d $(\u0027\u003cdiv class\u003d\"success\"\u003e\u003c/div\u003e\u0027);"},{"line_number":64,"context_line":"            self.td_element.find(\u0027.inline-edit-status\u0027).append(success);"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"            // var background_color \u003d self.td_element.css(\u0027background-color\u0027);"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"            // edit pencil will disappear and appear again once the check sign has faded"},{"line_number":69,"context_line":"            // also green background will disappear"}],"source_content_type":"text/javascript","patch_set":3,"id":"AAAAUH%2F9WPs%3D","line":66,"in_reply_to":"AAAAUH%2F9YNY%3D","updated":"2014-02-07 15:30:39.000000000","message":"git keeps track of it :-)  I agree with David; remove unused code instead of commenting it out.  I\u0027ve spent too much time staring at commented-out code wondering why it was commented out and wondering if it shouldn\u0027t be.","commit_id":"4c6f666de792bc0bc914c2c79a796e91bf7c76b9"},{"author":{"_account_id":7976,"name":"Maxime Vidori","email":"mvidori@redhat.com","username":"MaxV"},"change_message_id":"48793a13a8c59926fb627484b379818ac7dd02f4","unresolved":false,"context_lines":[{"line_number":63,"context_line":"            var success \u003d $(\u0027\u003cdiv class\u003d\"success\"\u003e\u003c/div\u003e\u0027);"},{"line_number":64,"context_line":"            self.td_element.find(\u0027.inline-edit-status\u0027).append(success);"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"            // var background_color \u003d self.td_element.css(\u0027background-color\u0027);"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"            // edit pencil will disappear and appear again once the check sign has faded"},{"line_number":69,"context_line":"            // also green background will disappear"}],"source_content_type":"text/javascript","patch_set":3,"id":"AAAAUH%2F9YNY%3D","line":66,"in_reply_to":"AAAAUH%2F9jUU%3D","updated":"2014-02-07 10:37:56.000000000","message":"This variable does not seem to be used, I did not found any side effect, but in case a bug is found, I thought that it can be interesting to keep a track of this line","commit_id":"4c6f666de792bc0bc914c2c79a796e91bf7c76b9"}]}