<div dir="ltr">Sound perfect for me.<div><br></div><div>Note : I spare a lots of time to review file contents and adjust file tittles for these bugs. FOr me the most important is described in titles, independantly than all comments that you will found in reports.</div><div><br></div><div>For some files, the history do not give a real plus to follow the right way in your project. For others one, all is fine. To resume, if you have a doubt for a file ask me to confirm.</div><div><br></div><div>But in all cases, i think to have titled properly all reports for your project.</div><div><br></div><div>Gilles</div></div><div class="gmail_extra"><br><div class="gmail_quote">2015-05-19 23:36 GMT+02:00 Shourya Singh Gupta <span dir="ltr"><<a href="mailto:shouryasgupta@gmail.com" target="_blank">shouryasgupta@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Sir<br><br>Yes, I got mails about the relevant bug reports. While having a look at these bugs I noticed that quite many demand feature that is common to all web export plugins. So for these bugs what we can do is that when we make common classes in libkipiplugis, then in those classes we can create functions that has the required common capabilities that solves these bugs. So basically we will be solving these bugs at the time of common classes creation in libkipiplugins.<br><br>And the bugs that are specific to a particular plugin, I will fix them first before porting that plugin to KF5.<br><br>Sounds ok ?</div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Tue, May 19, 2015 at 9:09 PM, Gilles Caulier <span dir="ltr"><<a href="mailto:caulier.gilles@gmail.com" target="_blank">caulier.gilles@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Shourya,<div><br></div><div>I review all entries in bugzilla about kipiplugins.</div><div><br></div><div>Look all entries relevant more and less of your project this summer :</div><div><br></div><div><a href="https://bugs.kde.org/buglist.cgi?bug_status=UNCONFIRMED&bug_status=CONFIRMED&bug_status=ASSIGNED&bug_status=REOPENED&component=general&list_id=1264782&product=kipiplugins&query_format=advanced" target="_blank">https://bugs.kde.org/buglist.cgi?bug_status=UNCONFIRMED&bug_status=CONFIRMED&bug_status=ASSIGNED&bug_status=REOPENED&component=general&list_id=1264782&product=kipiplugins&query_format=advanced</a><br></div><div><br></div><div>We must talk about...</div><span><font color="#888888"><div><br></div><div>Gilles</div></font></span></div><div><div><div class="gmail_extra"><br><div class="gmail_quote">2015-05-19 12:40 GMT+02:00 Gilles Caulier <span dir="ltr"><<a href="mailto:caulier.gilles@gmail.com" target="_blank">caulier.gilles@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><span style="font-size:13px;white-space:nowrap;background-color:rgb(245,245,245)">Shourya,</span><br><div><span style="font-size:13px;white-space:nowrap;background-color:rgb(245,245,245)"><br></span></div><div><span style="font-size:13px;white-space:nowrap;background-color:rgb(245,245,245)">In bugzilla, i set in FlickrExport components a sets of files relevant of you project for this summer.</span></div><div><span style="font-size:13px;white-space:nowrap;background-color:rgb(245,245,245)">All use prefix "FACTORING : " in report title.</span></div><div><span style="font-size:13px;white-space:nowrap;background-color:rgb(245,245,245)"><br></span></div><div><span style="background-color:rgb(245,245,245)"><span style="white-space:nowrap"><a href="https://bugs.kde.org/buglist.cgi?bug_status=UNCONFIRMED&bug_status=CONFIRMED&bug_status=ASSIGNED&bug_status=REOPENED&list_id=1264641&product=kipiplugins&query_format=advanced&short_desc=FACTORING&short_desc_type=allwordssubstr" target="_blank">https://bugs.kde.org/buglist.cgi?bug_status=UNCONFIRMED&bug_status=CONFIRMED&bug_status=ASSIGNED&bug_status=REOPENED&list_id=1264641&product=kipiplugins&query_format=advanced&short_desc=FACTORING&short_desc_type=allwordssubstr</a></span><span><font color="#888888"><br></font></span></span></div><span><font color="#888888"><div><span style="background-color:rgb(245,245,245)"><span style="white-space:nowrap"><br></span></span></div><div><span style="background-color:rgb(245,245,245)"><span style="white-space:nowrap">Gilles</span></span></div><div><span style="font-size:13px;white-space:nowrap;background-color:rgb(245,245,245)"><br></span></div></font></span></div><div><div><div class="gmail_extra"><br><div class="gmail_quote">2015-05-19 0:30 GMT+02:00 Shourya Singh Gupta <span dir="ltr"><<a href="mailto:shouryasgupta@gmail.com" target="_blank">shouryasgupta@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hello all<br><br>Yes, I think that's a good suggestion by Alexander that while factorizing a common API in libkipiplugin for mpform using QHttpMultiPart can be created. I will keep in mind while doing the factorization. Also, it would be interesting to see whether it provides improvement in memory usage or not.</div><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, May 18, 2015 at 6:26 PM, Gilles Caulier <span dir="ltr"><<a href="mailto:caulier.gilles@gmail.com" target="_blank">caulier.gilles@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div>2015-05-18 14:44 GMT+02:00 Alexander Potashev <<a href="mailto:aspotashev@gmail.com" target="_blank">aspotashev@gmail.com</a>>:<br>
> Hi Gilles,<br>
><br>
> Please find my comments below.<br>
><br>
> 2015-05-18 9:04 GMT+03:00 Gilles Caulier <<a href="mailto:caulier.gilles@gmail.com" target="_blank">caulier.gilles@gmail.com</a>>:<br>
>> 2015-05-17 23:45 GMT+02:00 Alexander Potashev <<a href="mailto:aspotashev@gmail.com" target="_blank">aspotashev@gmail.com</a>>:<br>
>>> Another commit I have just pushed removes the files mpform.* in favor<br>
>>> of my own code for HTTP multi-part requests using<br>
>>> QHttpMultiPart/QNetworkAccessManager.<br>
>><br>
>> This way permit to be logged to a remote service without to use an<br>
>> external session of  web browser as it's done in all web service kipi<br>
>> tools ? If yes, this solution must be the good way to apply in a<br>
>> common class from internal libkipiplugins (not libkipi) to generalize<br>
>> this authentification method.<br>
>><br>
>> Remember that we have a student who will work on kipi-plugins core to<br>
>> factorize all web service tool codes.<br>
><br>
> No, it is not about authentication. The multi-part request is only<br>
> used to actually upload photos.<br>
><br>
> But there are more advantages in using QHttpMultiPart besides licensing:<br>
>  1. We reduce the amount of code.<br>
>  2. This way we do not prepare the whole request (possibly<br>
> multi-megabyte) in a QByteArray. In my indestanding,<br>
> QNetworkAccessManager reads the images from disk as soon as they are<br>
> ready to be uploaded, thus preventing peaks in memory usage. I did not<br>
> actually check the memory usage though.<br>
><br>
> It may be interesting to factorize the code around QHttpMultiPart I<br>
> added in libkvkontakte/photopostjob.cpp and move it in a handy<br>
> function/class in libkipiplugins. I have not done this myself yet,<br>
> mainly because I didn't take time to have a look at the other export<br>
> plugins using mpform.{cpp,h} to see what would be the best API for<br>
> this in libkipiplugins. May be it should be just a rewritten version<br>
> of class MPForm taking advantage of QHttpMultiPart.<br>
<br>
</div></div>Shourya,<br>
<br>
Didi you seen Alexander proposal ?<br>
<span><font color="#888888"><br>
Gilles<br>
</font></span></blockquote></div><br></div>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br></div>