[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kopete-devel
Subject:    Re: [kopete-devel] History Import extension
From:       Olivier Goffart <ogoffart () kde ! org>
Date:       2008-12-20 21:42:42
Message-ID: 200812202242.48214.ogoffart () kde ! org
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


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.


["signature.asc" (application/pgp-signature)]

_______________________________________________
kopete-devel mailing list
kopete-devel@kde.org
https://mail.kde.org/mailman/listinfo/kopete-devel


[prev in list] [next in list] [prev in thread] [next in thread]