[sip-comm-dev] Item identification patch


#1

Hi all,
After some messing around with the source code and some 4 rewrites,
the patch for uniquely identifying feeds is here and Exception-free (I
hope :smiley: ). Briefly the patch:
聽聽* provides an (almost) foolproof way of identifying feed items that
allows for correct retrieval even for feeds that don't have the date
set (or it's incorrectly (a.k.a. unreadable) set). As it was discussed
previously, it uses item date or item url. For the feeds I have tried
(GoogleNews RSS & ATOM, various Wordpress feeds, Dr.Dobb, PhD Comic,
etc :wink: ) it works quite fine :slight_smile:
聽聽* fixes a performance issue in the favicon retrieval code.
Previously each time the conversation window was displayed we would
download and convert the favicon.ico to something readable. Now, after
it's first retrieved, it's stored, so that subsequent calls to
ContactRssImpl.getImage() return the stored image.
聽聽* code and Javadoc cleanup/format/spell-check & corrections (Long
live Eclipse :smiley: ).

That would be all for now.

All the best,
Mihai

UniqueID.tar.gz (11.3 KB)


#2

Hi Mihai,

It's a good work, feeds without date (i.e. http://www.freenews.fr/feeds/rss.php) are working now :slight_smile:

But I experienced strange behavior with feeds with date, which were previously working: i.e. http://news.google.com/nwshp?tab=wn&output=rss.
Try it with a new sip-communicator account, then create a Rss account, and add the google feeds. You will see the systray showing that he receives the news, then the ConversationPanel will appear but will stay empty.
Nevertheless, when a new news appears it will NOT be well displayed again if you have try to refresh it manually (sending a message) before.

By the way another bug appears with Elsevier feeds (feed://www1.elsevier.com/homepage/sac/cfp/compsci_cfp.rss). I'm running out of time to see it in details (Mihai can you have a look a it?).
See error dump following:
net.java.sip.communicator.service.contactlist.MetaContactListException: failed to create contactfeed://www1.elsevier.com/homepage/sac/cfp/compsci_cfp.rss
聽聽at net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.addNewContactToMetaContact(MetaContactListServiceImpl.java:417)
聽聽at net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.createMetaContact(MetaContactListServiceImpl.java:680)
聽聽at net.java.sip.communicator.impl.gui.main.contactlist.addcontact.AddContactWizard$CreateContact.run(AddContactWizard.java:88)
Caused by: java.lang.IllegalArgumentException: failed to create a URL for address feed://www1.elsevier.com/homepage/sac/cfp/compsci_cfp.rss. Error was: Unknown protocol: feed
聽聽at net.java.sip.communicator.impl.protocol.rss.OperationSetPersistentPresenceRssImpl.subscribe(OperationSetPersistentPresenceRssImpl.java:851)
聽聽at net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.addNewContactToMetaContact(MetaContactListServiceImpl.java:393)
聽聽... 2 more

Cheers,
Vincent

Mihai Balan wrote:

路路路

Hi all,
After some messing around with the source code and some 4 rewrites,
the patch for uniquely identifying feeds is here and Exception-free (I
hope :smiley: ). Briefly the patch:
聽聽* provides an (almost) foolproof way of identifying feed items that
allows for correct retrieval even for feeds that don't have the date
set (or it's incorrectly (a.k.a. unreadable) set). As it was discussed
previously, it uses item date or item url. For the feeds I have tried
(GoogleNews RSS & ATOM, various Wordpress feeds, Dr.Dobb, PhD Comic,
etc :wink: ) it works quite fine :slight_smile:
聽聽* fixes a performance issue in the favicon retrieval code.
Previously each time the conversation window was displayed we would
download and convert the favicon.ico to something readable. Now, after
it's first retrieved, it's stored, so that subsequent calls to
ContactRssImpl.getImage() return the stored image.
聽聽* code and Javadoc cleanup/format/spell-check & corrections (Long
live Eclipse :smiley: ).

That would be all for now.

All the best,
Mihai

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

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@sip-communicator.dev.java.net
For additional commands, e-mail: dev-help@sip-communicator.dev.java.net

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@sip-communicator.dev.java.net
For additional commands, e-mail: dev-help@sip-communicator.dev.java.net


#3

Hi Vincent,
A very quick glance the problem you're having with the Elsevier feeds
reveals the fact that you're trying to add a feed that uses the
feed:// protocol. Afaik the only protocols that ROME can deal with is
http:// ( https:// too?). By simply replacing feed:// with http:// in
the Elsevier feed adress everything works fine. Maybe some sort of
validation could be made upon adding a new RSS contact, but at the
moment, I have some other matters at hand.
I'm working on the user-requested update bug, I try to fix it asap :slight_smile:

All the best,
Mihai

路路路

On 8/16/07, Vincent Lucas <lucas@clarinet.u-strasbg.fr> wrote:

Hi Mihai,

It's a good work, feeds without date (i.e.
http://www.freenews.fr/feeds/rss.php) are working now :slight_smile:

But I experienced strange behavior with feeds with date, which were
previously working: i.e. http://news.google.com/nwshp?tab=wn&output=rss.
Try it with a new sip-communicator account, then create a Rss account,
and add the google feeds. You will see the systray showing that he
receives the news, then the ConversationPanel will appear but will stay
empty.
Nevertheless, when a new news appears it will NOT be well displayed
again if you have try to refresh it manually (sending a message) before.

By the way another bug appears with Elsevier feeds
(feed://www1.elsevier.com/homepage/sac/cfp/compsci_cfp.rss). I'm running
out of time to see it in details (Mihai can you have a look a it?).
See error dump following:
net.java.sip.communicator.service.contactlist.MetaContactListException:
failed to create
contactfeed://www1.elsevier.com/homepage/sac/cfp/compsci_cfp.rss
聽聽聽聽聽聽聽聽at
net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.addNewContactToMetaContact(MetaContactListServiceImpl.java:417)
聽聽聽聽聽聽聽聽at
net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.createMetaContact(MetaContactListServiceImpl.java:680)
聽聽聽聽聽聽聽聽at
net.java.sip.communicator.impl.gui.main.contactlist.addcontact.AddContactWizard$CreateContact.run(AddContactWizard.java:88)
Caused by: java.lang.IllegalArgumentException: failed to create a URL
for address feed://www1.elsevier.com/homepage/sac/cfp/compsci_cfp.rss.
Error was: Unknown protocol: feed
聽聽聽聽聽聽聽聽at
net.java.sip.communicator.impl.protocol.rss.OperationSetPersistentPresenceRssImpl.subscribe(OperationSetPersistentPresenceRssImpl.java:851)
聽聽聽聽聽聽聽聽at
net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.addNewContactToMetaContact(MetaContactListServiceImpl.java:393)
聽聽聽聽聽聽聽聽... 2 more

Cheers,
Vincent

Mihai Balan wrote:
> Hi all,
> After some messing around with the source code and some 4 rewrites,
> the patch for uniquely identifying feeds is here and Exception-free (I
> hope :smiley: ). Briefly the patch:
> * provides an (almost) foolproof way of identifying feed items that
> allows for correct retrieval even for feeds that don't have the date
> set (or it's incorrectly (a.k.a. unreadable) set). As it was discussed
> previously, it uses item date or item url. For the feeds I have tried
> (GoogleNews RSS & ATOM, various Wordpress feeds, Dr.Dobb, PhD Comic,
> etc :wink: ) it works quite fine :slight_smile:
> * fixes a performance issue in the favicon retrieval code.
> Previously each time the conversation window was displayed we would
> download and convert the favicon.ico to something readable. Now, after
> it's first retrieved, it's stored, so that subsequent calls to
> ContactRssImpl.getImage() return the stored image.
> * code and Javadoc cleanup/format/spell-check & corrections (Long
> live Eclipse :smiley: ).
>
> That would be all for now.
>
> All the best,
> Mihai
>
>
> ------------------------------------------------------------------------
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@sip-communicator.dev.java.net
> For additional commands, e-mail: dev-help@sip-communicator.dev.java.net

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@sip-communicator.dev.java.net
For additional commands, e-mail: dev-help@sip-communicator.dev.java.net

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@sip-communicator.dev.java.net
For additional commands, e-mail: dev-help@sip-communicator.dev.java.net


#4

Hi devs, we need your help :slight_smile:

The last patch from Mihai (the identification patch) give me a little bug that I can reproduce but Mihai can't.
That's why we need another tester to determine if this bug is only related to my configuration or if it's a real bug.
For this you just need to:
1) Patch your SC with the identification patch from Mihai.
2) Remove your .sip-communicator directory.
3) Launch SC.
4) Add a Rss account.
5) Create a group.
6) Add a new buddy: http://news.google.com/nwshp?tab=wn&output=rss
7) Tell us if the chat windows display some text. For my part the chat window appears but stay empty (the systray and the history work fine).

Thanks for your help,
Vincent

Mihai Balan wrote:

路路路

Hi Vincent,
A very quick glance the problem you're having with the Elsevier feeds
reveals the fact that you're trying to add a feed that uses the
feed:// protocol. Afaik the only protocols that ROME can deal with is
http:// ( https:// too?). By simply replacing feed:// with http:// in
the Elsevier feed adress everything works fine. Maybe some sort of
validation could be made upon adding a new RSS contact, but at the
moment, I have some other matters at hand.
I'm working on the user-requested update bug, I try to fix it asap :slight_smile:

All the best,
Mihai

On 8/16/07, Vincent Lucas <lucas@clarinet.u-strasbg.fr> wrote:

Hi Mihai,

It's a good work, feeds without date (i.e.
http://www.freenews.fr/feeds/rss.php) are working now :slight_smile:

But I experienced strange behavior with feeds with date, which were
previously working: i.e. http://news.google.com/nwshp?tab=wn&output=rss.
Try it with a new sip-communicator account, then create a Rss account,
and add the google feeds. You will see the systray showing that he
receives the news, then the ConversationPanel will appear but will stay
empty.
Nevertheless, when a new news appears it will NOT be well displayed
again if you have try to refresh it manually (sending a message) before.

