Ticket #1443 (closed defect: fixed)

Opened 2 years ago

Last modified 2 months ago

Smiley *JOKINGLY* does not work

Reported by: max Assigned to: emostar
Priority: normal Milestone: 1.3.5
Component: message Version: 1.3.4
Keywords: smiley icq5 Cc:

Description

When somebody sens me the smiley *JOKINGLY* (Mirabelis5-Smiley), i receive an empty message. But when i send it, it works - i can see the correct Smiley in the text...

Change History

2006-11-03 19:54:59 changed by root_42

Hm, this works for me, using Mirabilis 5 emoticon set. Can you elaborate more?

2006-11-15 18:28:09 changed by max

Hi!

Sorry, but i had no time before today. I testet it - between licq-clients it works. But when my partner has Windows-ICQ 5.1 and he sends me the *JOKINGLY*-Smiley, i just receive an empty message. I testet it with a lot of ICQ 5.1-clients - alsways the same - an empty message.

When I send him the Smiley, he receives it...

I could not test it with ICQ 5.0, MIRANDA or something like that.

Max

2006-11-15 18:33:31 changed by max

I will try more test this evening - maybe more smileys don't work, we will see.

2006-11-17 01:26:20 changed by erijo

Could you try attachment:ticket:1445:licq-1445-no-rtf.patch and see if it solves your problem?

2006-11-17 01:31:56 changed by erijo

  • keywords changed from Smiley to smiley icq5.
  • milestone set to 1.3.5.

2006-11-17 09:30:23 changed by max

OK, I will try your patch on sunday or monday, because I am not available till then. But I now testet all ICQ 5.1-smileys: three smileys have bugs:

First one:

:-[

You can see a picture of it here: == http://helios.wh2.tu-dresden.de/~max/Smiley1.jpg ==

just comes :- from Windows-clients and so no smiley is generated. When the Windows-ICQ-User sends me the string directly and does not use the smiley-collection to send me the smiley it works. But when he does it by this collection it just comes :- What do I mean with smiley-collection?: You have two ways to generate a smiley - just type in the string which is made to the smiley after pressing "send" or you choose one out of a collection if you do not know what the string is... I hope you know hat I mean. Between Windows-clients it works - does not matter if you use the smiley-collection or you type in the string. But from Windows-ICQ to licq it just works by typing the string. Test it yourself with Windows-ICQ. In licq, when you use the smiley-collection, the clicked smiley is directly made to a string in the message-put-in-field. If you send the message it is made to the smiley again. In Windows, if you use the smiley-collection, you can see the clicked smiley directly as a smiley in your message-put-in-field. Maybe there ist the problem - otherwise it would not work if you put in the string directly in the Windows-ICQ-message-put-in-field too.

Second one:

You know... *JOKINGLY*

Third one:

This smiley ist totally missed in licq Mirabelis 5. It ist this one:

http://helios.wh2.tu-dresden.de/~max/Smiley2.jpg

I also receive an empty message when this smiley is send to me by Windows-ICQ. And this smiley even is not integrated in the smiley-collection in licq.

2006-11-26 14:26:21 changed by emostar

  • status changed from new to assigned.
  • owner set to emostar.

ICQ 5.1 puts smileys in part of the RTF message, but uses a syntax like:

<##icqimage0015>

The smiley should be updated to include this string as one of the ways to display the display. We may have to check about allowing the < and > signs in the display, they are considered HTML, but should not be rendered as so.

2006-11-26 18:09:02 changed by emostar

  • status changed from assigned to closed.
  • resolution set to fixed.

Fixed in r4789.

2006-11-27 02:32:20 changed by max

OK, and what is with this one?

:-[

You can see a picture of it here: == http://helios.wh2.tu-dresden.de/~max/Smiley1.jpg ==

Is it also working now?

(follow-up: ↓ 11 ) 2006-11-27 03:07:34 changed by max

  • status changed from closed to reopened.
  • resolution deleted.

Hey, when I try to build a debian-package with the new rtf.cc in source, it gives me mistakes like that:

rtf.cc:5005: error: expected unqualified-id before '<' token rtf.cc:5005: error: expected constructor, destructor, or type conversion before '<' token

This mistake comes with many lines (not only 5005 like here)?

What to do?

And: Ist the Smiley *STOP* than also in Mirabelis-5-Emoticons-Set (as a picture) or does only the string work than?

(in reply to: ↑ 10 ) 2006-11-27 05:51:40 changed by emostar

Replying to max:

Hey, when I try to build a debian-package with the new rtf.cc in source, it gives me mistakes like that: rtf.cc:5005: error: expected unqualified-id before '<' token rtf.cc:5005: error: expected constructor, destructor, or type conversion before '<' token

What is your gcc version?

And: Ist the Smiley *STOP* than also in Mirabelis-5-Emoticons-Set (as a picture) or does only the string work than?

With SVN, both work. Until then only the string for STOP and JOKINGLY worked.

2006-11-27 07:21:23 changed by max

Hi,

I got latest version 4.1.

2006-11-27 14:11:30 changed by emostar

On a side note, there aren't even 5005 lines. How many lines are ni the rtf.cc file that you have?

2006-11-27 17:08:57 changed by max

Hi!

I downloaded rtf.cc from here:

http://trac.licq.org/browser/trunk/licq/src/rtf.cc?rev=4789

And then:

max@ace2:~$ wc -l rtf.cc 5083 rtf.cc

2006-11-27 18:52:03 changed by emostar

I guess you saved the file as html then. You need the source file. Update your SVN or download this:

http://trac.licq.org/browser/trunk/licq/src/rtf.cc?rev=4789&format=raw

$ wc -l rtf.cc
    2463 rtf.cc

(follow-up: ↓ 17 ) 2006-11-27 20:35:18 changed by max

  • status changed from reopened to closed.
  • resolution set to fixed.

Ok, it was my mistake. Now, Smileys *JOKINGLY* and :-[ work now, but *STOP* just comes as string - but if I understood correctly, this is normal and will be working with next version.

Thank you for fixing this!

By whe way - I got MSN-Connection problem like here http://www.mail-archive.com/licq-dev@googlegroups.com/msg00241.html

Do YOU know what to do? Maybe you have to open a new ticket...

(in reply to: ↑ 16 ; follow-up: ↓ 18 ) 2006-11-28 13:44:44 changed by emostar

Replying to max:

Ok, it was my mistake. Now, Smileys *JOKINGLY* and :-[ work now, but *STOP* just comes as string - but if I understood correctly, this is normal and will be working with next version.

It may not be a part of the icon set you are using. I know it is in Mirabilis5.1. It may not be in any other icon sets.

By whe way - I got MSN-Connection problem like here http://www.mail-archive.com/licq-dev@googlegroups.com/msg00241.html Do YOU know what to do? Maybe you have to open a new ticket...

If there isn't a ticket for it already, I will. Not sure why it happens, but it does go away the second time. If you want to discuss it more, please find the ticket or open a new ticket and discuss it there.

(in reply to: ↑ 17 ) 2006-11-29 20:43:57 changed by max

Replying to emostar:

Replying to max:

Ok, it was my mistake. Now, Smileys *JOKINGLY* and :-[ work now, but *STOP* just comes as string - but if I understood correctly, this is normal and will be working with next version.

It may not be a part of the icon set you are using. I know it is in Mirabilis5.1. It may not be in any other icon sets.

And it won't be in Mirabilis5 in newer licq-Versions like 1.3.5? Iam asking me why not, becuase this ist the only one missing in comparison with Windows-ICQ-5.0-Smileys.

By whe way - I got MSN-Connection problem like here http://www.mail-archive.com/licq-dev@googlegroups.com/msg00241.html Do YOU know what to do? Maybe you have to open a new ticket...

If there isn't a ticket for it already, I will. Not sure why it happens, but it does go away the second time. If you want to discuss it more, please find the ticket or open a new ticket and discuss it there.

Ok, I will try my best.