Pidgin does not display pictures recieved from Gadu Gadu

[MarkS]

Adium ticket: 4702

GG lets you send a picture (max size between 20 KB and 255 KB depending on the receiver's preferences) to a user. It is displayed in the chat window. However, Pidgin does not display the picture.

[MiKom]

I'll just add that sending images is implemented in libgadu.

[evands]

From sf.net patch #1575266, here is the patch from biedron submitted 2006-10-11 which he wrote fixed this.

[evands]

patch from #1575266: "Gadu Gadu images + txt format"

[MarkS]

This patch has not been committed in over a year(2006-10-11). Just curious if you found a reason this patch hasn't been committed. No presure but it would be great to hear some feedback. Thanks.

[nosnilmot]

This patch needs porting from Gaim -> libpurple, and updating for the changes in the imgstore API.

I have done a quick update of it but have no way to test this (it compiles, although with more warnings than I would like).

[nosnilmot]

Patch updated for libpurple

[nosnilmot]

I just noticed this leaks, which will need fixing if it is applied:

msg = g_string_new(g_markup_escape_text(tmp, -1));

[liviopl]

I wonder if libpurple could simply use features from libgadu.

[MarkS]

Replying to liviopl:

I wonder if libpurple could simply use features from libgadu.

I was wondering the same thing. http://toxygen.net/libgadu/releases/1.8.0.html

[liviopl]

And there are such nice news!

* file transfer with Gadu-Gadu 7, * authorization with SHA-1, * support for XML-based info sent by server...

Pidgin treats all protocols as orphans (treats them unfair).

[evands]

Building with the libgadu 1.8.0 files works for me without warnings, but I have no ability (or time) to test this configuration thoroughly. Please try the same and let us know what your results are.

[AlexJ]

Gadu-Gadu attends images with max size of 255kB, not 20kB ;). 20kB is the smallest-size. The max img size is set-up by the Gadu-Gadu user.

[sabby7890]

Ok, made some changes. Gadu-Gadu right now IS displaying the picture received from Gadu-Gadu (with formatting, too, so someone can send you pic like this:

message(pic)messagemessage(pic).

Just download my gg.c and gg.h files I've put here and receiving should work flawlessly. I am also working on sending images - works as for now (with original Gadu-Gadu client, just remember to be online), but without formatting and with <IMG ID="x"> message. Stay tuned, don't have time to create diffs, so just download gg.c and gg.h files, put them in your pidgin-2.4.1/libpurple/protocols/gg/ directory and it should work.

If someone wants to have the latest version of my work, go here:

http://dp0154.debowypark.waw.pl/gg.c
http://dp0154.debowypark.waw.pl/gg.h

Cheers!

[ono]

@sabby7890: Please put your patch as single file as described in TipsForPatchSubmissions

Regarding your latest patch:

1. msg = charset_convert(msg, "UTF-8", "CP1250"); and some other g_strdup_printf in ggp_send_im_richtext are IMHO memory leaks, since you don't free anywhere old msg memory buffer.

2. UTF-8 -> CP1250 should happen at the very end just before sending the buffer to libgadu function, as all glib functions such as g_strsplit work in UTF-8 locale, so doing conversion to CP1250 first then working with UTF-8 aware functions if pretty bad idea since it may lead to unpredictable behavior.

Anyway I really appreciate your work and looking towards putting it into repository once all the memory related things are sorted out.

[sabby7890]

@ono

Sorry for long time to reply, I'm getting to work right now:) Thanks for your tips, everything will be fixed shortly (I believe today)

[AlexJ]

Would anybody change the Ticket's description, where is written "max size 20KB" to the true one? In original client, user can specify how much big images he wants to receive (he can choose between 20 KiB - 255 KiB), and if somebody sends him too big image, the sender's client shows message about it and probably the reveiver also get know about it. I also have a proffer about sending a part of screenshot (made automatically) to the reveiver which is a very useful function (didn't it?). I hope that images support in Gadu-Gadu protocol will be implemented in the near future, maybe next stable version of Pidgin (it's very annoying when somebody sends you a picture, but you only see an empty message, or he tells you that you should change the max img size in the Gadu-Gadu client settings to 255 KiB). Thanks, and cheers!

[sabby7890]

Ok, I'm still working on it, so if anyone is using my patch right now please, contact with me on Gadu-Gadu: 3700457 or Google Talk: tsalacinski@… to report bugs before I'll release it. I need to do a couple of things to be sure this patch will be rock stable.

[sabby7890]

Ok, done some work, right now we should be freeing everything (I think?).

Please, check this out. Also - the patches are in unified (diff -u) format.

If you have any suggestions, please feel free to IM me. Sorry it takes so long, I have some other things to do that need to be done first:( Cheers!

Fixed: few memory leaks and character conversion, also when someone is sending us an image there is no null message (for ex. when someone sends an image to us, first there is a message without anything, then it shows the valid message).

[sabby7890]

GG Image patches (diff -u)

[sabby7890]

Ok, uploaded new patch file. Should be fine - I hope there are no memory leaks, image support should be better (but from dev point of view - I know, far from perfect;) ).

[sabby7890]

Updated (uses GList)

[sabby7890]

Fixed receiving multiple images

[tpg]

Looks like the patch is working like a charm ;)