By the way another bug appears with Elsevier feeds
(feed://www1.elsevier.com/homepage/sac/cfp/compsci_cfp.rss). I'm running
out of time to see it in details (Mihai can you have a look a it?).
See error dump following:
net.java.sip.communicator.service.contactlist.MetaContactListException:
failed to create
contactfeed://www1.elsevier.com/homepage/sac/cfp/compsci_cfp.rss
聽聽聽聽聽聽聽聽at
net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.addNewContactToMetaContact(MetaContactListServiceImpl.java:417)
聽聽聽聽聽聽聽聽at
net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.createMetaContact(MetaContactListServiceImpl.java:680)
聽聽聽聽聽聽聽聽at
net.java.sip.communicator.impl.gui.main.contactlist.addcontact.AddContactWizard$CreateContact.run(AddContactWizard.java:88)
Caused by: java.lang.IllegalArgumentException: failed to create a URL
for address feed://www1.elsevier.com/homepage/sac/cfp/compsci_cfp.rss.
Error was: Unknown protocol: feed
聽聽聽聽聽聽聽聽at
net.java.sip.communicator.impl.protocol.rss.OperationSetPersistentPresenceRssImpl.subscribe(OperationSetPersistentPresenceRssImpl.java:851)
聽聽聽聽聽聽聽聽at
net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.addNewContactToMetaContact(MetaContactListServiceImpl.java:393)
聽聽聽聽聽聽聽聽... 2 more

Cheers,
Vincent

Mihai Balan wrote:

Hi all,
After some messing around with the source code and some 4 rewrites,
the patch for uniquely identifying feeds is here and Exception-free (I
hope :smiley: ). Briefly the patch:
聽聽* provides an (almost) foolproof way of identifying feed items that
allows for correct retrieval even for feeds that don't have the date
set (or it's incorrectly (a.k.a. unreadable) set). As it was discussed
previously, it uses item date or item url. For the feeds I have tried
(GoogleNews RSS & ATOM, various Wordpress feeds, Dr.Dobb, PhD Comic,
etc :wink: ) it works quite fine :slight_smile:
聽聽* fixes a performance issue in the favicon retrieval code.
Previously each time the conversation window was displayed we would
download and convert the favicon.ico to something readable. Now, after
it's first retrieved, it's stored, so that subsequent calls to
ContactRssImpl.getImage() return the stored image.
聽聽* code and Javadoc cleanup/format/spell-check & corrections (Long
live Eclipse :smiley: ).

That would be all for now.

All the best,
Mihai

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

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@sip-communicator.dev.java.net
For additional commands, e-mail: dev-help@sip-communicator.dev.java.net

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@sip-communicator.dev.java.net
For additional commands, e-mail: dev-help@sip-communicator.dev.java.net

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@sip-communicator.dev.java.net
For additional commands, e-mail: dev-help@sip-communicator.dev.java.net

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@sip-communicator.dev.java.net
For additional commands, e-mail: dev-help@sip-communicator.dev.java.net


#5

Hi guys,

I just checked and I am seeing the exact same behavior as you describe:
I first get an empty window and then the updates succeed. There were no
exceptions in neither the console nor the log files.

Personally, I'd bet on another problem with the swing html renderer.

I had also noticed a similar problem with the FOSDEM RSS flow (
http://fosdem.org/2007/rss.xml ). It's there with or without the patch
and it looks a lot worse as it would sometimes even mess the way the
chat or history windows look. This one spits an exception in the logs
(though judging from how deep in swing it appears I don't think the
stacktrace would help a lot)

Here's what the exception looks like:

java.lang.NullPointerException
聽聽聽聽at javax.swing.text.FlowView.insertUpdate(FlowView.java:238)
聽聽聽聽at javax.swing.text.View.forwardUpdateToView(View.java:1186)
聽聽聽聽at javax.swing.text.View.forwardUpdate(View.java:1161)
聽聽聽聽at javax.swing.text.BoxView.forwardUpdate(BoxView.java:223)
聽聽聽聽at javax.swing.text.View.insertUpdate(View.java:685)
聽聽聽聽at javax.swing.text.View.forwardUpdateToView(View.java:1186)
聽聽聽聽at javax.swing.text.View.forwardUpdate(View.java:1161)
聽聽聽聽at javax.swing.text.BoxView.forwardUpdate(BoxView.java:223)
聽聽聽聽at javax.swing.text.View.insertUpdate(View.java:685)
聽聽聽聽at javax.swing.plaf.basic.BasicTextUI$RootView.insertUpdate(BasicTextUI.java:1589)
聽聽聽聽at javax.swing.plaf.basic.BasicTextUI$UpdateHandler.insertUpdate(BasicTextUI.java:1848)
聽聽聽聽at javax.swing.text.AbstractDocument.fireInsertUpdate(AbstractDocument.java:185)
聽聽聽聽at javax.swing.text.DefaultStyledDocument.insert(DefaultStyledDocument.java:206)
聽聽聽聽at javax.swing.text.html.HTMLDocument.insert(HTMLDocument.java:242)
聽聽聽聽at javax.swing.text.html.HTMLDocument$HTMLReader.flushBuffer(HTMLDocument.java:3325)
聽聽聽聽at javax.swing.text.html.HTMLDocument$HTMLReader.flush(HTMLDocument.java:2133)
聽聽聽聽at javax.swing.text.html.HTMLDocument.insertHTML(HTMLDocument.java:1160)
聽聽聽聽at javax.swing.text.html.HTMLDocument.insertAfterEnd(HTMLDocument.java:989)
聽聽聽聽at net.java.sip.communicator.impl.gui.main.chat.ChatConversationPanel.appendMessageToEnd(ChatConversationPanel.java:378)
聽聽聽聽at net.java.sip.communicator.impl.gui.main.chat.ChatPanel.processMessage(ChatPanel.java:375)
聽聽聽聽at net.java.sip.communicator.impl.gui.main.contactlist.ContactListPanel.messageDelivered(ContactListPanel.java:288)
聽聽聽聽at net.java.sip.communicator.impl.protocol.rss.OperationSetBasicInstantMessagingRssImpl.fireMessageDelivered(OperationSetBasicInstantMessagingRssImpl.java:318)
聽聽聽聽at net.java.sip.communicator.impl.protocol.rss.OperationSetBasicInstantMessagingRssImpl.sendInstantMessage(OperationSetBasicInstantMessagingRssImpl.java:289)
聽聽聽聽at net.java.sip.communicator.impl.gui.main.chat.MetaContactChatPanel.sendMessage(MetaContactChatPanel.java:369)
聽聽聽聽at net.java.sip.communicator.impl.gui.main.chat.ChatSendPanel$1.run(ChatSendPanel.java:84)

I guess the best way to resolve the problem is to simply experiment and
try to discover the exact html tags that swing doesn't like.

Good luck!

Emil

Vincent Lucas wrote:

路路路

Hi devs, we need your help :slight_smile:

The last patch from Mihai (the identification patch) give me a little
bug that I can reproduce but Mihai can't.
That's why we need another tester to determine if this bug is only
related to my configuration or if it's a real bug.
For this you just need to:
1) Patch your SC with the identification patch from Mihai.
2) Remove your .sip-communicator directory.
3) Launch SC.
4) Add a Rss account.
5) Create a group.
6) Add a new buddy: http://news.google.com/nwshp?tab=wn&output=rss
7) Tell us if the chat windows display some text. For my part the chat
window appears but stay empty (the systray and the history work fine).

Thanks for your help,
Vincent

Mihai Balan wrote:

Hi Vincent,
A very quick glance the problem you're having with the Elsevier feeds
reveals the fact that you're trying to add a feed that uses the
feed:// protocol. Afaik the only protocols that ROME can deal with is
http:// ( https:// too?). By simply replacing feed:// with http:// in
the Elsevier feed adress everything works fine. Maybe some sort of
validation could be made upon adding a new RSS contact, but at the
moment, I have some other matters at hand.
I'm working on the user-requested update bug, I try to fix it asap :slight_smile:

All the best,
Mihai

On 8/16/07, Vincent Lucas <lucas@clarinet.u-strasbg.fr> wrote:

Hi Mihai,

It's a good work, feeds without date (i.e.
http://www.freenews.fr/feeds/rss.php) are working now :slight_smile:

But I experienced strange behavior with feeds with date, which were
previously working: i.e. http://news.google.com/nwshp?tab=wn&output=rss.
Try it with a new sip-communicator account, then create a Rss account,
and add the google feeds. You will see the systray showing that he
receives the news, then the ConversationPanel will appear but will stay
empty.
Nevertheless, when a new news appears it will NOT be well displayed
again if you have try to refresh it manually (sending a message) before.

