)]}'
{"tuskar_ui/infrastructure/resource_management/resource_classes/tables.py":[{"author":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"change_message_id":"ea25bc814908188be50a11cc2ffcb91a54397d0b","unresolved":false,"context_lines":[{"line_number":153,"context_line":"        return \"%s?step\u003d%s\" % ("},{"line_number":154,"context_line":"            urlresolvers.reverse("},{"line_number":155,"context_line":"                url,"},{"line_number":156,"context_line":"                args\u003d(self.table.kwargs.get(\u0027resource_class_id\u0027),)),"},{"line_number":157,"context_line":"            resource_classes.workflows.ResourceClassFlavorsAction.slug)"},{"line_number":158,"context_line":""},{"line_number":159,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATX%2F%2F%2B7I%3D","line":156,"updated":"2013-10-23 10:14:40.000000000","message":"args\u003d... may fit on the same line as url.","commit_id":"72fa4abf865f0eeeb6e25da9d22af4a772d69de7"}],"tuskar_ui/infrastructure/resource_management/resource_classes/workflows.py":[{"author":{"_account_id":7585,"name":"Ladislav Smola","email":"lsmola@redhat.com","username":"lsmola"},"change_message_id":"f5225d62281ad4c7c98b0c37a541f2239a18780b","unresolved":false,"context_lines":[{"line_number":114,"context_line":""},{"line_number":115,"context_line":"    action_class \u003d ResourceClassFlavorsAction"},{"line_number":116,"context_line":"    template_name \u003d (\u0027infrastructure/resource_management/resource_classes/\u0027"},{"line_number":117,"context_line":"                        \u0027_flavors_step.html\u0027)"},{"line_number":118,"context_line":"    contributes \u003d (\"flavors\",)"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"    def get_flavors_data(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATH%2F%2FJQM%3D","line":117,"updated":"2013-10-22 15:02:09.000000000","message":"visual indentation","commit_id":"fedda0ad6937fd1adfc8030df9734f984ffd19db"},{"author":{"_account_id":7585,"name":"Ladislav Smola","email":"lsmola@redhat.com","username":"lsmola"},"change_message_id":"f5225d62281ad4c7c98b0c37a541f2239a18780b","unresolved":false,"context_lines":[{"line_number":162,"context_line":"    action_class \u003d RacksAction"},{"line_number":163,"context_line":"    contributes \u003d (\"racks_object_ids\")"},{"line_number":164,"context_line":"    template_name \u003d (\u0027infrastructure/resource_management/\u0027"},{"line_number":165,"context_line":"                        \u0027resource_classes/_table_step.html\u0027)"},{"line_number":166,"context_line":""},{"line_number":167,"context_line":"    def contribute(self, data, context):"},{"line_number":168,"context_line":"        context.update(data)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATH%2F%2FJQY%3D","line":165,"updated":"2013-10-22 15:02:09.000000000","message":"visual indentation","commit_id":"fedda0ad6937fd1adfc8030df9734f984ffd19db"},{"author":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"change_message_id":"ea25bc814908188be50a11cc2ffcb91a54397d0b","unresolved":false,"context_lines":[{"line_number":116,"context_line":"        return cleaned_data"},{"line_number":117,"context_line":""},{"line_number":118,"context_line":"    class Meta:"},{"line_number":119,"context_line":"        name \u003d _(\"Flavors\")"},{"line_number":120,"context_line":"        help_text \u003d _(\"From here you can add flavors to the class.\")"},{"line_number":121,"context_line":""},{"line_number":122,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATX%2F%2F%2B6o%3D","line":119,"updated":"2013-10-23 10:14:40.000000000","message":"I\u0027m not entirely sure that parse time evaluation will translate properly. I realise this isn\u0027t a new idiom here, so if it\u0027s known to work, cool.","commit_id":"72fa4abf865f0eeeb6e25da9d22af4a772d69de7"},{"author":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"change_message_id":"ea25bc814908188be50a11cc2ffcb91a54397d0b","unresolved":false,"context_lines":[{"line_number":173,"context_line":"    action_class \u003d RacksAction"},{"line_number":174,"context_line":"    contributes \u003d (\"racks_object_ids\")"},{"line_number":175,"context_line":"    template_name \u003d (\u0027infrastructure/resource_management/resource_classes/\u0027"},{"line_number":176,"context_line":"        \u0027_table_step.html\u0027)"},{"line_number":177,"context_line":""},{"line_number":178,"context_line":"    def contribute(self, data, context):"},{"line_number":179,"context_line":"        context.update(data)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATX%2F%2F%2B5s%3D","line":176,"updated":"2013-10-23 10:14:40.000000000","message":"It occurs to me that if you made a constant string with the prefix here the template names could be short:\n\n    template_name \u003d MODULE_PREFIX + \u0027_table_step.html\u0027","commit_id":"72fa4abf865f0eeeb6e25da9d22af4a772d69de7"},{"author":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"change_message_id":"ea25bc814908188be50a11cc2ffcb91a54397d0b","unresolved":false,"context_lines":[{"line_number":353,"context_line":""},{"line_number":354,"context_line":"class UpdateRacksWorkflow(UpdateResourceClass):"},{"line_number":355,"context_line":"    def get_index_url(self):"},{"line_number":356,"context_line":"        \"\"\"This url is used both as success and failure url\"\"\""},{"line_number":357,"context_line":"        url \u003d \"horizon:infrastructure:resource_management:resource_classes:\"\\"},{"line_number":358,"context_line":"              \"detail\""},{"line_number":359,"context_line":"        return \"%s?tab\u003dresource_class_details__racks\" % ("}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATX%2F%2F%2B5k%3D","line":356,"updated":"2013-10-23 10:14:40.000000000","message":"This looks like a comment, not a docstring: it isn\u0027t documenting the contract, rather it\u0027s documenting an implementation quirk.","commit_id":"72fa4abf865f0eeeb6e25da9d22af4a772d69de7"}],"tuskar_ui/infrastructure/resource_management/templates/resource_management/resource_classes/_flavors_step.html":[{"author":{"_account_id":7585,"name":"Ladislav Smola","email":"lsmola@redhat.com","username":"lsmola"},"change_message_id":"f5225d62281ad4c7c98b0c37a541f2239a18780b","unresolved":false,"context_lines":[{"line_number":11,"context_line":"          $table_block \u003d $(\u0027#flavors-table-block\u0027),"},{"line_number":12,"context_line":"          $tab \u003d $("},{"line_number":13,"context_line":"            \u0027a[data-target\u003d\"#create_resource_class__\u0027 +"},{"line_number":14,"context_line":"            \u0027resourceclassflavorsaction\"]\u0027"},{"line_number":15,"context_line":"          ).parent(\u0027li\u0027);"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"      $select.change(function () {"}],"source_content_type":"text/html","patch_set":1,"id":"AAAATH%2F%2FJO0%3D","line":14,"updated":"2013-10-22 15:02:09.000000000","message":"this actually works only for creating resource class\n\nedit has different url:\n\u003ca href\u003d\"#update_resource_class__resourceclassflavorsaction\" data-toggle\u003d\"tab\" data-target\u003d\"#update_resource_class__resourceclassflavorsaction\"\u003eFlavors\u003c/a\u003e","commit_id":"fedda0ad6937fd1adfc8030df9734f984ffd19db"}]}