Re: Review Request 130154: Re-enable XML-based history plugin, fix history file saving

2017-10-05 Thread Paulo Lieuthier
marked as submitted. Review request for Kopete, Laurent Montel and Pali Rohár. Changes --- Submitted with commit fcc69d5ad1fa6cca9d567b2f756d251d745f7c57 by Luigi Toscano on behalf of Paulo Lieuthier to branch kf5. Repository: kopete Description --- The history plugin port was not

Chat history progress - GSoC 2017

2017-06-25 Thread Paulo Lieuthier
Hi all, In this week I have worked on improving the façade abstraction for the history plugin's backends, and also adding a new, SQLite-based backend. It's still in the early stages, but saving and loading recent messages for 1:1 conversations already work. Group chats, Jabber rosters, SQL i

Re: Chat history progress - GSoC 2017

2017-06-25 Thread Paulo Lieuthier
lready submitted? Please, tell me if there is anything to improve on them so we can get them merged. Thanks, Paulo Lieuthier signature.asc Description: OpenPGP digital signature

Re: Review Request 130166: Add initial SQLite-based backend for the history plugin

2017-06-25 Thread Paulo Lieuthier
#file496733line49> > > > > SQL injections Good point. I'll add that to the timeline. - Paulo --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/130166/#review

Re: Review Request 130161: Replace KDebug by Qt's categorical logging in Kopete's protocols

2017-06-25 Thread Paulo Lieuthier
Testing --- Testbed account and test user creation and simple conversation. Only logging was changed. Thanks, Paulo Lieuthier

Review Request 130166: Add initial SQLite-based backend for the history plugin

2017-06-25 Thread Paulo Lieuthier
/historyplugin.cpp 48cd98010c2f3e65ff76ce3b6e5a43c92e988478 plugins/history/historypreferences.cpp 9c64b44184923aa92fe9fb252bfc5dc4d868b908 Diff: https://git.reviewboard.kde.org/r/130166/diff/ Testing --- I only tested 1:1 conversations so far, using the Testbed protocol. Thanks, Paulo

Re: Review Request 130164: Initial work on creating a backend façade for history and isolating XML-related code as a specific implementation

2017-06-24 Thread Paulo Lieuthier
rences.cpp 9c64b44184923aa92fe9fb252bfc5dc4d868b908 Diff: https://git.reviewboard.kde.org/r/130164/diff/ Testing --- Thanks, Paulo Lieuthier

Re: Chat history progress - GSoC 2017