By the way another bug appears with Elsevier feeds
(feed://www1.elsevier.com/homepage/sac/cfp/compsci_cfp.rss). I'm running
out of time to see it in details (Mihai can you have a look a it?).
See error dump following:
net.java.sip.communicator.service.contactlist.MetaContactListException:
failed to create
contactfeed://www1.elsevier.com/homepage/sac/cfp/compsci_cfp.rss
聽聽聽聽聽聽聽聽at
net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.addNewContactToMetaContact(MetaContactListServiceImpl.java:417)
聽聽聽聽聽聽聽聽at
net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.createMetaContact(MetaContactListServiceImpl.java:680)
聽聽聽聽聽聽聽聽at
net.java.sip.communicator.impl.gui.main.contactlist.addcontact.AddContactWizard$CreateContact.run(AddContactWizard.java:88)
Caused by: java.lang.IllegalArgumentException: failed to create a URL
for address feed://www1.elsevier.com/homepage/sac/cfp/compsci_cfp.rss.
Error was: Unknown protocol: feed
聽聽聽聽聽聽聽聽at
net.java.sip.communicator.impl.protocol.rss.OperationSetPersistentPresenceRssImpl.subscribe(OperationSetPersistentPresenceRssImpl.java:851)
聽聽聽聽聽聽聽聽at
net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.addNewContactToMetaContact(MetaContactListServiceImpl.java:393)
聽聽聽聽聽聽聽聽... 2 more

Cheers,
Vincent

Mihai Balan wrote:

Hi all,
After some messing around with the source code and some 4 rewrites,
the patch for uniquely identifying feeds is here and Exception-free (I
hope :smiley: ). Briefly the patch:
聽聽* provides an (almost) foolproof way of identifying feed items that
allows for correct retrieval even for feeds that don't have the date
set (or it's incorrectly (a.k.a. unreadable) set). As it was discussed
previously, it uses item date or item url. For the feeds I have tried
(GoogleNews RSS & ATOM, various Wordpress feeds, Dr.Dobb, PhD Comic,
etc :wink: ) it works quite fine :slight_smile:
聽聽* fixes a performance issue in the favicon retrieval code.
Previously each time the conversation window was displayed we would
download and convert the favicon.ico to something readable. Now, after
it's first retrieved, it's stored, so that subsequent calls to
ContactRssImpl.getImage() return the stored image.
聽聽* code and Javadoc cleanup/format/spell-check & corrections (Long
live Eclipse :smiley: ).

That would be all for now.

All the best,
Mihai

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

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@sip-communicator.dev.java.net
For additional commands, e-mail: dev-help@sip-communicator.dev.java.net

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@sip-communicator.dev.java.net
For additional commands, e-mail: dev-help@sip-communicator.dev.java.net

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@sip-communicator.dev.java.net
For additional commands, e-mail: dev-help@sip-communicator.dev.java.net

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@sip-communicator.dev.java.net
For additional commands, e-mail: dev-help@sip-communicator.dev.java.net

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@sip-communicator.dev.java.net
For additional commands, e-mail: dev-help@sip-communicator.dev.java.net


#6

Hi Emil, all,
I've done some more research on the subject and I hopefully fixed the
issues you mentioned here. The problem with the FOSDEM flow was in the
handling of the images that had the src attribute relative to the
server root. The current implementation interpreted (on Linux) such
addresses as local, hence the exceptions and over-all erratic
behaviour.

This version of the patch mainly adds the following:
聽聽* fixing manual refresh of feeds
聽聽* allows feed addresses to start with feed:// (Vincent's request :stuck_out_tongue: )
聽聽* fixes FOSDEM feeds
聽聽* fixes wordpress display of smilies by disabling smiley
processing/substitution for HTML. If this behaviour somewhat offends
you and/or makes a protocol work unexpected, just ping me and I'll try
to find an alternate way to fix things.

All the best,
Mihai

UniqueID-v3.patch (47.6 KB)

路路路

On 8/17/07, Emil Ivov <emcho@emcho.com> wrote:

Hi guys,

I just checked and I am seeing the exact same behavior as you describe:
I first get an empty window and then the updates succeed. There were no
exceptions in neither the console nor the log files.

Personally, I'd bet on another problem with the swing html renderer.

I had also noticed a similar problem with the FOSDEM RSS flow (
http://fosdem.org/2007/rss.xml ). It's there with or without the patch
and it looks a lot worse as it would sometimes even mess the way the
chat or history windows look. This one spits an exception in the logs
(though judging from how deep in swing it appears I don't think the
stacktrace would help a lot)

Here's what the exception looks like:

> java.lang.NullPointerException
> at javax.swing.text.FlowView.insertUpdate(FlowView.java:238)
> at javax.swing.text.View.forwardUpdateToView(View.java:1186)
> at javax.swing.text.View.forwardUpdate(View.java:1161)
> at javax.swing.text.BoxView.forwardUpdate(BoxView.java:223)
> at javax.swing.text.View.insertUpdate(View.java:685)
> at javax.swing.text.View.forwardUpdateToView(View.java:1186)
> at javax.swing.text.View.forwardUpdate(View.java:1161)
> at javax.swing.text.BoxView.forwardUpdate(BoxView.java:223)
> at javax.swing.text.View.insertUpdate(View.java:685)
> at javax.swing.plaf.basic.BasicTextUI$RootView.insertUpdate(BasicTextUI.java:1589)
> at javax.swing.plaf.basic.BasicTextUI$UpdateHandler.insertUpdate(BasicTextUI.java:1848)
> at javax.swing.text.AbstractDocument.fireInsertUpdate(AbstractDocument.java:185)
> at javax.swing.text.DefaultStyledDocument.insert(DefaultStyledDocument.java:206)
> at javax.swing.text.html.HTMLDocument.insert(HTMLDocument.java:242)
> at javax.swing.text.html.HTMLDocument$HTMLReader.flushBuffer(HTMLDocument.java:3325)
> at javax.swing.text.html.HTMLDocument$HTMLReader.flush(HTMLDocument.java:2133)
> at javax.swing.text.html.HTMLDocument.insertHTML(HTMLDocument.java:1160)
> at javax.swing.text.html.HTMLDocument.insertAfterEnd(HTMLDocument.java:989)
> at net.java.sip.communicator.impl.gui.main.chat.ChatConversationPanel.appendMessageToEnd(ChatConversationPanel.java:378)
> at net.java.sip.communicator.impl.gui.main.chat.ChatPanel.processMessage(ChatPanel.java:375)
> at net.java.sip.communicator.impl.gui.main.contactlist.ContactListPanel.messageDelivered(ContactListPanel.java:288)
> at net.java.sip.communicator.impl.protocol.rss.OperationSetBasicInstantMessagingRssImpl.fireMessageDelivered(OperationSetBasicInstantMessagingRssImpl.java:318)
> at net.java.sip.communicator.impl.protocol.rss.OperationSetBasicInstantMessagingRssImpl.sendInstantMessage(OperationSetBasicInstantMessagingRssImpl.java:289)
> at net.java.sip.communicator.impl.gui.main.chat.MetaContactChatPanel.sendMessage(MetaContactChatPanel.java:369)
> at net.java.sip.communicator.impl.gui.main.chat.ChatSendPanel$1.run(ChatSendPanel.java:84)

I guess the best way to resolve the problem is to simply experiment and
try to discover the exact html tags that swing doesn't like.

Good luck!

Emil

Vincent Lucas wrote:
> Hi devs, we need your help :slight_smile:
>
> The last patch from Mihai (the identification patch) give me a little
> bug that I can reproduce but Mihai can't.
> That's why we need another tester to determine if this bug is only
> related to my configuration or if it's a real bug.
> For this you just need to:
> 1) Patch your SC with the identification patch from Mihai.
> 2) Remove your .sip-communicator directory.
> 3) Launch SC.
> 4) Add a Rss account.
> 5) Create a group.
> 6) Add a new buddy: http://news.google.com/nwshp?tab=wn&output=rss
> 7) Tell us if the chat windows display some text. For my part the chat
> window appears but stay empty (the systray and the history work fine).
>
> Thanks for your help,
> Vincent
>
>
> Mihai Balan wrote:
>> Hi Vincent,
>> A very quick glance the problem you're having with the Elsevier feeds
>> reveals the fact that you're trying to add a feed that uses the
>> feed:// protocol. Afaik the only protocols that ROME can deal with is
>> http:// ( https:// too?). By simply replacing feed:// with http:// in
>> the Elsevier feed adress everything works fine. Maybe some sort of
>> validation could be made upon adding a new RSS contact, but at the
>> moment, I have some other matters at hand.
>> I'm working on the user-requested update bug, I try to fix it asap :slight_smile:
>>
>> All the best,
>> Mihai
>>
>> On 8/16/07, Vincent Lucas <lucas@clarinet.u-strasbg.fr> wrote:
>>> Hi Mihai,
>>>
>>> It's a good work, feeds without date (i.e.
>>> http://www.freenews.fr/feeds/rss.php) are working now :slight_smile:
>>>
>>> But I experienced strange behavior with feeds with date, which were
>>> previously working: i.e. http://news.google.com/nwshp?tab=wn&output=rss.
>>> Try it with a new sip-communicator account, then create a Rss account,
>>> and add the google feeds. You will see the systray showing that he
>>> receives the news, then the ConversationPanel will appear but will stay
>>> empty.
>>> Nevertheless, when a new news appears it will NOT be well displayed
>>> again if you have try to refresh it manually (sending a message) before.
>>>
>>> By the way another bug appears with Elsevier feeds
>>> (feed://www1.elsevier.com/homepage/sac/cfp/compsci_cfp.rss). I'm running
>>> out of time to see it in details (Mihai can you have a look a it?).
>>> See error dump following:
>>> net.java.sip.communicator.service.contactlist.MetaContactListException:
>>> failed to create
>>> contactfeed://www1.elsevier.com/homepage/sac/cfp/compsci_cfp.rss
>>> at
>>> net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.addNewContactToMetaContact(MetaContactListServiceImpl.java:417)
>>> at
>>> net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.createMetaContact(MetaContactListServiceImpl.java:680)
>>> at
>>> net.java.sip.communicator.impl.gui.main.contactlist.addcontact.AddContactWizard$CreateContact.run(AddContactWizard.java:88)
>>> Caused by: java.lang.IllegalArgumentException: failed to create a URL
>>> for address feed://www1.elsevier.com/homepage/sac/cfp/compsci_cfp.rss.
>>> Error was: Unknown protocol: feed
>>> at
>>> net.java.sip.communicator.impl.protocol.rss.OperationSetPersistentPresenceRssImpl.subscribe(OperationSetPersistentPresenceRssImpl.java:851)
>>> at
>>> net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.addNewContactToMetaContact(MetaContactListServiceImpl.java:393)
>>> ... 2 more
>>>
>>> Cheers,
>>> Vincent


#7

Hi, it's me again :smiley:
With the risk of becoming annoying, I send this mail too. I had a talk
with Emil and Yana so the previous patch got two little modifications:
聽聽* it seems that discarding smilies processing for HTML is not a
solution, so this part has been discarded from the patch. I'll work on
a different solution.
聽聽* the src attribute problem was solved in a different, more
general/elegant way.

A beautiful evening to you all,
Mihai

PS: Thus being said, please ignore the previous patch and consider
this one as the (I hope) final version :slight_smile:

UniqueID-v3.01.patch (47.5 KB)

路路路

This version of the patch mainly adds the following:
聽聽* fixing manual refresh of feeds
聽聽* allows feed addresses to start with feed:// (Vincent's request :stuck_out_tongue: )
聽聽* fixes FOSDEM feeds
聽聽* fixes wordpress display of smilies by disabling smiley
processing/substitution for HTML. If this behaviour somewhat offends
you and/or makes a protocol work unexpected, just ping me and I'll try
to find an alternate way to fix things.

All the best,
Mihai

On 8/17/07, Emil Ivov <emcho@emcho.com> wrote:
> Hi guys,
>
> I just checked and I am seeing the exact same behavior as you describe:
> I first get an empty window and then the updates succeed. There were no
> exceptions in neither the console nor the log files.
>
> Personally, I'd bet on another problem with the swing html renderer.
>
> I had also noticed a similar problem with the FOSDEM RSS flow (
> http://fosdem.org/2007/rss.xml ). It's there with or without the patch
> and it looks a lot worse as it would sometimes even mess the way the
> chat or history windows look. This one spits an exception in the logs
> (though judging from how deep in swing it appears I don't think the
> stacktrace would help a lot)
>
> Here's what the exception looks like:
>
> > java.lang.NullPointerException
> > at javax.swing.text.FlowView.insertUpdate(FlowView.java:238)
> > at javax.swing.text.View.forwardUpdateToView(View.java:1186)
> > at javax.swing.text.View.forwardUpdate(View.java:1161)
> > at javax.swing.text.BoxView.forwardUpdate(BoxView.java:223)
> > at javax.swing.text.View.insertUpdate(View.java:685)
> > at javax.swing.text.View.forwardUpdateToView(View.java:1186)
> > at javax.swing.text.View.forwardUpdate(View.java:1161)
> > at javax.swing.text.BoxView.forwardUpdate(BoxView.java:223)
> > at javax.swing.text.View.insertUpdate(View.java:685)
> > at javax.swing.plaf.basic.BasicTextUI$RootView.insertUpdate(BasicTextUI.java:1589)
> > at javax.swing.plaf.basic.BasicTextUI$UpdateHandler.insertUpdate(BasicTextUI.java:1848)
> > at javax.swing.text.AbstractDocument.fireInsertUpdate(AbstractDocument.java:185)
> > at javax.swing.text.DefaultStyledDocument.insert(DefaultStyledDocument.java:206)
> > at javax.swing.text.html.HTMLDocument.insert(HTMLDocument.java:242)
> > at javax.swing.text.html.HTMLDocument$HTMLReader.flushBuffer(HTMLDocument.java:3325)
> > at javax.swing.text.html.HTMLDocument$HTMLReader.flush(HTMLDocument.java:2133)
> > at javax.swing.text.html.HTMLDocument.insertHTML(HTMLDocument.java:1160)
> > at javax.swing.text.html.HTMLDocument.insertAfterEnd(HTMLDocument.java:989)
> > at net.java.sip.communicator.impl.gui.main.chat.ChatConversationPanel.appendMessageToEnd(ChatConversationPanel.java:378)
> > at net.java.sip.communicator.impl.gui.main.chat.ChatPanel.processMessage(ChatPanel.java:375)
> > at net.java.sip.communicator.impl.gui.main.contactlist.ContactListPanel.messageDelivered(ContactListPanel.java:288)
> > at net.java.sip.communicator.impl.protocol.rss.OperationSetBasicInstantMessagingRssImpl.fireMessageDelivered(OperationSetBasicInstantMessagingRssImpl.java:318)
> > at net.java.sip.communicator.impl.protocol.rss.OperationSetBasicInstantMessagingRssImpl.sendInstantMessage(OperationSetBasicInstantMessagingRssImpl.java:289)
> > at net.java.sip.communicator.impl.gui.main.chat.MetaContactChatPanel.sendMessage(MetaContactChatPanel.java:369)
> > at net.java.sip.communicator.impl.gui.main.chat.ChatSendPanel$1.run(ChatSendPanel.java:84)
>
> I guess the best way to resolve the problem is to simply experiment and
> try to discover the exact html tags that swing doesn't like.
>
> Good luck!
>
> Emil
>
> Vincent Lucas wrote:
> > Hi devs, we need your help :slight_smile:
> >
> > The last patch from Mihai (the identification patch) give me a little
> > bug that I can reproduce but Mihai can't.
> > That's why we need another tester to determine if this bug is only
> > related to my configuration or if it's a real bug.
> > For this you just need to:
> > 1) Patch your SC with the identification patch from Mihai.
> > 2) Remove your .sip-communicator directory.
> > 3) Launch SC.
> > 4) Add a Rss account.
> > 5) Create a group.
> > 6) Add a new buddy: http://news.google.com/nwshp?tab=wn&output=rss
> > 7) Tell us if the chat windows display some text. For my part the chat
> > window appears but stay empty (the systray and the history work fine).
> >
> > Thanks for your help,
> > Vincent
> >
> >
> > Mihai Balan wrote:
> >> Hi Vincent,
> >> A very quick glance the problem you're having with the Elsevier feeds
> >> reveals the fact that you're trying to add a feed that uses the
> >> feed:// protocol. Afaik the only protocols that ROME can deal with is
> >> http:// ( https:// too?). By simply replacing feed:// with http:// in
> >> the Elsevier feed adress everything works fine. Maybe some sort of
> >> validation could be made upon adding a new RSS contact, but at the
> >> moment, I have some other matters at hand.
> >> I'm working on the user-requested update bug, I try to fix it asap :slight_smile:
> >>
> >> All the best,
> >> Mihai
> >>
> >> On 8/16/07, Vincent Lucas <lucas@clarinet.u-strasbg.fr> wrote:
> >>> Hi Mihai,
> >>>
> >>> It's a good work, feeds without date (i.e.
> >>> http://www.freenews.fr/feeds/rss.php) are working now :slight_smile:
> >>>
> >>> But I experienced strange behavior with feeds with date, which were
> >>> previously working: i.e. http://news.google.com/nwshp?tab=wn&output=rss.
> >>> Try it with a new sip-communicator account, then create a Rss account,
> >>> and add the google feeds. You will see the systray showing that he
> >>> receives the news, then the ConversationPanel will appear but will stay
> >>> empty.
> >>> Nevertheless, when a new news appears it will NOT be well displayed
> >>> again if you have try to refresh it manually (sending a message) before.
> >>>
> >>> By the way another bug appears with Elsevier feeds
> >>> (feed://www1.elsevier.com/homepage/sac/cfp/compsci_cfp.rss). I'm running
> >>> out of time to see it in details (Mihai can you have a look a it?).
> >>> See error dump following:
> >>> net.java.sip.communicator.service.contactlist.MetaContactListException:
> >>> failed to create
> >>> contactfeed://www1.elsevier.com/homepage/sac/cfp/compsci_cfp.rss
> >>> at
> >>> net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.addNewContactToMetaContact(MetaContactListServiceImpl.java:417)
> >>> at
> >>> net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.createMetaContact(MetaContactListServiceImpl.java:680)
> >>> at
> >>> net.java.sip.communicator.impl.gui.main.contactlist.addcontact.AddContactWizard$CreateContact.run(AddContactWizard.java:88)
> >>> Caused by: java.lang.IllegalArgumentException: failed to create a URL
> >>> for address feed://www1.elsevier.com/homepage/sac/cfp/compsci_cfp.rss.
> >>> Error was: Unknown protocol: feed
> >>> at
> >>> net.java.sip.communicator.impl.protocol.rss.OperationSetPersistentPresenceRssImpl.subscribe(OperationSetPersistentPresenceRssImpl.java:851)
> >>> at
> >>> net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.addNewContactToMetaContact(MetaContactListServiceImpl.java:393)
> >>> ... 2 more
> >>>
> >>> Cheers,
> >>> Vincent


#8

Hi,
There's definitely something wrong with me today. The actual patch
file is attached below.

Sory for spamming the list O:-)

