Skip Menu |
 

This queue is for tickets about the RT-Extension-ExtractCustomFieldValues CPAN distribution.

Report information
The Basics
Id: 86978
Status: resolved
Priority: 0/
Queue: RT-Extension-ExtractCustomFieldValues

People
Owner: Nobody in particular
Requestors: kriggle [...] akamai.com
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)

Attachments
0001-add-a-option-to-capture-multiple-values.patch
add-a-option-to-capture-multiple-values.patch



Subject: [Patch] add option to extract multiple values
Date: Tue, 16 Jul 2013 18:29:10 -0400
To: "bug-RT-Extension-ExtractCustomFieldValues [...] rt.cpan.org" <bug-RT-Extension-ExtractCustomFieldValues [...] rt.cpan.org>
From: "Riggle, Kevin" <kriggle [...] akamai.com>
Download (untitled) / with headers
text/plain 650b
One of the things I've needed for my use of RT::Extension::ExtractCustomFieldValues is the ability to extract multiple values from the same header into a multiple value CF, eg. pulling two or three subject tags out of the same Subject: header. I've attached the patch I cooked up to do it, which includes documentation describing its behavior further. I've been running with it in place for a few weeks and haven't encountered any problems so far. (The patch is based on 633739c9 from the Github repository. I'm not quite sure how you prefer taking patches these days -- I can file a pull request if you'd rather.) Best, - Kevin

Message body is not shown because sender requested not to inline it.

Subject: Re: [rt.cpan.org #86978] [Patch] add option to extract multiple values
Date: Tue, 23 Jul 2013 17:28:15 -0700
To: bug-RT-Extension-ExtractCustomFieldValues [...] rt.cpan.org
From: Thomas Sibley <trs [...] bestpractical.com>
On 07/16/2013 03:29 PM, Riggle, Kevin via RT wrote: Show quoted text
> One of the things I've needed for my use of > RT::Extension::ExtractCustomFieldValues is the ability to extract multiple > values from the same header into a multiple value CF, eg. pulling two or > three subject tags out of the same Subject: header. I've attached the > patch I cooked up to do it, which includes documentation describing its > behavior further. I've been running with it in place for a few weeks and > haven't encountered any problems so far.
The patch you sent needs a little tweaking before being applied. Notably: • Doc was added to the README, but it's generated from the POD in the extension's primary lib file. The doc needs to go into the POD instead, not the README directly. • The match needs to be surrounded by a capture group for $2 || $1 to make sense. Show quoted text
> (The patch is based on 633739c9 from the Github repository. I'm not quite > sure how you prefer taking patches these days -- I can file a pull request > if you'd rather.)
Patches and pull requests are both great.
Subject: Re: [rt.cpan.org #86978] [Patch] add option to extract multiple values
Date: Tue, 30 Jul 2013 17:01:00 -0500
To: "bug-RT-Extension-ExtractCustomFieldValues [...] rt.cpan.org" <bug-RT-Extension-ExtractCustomFieldValues [...] rt.cpan.org>
From: "Riggle, Kevin" <kriggle [...] akamai.com>
Download (untitled) / with headers
text/plain 1.3k
Updated patch attached. Thanks! - Kevin On 2013-7-23 8:28 PM, "trs@bestpractical.com via RT" <bug-RT-Extension-ExtractCustomFieldValues@rt.cpan.org> wrote: Show quoted text
><URL: https://rt.cpan.org/Ticket/Display.html?id=86978 > > >On 07/16/2013 03:29 PM, Riggle, Kevin via RT wrote:
>> One of the things I've needed for my use of >> RT::Extension::ExtractCustomFieldValues is the ability to extract >>multiple >> values from the same header into a multiple value CF, eg. pulling two or >> three subject tags out of the same Subject: header. I've attached the >> patch I cooked up to do it, which includes documentation describing its >> behavior further. I've been running with it in place for a few weeks >>and >> haven't encountered any problems so far.
> >The patch you sent needs a little tweaking before being applied. Notably: > >• Doc was added to the README, but it's generated from the POD in the >extension's primary lib file. The doc needs to go into the POD instead, >not the README directly. > >• The match needs to be surrounded by a capture group for $2 || $1 to >make sense. >
>> (The patch is based on 633739c9 from the Github repository. I'm not >>quite >> sure how you prefer taking patches these days -- I can file a pull >>request >> if you'd rather.)
> >Patches and pull requests are both great. >

Message body is not shown because sender requested not to inline it.

Thanks! Applied and released as 3.08.
Subject: Re: [rt.cpan.org #86978] [Patch] add option to extract multiple values
Date: Tue, 30 Jul 2013 20:19:08 -0700
To: "bug-RT-Extension-ExtractCustomFieldValues [...] rt.cpan.org" <bug-RT-Extension-ExtractCustomFieldValues [...] rt.cpan.org>
From: Thomas Sibley <trs [...] bestpractical.com>
It'll be in 3.09 instead; there were permission issues with 3.08.


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.