From kopete-devel Sun Jan 11 19:33:43 2009 From: =?ISO-8859-1?Q?Timo_Schl=FC=DFler?= Date: Sun, 11 Jan 2009 19:33:43 +0000 To: kopete-devel Subject: Re: [kopete-devel] History Import extension Message-Id: <496A4997.8000305 () schluessler ! org> X-MARC-Message: https://marc.info/?l=kopete-devel&m=123170248325845 MIME-Version: 1 Content-Type: multipart/mixed; boundary="--===============1454358238==" This is a multi-part message in MIME format. --===============1454358238== Content-Type: multipart/alternative; boundary="------------060302040904000309080808" This is a multi-part message in MIME format. --------------060302040904000309080808 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 8bit Thank you for committing the patch :) regards, Timo Matt Rogers wrote: > On Monday 22 December 2008 11:07:06 Timo Schlüßler wrote: > >> attached is a version where i changed the things you mentioned in irc ... >> >> regards >> >> Olivier Goffart wrote: >> >>> Le mercredi 3 décembre 2008, Timo Schlüßler a écrit : >>> >>>> Hello, >>>> >>>> attached is a patch and the two files which add the ability to import >>>> logs from pidgin to kopete. The source-code files belong to >>>> kdenetwork/kopete/plugins/history. >>>> I changed HistoryLogger slightly so that appendMessage() uses the >>>> timestamp from the message that is to be imported but not >>>> QDate::currentDate(). >>>> >>>> I don't know what happens if you want to import multiuser chatlogs >>>> because i haven't got any ... i would be glad if you can check that. >>>> For me it works just fine, but the logs from pidgin are very unsteady. >>>> anyway i hope it works for all pidgin logs. >>>> >>> Hi, >>> >>> sorry for answering so late. >>> >>> You shouldn't change the reference date in the getDocument() function. >>> >>> You should understand that by definition, the 'int month' passed to the >>> getDocument function is the difference between the _current_ month and >>> the month of the document you ask. >>> >>> So you should not change the reference. >>> >>> What you should do instead is, in appendMessage, call the overloaded >>> getDocument, that take a QDate instead. >>> BUT then move the cache from the function that take an integer to this >>> one. >>> >>> >>> > > I committed your patch! Thanks for your contribution! > --------------060302040904000309080808 Content-Type: text/html; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Thank you for committing the patch :)

regards,
Timo

Matt Rogers wrote: 
On Monday 22 December 2008 11:07:06 Timo Schlüßler wrote:
  
attached is a version where i changed the things you mentioned in irc ...

regards

Olivier Goffart wrote:
    
Le mercredi 3 décembre 2008, Timo Schlüßler a écrit :
      
Hello,

attached is a patch and the two files which add the ability to import
logs from pidgin to kopete. The source-code files belong to
kdenetwork/kopete/plugins/history.
I changed HistoryLogger slightly so that appendMessage() uses the
timestamp from the message that is to be imported  but not
QDate::currentDate().

I don't know what happens if you want to import multiuser chatlogs
because i haven't got any ... i would be glad if you can check that.
For me it works just fine, but the logs from pidgin are very unsteady.
anyway i hope it works for all pidgin logs.
        
Hi,

sorry for answering so late.

You shouldn't change the reference date in the getDocument() function.

You should understand that by definition, the 'int month' passed to the
getDocument function is the difference between the _current_ month and
the month of the document you ask.

So you should not change the reference.

What you should do instead is, in appendMessage, call the overloaded
getDocument, that take a QDate instead.
BUT then move the cache from the function that take an integer to this
one.


      

I committed your patch! Thanks for your contribution!
--------------060302040904000309080808-- --===============1454358238== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ kopete-devel mailing list kopete-devel@kde.org https://mail.kde.org/mailman/listinfo/kopete-devel --===============1454358238==--