Best,
Mihai

UniqueID-v3.01.patch (46.7 KB)

路路路

On 8/17/07, Mihai Balan <mihai.balan@gmail.com> wrote:

Hi, it's me again :smiley:
With the risk of becoming annoying, I send this mail too. I had a talk
with Emil and Yana so the previous patch got two little modifications:
聽聽* it seems that discarding smilies processing for HTML is not a
solution, so this part has been discarded from the patch. I'll work on
a different solution.
聽聽* the src attribute problem was solved in a different, more
general/elegant way.

A beautiful evening to you all,
Mihai

PS: Thus being said, please ignore the previous patch and consider
this one as the (I hope) final version :slight_smile:

> This version of the patch mainly adds the following:
> * fixing manual refresh of feeds
> * allows feed addresses to start with feed:// (Vincent's request :stuck_out_tongue: )
> * fixes FOSDEM feeds
> * fixes wordpress display of smilies by disabling smiley
> processing/substitution for HTML. If this behaviour somewhat offends
> you and/or makes a protocol work unexpected, just ping me and I'll try
> to find an alternate way to fix things.
>
> All the best,
> Mihai
>
> On 8/17/07, Emil Ivov <emcho@emcho.com> wrote:
> > Hi guys,
> >
> > I just checked and I am seeing the exact same behavior as you describe:
> > I first get an empty window and then the updates succeed. There were no
> > exceptions in neither the console nor the log files.
> >
> > Personally, I'd bet on another problem with the swing html renderer.
> >
> > I had also noticed a similar problem with the FOSDEM RSS flow (
> > http://fosdem.org/2007/rss.xml ). It's there with or without the patch
> > and it looks a lot worse as it would sometimes even mess the way the
> > chat or history windows look. This one spits an exception in the logs
> > (though judging from how deep in swing it appears I don't think the
> > stacktrace would help a lot)
> >
> > Here's what the exception looks like:
> >
> > > java.lang.NullPointerException
> > > at javax.swing.text.FlowView.insertUpdate(FlowView.java:238)
> > > at javax.swing.text.View.forwardUpdateToView(View.java:1186)
> > > at javax.swing.text.View.forwardUpdate(View.java:1161)
> > > at javax.swing.text.BoxView.forwardUpdate(BoxView.java:223)
> > > at javax.swing.text.View.insertUpdate(View.java:685)
> > > at javax.swing.text.View.forwardUpdateToView(View.java:1186)
> > > at javax.swing.text.View.forwardUpdate(View.java:1161)
> > > at javax.swing.text.BoxView.forwardUpdate(BoxView.java:223)
> > > at javax.swing.text.View.insertUpdate(View.java:685)
> > > at javax.swing.plaf.basic.BasicTextUI$RootView.insertUpdate(BasicTextUI.java:1589)
> > > at javax.swing.plaf.basic.BasicTextUI$UpdateHandler.insertUpdate(BasicTextUI.java:1848)
> > > at javax.swing.text.AbstractDocument.fireInsertUpdate(AbstractDocument.java:185)
> > > at javax.swing.text.DefaultStyledDocument.insert(DefaultStyledDocument.java:206)
> > > at javax.swing.text.html.HTMLDocument.insert(HTMLDocument.java:242)
> > > at javax.swing.text.html.HTMLDocument$HTMLReader.flushBuffer(HTMLDocument.java:3325)
> > > at javax.swing.text.html.HTMLDocument$HTMLReader.flush(HTMLDocument.java:2133)
> > > at javax.swing.text.html.HTMLDocument.insertHTML(HTMLDocument.java:1160)
> > > at javax.swing.text.html.HTMLDocument.insertAfterEnd(HTMLDocument.java:989)
> > > at net.java.sip.communicator.impl.gui.main.chat.ChatConversationPanel.appendMessageToEnd(ChatConversationPanel.java:378)
> > > at net.java.sip.communicator.impl.gui.main.chat.ChatPanel.processMessage(ChatPanel.java:375)
> > > at net.java.sip.communicator.impl.gui.main.contactlist.ContactListPanel.messageDelivered(ContactListPanel.java:288)
> > > at net.java.sip.communicator.impl.protocol.rss.OperationSetBasicInstantMessagingRssImpl.fireMessageDelivered(OperationSetBasicInstantMessagingRssImpl.java:318)
> > > at net.java.sip.communicator.impl.protocol.rss.OperationSetBasicInstantMessagingRssImpl.sendInstantMessage(OperationSetBasicInstantMessagingRssImpl.java:289)
> > > at net.java.sip.communicator.impl.gui.main.chat.MetaContactChatPanel.sendMessage(MetaContactChatPanel.java:369)
> > > at net.java.sip.communicator.impl.gui.main.chat.ChatSendPanel$1.run(ChatSendPanel.java:84)
> >
> > I guess the best way to resolve the problem is to simply experiment and
> > try to discover the exact html tags that swing doesn't like.
> >
> > Good luck!
> >
> > Emil
> >
> > Vincent Lucas wrote:
> > > Hi devs, we need your help :slight_smile:
> > >
> > > The last patch from Mihai (the identification patch) give me a little
> > > bug that I can reproduce but Mihai can't.
> > > That's why we need another tester to determine if this bug is only
> > > related to my configuration or if it's a real bug.
> > > For this you just need to:
> > > 1) Patch your SC with the identification patch from Mihai.
> > > 2) Remove your .sip-communicator directory.
> > > 3) Launch SC.
> > > 4) Add a Rss account.
> > > 5) Create a group.
> > > 6) Add a new buddy: http://news.google.com/nwshp?tab=wn&output=rss
> > > 7) Tell us if the chat windows display some text. For my part the chat
> > > window appears but stay empty (the systray and the history work fine).
> > >
> > > Thanks for your help,
> > > Vincent
> > >
> > >
> > > Mihai Balan wrote:
> > >> Hi Vincent,
> > >> A very quick glance the problem you're having with the Elsevier feeds
> > >> reveals the fact that you're trying to add a feed that uses the
> > >> feed:// protocol. Afaik the only protocols that ROME can deal with is
> > >> http:// ( https:// too?). By simply replacing feed:// with http:// in
> > >> the Elsevier feed adress everything works fine. Maybe some sort of
> > >> validation could be made upon adding a new RSS contact, but at the
> > >> moment, I have some other matters at hand.
> > >> I'm working on the user-requested update bug, I try to fix it asap :slight_smile:
> > >>
> > >> All the best,
> > >> Mihai
> > >>
> > >> On 8/16/07, Vincent Lucas <lucas@clarinet.u-strasbg.fr> wrote:
> > >>> Hi Mihai,
> > >>>
> > >>> It's a good work, feeds without date (i.e.
> > >>> http://www.freenews.fr/feeds/rss.php) are working now :slight_smile:
> > >>>
> > >>> But I experienced strange behavior with feeds with date, which were
> > >>> previously working: i.e. http://news.google.com/nwshp?tab=wn&output=rss.
> > >>> Try it with a new sip-communicator account, then create a Rss account,
> > >>> and add the google feeds. You will see the systray showing that he
> > >>> receives the news, then the ConversationPanel will appear but will stay
> > >>> empty.
> > >>> Nevertheless, when a new news appears it will NOT be well displayed
> > >>> again if you have try to refresh it manually (sending a message) before.
> > >>>
> > >>> By the way another bug appears with Elsevier feeds
> > >>> (feed://www1.elsevier.com/homepage/sac/cfp/compsci_cfp.rss). I'm running
> > >>> out of time to see it in details (Mihai can you have a look a it?).
> > >>> See error dump following:
> > >>> net.java.sip.communicator.service.contactlist.MetaContactListException:
> > >>> failed to create
> > >>> contactfeed://www1.elsevier.com/homepage/sac/cfp/compsci_cfp.rss
> > >>> at
> > >>> net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.addNewContactToMetaContact(MetaContactListServiceImpl.java:417)
> > >>> at
> > >>> net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.createMetaContact(MetaContactListServiceImpl.java:680)
> > >>> at
> > >>> net.java.sip.communicator.impl.gui.main.contactlist.addcontact.AddContactWizard$CreateContact.run(AddContactWizard.java:88)
> > >>> Caused by: java.lang.IllegalArgumentException: failed to create a URL
> > >>> for address feed://www1.elsevier.com/homepage/sac/cfp/compsci_cfp.rss.
> > >>> Error was: Unknown protocol: feed
> > >>> at
> > >>> net.java.sip.communicator.impl.protocol.rss.OperationSetPersistentPresenceRssImpl.subscribe(OperationSetPersistentPresenceRssImpl.java:851)
> > >>> at
> > >>> net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.addNewContactToMetaContact(MetaContactListServiceImpl.java:393)
> > >>> ... 2 more
> > >>>
> > >>> Cheers,
> > >>> Vincent
>
>


#9

Hi Mihai and devs,

Today, Yana has patched the "display bug" of the chat panel (when the first message was too big). That's give me the opportunity to try your patch "UniqueID-v3.01.patch" (the last you send under this name).
I just not have patched the "src/net/java/sip/communicator/impl/gui/utils/SIPCommImageView.java" which was conflicting with the last version available (is this part f the patch really necessery? For what usage have been it designed?).

First, it's a great progress for RSS readers:
- No more old-feeds poping-up at each beginning of sip-communicator.
- The new ID management is a important improvement. This way SC si able to deal with most of the feeds I tried (All reports of unreadable feeds are welcome :wink: ).

Second, there's still some little bugs:
- When adding feeds beginning with "fedd://", the feed is displayed but not saved. When restarting SC the feed is no more in the buddy list. See dump #1 for feed "feed://www1.elsevier.com/homepage/sac/cfp/compsci_cfp.rss".
- Dump #2 and #3 are bugs appearing (pop-up error) when adding feeds, but I don't manage to reproduce it twice.

That's all for that report.

Hope this helps,
Vincent

PS: I have try your feed "http://gramos.wordpress.com/comments/feed" without your smiley-patch and haven't have a problem (smiley are well displayed). Can you send me an HTML exemple which can reproduce the smiley bug?

List of dumps:
-- dump #1
net.java.sip.communicator.service.contactlist.MetaContactListException: Failed to create a contact with address: feed://www1.elsevier.com/homepage/sac/cfp/compsci_cfp.rss
聽聽at net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.addNewContactToMetaContact(MetaContactListServiceImpl.java:433)
聽聽at net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.createMetaContact(MetaContactListServiceImpl.java:680)
聽聽at net.java.sip.communicator.impl.gui.main.contactlist.addcontact.AddContactWizard$CreateContact.run(AddContactWizard.java:88)

--dump #2
net.java.sip.communicator.service.contactlist.MetaContactListException: failed to create contacthttp://www.pheedo.com/f/drdobbs_all_articles
聽聽at net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.addNewContactToMetaContact(MetaContactListServiceImpl.java:417)
聽聽at net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.createMetaContact(MetaContactListServiceImpl.java:680)
聽聽at net.java.sip.communicator.impl.gui.main.contactlist.addcontact.AddContactWizard$CreateContact.run(AddContactWizard.java:88)
Caused by: java.lang.NullPointerException
聽聽at net.java.sip.communicator.impl.gui.main.chat.ProtocolContactSelectorBox.updateContactStatus(ProtocolContactSelectorBox.java:187)
聽聽at net.java.sip.communicator.impl.gui.main.chat.MetaContactChatPanel.contactPresenceStatusChanged(MetaContactChatPanel.java:303)
聽聽at net.java.sip.communicator.impl.protocol.rss.OperationSetPersistentPresenceRssImpl.fireContactPresenceStatusChangeEvent(OperationSetPersistentPresenceRssImpl.java:141)
聽聽at net.java.sip.communicator.impl.protocol.rss.OperationSetPersistentPresenceRssImpl.changePresenceStatusForContact(OperationSetPersistentPresenceRssImpl.java:637)
聽聽at net.java.sip.communicator.impl.protocol.rss.OperationSetPersistentPresenceRssImpl.subscribe(OperationSetPersistentPresenceRssImpl.java:884)
聽聽at net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.addNewContactToMetaContact(MetaContactListServiceImpl.java:393)
聽聽... 2 more

-- dump #3
net.java.sip.communicator.service.contactlist.MetaContactListException: failed to create contacthttp://feeds.reuters.com/reuters/topNews/
聽聽at net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.addNewContactToMetaContact(MetaContactListServiceImpl.java:417)
聽聽at net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.createMetaContact(MetaContactListServiceImpl.java:680)
聽聽at net.java.sip.communicator.impl.gui.main.contactlist.addcontact.AddContactWizard$CreateContact.run(AddContactWizard.java:88)
Caused by: java.lang.NullPointerException
聽聽at net.java.sip.communicator.impl.gui.main.chat.ProtocolContactSelectorBox.updateContactStatus(ProtocolContactSelectorBox.java:187)
聽聽at net.java.sip.communicator.impl.gui.main.chat.MetaContactChatPanel.contactPresenceStatusChanged(MetaContactChatPanel.java:303)
聽聽at net.java.sip.communicator.impl.protocol.rss.OperationSetPersistentPresenceRssImpl.fireContactPresenceStatusChangeEvent(OperationSetPersistentPresenceRssImpl.java:141)
聽聽at net.java.sip.communicator.impl.protocol.rss.OperationSetPersistentPresenceRssImpl.changePresenceStatusForContact(OperationSetPersistentPresenceRssImpl.java:637)
聽聽at net.java.sip.communicator.impl.protocol.rss.OperationSetPersistentPresenceRssImpl.subscribe(OperationSetPersistentPresenceRssImpl.java:884)
聽聽at net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.addNewContactToMetaContact(MetaContactListServiceImpl.java:393)
聽聽... 2 more

Mihai Balan wrote:

路路路

Hi,
There's definitely something wrong with me today. The actual patch
file is attached below.

Sory for spamming the list O:-)

