URL: https://github.com/freeipa/freeipa/pull/498 Author: HonzaCholasta Title: #498: compat: fix `Any` params in `batch` and `dnsrecord` Action: opened
PR body: """ The `methods` argument of `batch` and `dnsrecords` attribute of `dnsrecord` were incorrectly defined as `Str` instead of `Any`. https://fedorahosted.org/freeipa/ticket/6647 """ To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/498/head:pr498 git checkout pr498
From 3936af7513c1f39de254db05a9fbb8f3395dfaa6 Mon Sep 17 00:00:00 2001 From: Jan Cholasta <jchol...@redhat.com> Date: Thu, 23 Feb 2017 13:04:19 +0000 Subject: [PATCH] compat: fix `Any` params in `batch` and `dnsrecord` The `methods` argument of `batch` and `dnsrecords` attribute of `dnsrecord` were incorrectly defined as `Str` instead of `Any`. https://fedorahosted.org/freeipa/ticket/6647 --- ipaclient/remote_plugins/2_114/batch.py | 2 +- ipaclient/remote_plugins/2_114/dns.py | 2 +- ipaclient/remote_plugins/2_156/batch.py | 2 +- ipaclient/remote_plugins/2_156/dns.py | 2 +- ipaclient/remote_plugins/2_164/batch.py | 2 +- ipaclient/remote_plugins/2_164/dns.py | 2 +- ipaclient/remote_plugins/2_49/batch.py | 2 +- ipaclient/remote_plugins/2_49/dns.py | 2 +- 8 files changed, 8 insertions(+), 8 deletions(-) diff --git a/ipaclient/remote_plugins/2_114/batch.py b/ipaclient/remote_plugins/2_114/batch.py index 4a613b6..2709e59 100644 --- a/ipaclient/remote_plugins/2_114/batch.py +++ b/ipaclient/remote_plugins/2_114/batch.py @@ -50,7 +50,7 @@ class batch(Command): NO_CLI = True takes_args = ( - parameters.Str( + parameters.Any( 'methods', required=False, multivalue=True, diff --git a/ipaclient/remote_plugins/2_114/dns.py b/ipaclient/remote_plugins/2_114/dns.py index 5d91dbc..acb8a65 100644 --- a/ipaclient/remote_plugins/2_114/dns.py +++ b/ipaclient/remote_plugins/2_114/dns.py @@ -326,7 +326,7 @@ class dnsrecord(Object): 'dnsclass', required=False, ), - parameters.Str( + parameters.Any( 'dnsrecords', required=False, label=_(u'Records'), diff --git a/ipaclient/remote_plugins/2_156/batch.py b/ipaclient/remote_plugins/2_156/batch.py index 4a613b6..2709e59 100644 --- a/ipaclient/remote_plugins/2_156/batch.py +++ b/ipaclient/remote_plugins/2_156/batch.py @@ -50,7 +50,7 @@ class batch(Command): NO_CLI = True takes_args = ( - parameters.Str( + parameters.Any( 'methods', required=False, multivalue=True, diff --git a/ipaclient/remote_plugins/2_156/dns.py b/ipaclient/remote_plugins/2_156/dns.py index 39a0b26..bbfaa9f 100644 --- a/ipaclient/remote_plugins/2_156/dns.py +++ b/ipaclient/remote_plugins/2_156/dns.py @@ -326,7 +326,7 @@ class dnsrecord(Object): 'dnsclass', required=False, ), - parameters.Str( + parameters.Any( 'dnsrecords', required=False, label=_(u'Records'), diff --git a/ipaclient/remote_plugins/2_164/batch.py b/ipaclient/remote_plugins/2_164/batch.py index 4a613b6..2709e59 100644 --- a/ipaclient/remote_plugins/2_164/batch.py +++ b/ipaclient/remote_plugins/2_164/batch.py @@ -50,7 +50,7 @@ class batch(Command): NO_CLI = True takes_args = ( - parameters.Str( + parameters.Any( 'methods', required=False, multivalue=True, diff --git a/ipaclient/remote_plugins/2_164/dns.py b/ipaclient/remote_plugins/2_164/dns.py index b07a94f..244be87 100644 --- a/ipaclient/remote_plugins/2_164/dns.py +++ b/ipaclient/remote_plugins/2_164/dns.py @@ -326,7 +326,7 @@ class dnsrecord(Object): 'dnsclass', required=False, ), - parameters.Str( + parameters.Any( 'dnsrecords', required=False, label=_(u'Records'), diff --git a/ipaclient/remote_plugins/2_49/batch.py b/ipaclient/remote_plugins/2_49/batch.py index a1f351d..67e5978 100644 --- a/ipaclient/remote_plugins/2_49/batch.py +++ b/ipaclient/remote_plugins/2_49/batch.py @@ -50,7 +50,7 @@ class batch(Command): NO_CLI = True takes_args = ( - parameters.Str( + parameters.Any( 'methods', required=False, multivalue=True, diff --git a/ipaclient/remote_plugins/2_49/dns.py b/ipaclient/remote_plugins/2_49/dns.py index 07cef75..4b543a2 100644 --- a/ipaclient/remote_plugins/2_49/dns.py +++ b/ipaclient/remote_plugins/2_49/dns.py @@ -256,7 +256,7 @@ class dnsrecord(Object): label=_(u'Class'), doc=_(u'DNS class'), ), - parameters.Str( + parameters.Any( 'dnsrecords', required=False, label=_(u'Records'),
-- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code