[sabby7890]

Replying to tpg:

Looks like the patch is working like a charm ;)

Glad you like it. I'm still working on it, so try the new patch (preview of text formatting, receiving only - bold, italic, underline).

Any help appreciated.

[sabby7890]

Fixed bug that caused emoticon only messages (like <dresik>) to disappear

[MarkS]

sabby7890:

Is your latest patch (gg-patch.dff) feature complete and you feel the patch can be committed or do you feel it still needs more work.

By the way, good work and thanks for the hard work.

[sabby7890]

MarkS:

It is not feature complete (it doesn't support images in conference windows), but I think it can be committed.

[sabby7890]

I think this cannot be pushed yet. Messages in Gadu-Gadu sometimes crash Pidgin (it happened twice to me), not very often, but they do. I have to work on it.

[sabby7890]

Fixed patch that made Pidgin crash sometimes when speaking with person that was using Tlen messenger

[sabby7890]

I think it can be committed right now. I haven't noticed any crash or bad work when I was using this patch.

[ono]

+1

I think this patch is now alright, thanks for your hard work and testing Sabby.

[ono]

Please DON'T commit it yet. It has few bugs inside, sorry I just revised this patch.

[ono]

Fixed sabby gg-patch v4

[ono]

Just to not be "annoying teacher" I'm attaching fixes to sabby patch above. This needs to be tested though:

Fixes are:

• proper formatting and C / Doxy compatible comments
• pending_richtext_messages are now part of GG proto instance data
• pending_images hash instead of looking trough all images for valid image with crc to send
• few fixes, about not released string buffers.. or unnecessary variables, etc. just to make code smaller and cleaner

It is getting late, tommorow I'll be testing this patch hopefully. Waiting for your comments.

[sabby7890]

Ono, I've started to write a message to ask you for exactly the same thing you just did, then it showed me that someone else modified the bug:) Believe me, for me anyone can thrash my patch, tell me it's very bad etc, but if someone points me to the right direction/fixes my patch, it's VERY appreciated. I won't take a look at it today, though, it's late here too. Goodnight:)

[ono]

Rearanged and revisited image support patch, should be fine now.

[ono]

Okay I think the patch I just attached libpurple_gg_image_support.3.patch (reworked Sabby's) patch is good enough. Please check it, I've checked it with latest 7.7 client. @Sabby please install this patch on fresh mtn gg.c & gg.h releases, and try to test it. It should be safe enough to commit it into the trunk.

In comparison to previous patch:

• fixed UTF-8 positioning in ggp_recv_message_handler, so result string is valid UTF8 after injecting HTML tags
• fix incoming bold, italic, etc. formatting, GG is sending empty bogus formatting packets, that shouldn't be treated as formatting end
• richtext sending is now embedded in ggp_send_im no need for extra function, also safer return values
• purple_markup_find_tag used for detecting img tags -> safer than g_split
• fixed all purple_debug to include function names.. so they are more descriptive

[ono]

Clean patch, only image support (w/o purple_debug comment fixes)

[MarkS]

Just curious...Will the changes in the patch also fix the formatting issue that were reported in Ticket 222?

[sabby7890]

I don't know how to reproduce it, never had any problems with links in Pidgin.

[MarkS]

It was a more of a formating issue. It was related to this Adium ticket 6151. It explains the issue more clearly.

[ono]

Well right now, this patch just adds support for richtext message reception (bold, italic, underline + inline messages), while no richtext message sending but just image sending trough <img id="ID"> mapping.

Obviously we need to sort Adium out somehow, since it removes <img> tags too, so actually image sending works in pidgin while not in Adium AFAIK.

I'll try expend this patch to add <b>, <i>, <u> mapping, too, then I need to remove rest of html mapping. Probably this removal should happen directly in the plugin, right?

Then I'll try to add propriate patch to Adium too. So we got more complete GG support. Cheers!

[liviopl]

So we got more complete GG support.

Remember about "more". Audio (and video?) chats left, for example.

[sabby7890]

I think instead of just images this patch should contain complete richtext support (ie. colored text). I'll try to work on it, too.

[Sim-on]

hey guys, this image stuff would be really nice but at the moment the latest gadu-gadu version doesnt work at all... (#6232)

[MarkS]

ono:

Adium is based on libpurple so if it is fixed here once committed it should get automatically fixed in the Adium once Evan pushes the latest libpurple. This is why I hang out here to see what progress has been made if any. Try to test the new gadu libpurple features in Adium if there are any changes. I considered rich text a nice to have, but it was not considered an essential feature. However, having it would bring Pidgin one step closer to actually client. By the way, I want to thank you as well as sabby7890 for improving the Gadu Gadu experience on Pidgin/Aduim. Job well done.

[sabby7890]

MarkS:

Please tell me, what is needed to be done in order to improve this patch and commit it to Pidgin? AFAIK last time I used it it worked fine.

[MarkS]

I thought you were still working on the patch per this comment.

"Replying to sabby7890:

I think instead of just images this patch should contain complete richtext support (ie. colored text). I'll try to work on it, too."

If you feel this patch works fine, let see if on of the Pidgin developers can get this patch committed. Can any Pidgin developer on this distro list review sabby7890's latest revision of the patch?

Thanks for the hard work.

I have seen a quite a few patches for gadu that have been just sitting idle However, I would like to see yours committed.

=======================

Here are some example of patches for gadu that have been collecting dust. (Ex. 373, 4582, 5175, 6270, 6691)

[sabby7890]

I see these patches, and IMHO lot of them provide great functionality to Pidgin. They cannot be applied at once, though. First, upgrading libgadu is necessary (bug 373). Then we can apply rest of the patches. AFAIK my patch is working fine, I had some trouble with it on x64 machine. I'll try to merge these patches to one big patch and send it to pidgin devs. It will take some time though, I don't have much time (developing Alarm Clock - alarm-clock.54.pl) and I just wanted to have some personal life:) When I'll find some time, I'll just post a new bug with big patch.

[ono@…]

(In [bc507450c4a8e842eb0ab445428935652307df88]):
Support displaying buddy icons from Gadu-Gadu buddies. Fixes #220.

[MarkS]

Glad to see this committed but shouldn't the description say "Fixes sending/receiving a picture or image via a chat in Gadu Gadu" instead of "Support displaying buddy icons from Gadu-Gadu buddies." Since support for displaying icon was addressed in this ticket 4971 .

[rekkanoryo@…]

(In [57f0a80156490200a80b4787d0be27ded56040c7]):
I forgot to ChangeLog this earlier. Refs #220.

[nosnilmot@…]

(In [c156244970ba750784155b6bae31df621c20af02]):
I believe the patch from #220 actually implemented IM images for Gadu-Gadu, not something to do with Buddy Icons. References #220

[rekkanoryo@…]

(In [df6eba32e5b6b34d7483cbfb7e9f2e4c836ac35f]):
Correctly credit Tomasz Sałaciński for writing the original patch for IM images on Gadu-Gadu. I previously missed that the patch I applied was Tomasz's patch with modifications. Refs #220.