On 10/26/2011 7:09 AM, Petr Vobornik wrote:
1) add_dialog was renamed to entity_adder_dialog but its method add_dialog_create wasn't renamed
Fixed. -- Endi S. Dewata
From 66ba2ed6414886457ad11d0fc628081ead3575c4 Mon Sep 17 00:00:00 2001 From: Endi S. Dewata <edew...@redhat.com> Date: Mon, 24 Oct 2011 19:20:14 -0500 Subject: [PATCH] Fixed "enroll" labels. Labels using the word "enroll" (except for host enrollment) have been modified to use more relevant words. The IPA.add_dialog has been renamed into IPA.entity_adder_dialog for clarity. Ticket #1642 --- install/ui/add.js | 4 ++-- install/ui/association.js | 6 +++--- install/ui/automount.js | 4 ++-- install/ui/dialog.js | 2 +- install/ui/dns.js | 4 ++-- install/ui/entity.js | 2 +- install/ui/group.js | 2 +- install/ui/host.js | 4 ++-- install/ui/service.js | 6 +++--- install/ui/test/data/ipa_init.json | 9 ++++----- ipalib/plugins/internal.py | 9 ++++----- 11 files changed, 25 insertions(+), 27 deletions(-) diff --git a/install/ui/add.js b/install/ui/add.js index fd99b02c5eb77acc99484995a6fb0c65598128b7..65b7711e26a1080536d1a8dc3656aca7b8e970ab 100644 --- a/install/ui/add.js +++ b/install/ui/add.js @@ -23,7 +23,7 @@ /* REQUIRES: ipa.js */ -IPA.add_dialog = function (spec) { +IPA.entity_adder_dialog = function(spec) { spec = spec || {}; @@ -192,7 +192,7 @@ IPA.add_dialog = function (spec) { }; // methods that should be invoked by subclasses - that.add_dialog_create = that.create; + that.entity_adder_dialog_create = that.create; init(); diff --git a/install/ui/association.js b/install/ui/association.js index ebb6e421ff3b8538116471de240b1f972e08e6bf..d3b66132d5043b0dfe60b8847896e9f27f676059 100644 --- a/install/ui/association.js +++ b/install/ui/association.js @@ -889,7 +889,7 @@ IPA.association_facet = function (spec) { that.add_button = IPA.action_button({ name: 'add', - label: IPA.messages.buttons.enroll, + label: IPA.messages.buttons.add, icon: 'add-icon', click: function() { if (!that.add_button.hasClass('action-button-disabled')) { @@ -923,7 +923,7 @@ IPA.association_facet = function (spec) { }).appendTo(span); $('<label/>', { - text: IPA.messages.association.direct_enrollment, + text: IPA.messages.association.direct_membership, 'for': direct_id }).appendTo(span); @@ -944,7 +944,7 @@ IPA.association_facet = function (spec) { }).appendTo(span); $('<label/>', { - text: IPA.messages.association.indirect_enrollment, + text: IPA.messages.association.indirect_membership, 'for': indirect_id }).appendTo(span); } diff --git a/install/ui/automount.js b/install/ui/automount.js index 6b740d8e6d4abc1d0e402f5a1a3b79e8a4a77420..2dfb9ddcb948d284a7676b212a94cccb02faeaca 100644 --- a/install/ui/automount.js +++ b/install/ui/automount.js @@ -221,10 +221,10 @@ IPA.automount_key_column = function(spec) { IPA.automountmap_adder_dialog = function(spec) { - var that = IPA.add_dialog(spec); + var that = IPA.entity_adder_dialog(spec); that.create = function() { - that.add_dialog_create(); + that.entity_adder_dialog_create(); var method_field = that.get_field('method'); var indirect_section = that.get_section('indirect'); diff --git a/install/ui/dialog.js b/install/ui/dialog.js index b55cce715d9ba16b9062378603b4210301012872..41b35fb42120a015fcdeb56c10e80cf638456bd8 100644 --- a/install/ui/dialog.js +++ b/install/ui/dialog.js @@ -492,7 +492,7 @@ IPA.adder_dialog = function(spec) { var add_button = that.create_button({ name: 'add', - label: IPA.messages.buttons.enroll, + label: IPA.messages.buttons.add, click: function() { if (!add_button.is_enabled()) return; that.execute(); diff --git a/install/ui/dns.js b/install/ui/dns.js index 2b98f0dfec4c8f70770123930ea06081ff47633a..ddb284d7a7f64a5d9a7be0f0cc618dfae8975bcc 100644 --- a/install/ui/dns.js +++ b/install/ui/dns.js @@ -383,10 +383,10 @@ IPA.dnszone_adder_dialog = function(spec) { spec = spec || {}; - var that = IPA.add_dialog(spec); + var that = IPA.entity_adder_dialog(spec); that.create = function() { - that.add_dialog_create(); + that.entity_adder_dialog_create(); that.container.addClass('dnszone-adder-dialog'); }; diff --git a/install/ui/entity.js b/install/ui/entity.js index c82f4a8df727c1a3dd75b133d2b005867e0015ab..704b5c43b2ce7ef9dcc7221f5a73e4bf5df4bf15 100644 --- a/install/ui/entity.js +++ b/install/ui/entity.js @@ -914,7 +914,7 @@ IPA.entity_builder = function(){ }; that.adder_dialog = function(spec) { - spec.factory = spec.factory || IPA.add_dialog; + spec.factory = spec.factory || IPA.entity_adder_dialog; spec.name = spec.name || 'add'; if (!spec.title) { diff --git a/install/ui/group.js b/install/ui/group.js index a63a7800a0b668152188bfccfd372e8548e484fd..8c18c14a252f10c763bfed332b14d0248ecb1c25 100644 --- a/install/ui/group.js +++ b/install/ui/group.js @@ -140,7 +140,7 @@ IPA.group_adder_dialog = function (spec) { spec = spec || {}; - var that = IPA.add_dialog(spec); + var that = IPA.entity_adder_dialog(spec); var init = function() { diff --git a/install/ui/host.js b/install/ui/host.js index 4c0ce6ed0e461a38a565c1450cd483098b0c2dc7..63f45e043b0e95c8f517caf8da5a933b0ba71e5e 100644 --- a/install/ui/host.js +++ b/install/ui/host.js @@ -267,10 +267,10 @@ IPA.host_adder_dialog = function(spec) { spec = spec || {}; spec.retry = typeof spec.retry !== 'undefined' ? spec.retry : false; - var that = IPA.add_dialog(spec); + var that = IPA.entity_adder_dialog(spec); that.create = function() { - that.add_dialog_create(); + that.entity_adder_dialog_create(); that.container.addClass('host-adder-dialog'); }; diff --git a/install/ui/service.js b/install/ui/service.js index 0ac2b6bec131786ca3f0c80122e9d8483a9c8f6c..02b206e19fa5cb03e1cc5eebb5380d279fb8e9b3 100644 --- a/install/ui/service.js +++ b/install/ui/service.js @@ -79,18 +79,18 @@ IPA.entity_factories.service = function() { }). standard_association_facets(). adder_dialog({ - factory: IPA.service_add_dialog, + factory: IPA.service_adder_dialog, height: 350 }). build(); }; -IPA.service_add_dialog = function(spec) { +IPA.service_adder_dialog = function(spec) { spec = spec || {}; - var that = IPA.add_dialog(spec). + var that = IPA.entity_adder_dialog(spec). field(IPA.widget({ name: 'krbprincipalname', required: false, diff --git a/install/ui/test/data/ipa_init.json b/install/ui/test/data/ipa_init.json index dfd1fa68a2a4a65eef53ee2e265dac27ccc5589b..5ba1ad1f5bba627076183a06a1db347a3b6ad41c 100644 --- a/install/ui/test/data/ipa_init.json +++ b/install/ui/test/data/ipa_init.json @@ -24,8 +24,8 @@ "memberof": "Add ${entity} ${primary_key} into ${other_entity}", "sourcehost": "Add Source ${other_entity} into ${entity} ${primary_key}" }, - "direct_enrollment": "Direct Enrollment", - "indirect_enrollment": "Indirect Enrollment", + "direct_membership": "Direct Membership", + "indirect_membership": "Indirect Membership", "no_entries": "No entries.", "paging": "Showing ${start} to ${end} of ${total} entries.", "remove": { @@ -48,7 +48,6 @@ "add_many": "Add Many", "cancel": "Cancel", "close": "Close", - "enroll": "Enroll", "find": "Find", "get": "Get", "issue": "Issue", @@ -98,8 +97,8 @@ }, "facet_groups": { "managedby": "${primary_key} is managed by:", - "member": "Entities enrolled in ${primary_key}:", - "memberof": "${primary_key} is a member of these:" + "member": "${primary_key} members:", + "memberof": "${primary_key} is a member of:" }, "facets": { "details": "Settings", diff --git a/ipalib/plugins/internal.py b/ipalib/plugins/internal.py index e613bc4441b68a2e3c78b048bd1dfa0306f3b6b9..bd32442bc20e8855de96d2ae81c15451b6378200 100644 --- a/ipalib/plugins/internal.py +++ b/ipalib/plugins/internal.py @@ -114,8 +114,8 @@ class i18n_messages(Command): "memberof": _("Add ${entity} ${primary_key} into ${other_entity}"), "sourcehost": _("Add Source ${other_entity} into ${entity} ${primary_key}"), }, - "direct_enrollment": _("Direct Enrollment"), - "indirect_enrollment": _("Indirect Enrollment"), + "direct_membership": _("Direct Membership"), + "indirect_membership": _("Indirect Membership"), "no_entries": _("No entries."), "paging": _("Showing ${start} to ${end} of ${total} entries."), "remove": { @@ -138,7 +138,6 @@ class i18n_messages(Command): "add_many": _("Add Many"), "cancel": _("Cancel"), "close": _("Close"), - "enroll": _("Enroll"), "find": _("Find"), "get": _("Get"), "issue": _("Issue"), @@ -188,8 +187,8 @@ class i18n_messages(Command): }, "facet_groups": { "managedby": _("${primary_key} is managed by:"), - "member": _("Entities enrolled in ${primary_key}:"), - "memberof": _("${primary_key} is a member of these:"), + "member": _("${primary_key} members:"), + "memberof": _("${primary_key} is a member of:"), }, "facets": { "details": _("Settings"), -- 1.7.5.1
_______________________________________________ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel