)]}'
{"searchlight_ui/static/searchlight-ui/settings/search-settings.controller.js":[{"author":{"_account_id":7665,"name":"Travis Tripp","email":"os.travis.tripp@gmail.com","username":"travis.tripp"},"change_message_id":"8f29a2ddc8b3a4c5c8d0a0fb06373be463313d67","unresolved":false,"context_lines":[{"line_number":32,"context_line":"  // \u0027searchSettings\u0027 is a variable that MUST be present on some parent scope."},{"line_number":33,"context_line":"  // Look into injecting the search settings service itself, or better still,"},{"line_number":34,"context_line":"  // converting search settings into a directive that can accept attributes it"},{"line_number":35,"context_line":"  // can data bind."},{"line_number":36,"context_line":"  SettingsController.$inject \u003d ["},{"line_number":37,"context_line":"    \u0027$modalInstance\u0027,"},{"line_number":38,"context_line":"    \u0027searchSettings\u0027"}],"source_content_type":"text/javascript","patch_set":3,"id":"dab17558_2759455b","line":35,"updated":"2016-05-13 23:07:22.000000000","message":"I think this was meant to be \u0027searchlight-ui.settings.settingsService\u0027\n\nI\u0027m okay with making that change as an isolated change as this is about module / directory name changes to start decoupling the specific horizon panel this is deployed in from the actual logic.","commit_id":"aa6491204b157abb7e86a9553bd052e800553e2e"},{"author":{"_account_id":14307,"name":"Tyr Johanson","email":"tyr@hpe.com","username":"tyr"},"change_message_id":"66f88ad66d16be7805e31259ce4b9118ca747877","unresolved":false,"context_lines":[{"line_number":32,"context_line":"  // \u0027searchSettings\u0027 is a variable that MUST be present on some parent scope."},{"line_number":33,"context_line":"  // Look into injecting the search settings service itself, or better still,"},{"line_number":34,"context_line":"  // converting search settings into a directive that can accept attributes it"},{"line_number":35,"context_line":"  // can data bind."},{"line_number":36,"context_line":"  SettingsController.$inject \u003d ["},{"line_number":37,"context_line":"    \u0027$modalInstance\u0027,"},{"line_number":38,"context_line":"    \u0027searchSettings\u0027"}],"source_content_type":"text/javascript","patch_set":3,"id":"dab17558_ba686cf6","line":35,"in_reply_to":"dab17558_2759455b","updated":"2016-05-16 16:42:39.000000000","message":"I thought so too! But, in fact, this breaks unless you inject \"searchSettings\" which Angular satisfies by pulling in the parent scope variable \"ctrl.searchSettings\". Very unexpected...and hence the comment above :)","commit_id":"aa6491204b157abb7e86a9553bd052e800553e2e"}]}
