Re: API for history plugin

On Wednesday 24 June 2015 14:14:15 Pali Rohár wrote:
> (now breaking email thread)
> 
> On Wednesday 24 June 2015 14:23:52 Joshua Joseph wrote:
> > It is fixed now.
> > 
> 
> I do not thing so...
> 
>   void logMessage(Kopete::Message *message, Kopete::Protocol *protocol, bool isGroup = false);
> 
> Why are there protocol and isGroup variables?
> 
> 
>   QList<Kopete::Message> search(Kopete::Contact *contact, bool showFrom = true);
> 
> Do you think that such function API describes for what is designed? And
> is easy to use? Have you tried to imagine some code which will use it?
> 
> If I write something like that:
> 
>   messages = history->search(account->myself(), true);
> 
> Without deep knowledge of what method search is doing and how is
> implemented I have really no idea what will be to "messages" set.
> 

And I would like to know... when will be API and your code ready?
Because now is June 24, mid term evaluation starts in few days and
except database scheme I do not see nothing more to be done yet...

According to your (updated!) timeline you proposed that by this day we
would see working chat logging and some GUI viewer for it.

-- 
Pali Rohár
[email protected]
_______________________________________________
kopete-devel mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/kopete-devel