Best,
Mihai

On 8/17/07, Mihai Balan <mihai.balan@gmail.com> wrote:

Hi, it's me again :smiley:
With the risk of becoming annoying, I send this mail too. I had a talk
with Emil and Yana so the previous patch got two little modifications:
聽聽* it seems that discarding smilies processing for HTML is not a
solution, so this part has been discarded from the patch. I'll work on
a different solution.
聽聽* the src attribute problem was solved in a different, more
general/elegant way.

A beautiful evening to you all,
Mihai

PS: Thus being said, please ignore the previous patch and consider
this one as the (I hope) final version :slight_smile:

This version of the patch mainly adds the following:
聽聽* fixing manual refresh of feeds
聽聽* allows feed addresses to start with feed:// (Vincent's request :stuck_out_tongue: )
聽聽* fixes FOSDEM feeds
聽聽* fixes wordpress display of smilies by disabling smiley
processing/substitution for HTML. If this behaviour somewhat offends
you and/or makes a protocol work unexpected, just ping me and I'll try
to find an alternate way to fix things.

All the best,
Mihai

On 8/17/07, Emil Ivov <emcho@emcho.com> wrote:

Hi guys,

I just checked and I am seeing the exact same behavior as you describe:
I first get an empty window and then the updates succeed. There were no
exceptions in neither the console nor the log files.

Personally, I'd bet on another problem with the swing html renderer.

I had also noticed a similar problem with the FOSDEM RSS flow (
http://fosdem.org/2007/rss.xml ). It's there with or without the patch
and it looks a lot worse as it would sometimes even mess the way the
chat or history windows look. This one spits an exception in the logs
(though judging from how deep in swing it appears I don't think the
stacktrace would help a lot)

Here's what the exception looks like:

java.lang.NullPointerException
聽聽聽聽at javax.swing.text.FlowView.insertUpdate(FlowView.java:238)
聽聽聽聽at javax.swing.text.View.forwardUpdateToView(View.java:1186)
聽聽聽聽at javax.swing.text.View.forwardUpdate(View.java:1161)
聽聽聽聽at javax.swing.text.BoxView.forwardUpdate(BoxView.java:223)
聽聽聽聽at javax.swing.text.View.insertUpdate(View.java:685)
聽聽聽聽at javax.swing.text.View.forwardUpdateToView(View.java:1186)
聽聽聽聽at javax.swing.text.View.forwardUpdate(View.java:1161)
聽聽聽聽at javax.swing.text.BoxView.forwardUpdate(BoxView.java:223)
聽聽聽聽at javax.swing.text.View.insertUpdate(View.java:685)
聽聽聽聽at javax.swing.plaf.basic.BasicTextUI$RootView.insertUpdate(BasicTextUI.java:1589)
聽聽聽聽at javax.swing.plaf.basic.BasicTextUI$UpdateHandler.insertUpdate(BasicTextUI.java:1848)
聽聽聽聽at javax.swing.text.AbstractDocument.fireInsertUpdate(AbstractDocument.java:185)
聽聽聽聽at javax.swing.text.DefaultStyledDocument.insert(DefaultStyledDocument.java:206)
聽聽聽聽at javax.swing.text.html.HTMLDocument.insert(HTMLDocument.java:242)
聽聽聽聽at javax.swing.text.html.HTMLDocument$HTMLReader.flushBuffer(HTMLDocument.java:3325)
聽聽聽聽at javax.swing.text.html.HTMLDocument$HTMLReader.flush(HTMLDocument.java:2133)
聽聽聽聽at javax.swing.text.html.HTMLDocument.insertHTML(HTMLDocument.java:1160)
聽聽聽聽at javax.swing.text.html.HTMLDocument.insertAfterEnd(HTMLDocument.java:989)
聽聽聽聽at net.java.sip.communicator.impl.gui.main.chat.ChatConversationPanel.appendMessageToEnd(ChatConversationPanel.java:378)
聽聽聽聽at net.java.sip.communicator.impl.gui.main.chat.ChatPanel.processMessage(ChatPanel.java:375)
聽聽聽聽at net.java.sip.communicator.impl.gui.main.contactlist.ContactListPanel.messageDelivered(ContactListPanel.java:288)
聽聽聽聽at net.java.sip.communicator.impl.protocol.rss.OperationSetBasicInstantMessagingRssImpl.fireMessageDelivered(OperationSetBasicInstantMessagingRssImpl.java:318)
聽聽聽聽at net.java.sip.communicator.impl.protocol.rss.OperationSetBasicInstantMessagingRssImpl.sendInstantMessage(OperationSetBasicInstantMessagingRssImpl.java:289)
聽聽聽聽at net.java.sip.communicator.impl.gui.main.chat.MetaContactChatPanel.sendMessage(MetaContactChatPanel.java:369)
聽聽聽聽at net.java.sip.communicator.impl.gui.main.chat.ChatSendPanel$1.run(ChatSendPanel.java:84)

I guess the best way to resolve the problem is to simply experiment and
try to discover the exact html tags that swing doesn't like.

Good luck!

Emil

Vincent Lucas wrote:

Hi devs, we need your help :slight_smile:

The last patch from Mihai (the identification patch) give me a little
bug that I can reproduce but Mihai can't.
That's why we need another tester to determine if this bug is only
related to my configuration or if it's a real bug.
For this you just need to:
1) Patch your SC with the identification patch from Mihai.
2) Remove your .sip-communicator directory.
3) Launch SC.
4) Add a Rss account.
5) Create a group.
6) Add a new buddy: http://news.google.com/nwshp?tab=wn&output=rss
7) Tell us if the chat windows display some text. For my part the chat
window appears but stay empty (the systray and the history work fine).

Thanks for your help,
Vincent

Mihai Balan wrote:

Hi Vincent,
A very quick glance the problem you're having with the Elsevier feeds
reveals the fact that you're trying to add a feed that uses the
feed:// protocol. Afaik the only protocols that ROME can deal with is
http:// ( https:// too?). By simply replacing feed:// with http:// in
the Elsevier feed adress everything works fine. Maybe some sort of
validation could be made upon adding a new RSS contact, but at the
moment, I have some other matters at hand.
I'm working on the user-requested update bug, I try to fix it asap :slight_smile:

All the best,
Mihai

On 8/16/07, Vincent Lucas <lucas@clarinet.u-strasbg.fr> wrote:

Hi Mihai,

It's a good work, feeds without date (i.e.
http://www.freenews.fr/feeds/rss.php) are working now :slight_smile:

But I experienced strange behavior with feeds with date, which were
previously working: i.e. http://news.google.com/nwshp?tab=wn&output=rss.
Try it with a new sip-communicator account, then create a Rss account,
and add the google feeds. You will see the systray showing that he
receives the news, then the ConversationPanel will appear but will stay
empty.
Nevertheless, when a new news appears it will NOT be well displayed
again if you have try to refresh it manually (sending a message) before.

By the way another bug appears with Elsevier feeds
(feed://www1.elsevier.com/homepage/sac/cfp/compsci_cfp.rss). I'm running
out of time to see it in details (Mihai can you have a look a it?).
See error dump following:
net.java.sip.communicator.service.contactlist.MetaContactListException:
failed to create
contactfeed://www1.elsevier.com/homepage/sac/cfp/compsci_cfp.rss
聽聽聽聽聽聽聽聽at
net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.addNewContactToMetaContact(MetaContactListServiceImpl.java:417)
聽聽聽聽聽聽聽聽at
net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.createMetaContact(MetaContactListServiceImpl.java:680)
聽聽聽聽聽聽聽聽at
net.java.sip.communicator.impl.gui.main.contactlist.addcontact.AddContactWizard$CreateContact.run(AddContactWizard.java:88)
Caused by: java.lang.IllegalArgumentException: failed to create a URL
for address feed://www1.elsevier.com/homepage/sac/cfp/compsci_cfp.rss.
Error was: Unknown protocol: feed
聽聽聽聽聽聽聽聽at
net.java.sip.communicator.impl.protocol.rss.OperationSetPersistentPresenceRssImpl.subscribe(OperationSetPersistentPresenceRssImpl.java:851)
聽聽聽聽聽聽聽聽at
net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.addNewContactToMetaContact(MetaContactListServiceImpl.java:393)
聽聽聽聽聽聽聽聽... 2 more

Cheers,
Vincent

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

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@sip-communicator.dev.java.net
For additional commands, e-mail: dev-help@sip-communicator.dev.java.net

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@sip-communicator.dev.java.net
For additional commands, e-mail: dev-help@sip-communicator.dev.java.net


#10

Hi,

there is some problem with rss that don't have dates. I add such rss and everything is ok. But when I stop and start
sip-communicator and try to refresh the feed by sending something in message dialog I got an exception :

聽聽聽聽聽[java] java.lang.NullPointerException
聽聽聽聽聽[java] at net.java.sip.communicator.impl.protocol.rss.RssFeedReader.feedToString(RssFeedReader.java:139)
聽聽聽聽聽[java] at net.java.sip.communicator.impl.protocol.rss.OperationSetBasicInstantMessagingRssImpl.submitRssQuery(OperationSetBasicInstantMessagingRssImpl.java:179)
聽聽聽聽聽[java] at net.java.sip.communicator.impl.protocol.rss.OperationSetBasicInstantMessagingRssImpl.refreshRssFeed(OperationSetBasicInstantMessagingRssImpl.java:244)
聽聽聽聽聽[java] at net.java.sip.communicator.impl.protocol.rss.RssThread.run(RssThread.java:45)

I am testing with this : http://linux-bg.org/linux-bg-forum.rdf

damencho

Mihai Balan wrote:

路路路

Hi,
There's definitely something wrong with me today. The actual patch
file is attached below.

Sory for spamming the list O:-)

