Agreed, but we will deal with that if/when it comes to it.
Thanks,
Cole
On 04/08/2015 02:55 PM, Dustin Mitchell wrote:
Rock on, thanks!
By the way, I checked with the BMO folks, and they made this change
because the site is a mix of 4.2, 4.4, 4.5, and a whole bunch of local
hacks. So it may soon come to pass that you need a custom subclass
for the site. For what I need (well, what bugwarrior needs),
Bugzilla42 is sufficient at the moment.
Dustin
On Wed, Apr 8, 2015 at 2:52 PM, Cole Robinson <crobinso(a)redhat.com> wrote:
> On 04/07/2015 01:42 PM, Dustin J. Mitchell wrote:
>> ---
>> bugzilla/__init__.py | 5 ++++-
>> 1 file changed, 4 insertions(+), 1 deletion(-)
>>
>> diff --git a/bugzilla/__init__.py b/bugzilla/__init__.py
>> index 138bd3b..52f77bc 100644
>> --- a/bugzilla/__init__.py
>> +++ b/bugzilla/__init__.py
>> @@ -41,18 +41,21 @@ def _getBugzillaClassForURL(url, sslverify):
>> rhbz = False
>> bzversion = ''
>> c = None
>>
>> if "bugzilla.redhat.com" in url:
>> log.info("Using RHBugzilla for URL containing
bugzilla.redhat.com")
>> return RHBugzilla
>> if "bugzilla.novell.com" in url:
>> - log.info("Using NovellBugzilla for URL containing
novell.com")
>> + log.info("Using NovellBugzilla for URL containing
bugzilla.novell.com")
>> return NovellBugzilla
>> + if "bugzilla.mozilla.org" in url:
>> + log.info("Using Bugzilla42 for URL containing
bugzilla.mozilla.org")
>> + return Bugzilla42
>>
>> # Check for a Red Hat extension
>> try:
>> log.debug("Checking for Red Hat Bugzilla extension")
>> extensions = s.Bugzilla.extensions()
>> if extensions.get('extensions', {}).get('RedHat',
False):
>> rhbz = True
>> except Fault:
>>
>
> Works for me, I've pushed this now, and added a testcase for it too.
>
> - Cole