2017-06-21 Thread Paulo Lieuthier
On 21-06-2017 11:45, Pali Rohár wrote: On Tuesday 20 June 2017 08:38:01 Paulo Lieuthier wrote: On 20-06-2017 05:20, Pali Rohár wrote: On Tuesday 20 June 2017 01:24:10 Paulo Lieuthier wrote: The new iteration: Conversations: * id * subconversation_id * account * type (1:1, group, channel

Re: Chat history progress - GSoC 2017

2017-06-20 Thread Paulo Lieuthier
On 20-06-2017 05:20, Pali Rohár wrote: > On Tuesday 20 June 2017 01:24:10 Paulo Lieuthier wrote: >> The new iteration: >> >> Conversations: >> * id >> * subconversation_id >> * account >> * type (1:1, group, channel) >> * entity_identifier (id of

Re: Review Request 130161: Replace KDebug by Qt's categorical logging in Kopete's protocols

2017-06-20 Thread Paulo Lieuthier
> On June 19, 2017, 1:51 a.m., Laurent Montel wrote: > > protocols/gadu/gaduaccount.cpp, line 287 > > <https://git.reviewboard.kde.org/r/130161/diff/1/?file=496313#file496313line287> > > > > it's not debug categories changes no ?:) > > Paulo Lieu

Re: Chat history progress - GSoC 2017

2017-06-19 Thread Paulo Lieuthier
) * sender_display_name (sender name, if name not available through id) * type (text, event, file, voice clip) * content (html) * importance * state On 12-06-2017 07:33, Pali Rohár wrote:> On Sunday 11 June 2017 20:22:24 Pali Rohár wrote: >> On Sunday 11 June 2017 19:25:52 Paulo Lieuthier wrote: >>&

Chat history improvements - GSoC 2017 - Progress

2017-06-19 Thread Paulo Lieuthier
a new branch in Github [2]. Paulo Lieuthier [1] https://git.reviewboard.kde.org/r/130164/ [2] https://github.com/paulolieuthier/kopete/commits/initial-facade signature.asc Description: OpenPGP digital signature

Review Request 130164: Initial work on creating a backend façade for history and isolating XML-related code as a specific implementation

2017-06-19 Thread Paulo Lieuthier
logger.cpp 66b627030f9baf78d98c010c3cc2dc3e7cc08c99 plugins/history/historyplugin.cpp 48cd98010c2f3e65ff76ce3b6e5a43c92e988478 plugins/history/historypreferences.cpp 9c64b44184923aa92fe9fb252bfc5dc4d868b908 Diff: https://git.reviewboard.kde.org/r/130164/diff/ Testing --- Thanks, Paulo Lieuthier

Re: Review Request 130161: Replace KDebug by Qt's categorical logging in Kopete's protocols

2017-06-19 Thread Paulo Lieuthier
? I removed all empty debugs. Are they worth keeping, even with no message? > On June 19, 2017, 1:51 a.m., Laurent Montel wrote: > > protocols/yahoo/libkyahoo/receivefiletask.cpp, line 173 > > <https://git.reviewboard.kde.org/r/130161/diff/1/?file=496620#file496620line

Re: Review Request 130154: Re-enable XML-based history plugin, fix history file saving

2017-06-18 Thread Paulo Lieuthier
> On June 10, 2017, 1:26 p.m., Laurent Montel wrote: > > > > Paulo Lieuthier wrote: > I had already started translating those (and all others) in my main > branch, and didn't want to mess with that here. Next review request will be > such translation, isn&#

Re: Review Request 130161: Replace KDebug by Qt's categorical logging in Kopete's protocols

2017-06-18 Thread Paulo Lieuthier
it is. - Paulo Lieuthier On June 18, 2017, 11:09 a.m., Paulo Lieuthier wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard

Review Request 130161: Replace KDebug by Qt's categorical logging in Kopete's protocols

2017-06-18 Thread Paulo Lieuthier
c60f3cf63 Diff: https://git.reviewboard.kde.org/r/130161/diff/ Testing --- Testbed account and test user creation and simple conversation. Only logging was changed. Thanks, Paulo Lieuthier

Re: Chat history progress - GSoC 2017

2017-06-11 Thread Paulo Lieuthier
On 11-06-2017 08:28, Pali Rohár wrote> What does the "state" mean? Whether the message was really sent or failed to be sent. > Also there is missing indication about direction. How to store message > which user of Kopete sent to IRC person? Or how to store message that > was sent from user to

Re: Review Request 130154: Re-enable XML-based history plugin, fix history file saving

2017-06-11 Thread Paulo Lieuthier
> On June 10, 2017, 1:26 p.m., Laurent Montel wrote: > > > > Paulo Lieuthier wrote: > I had already started translating those (and all others) in my main > branch, and didn't want to mess with that here. Next review request will be > such translation, isn&#

Re: Chat history progress - GSoC 2017

2017-06-11 Thread Paulo Lieuthier
is complete, but I think this is a good starting point. This next week I'll be working on the storage façade, and probably send a few more patches regarding Qt5 and KF5 migration! Paulo On 06-06-2017 11:34, Pali Rohár wrote: > Hi! > > On Tuesday 06 June 2017 11:05:26 Paulo Lieuthier wr

Re: Review Request 130154: Re-enable XML-based history plugin, fix history file saving

2017-06-11 Thread Paulo Lieuthier
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/130154/#review103308 --- Ship it! Ship It! - Paulo Lieuthier On June 10, 2017

Re: Review Request 130154: Re-enable XML-based history plugin, fix history file saving

2017-06-11 Thread Paulo Lieuthier
> On June 10, 2017, 1:26 p.m., Laurent Montel wrote: > > > > Paulo Lieuthier wrote: > I had already started translating those (and all others) in my main > branch, and didn't want to mess with that here. Next review request will be > such translation, isn&#

Re: Review Request 130154: Re-enable XML-based history plugin, fix history file saving

2017-06-10 Thread Paulo Lieuthier
.reviewboard.kde.org/r/130154/#review103302 --- On June 10, 2017, noon, Paulo Lieuthier wrote: > > --- > This is an automatically generated e-mail. To reply, visit: >

Review Request 130154: Re-enable XML-based history plugin, fix history file saving

2017-06-10 Thread Paulo Lieuthier
plugin.cpp 48cd98010 plugins/history/historypreferences.cpp 9c64b4418 Diff: https://git.reviewboard.kde.org/r/130154/diff/ Testing --- Tested last messages appearance on reopening a chat window, using Jabber and Test Protocol. Thanks, Paulo Lieuthier

Chat history progress - GSoC 2017 (was: Timeline for GSoC 2017 project (chat history improvements))

2017-06-06 Thread Paulo Lieuthier
l#tokenizers On 05-06-2017 16:05, Pali Rohár wrote: > Hi Paulo! > > Now one week passed and we would like to hear from you some status > update... > > On Monday 29 May 2017 04:04:57 Paulo Lieuthier wrote: >> May 29 – June 2 >> * Become familiar with Kopete’s current his

Re: Timeline for GSoC 2017 project (chat history improvements)

2017-05-31 Thread Paulo Lieuthier
On 29-05-2017 04:41, Pali Rohár wrote: > On Sunday 28 May 2017 23:04:57 Paulo Lieuthier wrote: >> Hey, Pali. I thought history2 was the best option today. > > history2 does not support rich text messages, so no is not a good. > >> I just updated my >> timetable a

Re: Timeline for GSoC 2017 project (chat history improvements)

2017-05-28 Thread Paulo Lieuthier
On 27-05-2017 06:12, Pali Rohár wrote: > On Friday 26 May 2017 16:41:15 Paulo Lieuthier wrote: > > Hi all, > > > > I'm really sorry it took this long. I've been really busy with my > > undergraduate thesis, as I'm graduating this semester. The final >

Timeline for GSoC 2017 project (chat history improvements)

2017-05-26 Thread Paulo Lieuthier
[1] that I think is reasonable, but I really need everyone's opinions and expectations about it. Is it sufficient? Please review it and comment on the document. I will update the timeline during the project so it stays always up to date. Thanks, Paulo Lieuthier [1] https://goo.gl/vs7dwG 0

CMake 3+, C++11 in Kopete

2017-05-14 Thread Paulo Lieuthier
Hi everyone, Is there a reason not to upgrade to CMake 3.0.2 or to use C++11 in Kopete code, following KDE general path (both cmake 3.0.2 and C++11-capable GCC are present in Debian stable repositories)? I'm working in Kopete for GSoC (history plugin), and will be sending my schedule in the next

Re: GSoC 2017 - Announcement of Kopete projects

2017-05-07 Thread Paulo Lieuthier
Hi all, Thanks for the oportunity. I'm finishing my timeline and will send to the ML soon. Congratulations, Vijay! Let's do it! Paulo On 05/05/17 17:25, Pali Rohár wrote: > On Friday 05 May 2017 22:16:48 vijay krishnavanshi wrote: >> Do we have to give updates on kde-devel also or just updates

Re: Will code for mentorship (I need a mentor for GSoC)

2017-04-04 Thread Paulo Lieuthier
d I'm very interested to implement them. I myself work with implementing server to server exchange protocols in my day job. I hope this can help me introducing myself and the ideas. Best, Paulo Lieuthier On Tue, Mar 28, 2017 at 5:17 AM Pali Rohár wrote: > On Saturday 25 March 2017 2

Will code for mentorship (I need a mentor for GSoC)

2017-03-25 Thread Paulo Lieuthier
o ask for some help, if needed be. Please, be my mentor or point me to someone. :) Best, Paulo Lieuthier [1] https://git.reviewboard.kde.org/users/paulolieuthier/ [2] https://docs.google.com/document/d/10l5kIePsN69Amvk9I1WYLsU3nuReVZzvtp1fyVIV22E/edit?usp=sharing [3] https://docs.google.com/docum

