diff --git a/install/ui/src/freeipa/details.js b/install/ui/src/freeipa/details.js index 7aa4c0ef6..e428dc908 100644 --- a/install/ui/src/freeipa/details.js +++ b/install/ui/src/freeipa/details.js @@ -453,8 +453,8 @@ exp.facet_policies = IPA.facet_policies = function(spec) { * - sets name, title, label if not present * - adds default actions and related buttons * - refresh - * - reset - * - update + * - revert + * - save * - adds dirty state evaluator * * @member details @@ -472,21 +472,21 @@ exp.details_facet_pre_op = function(spec, context) { spec.actions = spec.actions || []; spec.actions.unshift( 'refresh', - 'reset', - 'update'); + 'revert', + 'save'); spec.control_buttons = spec.control_buttons || []; if (!spec.no_update) { spec.control_buttons.unshift( { - name: 'reset', - label: '@i18n:buttons.reset', + name: 'revert', + label: '@i18n:buttons.revert', icon: 'fa-undo' }, { - name: 'update', - label: '@i18n:buttons.update', + name: 'save', + label: '@i18n:buttons.save', icon: 'fa-upload' }); } @@ -1404,8 +1404,8 @@ exp.refresh_action = IPA.refresh_action = function(spec) { exp.reset_action = IPA.reset_action = function(spec) { spec = spec || {}; - spec.name = spec.name || 'reset'; - spec.label = spec.label || '@i18n:buttons.reset'; + spec.name = spec.name || 'revert'; + spec.label = spec.label || '@i18n:buttons.revert'; spec.enable_cond = spec.enable_cond || ['dirty']; var that = IPA.action(spec); @@ -1426,8 +1426,8 @@ exp.reset_action = IPA.reset_action = function(spec) { exp.update_action = IPA.update_action = function(spec) { spec = spec || {}; - spec.name = spec.name || 'update'; - spec.label = spec.label || '@i18n:buttons.update'; + spec.name = spec.name || 'save'; + spec.label = spec.label || '@i18n:buttons.save'; spec.needs_confirm = spec.needs_confirm !== undefined ? spec.needs_confirm : false; spec.enable_cond = spec.enable_cond || ['dirty']; @@ -2007,8 +2007,8 @@ exp.register = function() { var f = reg.facet; a.register('refresh', exp.refresh_action); - a.register('reset', exp.reset_action); - a.register('update', exp.update_action); + a.register('revert', exp.reset_action); + a.register('save', exp.update_action); a.register('object', exp.object_action); a.register('enable', exp.enable_action); a.register('disable', exp.disable_action); @@ -2026,4 +2026,4 @@ exp.register = function() { phases.on('registration', exp.register); return exp; -}); \ No newline at end of file +}); diff --git a/install/ui/src/freeipa/dns.js b/install/ui/src/freeipa/dns.js index 7b66c8674..8dce6a96d 100644 --- a/install/ui/src/freeipa/dns.js +++ b/install/ui/src/freeipa/dns.js @@ -1961,7 +1961,7 @@ IPA.dns.record_type_table_widget = function(spec) { dialog.create_button({ name: 'modify', - label: '@i18n:buttons.update', + label: '@i18n:buttons.save', click: function() { dialog.modify(); } diff --git a/install/ui/src/freeipa/ipa.js b/install/ui/src/freeipa/ipa.js index a78d1f063..836ad689b 100644 --- a/install/ui/src/freeipa/ipa.js +++ b/install/ui/src/freeipa/ipa.js @@ -754,8 +754,8 @@ IPA.dirty_dialog = function(spec) { }; that.create_button({ - name: 'update', - label: '@i18n:buttons.update', + name: 'save', + label: '@i18n:buttons.save', click: function() { that.facet.update(function() { that.close(); @@ -765,8 +765,8 @@ IPA.dirty_dialog = function(spec) { }); that.create_button({ - name: 'reset', - label: '@i18n:buttons.reset', + name: 'revert', + label: '@i18n:buttons.revert', click: function() { that.facet.reset(); that.close(); diff --git a/install/ui/test/data/ipa_init.json b/install/ui/test/data/ipa_init.json index 036543129..f969a874f 100644 --- a/install/ui/test/data/ipa_init.json +++ b/install/ui/test/data/ipa_init.json @@ -81,7 +81,9 @@ "reset_password_and_login": "Reset Password and Login", "restore": "Restore", "retry": "Retry", + "revert": "Revert", "revoke": "Revoke", + "save": "Save", "set": "Set", "unapply": "Un-apply", "update": "Update", diff --git a/install/ui/util/make-ui.sh b/install/ui/util/make-ui.sh index d842e955b..088fa8c3c 100755 --- a/install/ui/util/make-ui.sh +++ b/install/ui/util/make-ui.sh @@ -33,4 +33,4 @@ $DIR/build.sh webui # don't stop at error. Some dependency errors are expected. $DIR/compile.sh --release lib --layer freeipa/core --output $IPA_DIR/core.js $DIR/compile.sh --release lib --layer freeipa/app --output $IPA_DIR/app.js -$DIR/clean.sh \ No newline at end of file +$DIR/clean.sh diff --git a/ipalib/plugins/internal.py b/ipalib/plugins/internal.py index e04e84340..a76a3180d 100644 --- a/ipalib/plugins/internal.py +++ b/ipalib/plugins/internal.py @@ -223,7 +223,9 @@ class i18n_messages(Command): "reset_password_and_login": _("Reset Password and Login"), "restore": _("Restore"), "retry": _("Retry"), + "revert": _("Revert"), "revoke": _("Revoke"), + "save": _("Save"), "set": _("Set"), "unapply": ("Un-apply"), "update": _("Update"), diff --git a/ipatests/test_webui/test_realmdomains.py b/ipatests/test_webui/test_realmdomains.py index 8e8f13a5c..87af690c7 100644 --- a/ipatests/test_webui/test_realmdomains.py +++ b/ipatests/test_webui/test_realmdomains.py @@ -39,13 +39,13 @@ class test_realmdomains(UI_driver): # add self.add_multivalued('associateddomain', 'itest.bar') - self.facet_button_click('update') + self.facet_button_click('save') self.dialog_button_click('force') self.wait_for_request() # delete self.del_multivalued('associateddomain', 'itest.bar') - self.facet_button_click('update') + self.facet_button_click('save') self.dialog_button_click('force') self.wait_for_request() self.wait_for_request() diff --git a/ipatests/test_webui/ui_driver.py b/ipatests/test_webui/ui_driver.py index 451e61cc0..7335f115a 100644 --- a/ipatests/test_webui/ui_driver.py +++ b/ipatests/test_webui/ui_driver.py @@ -1218,7 +1218,7 @@ class UI_driver(object): new_count = len(self.get_rows()) self.assert_row_count(count, new_count) - def mod_record(self, entity, data, facet='details', facet_btn='update'): + def mod_record(self, entity, data, facet='details', facet_btn='save'): """ Mod record @@ -1243,7 +1243,7 @@ class UI_driver(object): add_facet_btn='add', add_dialog_btn='add', add_dialog_name='add', - update_btn='update', + update_btn='save', breadcrumb=None, navigate=True, delete=True): @@ -1461,7 +1461,7 @@ class UI_driver(object): # update self.assert_rule_tables_enabled(t_list, False) - self.facet_button_click('update') + self.facet_button_click('save') self.wait_for_request(n=3, d=0.3) self.assert_rule_tables_enabled(t_list, False) @@ -1480,7 +1480,7 @@ class UI_driver(object): for cat in categories: self.check_option(cat, '') self.assert_rule_tables_enabled(t_list, True) - self.facet_button_click('update') + self.facet_button_click('save') self.wait_for_request(n=3, d=0.3) self.assert_rule_tables_enabled(t_list, True)