* Do not use RssThred
- All timer tasks are being run in a thread of their own, created by the java.util.Timer. Hence, there's no need to be executing a new thread
* Renamded recupFlux() to retrieveFlow()
- nothing but English in source code!
* Added an OperationFailedException to retrieveFlow()
- we need to have some kind of indication if retrieving a flow fails so that we could act accordingly (e.g. not add a contact to the contact list)
* The reader should be created with a URL object and not an unparsed string.
- URL objects must be created as early as possible (e.g. at the time a contact is added to the contact list) so that we could parse and test the syntax of the URL and ignore the contact if we encounter any problems.
* Made the sortItems() method private
- seems that there's no one else using it.
* Made sortItems use Arrays.sort()
- thought it would be better this way rather than let it implement the sort itself. I've therefore also added a comparatator class.
* ReReversed sort order for RSS posts.
- given the way SC wokrs and what it is used for, the user is most likely to only be seeing the last messages being printed out. In the meantime, the msgs that a user would be most likely to be looking for are the newest, so it would be better if they were printed last.
* Removed calls to System.out.println()
- we should be using net.java.sip.communicator.util.Logger instead.
* Removed the deliverAuthentication() method
- seems like Gibberish residue so I decided to drop it. I don't think we handle or need authorizations in RSS do we?
* Remove the createVolatileContact() method.
- we don't really have that for RSS
* Remove the getNonPersistentGroup() method
- unused
* Remove the findProviderForRssID() method
- unused. (Gibberish residue?)
* Removed java.awt imports
- unnecessary
* Removed the deliverMessage() method.
- I guess this too was a residue from gibberish that we don't need here.
* The sendInstantMessage() method was creating a MessageDeliveredEvent instance that it was not using
- the event instance is actually created by the fireMessageDelivered() method
* Store a reference of the source RSS url and RssFeedReader
- did this so that OperationSetBasicInstantMessagingRssImpl could then reuse them.
* Removed the getToken() method
- seems no one was using it.
This test case may suffer from a problem in a specific situation :
it is possible for an authenticated message to being resend after some time (and to be re-handled).
We will activate these tests later. Benoit already sent us a patch for that.
- transformerFactory.setAttribute("indent-number",4); - works for jdk 1.5 and 1.6, but on 1.4 throws Exception
- serializer.setOutputProperty("{http://xml.apache.org/xalan}indent-amount", "4"); - works for 1.4 and 1.6. On 1.5 there is bug report which hasn't been fixed
2. Fix resource loading for jabberaccregwizz