Best,
Mihai

On 8/17/07, Mihai Balan <mihai.balan@gmail.com> wrote:
聽聽

Hi, it's me again :smiley:
With the risk of becoming annoying, I send this mail too. I had a talk
with Emil and Yana so the previous patch got two little modifications:
聽聽* it seems that discarding smilies processing for HTML is not a
solution, so this part has been discarded from the patch. I'll work on
a different solution.
聽聽* the src attribute problem was solved in a different, more
general/elegant way.

A beautiful evening to you all,
Mihai

PS: Thus being said, please ignore the previous patch and consider
this one as the (I hope) final version :slight_smile:

This version of the patch mainly adds the following:
聽聽* fixing manual refresh of feeds
聽聽* allows feed addresses to start with feed:// (Vincent's request :stuck_out_tongue: )
聽聽* fixes FOSDEM feeds
聽聽* fixes wordpress display of smilies by disabling smiley
processing/substitution for HTML. If this behaviour somewhat offends
you and/or makes a protocol work unexpected, just ping me and I'll try
to find an alternate way to fix things.

All the best,
Mihai

On 8/17/07, Emil Ivov <emcho@emcho.com> wrote:
聽聽聽聽聽聽

Hi guys,

I just checked and I am seeing the exact same behavior as you describe:
I first get an empty window and then the updates succeed. There were no
exceptions in neither the console nor the log files.

Personally, I'd bet on another problem with the swing html renderer.

I had also noticed a similar problem with the FOSDEM RSS flow (
http://fosdem.org/2007/rss.xml ). It's there with or without the patch
and it looks a lot worse as it would sometimes even mess the way the
chat or history windows look. This one spits an exception in the logs
(though judging from how deep in swing it appears I don't think the
stacktrace would help a lot)

Here's what the exception looks like:

java.lang.NullPointerException
聽聽聽聽at javax.swing.text.FlowView.insertUpdate(FlowView.java:238)
聽聽聽聽at javax.swing.text.View.forwardUpdateToView(View.java:1186)
聽聽聽聽at javax.swing.text.View.forwardUpdate(View.java:1161)
聽聽聽聽at javax.swing.text.BoxView.forwardUpdate(BoxView.java:223)
聽聽聽聽at javax.swing.text.View.insertUpdate(View.java:685)
聽聽聽聽at javax.swing.text.View.forwardUpdateToView(View.java:1186)
聽聽聽聽at javax.swing.text.View.forwardUpdate(View.java:1161)
聽聽聽聽at javax.swing.text.BoxView.forwardUpdate(BoxView.java:223)
聽聽聽聽at javax.swing.text.View.insertUpdate(View.java:685)
聽聽聽聽at javax.swing.plaf.basic.BasicTextUI$RootView.insertUpdate(BasicTextUI.java:1589)
聽聽聽聽at javax.swing.plaf.basic.BasicTextUI$UpdateHandler.insertUpdate(BasicTextUI.java:1848)
聽聽聽聽at javax.swing.text.AbstractDocument.fireInsertUpdate(AbstractDocument.java:185)
聽聽聽聽at javax.swing.text.DefaultStyledDocument.insert(DefaultStyledDocument.java:206)
聽聽聽聽at javax.swing.text.html.HTMLDocument.insert(HTMLDocument.java:242)
聽聽聽聽at javax.swing.text.html.HTMLDocument$HTMLReader.flushBuffer(HTMLDocument.java:3325)
聽聽聽聽at javax.swing.text.html.HTMLDocument$HTMLReader.flush(HTMLDocument.java:2133)
聽聽聽聽at javax.swing.text.html.HTMLDocument.insertHTML(HTMLDocument.java:1160)
聽聽聽聽at javax.swing.text.html.HTMLDocument.insertAfterEnd(HTMLDocument.java:989)
聽聽聽聽at net.java.sip.communicator.impl.gui.main.chat.ChatConversationPanel.appendMessageToEnd(ChatConversationPanel.java:378)
聽聽聽聽at net.java.sip.communicator.impl.gui.main.chat.ChatPanel.processMessage(ChatPanel.java:375)
聽聽聽聽at net.java.sip.communicator.impl.gui.main.contactlist.ContactListPanel.messageDelivered(ContactListPanel.java:288)
聽聽聽聽at net.java.sip.communicator.impl.protocol.rss.OperationSetBasicInstantMessagingRssImpl.fireMessageDelivered(OperationSetBasicInstantMessagingRssImpl.java:318)
聽聽聽聽at net.java.sip.communicator.impl.protocol.rss.OperationSetBasicInstantMessagingRssImpl.sendInstantMessage(OperationSetBasicInstantMessagingRssImpl.java:289)
聽聽聽聽at net.java.sip.communicator.impl.gui.main.chat.MetaContactChatPanel.sendMessage(MetaContactChatPanel.java:369)
聽聽聽聽at net.java.sip.communicator.impl.gui.main.chat.ChatSendPanel$1.run(ChatSendPanel.java:84)
聽聽聽聽聽聽聽聽聽聽

I guess the best way to resolve the problem is to simply experiment and
try to discover the exact html tags that swing doesn't like.

Good luck!

Emil

Vincent Lucas wrote:
聽聽聽聽聽聽聽聽

Hi devs, we need your help :slight_smile:

The last patch from Mihai (the identification patch) give me a little
bug that I can reproduce but Mihai can't.
That's why we need another tester to determine if this bug is only
related to my configuration or if it's a real bug.
For this you just need to:
1) Patch your SC with the identification patch from Mihai.
2) Remove your .sip-communicator directory.
3) Launch SC.
4) Add a Rss account.
5) Create a group.
6) Add a new buddy: http://news.google.com/nwshp?tab=wn&output=rss
7) Tell us if the chat windows display some text. For my part the chat
window appears but stay empty (the systray and the history work fine).

Thanks for your help,
Vincent

Mihai Balan wrote:
聽聽聽聽聽聽聽聽聽聽

Hi Vincent,
A very quick glance the problem you're having with the Elsevier feeds
reveals the fact that you're trying to add a feed that uses the
feed:// protocol. Afaik the only protocols that ROME can deal with is
http:// ( https:// too?). By simply replacing feed:// with http:// in
the Elsevier feed adress everything works fine. Maybe some sort of
validation could be made upon adding a new RSS contact, but at the
moment, I have some other matters at hand.
I'm working on the user-requested update bug, I try to fix it asap :slight_smile:

All the best,
Mihai

On 8/16/07, Vincent Lucas <lucas@clarinet.u-strasbg.fr> wrote:
聽聽聽聽聽聽聽聽聽聽聽聽

Hi Mihai,

It's a good work, feeds without date (i.e.
http://www.freenews.fr/feeds/rss.php) are working now :slight_smile:

But I experienced strange behavior with feeds with date, which were
previously working: i.e. http://news.google.com/nwshp?tab=wn&output=rss.
Try it with a new sip-communicator account, then create a Rss account,
and add the google feeds. You will see the systray showing that he
receives the news, then the ConversationPanel will appear but will stay
empty.
Nevertheless, when a new news appears it will NOT be well displayed
again if you have try to refresh it manually (sending a message) before.

By the way another bug appears with Elsevier feeds
(feed://www1.elsevier.com/homepage/sac/cfp/compsci_cfp.rss). I'm running
out of time to see it in details (Mihai can you have a look a it?).
See error dump following:
net.java.sip.communicator.service.contactlist.MetaContactListException:
failed to create
contactfeed://www1.elsevier.com/homepage/sac/cfp/compsci_cfp.rss
聽聽聽聽聽聽聽聽at
net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.addNewContactToMetaContact(MetaContactListServiceImpl.java:417)
聽聽聽聽聽聽聽聽at
net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.createMetaContact(MetaContactListServiceImpl.java:680)
聽聽聽聽聽聽聽聽at
net.java.sip.communicator.impl.gui.main.contactlist.addcontact.AddContactWizard$CreateContact.run(AddContactWizard.java:88)
Caused by: java.lang.IllegalArgumentException: failed to create a URL
for address feed://www1.elsevier.com/homepage/sac/cfp/compsci_cfp.rss.
Error was: Unknown protocol: feed
聽聽聽聽聽聽聽聽at
net.java.sip.communicator.impl.protocol.rss.OperationSetPersistentPresenceRssImpl.subscribe(OperationSetPersistentPresenceRssImpl.java:851)
聽聽聽聽聽聽聽聽at
net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.addNewContactToMetaContact(MetaContactListServiceImpl.java:393)
聽聽聽聽聽聽聽聽... 2 more

Cheers,
Vincent
聽聽聽聽聽聽聽聽聽聽聽聽聽聽

聽聽聽聽------------------------------------------------------------------------

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@sip-communicator.dev.java.net
For additional commands, e-mail: dev-help@sip-communicator.dev.java.net

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@sip-communicator.dev.java.net
For additional commands, e-mail: dev-help@sip-communicator.dev.java.net


#11

Hi Vincent and Mihai,

Vincent, actually the conflict came from the fact that I've already applied and committed the SIPCommImageView fixes. Sorry for this inconvenience.

Mihai, thanks for the patches. And also for all fixes that you have provided so far for the ChatConversationPanel. You really helped us to improve the chat!

So on the question now:) I have applied the two patches. The one for the smilies and the item identification one. There are two minor changes that I have made. I have removed the RSSItemKey class from the RSSFeedReader file in a separate file, because I think it's a little confusing for someone who looks at the code (someone like me:). I have also replaced the StringBuilder in the RSSItemKey.toString with StringBuffer (unfortunately the StringBuilder is only in JDK1.5).

One more thing. The white-window-bug was not so simple as I expected, because of some other constraints, so it need a good testing. Let me know if something go wrong again.

Thanks again Mihai!

Yana

Vincent Lucas wrote:

路路路

Hi Mihai and devs,

Today, Yana has patched the "display bug" of the chat panel (when the first message was too big). That's give me the opportunity to try your patch "UniqueID-v3.01.patch" (the last you send under this name).
I just not have patched the "src/net/java/sip/communicator/impl/gui/utils/SIPCommImageView.java" which was conflicting with the last version available (is this part f the patch really necessery? For what usage have been it designed?).

First, it's a great progress for RSS readers:
- No more old-feeds poping-up at each beginning of sip-communicator.
- The new ID management is a important improvement. This way SC si able to deal with most of the feeds I tried (All reports of unreadable feeds are welcome :wink: ).

Second, there's still some little bugs:
- When adding feeds beginning with "fedd://", the feed is displayed but not saved. When restarting SC the feed is no more in the buddy list. See dump #1 for feed "feed://www1.elsevier.com/homepage/sac/cfp/compsci_cfp.rss".
- Dump #2 and #3 are bugs appearing (pop-up error) when adding feeds, but I don't manage to reproduce it twice.

