Re: [Freeipa-devel] [PATCH] 442 Hide 'New Certificate' action on CA-less install

2013-08-22 Thread Petr Vobornik

On 08/19/2013 06:57 PM, Ana Krivokapic wrote:

On 08/15/2013 04:00 PM, Petr Vobornik wrote:

This action calls cert-request command which is not available on CA-less
installs. Thus this action won't be enabled and therefore there is no reason
to keep it visible.

https://fedorahosted.org/freeipa/ticket/3363


ACK



Pushed to ipa-3-3 and master.
--
Petr Vobornik

___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel


Re: [Freeipa-devel] [PATCH] 442 Hide 'New Certificate' action on CA-less install

2013-08-19 Thread Ana Krivokapic
On 08/15/2013 04:00 PM, Petr Vobornik wrote:
 This action calls cert-request command which is not available on CA-less
 installs. Thus this action won't be enabled and therefore there is no reason
 to keep it visible.

 https://fedorahosted.org/freeipa/ticket/3363


 ___
 Freeipa-devel mailing list
 Freeipa-devel@redhat.com
 https://www.redhat.com/mailman/listinfo/freeipa-devel

ACK

-- 
Regards,

Ana Krivokapic
Associate Software Engineer
FreeIPA team
Red Hat Inc.

___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

[Freeipa-devel] [PATCH] 442 Hide 'New Certificate' action on CA-less install

2013-08-15 Thread Petr Vobornik
This action calls cert-request command which is not available on CA-less 
installs. Thus this action won't be enabled and therefore there is no 
reason to keep it visible.


https://fedorahosted.org/freeipa/ticket/3363
--
Petr Vobornik
From 36ae1ba3551911de931152460c292079a54df76b Mon Sep 17 00:00:00 2001
From: Petr Vobornik pvobo...@redhat.com
Date: Thu, 15 Aug 2013 14:05:50 +0200
Subject: [PATCH] Hide 'New Certificate' action on CA-less install

This action calls cert-request command which is not available on CA-less installs. Thus this action won't be enabled and therefore there is no reason to keep it visible.

https://fedorahosted.org/freeipa/ticket/3363
---
 install/ui/src/freeipa/certificate.js | 1 +
 1 file changed, 1 insertion(+)

diff --git a/install/ui/src/freeipa/certificate.js b/install/ui/src/freeipa/certificate.js
index 8979390871e5cd2b75a9deb8a5b79818a4131f18..2fab64b7f2c1ef64209cbe54543dca26f7ee8bb7 100755
--- a/install/ui/src/freeipa/certificate.js
+++ b/install/ui/src/freeipa/certificate.js
@@ -615,6 +615,7 @@ IPA.cert.request_action = function(spec) {
 spec.name = spec.name || 'request_cert';
 spec.label = spec.label || '@i18n:objects.cert.new_certificate';
 spec.enable_cond = spec.enable_cond || ['ra_enabled'];
+spec.hide_cond = spec.hide_cond || ['ra_disabled'];
 
 var that = IPA.action(spec);
 that.entity_label = spec.entity_label;
-- 
1.8.3.1

___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel