Timeframe: 30 days, Total Commits: 10 Total Number of Files Changed: 14
Date | Author | File/Message |
---|---|---|
2006-02-02 02:22:01 | chanezon | added maven properties to compile for 1.4: without that it compiles for 1.5 on a 1.5 system |
2006-02-01 18:45:24 | rjray |
src/test/com/sun/syndication/unittest/TestSyndFeedAtom03DCSyModules.java v 1.3 Changed the test-value for "creator" property, now that the buf has been fixed in SyndEntryImpl.java. CVS: ---------------------------------------------------------------------- CVS: Issue number: CVS: If this change addresses one or more issues, CVS: then enter the issue number(s) here. CVS: Obtained from: CVS: If this change has been taken from another system, CVS: then name the system in this line, otherwise delete it. CVS: Submitted by: CVS: If this code has been contributed to the project by someone else; i.e., CVS: they sent us a patch or a set of diffs, then include their name/email CVS: address here. If this is your work then delete this line. CVS: Reviewed by: CVS: If we are doing pre-commit code reviews and someone else has CVS: reviewed your changes, include their name(s) here. CVS: If you have not had it reviewed then delete this line. |
2006-02-01 18:43:51 | rjray |
src/java/com/sun/syndication/feed/synd/SyndEntryImpl.java v 1.16 Re-did the setting of author (singular) and retrieval to juggle between DC module and actual "authors" list. CVS: ---------------------------------------------------------------------- CVS: Issue number: CVS: If this change addresses one or more issues, CVS: then enter the issue number(s) here. CVS: Obtained from: CVS: If this change has been taken from another system, CVS: then name the system in this line, otherwise delete it. CVS: Submitted by: CVS: If this code has been contributed to the project by someone else; i.e., CVS: they sent us a patch or a set of diffs, then include their name/email CVS: address here. If this is your work then delete this line. CVS: Reviewed by: CVS: If we are doing pre-commit code reviews and someone else has CVS: reviewed your changes, include their name(s) here. CVS: If you have not had it reviewed then delete this line. |
2006-02-01 07:29:24 | chanezon |
src/java/com/sun/syndication/feed/synd/SyndEntryImpl.java v 1.15 Issue number: Obtained from: Submitted by: Reviewed by: CVS: ---------------------------------------------------------------------- CVS: Issue number: CVS: If this change addresses one or more issues, CVS: then enter the issue number(s) here. CVS: Obtained from: CVS: If this change has been taken from another system, CVS: then name the system in this line, otherwise delete it. CVS: Submitted by: CVS: If this code has been contributed to the project by someone else; i.e., CVS: they sent us a patch or a set of diffs, then include their name/email CVS: address here. If this is your work then delete this line. CVS: Reviewed by: CVS: If we are doing pre-commit code reviews and someone else has CVS: reviewed your changes, include their name(s) here. CVS: If you have not had it reviewed then delete this line. removed "categories" from the ignore list. OK the culprit is a checkin from september, where categories in SyndEntryImpl was changed from a convenience property mapped to dc module, thus needing to be ignored when cloning, to a _catgories List property. The person who checked that in forgot to remove categories from the ignore list (which is very forgivable considering the clunky design, but ignoring these 5 failed unit tests is, how would I say,.... I would say nothing because I did not commit anything in a long time so I'm the last one to give any lessons here:-). https://rome.dev.java.net/source/browse/rome/src/java/com/sun/syndication/feed/synd/SyndEntryImpl.java?r1=1.12&r2=1.13 That means that Randy is right, removing line 65 fixes the bug without any side effects. |
2006-01-30 22:45:57 | vanvalen |
src/java/com/sun/syndication/feed/synd/impl/ConverterForAtom10.java v 1.3 FIXED THE FOLLOWING ISSUES in ConverterForAtom10.java 1) When converting between SyndFeed and WireFeed and back, there are some mix ups in that links with rel=null or rel="" aren't treated as alternate links, but they should be since those translate by default to rel="alternate" according to atom spec. In a few cases in ConverterForAtom10, testing for alternate links is done by "alternate".equals... when it should be == null || "".equals... || "alternate".equals as is done in Atom10Parser. 2) When converting from SyndFeed to WireFeed and back, the Atom "contents" get dropped on the floor. 3) There is a little bug where when converting from SyndFeed to WireFeed and back, some entry "links" don't have the type copied over. 4) Change List.add(List) to be List.addAll(List) in a few places in ConverterForAtom10 to get the expected behavior. There were some class cast exceptions when pulling things out of the List because it contained another List instead of the other List's contents. |
2006-01-30 10:03:08 | rjray | Added getAuthor() to the interface. This was implemented in classes that implement SyndFeed, but classes USING these object via interface handles could not access the method. CVS: ---------------------------------------------------------------------- CVS: Issue number: CVS: If this change addresses one or more issues, CVS: then enter the issue number(s) here. CVS: Obtained from: CVS: If this change has been taken from another system, CVS: then name the system in this line, otherwise delete it. CVS: Submitted by: CVS: If this code has been contributed to the project by someone else; i.e., CVS: they sent us a patch or a set of diffs, then include their name/email CVS: address here. If this is your work then delete this line. CVS: Reviewed by: CVS: If we are doing pre-commit code reviews and someone else has CVS: reviewed your changes, include their name(s) here. CVS: If you have not had it reviewed then delete this line. |
2006-01-30 06:02:27 | rjray |
src/test/com/sun/syndication/unittest/TestSyndFeedAtom10.java v 1.3 Fixed a typo in the atom_1.0 test-input, and several typos in the JUnit suite for Atom 1.0. CVS: ---------------------------------------------------------------------- CVS: Issue number: CVS: If this change addresses one or more issues, CVS: then enter the issue number(s) here. CVS: Obtained from: CVS: If this change has been taken from another system, CVS: then name the system in this line, otherwise delete it. CVS: Submitted by: CVS: If this code has been contributed to the project by someone else; i.e., CVS: they sent us a patch or a set of diffs, then include their name/email CVS: address here. If this is your work then delete this line. CVS: Reviewed by: CVS: If we are doing pre-commit code reviews and someone else has CVS: reviewed your changes, include their name(s) here. CVS: If you have not had it reviewed then delete this line. |
2006-01-30 05:57:03 | rjray |
src/java/com/sun/syndication/feed/synd/impl/ConverterForRSS10.java v 1.10 The source and destination were backwards when copying the "uri" property while cloning. CVS: ---------------------------------------------------------------------- CVS: Issue number: CVS: If this change addresses one or more issues, CVS: then enter the issue number(s) here. CVS: Obtained from: CVS: If this change has been taken from another system, CVS: then name the system in this line, otherwise delete it. CVS: Submitted by: CVS: If this code has been contributed to the project by someone else; i.e., CVS: they sent us a patch or a set of diffs, then include their name/email CVS: address here. If this is your work then delete this line. CVS: Reviewed by: CVS: If we are doing pre-commit code reviews and someone else has CVS: reviewed your changes, include their name(s) here. CVS: If you have not had it reviewed then delete this line. |
2006-01-05 15:36:21 | snoopdave |
src/test/com/sun/syndication/unittest/TestOpsAtom03.java v 1.5 src/test/com/sun/syndication/unittest/TestOpsAtom10.java v 1.2 src/test/com/sun/syndication/unittest/TestSyndFeedAtom10.java v 1.2 Updated tests and test file for Atom 1.0, now need to figure out why they are failing |
2006-01-03 04:16:53 | nicklothian |
src/java/com/sun/syndication/feed/synd/impl/ConverterForRSS10.java v 1.9 Fix for problem converting to RSS 1.0 if not URI is specified at the channel level (it will now attempt to use the Link element) |