That's all for that report.

Hope this helps,
Vincent

PS: I have try your feed "http://gramos.wordpress.com/comments/feed" without your smiley-patch and haven't have a problem (smiley are well displayed). Can you send me an HTML exemple which can reproduce the smiley bug?

List of dumps:
-- dump #1
net.java.sip.communicator.service.contactlist.MetaContactListException: Failed to create a contact with address: feed://www1.elsevier.com/homepage/sac/cfp/compsci_cfp.rss
聽聽聽聽at net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.addNewContactToMetaContact(MetaContactListServiceImpl.java:433)

聽聽聽聽at net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.createMetaContact(MetaContactListServiceImpl.java:680)

聽聽聽聽at net.java.sip.communicator.impl.gui.main.contactlist.addcontact.AddContactWizard$CreateContact.run(AddContactWizard.java:88)

--dump #2
net.java.sip.communicator.service.contactlist.MetaContactListException: failed to create contacthttp://www.pheedo.com/f/drdobbs_all_articles
聽聽聽聽at net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.addNewContactToMetaContact(MetaContactListServiceImpl.java:417)

聽聽聽聽at net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.createMetaContact(MetaContactListServiceImpl.java:680)

聽聽聽聽at net.java.sip.communicator.impl.gui.main.contactlist.addcontact.AddContactWizard$CreateContact.run(AddContactWizard.java:88)

Caused by: java.lang.NullPointerException
聽聽聽聽at net.java.sip.communicator.impl.gui.main.chat.ProtocolContactSelectorBox.updateContactStatus(ProtocolContactSelectorBox.java:187)

聽聽聽聽at net.java.sip.communicator.impl.gui.main.chat.MetaContactChatPanel.contactPresenceStatusChanged(MetaContactChatPanel.java:303)

聽聽聽聽at net.java.sip.communicator.impl.protocol.rss.OperationSetPersistentPresenceRssImpl.fireContactPresenceStatusChangeEvent(OperationSetPersistentPresenceRssImpl.java:141)

聽聽聽聽at net.java.sip.communicator.impl.protocol.rss.OperationSetPersistentPresenceRssImpl.changePresenceStatusForContact(OperationSetPersistentPresenceRssImpl.java:637)

聽聽聽聽at net.java.sip.communicator.impl.protocol.rss.OperationSetPersistentPresenceRssImpl.subscribe(OperationSetPersistentPresenceRssImpl.java:884)

聽聽聽聽at net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.addNewContactToMetaContact(MetaContactListServiceImpl.java:393)

聽聽聽聽... 2 more

-- dump #3
net.java.sip.communicator.service.contactlist.MetaContactListException: failed to create contacthttp://feeds.reuters.com/reuters/topNews/
聽聽聽聽at net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.addNewContactToMetaContact(MetaContactListServiceImpl.java:417)

聽聽聽聽at net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.createMetaContact(MetaContactListServiceImpl.java:680)

聽聽聽聽at net.java.sip.communicator.impl.gui.main.contactlist.addcontact.AddContactWizard$CreateContact.run(AddContactWizard.java:88)

Caused by: java.lang.NullPointerException
聽聽聽聽at net.java.sip.communicator.impl.gui.main.chat.ProtocolContactSelectorBox.updateContactStatus(ProtocolContactSelectorBox.java:187)

聽聽聽聽at net.java.sip.communicator.impl.gui.main.chat.MetaContactChatPanel.contactPresenceStatusChanged(MetaContactChatPanel.java:303)

聽聽聽聽at net.java.sip.communicator.impl.protocol.rss.OperationSetPersistentPresenceRssImpl.fireContactPresenceStatusChangeEvent(OperationSetPersistentPresenceRssImpl.java:141)

聽聽聽聽at net.java.sip.communicator.impl.protocol.rss.OperationSetPersistentPresenceRssImpl.changePresenceStatusForContact(OperationSetPersistentPresenceRssImpl.java:637)

聽聽聽聽at net.java.sip.communicator.impl.protocol.rss.OperationSetPersistentPresenceRssImpl.subscribe(OperationSetPersistentPresenceRssImpl.java:884)

聽聽聽聽at net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.addNewContactToMetaContact(MetaContactListServiceImpl.java:393)

聽聽聽聽... 2 more

Mihai Balan wrote:

Hi,
There's definitely something wrong with me today. The actual patch
file is attached below.

Sory for spamming the list O:-)

Best,
Mihai

On 8/17/07, Mihai Balan <mihai.balan@gmail.com> wrote:

Hi, it's me again :smiley:
With the risk of becoming annoying, I send this mail too. I had a talk
with Emil and Yana so the previous patch got two little modifications:
聽聽* it seems that discarding smilies processing for HTML is not a
solution, so this part has been discarded from the patch. I'll work on
a different solution.
聽聽* the src attribute problem was solved in a different, more
general/elegant way.

A beautiful evening to you all,
Mihai

PS: Thus being said, please ignore the previous patch and consider
this one as the (I hope) final version :slight_smile:

This version of the patch mainly adds the following:
聽聽* fixing manual refresh of feeds
聽聽* allows feed addresses to start with feed:// (Vincent's request :stuck_out_tongue: )
聽聽* fixes FOSDEM feeds
聽聽* fixes wordpress display of smilies by disabling smiley
processing/substitution for HTML. If this behaviour somewhat offends
you and/or makes a protocol work unexpected, just ping me and I'll try
to find an alternate way to fix things.

All the best,
Mihai

On 8/17/07, Emil Ivov <emcho@emcho.com> wrote:

Hi guys,

I just checked and I am seeing the exact same behavior as you describe:
I first get an empty window and then the updates succeed. There were no
exceptions in neither the console nor the log files.

Personally, I'd bet on another problem with the swing html renderer.

I had also noticed a similar problem with the FOSDEM RSS flow (
http://fosdem.org/2007/rss.xml ). It's there with or without the patch
and it looks a lot worse as it would sometimes even mess the way the
chat or history windows look. This one spits an exception in the logs
(though judging from how deep in swing it appears I don't think the
stacktrace would help a lot)

Here's what the exception looks like:

java.lang.NullPointerException
聽聽聽聽at javax.swing.text.FlowView.insertUpdate(FlowView.java:238)
聽聽聽聽at javax.swing.text.View.forwardUpdateToView(View.java:1186)
聽聽聽聽at javax.swing.text.View.forwardUpdate(View.java:1161)
聽聽聽聽at javax.swing.text.BoxView.forwardUpdate(BoxView.java:223)
聽聽聽聽at javax.swing.text.View.insertUpdate(View.java:685)
聽聽聽聽at javax.swing.text.View.forwardUpdateToView(View.java:1186)
聽聽聽聽at javax.swing.text.View.forwardUpdate(View.java:1161)
聽聽聽聽at javax.swing.text.BoxView.forwardUpdate(BoxView.java:223)
聽聽聽聽at javax.swing.text.View.insertUpdate(View.java:685)
聽聽聽聽at javax.swing.plaf.basic.BasicTextUI$RootView.insertUpdate(BasicTextUI.java:1589)

聽聽聽聽at javax.swing.plaf.basic.BasicTextUI$UpdateHandler.insertUpdate(BasicTextUI.java:1848)

聽聽聽聽at javax.swing.text.AbstractDocument.fireInsertUpdate(AbstractDocument.java:185)

聽聽聽聽at javax.swing.text.DefaultStyledDocument.insert(DefaultStyledDocument.java:206)

聽聽聽聽at javax.swing.text.html.HTMLDocument.insert(HTMLDocument.java:242)
聽聽聽聽at javax.swing.text.html.HTMLDocument$HTMLReader.flushBuffer(HTMLDocument.java:3325)

聽聽聽聽at javax.swing.text.html.HTMLDocument$HTMLReader.flush(HTMLDocument.java:2133)

聽聽聽聽at javax.swing.text.html.HTMLDocument.insertHTML(HTMLDocument.java:1160)
聽聽聽聽at javax.swing.text.html.HTMLDocument.insertAfterEnd(HTMLDocument.java:989)

聽聽聽聽at net.java.sip.communicator.impl.gui.main.chat.ChatConversationPanel.appendMessageToEnd(ChatConversationPanel.java:378)

聽聽聽聽at net.java.sip.communicator.impl.gui.main.chat.ChatPanel.processMessage(ChatPanel.java:375)

聽聽聽聽at net.java.sip.communicator.impl.gui.main.contactlist.ContactListPanel.messageDelivered(ContactListPanel.java:288)

聽聽聽聽at net.java.sip.communicator.impl.protocol.rss.OperationSetBasicInstantMessagingRssImpl.fireMessageDelivered(OperationSetBasicInstantMessagingRssImpl.java:318)

聽聽聽聽at net.java.sip.communicator.impl.protocol.rss.OperationSetBasicInstantMessagingRssImpl.sendInstantMessage(OperationSetBasicInstantMessagingRssImpl.java:289)

聽聽聽聽at net.java.sip.communicator.impl.gui.main.chat.MetaContactChatPanel.sendMessage(MetaContactChatPanel.java:369)

聽聽聽聽at net.java.sip.communicator.impl.gui.main.chat.ChatSendPanel$1.run(ChatSendPanel.java:84)

I guess the best way to resolve the problem is to simply experiment and
try to discover the exact html tags that swing doesn't like.

Good luck!

Emil

Vincent Lucas wrote:

Hi devs, we need your help :slight_smile:

The last patch from Mihai (the identification patch) give me a little
bug that I can reproduce but Mihai can't.
That's why we need another tester to determine if this bug is only
related to my configuration or if it's a real bug.
For this you just need to:
1) Patch your SC with the identification patch from Mihai.
2) Remove your .sip-communicator directory.
3) Launch SC.
4) Add a Rss account.
5) Create a group.
6) Add a new buddy: http://news.google.com/nwshp?tab=wn&output=rss
7) Tell us if the chat windows display some text. For my part the chat
window appears but stay empty (the systray and the history work fine).

Thanks for your help,
Vincent

Mihai Balan wrote:

Hi Vincent,
A very quick glance the problem you're having with the Elsevier feeds
reveals the fact that you're trying to add a feed that uses the
feed:// protocol. Afaik the only protocols that ROME can deal with is
http:// ( https:// too?). By simply replacing feed:// with http:// in
the Elsevier feed adress everything works fine. Maybe some sort of
validation could be made upon adding a new RSS contact, but at the
moment, I have some other matters at hand.
I'm working on the user-requested update bug, I try to fix it asap :slight_smile:

All the best,
Mihai

On 8/16/07, Vincent Lucas <lucas@clarinet.u-strasbg.fr> wrote:

Hi Mihai,

It's a good work, feeds without date (i.e.
http://www.freenews.fr/feeds/rss.php) are working now :slight_smile:

But I experienced strange behavior with feeds with date, which were
previously working: i.e. http://news.google.com/nwshp?tab=wn&output=rss.
Try it with a new sip-communicator account, then create a Rss account,
and add the google feeds. You will see the systray showing that he
receives the news, then the ConversationPanel will appear but will stay
empty.
Nevertheless, when a new news appears it will NOT be well displayed
again if you have try to refresh it manually (sending a message) before.

