Shindig Feed Processor: Parse MediaRSS items into optional Media element

It can be useful when displaying the results of FEED type makeRequests to be able to see if any <media:content /> or <media:thumbnail /> elements were included.

This patch adds support for this while maintaining all existing behaviour.

[mat.mannion, 2010-04-27 13:19:15]

Apologies, uploaded wrong patch set

[Paul Lindner, 2010-04-28 17:54:14]

Looks good to me.

I can fix up the tabs vs spaces.

Anyone opposed to adding rome-modules as a dependency?


On Tue, Apr 27, 2010 at 6:19 AM, <mat.mannion@gmail.com> wrote:

> Apologies, uploaded wrong patch set
>
>
> http://codereview.appspot.com/1019041/show
>

[mat.mannion, 2010-04-28 20:01:16]

On 28 April 2010 18:54, Paul Lindner <lindner@inuus.com> wrote:
> I can fix up the tabs vs spaces.

Thanks, I was working off an Eclipse formatter XML that I found
through Google (can't remember exactly where or what I searched for
but it definitely said Shindig)... is there an updated one I can grab
so the tabs vs. spaces thing doesn't come up again?

[Paul Lindner, 2010-05-04 20:08:16]

+dev@shindig.apache.org

Getting one more round of feedback, if no objections I'll commit this
tomorrow via lazy consensus.

On Wed, Apr 28, 2010 at 10:54 AM, Paul Lindner <lindner@inuus.com> wrote:

> Looks good to me.
>
> I can fix up the tabs vs spaces.
>
> Anyone opposed to adding rome-modules as a dependency?
>
>
> On Tue, Apr 27, 2010 at 6:19 AM, <mat.mannion@gmail.com> wrote:
>
>> Apologies, uploaded wrong patch set
>>
>>
>> http://codereview.appspot.com/1019041/show
>>
>
>

[Paul Lindner, 2010-05-06 14:26:41]

patch applied.  Could you also file a jira with the patch so we can get proper
IP clearance?

Thanks!

[mat.mannion, 2010-05-06 14:30:07]

On 2010/05/06 14:26:41, Paul Lindner wrote:
> patch applied.  Could you also file a jira with the patch so we can get proper
> IP clearance?
> 
> Thanks!

No probs; done (SHINDIG-1323)