Re: Review Request 127357: Fix layout bug in jabber's service registration dialog

2016-03-25 Thread Paulo Lieuthier
abber/jabberformtranslator.cpp 520cb45 protocols/jabber/jabberxdatawidget.cpp 5d3bd7c Diff: https://git.reviewboard.kde.org/r/127357/diff/ Testing --- Thanks, Paulo Lieuthier ___ kopete-devel mailing list kopete-devel@kde.org https://mail.kde.org/ma

Re: Review Request 127358: Resize 'New Account' dialog when changing page

2016-03-25 Thread Paulo Lieuthier
shown in the referenced bug report. The patch makes the wizzard have its size adjusted on chaging page. Diffs - kopete/addaccountwizard/addaccountwizard.cpp fd0506f Diff: https://git.reviewboard.kde.org/r/127358/diff/ Testing --- Thanks, Paulo Lieu

Re: Review Request 127358: Resize 'New Account' dialog when changing page

2016-03-15 Thread Paulo Lieuthier
sting --- Thanks, Paulo Lieuthier ___ kopete-devel mailing list kopete-devel@kde.org https://mail.kde.org/mailman/listinfo/kopete-devel

Re: Review Request 127358: Resize 'New Account' dialog when changing page

2016-03-14 Thread Paulo Lieuthier
disabling word wrap. I'll keep searching. - Paulo --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127358/#review93474 ------- On Mar

Re: Review Request 127358: Resize 'New Account' dialog when changing page

2016-03-13 Thread Paulo Lieuthier
have its size adjusted on chaging page. Diffs - kopete/addaccountwizard/addaccountwizard.cpp cc9fc8a Diff: https://git.reviewboard.kde.org/r/127358/diff/ Testing --- Thanks, Paulo Lieuthier ___ kopete-devel mailing list kopete-devel@kd

Re: Review Request 127357: Fix layout bug in jabber's service registration dialog

2016-03-13 Thread Paulo Lieuthier
t.cpp 5d3bd7c Diff: https://git.reviewboard.kde.org/r/127357/diff/ Testing --- Thanks, Paulo Lieuthier ___ kopete-devel mailing list kopete-devel@kde.org https://mail.kde.org/mailman/listinfo/kopete-devel