By the way another bug appears with Elsevier feeds
(feed://www1.elsevier.com/homepage/sac/cfp/compsci_cfp.rss). I'm running
out of time to see it in details (Mihai can you have a look a it?).
See error dump following:
net.java.sip.communicator.service.contactlist.MetaContactListException:

failed to create
contactfeed://www1.elsevier.com/homepage/sac/cfp/compsci_cfp.rss
聽聽聽聽聽聽聽聽at
net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.addNewContactToMetaContact(MetaContactListServiceImpl.java:417)

聽聽聽聽聽聽聽聽at
net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.createMetaContact(MetaContactListServiceImpl.java:680)

聽聽聽聽聽聽聽聽at
net.java.sip.communicator.impl.gui.main.contactlist.addcontact.AddContactWizard$CreateContact.run(AddContactWizard.java:88)

Caused by: java.lang.IllegalArgumentException: failed to create a URL
for address feed://www1.elsevier.com/homepage/sac/cfp/compsci_cfp.rss.
Error was: Unknown protocol: feed
聽聽聽聽聽聽聽聽at
net.java.sip.communicator.impl.protocol.rss.OperationSetPersistentPresenceRssImpl.subscribe(OperationSetPersistentPresenceRssImpl.java:851)

聽聽聽聽聽聽聽聽at
net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.addNewContactToMetaContact(MetaContactListServiceImpl.java:393)

聽聽聽聽聽聽聽聽... 2 more

Cheers,
Vincent

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

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@sip-communicator.dev.java.net
For additional commands, e-mail: dev-help@sip-communicator.dev.java.net

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@sip-communicator.dev.java.net
For additional commands, e-mail: dev-help@sip-communicator.dev.java.net

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@sip-communicator.dev.java.net
For additional commands, e-mail: dev-help@sip-communicator.dev.java.net


#12

Hi Damian,
I'm having the same problem you describe. I'll have a look into it as
soon as I finish writing the unit tests.

All the best,
Mihai

路路路

On 8/23/07, Damian Minkov <damencho@damencho.com> wrote:

Hi,

there is some problem with rss that don't have dates. I add such rss and
everything is ok. But when I stop and start
sip-communicator and try to refresh the feed by sending something in
message dialog I got an exception :

聽聽聽聽聽[java] java.lang.NullPointerException
聽聽聽聽聽[java] at
net.java.sip.communicator.impl.protocol.rss.RssFeedReader.feedToString(RssFeedReader.java:139)
聽聽聽聽聽[java] at
net.java.sip.communicator.impl.protocol.rss.OperationSetBasicInstantMessagingRssImpl.submitRssQuery(OperationSetBasicInstantMessagingRssImpl.java:179)
聽聽聽聽聽[java] at
net.java.sip.communicator.impl.protocol.rss.OperationSetBasicInstantMessagingRssImpl.refreshRssFeed(OperationSetBasicInstantMessagingRssImpl.java:244)
聽聽聽聽聽[java] at
net.java.sip.communicator.impl.protocol.rss.RssThread.run(RssThread.java:45)

I am testing with this : http://linux-bg.org/linux-bg-forum.rdf

damencho

Mihai Balan wrote:
> Hi,
> There's definitely something wrong with me today. The actual patch
> file is attached below.
>
> Sory for spamming the list O:-)
>
> Best,
> Mihai
>
> On 8/17/07, Mihai Balan <mihai.balan@gmail.com> wrote:
>
>> Hi, it's me again :smiley:
>> With the risk of becoming annoying, I send this mail too. I had a talk
>> with Emil and Yana so the previous patch got two little modifications:
>> * it seems that discarding smilies processing for HTML is not a
>> solution, so this part has been discarded from the patch. I'll work on
>> a different solution.
>> * the src attribute problem was solved in a different, more
>> general/elegant way.
>>
>> A beautiful evening to you all,
>> Mihai
>>
>> PS: Thus being said, please ignore the previous patch and consider
>> this one as the (I hope) final version :slight_smile:
>>
>>
>>> This version of the patch mainly adds the following:
>>> * fixing manual refresh of feeds
>>> * allows feed addresses to start with feed:// (Vincent's request :stuck_out_tongue: )
>>> * fixes FOSDEM feeds
>>> * fixes wordpress display of smilies by disabling smiley
>>> processing/substitution for HTML. If this behaviour somewhat offends
>>> you and/or makes a protocol work unexpected, just ping me and I'll try
>>> to find an alternate way to fix things.
>>>
>>> All the best,
>>> Mihai
>>>
>>> On 8/17/07, Emil Ivov <emcho@emcho.com> wrote:
>>>
>>>> Hi guys,
>>>>
>>>> I just checked and I am seeing the exact same behavior as you describe:
>>>> I first get an empty window and then the updates succeed. There were no
>>>> exceptions in neither the console nor the log files.
>>>>
>>>> Personally, I'd bet on another problem with the swing html renderer.
>>>>
>>>> I had also noticed a similar problem with the FOSDEM RSS flow (
>>>> http://fosdem.org/2007/rss.xml ). It's there with or without the patch
>>>> and it looks a lot worse as it would sometimes even mess the way the
>>>> chat or history windows look. This one spits an exception in the logs
>>>> (though judging from how deep in swing it appears I don't think the
>>>> stacktrace would help a lot)
>>>>
>>>> Here's what the exception looks like:
>>>>
>>>>
>>>>> java.lang.NullPointerException
>>>>> at javax.swing.text.FlowView.insertUpdate(FlowView.java:238)
>>>>> at javax.swing.text.View.forwardUpdateToView(View.java:1186)
>>>>> at javax.swing.text.View.forwardUpdate(View.java:1161)
>>>>> at javax.swing.text.BoxView.forwardUpdate(BoxView.java:223)
>>>>> at javax.swing.text.View.insertUpdate(View.java:685)
>>>>> at javax.swing.text.View.forwardUpdateToView(View.java:1186)
>>>>> at javax.swing.text.View.forwardUpdate(View.java:1161)
>>>>> at javax.swing.text.BoxView.forwardUpdate(BoxView.java:223)
>>>>> at javax.swing.text.View.insertUpdate(View.java:685)
>>>>> at javax.swing.plaf.basic.BasicTextUI$RootView.insertUpdate(BasicTextUI.java:1589)
>>>>> at javax.swing.plaf.basic.BasicTextUI$UpdateHandler.insertUpdate(BasicTextUI.java:1848)
>>>>> at javax.swing.text.AbstractDocument.fireInsertUpdate(AbstractDocument.java:185)
>>>>> at javax.swing.text.DefaultStyledDocument.insert(DefaultStyledDocument.java:206)
>>>>> at javax.swing.text.html.HTMLDocument.insert(HTMLDocument.java:242)
>>>>> at javax.swing.text.html.HTMLDocument$HTMLReader.flushBuffer(HTMLDocument.java:3325)
>>>>> at javax.swing.text.html.HTMLDocument$HTMLReader.flush(HTMLDocument.java:2133)
>>>>> at javax.swing.text.html.HTMLDocument.insertHTML(HTMLDocument.java:1160)
>>>>> at javax.swing.text.html.HTMLDocument.insertAfterEnd(HTMLDocument.java:989)
>>>>> at net.java.sip.communicator.impl.gui.main.chat.ChatConversationPanel.appendMessageToEnd(ChatConversationPanel.java:378)
>>>>> at net.java.sip.communicator.impl.gui.main.chat.ChatPanel.processMessage(ChatPanel.java:375)
>>>>> at net.java.sip.communicator.impl.gui.main.contactlist.ContactListPanel.messageDelivered(ContactListPanel.java:288)
>>>>> at net.java.sip.communicator.impl.protocol.rss.OperationSetBasicInstantMessagingRssImpl.fireMessageDelivered(OperationSetBasicInstantMessagingRssImpl.java:318)
>>>>> at net.java.sip.communicator.impl.protocol.rss.OperationSetBasicInstantMessagingRssImpl.sendInstantMessage(OperationSetBasicInstantMessagingRssImpl.java:289)
>>>>> at net.java.sip.communicator.impl.gui.main.chat.MetaContactChatPanel.sendMessage(MetaContactChatPanel.java:369)
>>>>> at net.java.sip.communicator.impl.gui.main.chat.ChatSendPanel$1.run(ChatSendPanel.java:84)
>>>>>
>>>> I guess the best way to resolve the problem is to simply experiment and
>>>> try to discover the exact html tags that swing doesn't like.
>>>>
>>>> Good luck!
>>>>
>>>> Emil
>>>>
>>>> Vincent Lucas wrote:
>>>>
>>>>> Hi devs, we need your help :slight_smile:
>>>>>
>>>>> The last patch from Mihai (the identification patch) give me a little
>>>>> bug that I can reproduce but Mihai can't.
>>>>> That's why we need another tester to determine if this bug is only
>>>>> related to my configuration or if it's a real bug.
>>>>> For this you just need to:
>>>>> 1) Patch your SC with the identification patch from Mihai.
>>>>> 2) Remove your .sip-communicator directory.
>>>>> 3) Launch SC.
>>>>> 4) Add a Rss account.
>>>>> 5) Create a group.
>>>>> 6) Add a new buddy: http://news.google.com/nwshp?tab=wn&output=rss
>>>>> 7) Tell us if the chat windows display some text. For my part the chat
>>>>> window appears but stay empty (the systray and the history work fine).
>>>>>
>>>>> Thanks for your help,
>>>>> Vincent
>>>>>
>>>>>
>>>>> Mihai Balan wrote:
>>>>>
>>>>>> Hi Vincent,
>>>>>> A very quick glance the problem you're having with the Elsevier feeds
>>>>>> reveals the fact that you're trying to add a feed that uses the
>>>>>> feed:// protocol. Afaik the only protocols that ROME can deal with is
>>>>>> http:// ( https:// too?). By simply replacing feed:// with http:// in
>>>>>> the Elsevier feed adress everything works fine. Maybe some sort of
>>>>>> validation could be made upon adding a new RSS contact, but at the
>>>>>> moment, I have some other matters at hand.
>>>>>> I'm working on the user-requested update bug, I try to fix it asap :slight_smile:
>>>>>>
>>>>>> All the best,
>>>>>> Mihai
>>>>>>
>>>>>> On 8/16/07, Vincent Lucas <lucas@clarinet.u-strasbg.fr> wrote:
>>>>>>
>>>>>>> Hi Mihai,
>>>>>>>
>>>>>>> It's a good work, feeds without date (i.e.
>>>>>>> http://www.freenews.fr/feeds/rss.php) are working now :slight_smile:
>>>>>>>
>>>>>>> But I experienced strange behavior with feeds with date, which were
>>>>>>> previously working: i.e. http://news.google.com/nwshp?tab=wn&output=rss.
>>>>>>> Try it with a new sip-communicator account, then create a Rss account,
>>>>>>> and add the google feeds. You will see the systray showing that he
>>>>>>> receives the news, then the ConversationPanel will appear but will stay
>>>>>>> empty.
>>>>>>> Nevertheless, when a new news appears it will NOT be well displayed
>>>>>>> again if you have try to refresh it manually (sending a message) before.
>>>>>>>
>>>>>>> By the way another bug appears with Elsevier feeds
>>>>>>> (feed://www1.elsevier.com/homepage/sac/cfp/compsci_cfp.rss). I'm running
>>>>>>> out of time to see it in details (Mihai can you have a look a it?).
>>>>>>> See error dump following:
>>>>>>> net.java.sip.communicator.service.contactlist.MetaContactListException:
>>>>>>> failed to create
>>>>>>> contactfeed://www1.elsevier.com/homepage/sac/cfp/compsci_cfp.rss
>>>>>>> at
>>>>>>> net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.addNewContactToMetaContact(MetaContactListServiceImpl.java:417)
>>>>>>> at
>>>>>>> net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.createMetaContact(MetaContactListServiceImpl.java:680)
>>>>>>> at
>>>>>>> net.java.sip.communicator.impl.gui.main.contactlist.addcontact.AddContactWizard$CreateContact.run(AddContactWizard.java:88)
>>>>>>> Caused by: java.lang.IllegalArgumentException: failed to create a URL
>>>>>>> for address feed://www1.elsevier.com/homepage/sac/cfp/compsci_cfp.rss.
>>>>>>> Error was: Unknown protocol: feed
>>>>>>> at
>>>>>>> net.java.sip.communicator.impl.protocol.rss.OperationSetPersistentPresenceRssImpl.subscribe(OperationSetPersistentPresenceRssImpl.java:851)
>>>>>>> at
>>>>>>> net.java.sip.communicator.impl.contactlist.MetaContactListServiceImpl.addNewContactToMetaContact(MetaContactListServiceImpl.java:393)
>>>>>>> ... 2 more
>>>>>>>
>>>>>>> Cheers,
>>>>>>> Vincent

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@sip-communicator.dev.java.net
For additional commands, e-mail: dev-help@sip-communicator.dev.java.net