ROME open issues

[Alejandro Abdelnur-2]

I finally made some time to housekeep ROME and I'm trying to get rid
of all issues/bugs.

Once we take care of the following we could do another release as we
have 25+ fixes since 0.9.

Please let me know what is your opinion about the following issues.

Thxs.

A

1 --------------------------------------------------------------------------
https://rome.dev.java.net/issues/show_bug.cgi?id=88

I don't understand this bug

2 --------------------------------------------------------------------------
https://rome.dev.java.net/issues/show_bug.cgi?id=94

I've found the problem to this one:

When adding support for content:encoded this has been wrongfully done
in the RSS parsers and generators, it should have been done in a
module parser/generator.

Because there is not module registered for that namespace it is
treated as foreign markup as well as as the content:encoded property
of the item thus the ocurrences increment on every read/write.

I personally don't like how the bean model has evolved, is to convoluted.

A simple fix for this is to make the RSS090Parser parseItem() to
discard content:encoded elements when computing foreign markup for the
item.

I'll commit the mentioned fix on Monday if I don't hear an objection.

3 --------------------------------------------------------------------------
https://rome.dev.java.net/issues/show_bug.cgi?id=95

I've tested the head of CVS with JDOM 1.1 and all testcases work fine.

I need somebody to verify there is a problem, else I'll close it as invalid.

--------------------------------------------------------------------------

---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@...
For additional commands, e-mail: users-help@...

[Alejandro Abdelnur-2]

I have not seen any comments on these issues yet.

Thxs.

A

On Thu, Apr 3, 2008 at 9:31 PM, Alejandro Abdelnur <tucu00@...> wrote:

> I finally made some time to housekeep ROME and I'm trying to get rid
>  of all issues/bugs.
>
>  Once we take care of the following we could do another release as we
>  have 25+ fixes since 0.9.
>
>  Please let me know what is your opinion about the following issues.
>
>  Thxs.
>
>  A
>
>  1 --------------------------------------------------------------------------
>  https://rome.dev.java.net/issues/show_bug.cgi?id=88
>
>  I don't understand this bug
>
>  2 --------------------------------------------------------------------------
>  https://rome.dev.java.net/issues/show_bug.cgi?id=94
>
>  I've found the problem to this one:
>
>  When adding support for content:encoded this has been wrongfully done
>  in the RSS parsers and generators, it should have been done in a
>  module parser/generator.
>
>  Because there is not module registered for that namespace it is
>  treated as foreign markup as well as as the content:encoded property
>  of the item thus the ocurrences increment on every read/write.
>
>  I personally don't like how the bean model has evolved, is to convoluted.
>
>  A simple fix for this is to make the RSS090Parser parseItem() to
>  discard content:encoded elements when computing foreign markup for the
>  item.
>
>  I'll commit the mentioned fix on Monday if I don't hear an objection.
>
>  3 --------------------------------------------------------------------------
>  https://rome.dev.java.net/issues/show_bug.cgi?id=95
>
>  I've tested the head of CVS with JDOM 1.1 and all testcases work fine.
>
>  I need somebody to verify there is a problem, else I'll close it as invalid.
>
>  --------------------------------------------------------------------------
>

---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@...
For additional commands, e-mail: users-help@...

[Dave Johnson-8]

On Thu, Apr 3, 2008 at 6:01 PM, Alejandro Abdelnur <tucu00@...> wrote:
> I finally made some time to housekeep ROME and I'm trying to get rid
>  of all issues/bugs.

>  2 --------------------------------------------------------------------------
>  https://rome.dev.java.net/issues/show_bug.cgi?id=94
>
>  I've found the problem to this one:
>
>  When adding support for content:encoded this has been wrongfully done
>  in the RSS parsers and generators, it should have been done in a
>  module parser/generator.
>
>  Because there is not module registered for that namespace it is
>  treated as foreign markup as well as as the content:encoded property
>  of the item thus the ocurrences increment on every read/write.
>
>  I personally don't like how the bean model has evolved, is to convoluted.
>
>  A simple fix for this is to make the RSS090Parser parseItem() to
>  discard content:encoded elements when computing foreign markup for the
>  item.

We discussed this one on the list. I think we decided that
content:encoded was so common in RSS that it should be built into
standard ROME and not implemented as a separate module. There is code
that depends on the current behavior in Roller's Planet aggregator and
other places.

- Dave

---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@...
For additional commands, e-mail: users-help@...

[Alejandro Abdelnur-2]

Mhhh, OK. So, the fix should be alright, right?

A

On Wed, Apr 9, 2008 at 4:32 PM, Dave <snoopdave@...> wrote:

> On Thu, Apr 3, 2008 at 6:01 PM, Alejandro Abdelnur <tucu00@...> wrote:
>  > I finally made some time to housekeep ROME and I'm trying to get rid
>  >  of all issues/bugs.
>
>
> >  2 --------------------------------------------------------------------------
>  >  https://rome.dev.java.net/issues/show_bug.cgi?id=94
>  >
>  >  I've found the problem to this one:
>  >
>  >  When adding support for content:encoded this has been wrongfully done
>  >  in the RSS parsers and generators, it should have been done in a
>  >  module parser/generator.
>  >
>  >  Because there is not module registered for that namespace it is
>  >  treated as foreign markup as well as as the content:encoded property
>  >  of the item thus the ocurrences increment on every read/write.
>  >
>  >  I personally don't like how the bean model has evolved, is to convoluted.
>  >
>  >  A simple fix for this is to make the RSS090Parser parseItem() to
>  >  discard content:encoded elements when computing foreign markup for the
>  >  item.
>
>  We discussed this one on the list. I think we decided that
>  content:encoded was so common in RSS that it should be built into
>  standard ROME and not implemented as a separate module. There is code
>  that depends on the current behavior in Roller's Planet aggregator and
>  other places.
>
>  - Dave
>
>  ---------------------------------------------------------------------
>  To unsubscribe, e-mail: users-unsubscribe@...
>  For additional commands, e-mail: users-help@...
>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@...
For additional commands, e-mail: users-help@...

[Dave Johnson-8]

On Wed, Apr 9, 2008 at 1:06 PM, Alejandro Abdelnur <tucu00@...> wrote:
> Mhhh, OK. So, the fix should be alright, right?

Sorry, I didn't address the issue at hand ;-)

Yes, I think the fix is correct.

- Dave

---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@...
For additional commands, e-mail: users-help@...