<div dir="ltr">Wait is better...<div><br></div><div style>Gilles Caulier</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/3/4 Saurabh Patel <span dir="ltr"><<a href="mailto:saurabhpatel7717@gmail.com" target="_blank">saurabhpatel7717@gmail.com</a>></span><br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello,<br>    So what should be done.TODO should be removed or wait for the reply of one who coded<br><br>Thanks<div class="HOEnZb">

<div class="h5"><br><br><div class="gmail_quote">On Mon, Mar 4, 2013 at 5:45 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">I think <b style="font-family:arial,sans-serif;font-size:13px">m_talker->loggedIn() is a method to check if talker is already logged (as it return bool).</b><div>


<b style="font-family:arial,sans-serif;font-size:13px"><br>

</b></div><div><b style="font-family:arial,sans-serif;font-size:13px">I don't coded this tool, so i'm not sure...</b></div><div><b style="font-family:arial,sans-serif;font-size:13px"><br></b></div><div>

<b style="font-family:arial,sans-serif;font-size:13px">As well, for me, code commented can be removed...</b></div><div><b style="font-family:arial,sans-serif;font-size:13px"><br></b></div><div><b style="font-family:arial,sans-serif;font-size:13px">Gilles Caulier</b></div>




</div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/3/4 Saurabh Patel <span dir="ltr"><<a href="mailto:saurabhpatel7717@gmail.com" target="_blank">saurabhpatel7717@gmail.com</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<div><div>

Hello,<br>    I was going through kipi-plugins in digikam.In kipi-plugin for picasawebexport I found a TODO on line 320 in picasawebwindow.cpp.I am pasting the whole method here.<br><br>void PicasawebWindow::slotLoginDone(int errCode, const QString& errMsg)<br>





{<br>    m_widget->progressBar()->hide();<br><br>   <b> if (errCode == 0 /*TODO&& m_talker->loggedIn()*/)</b><br>    {<br>        buttonStateChange(true);<br>        m_token = m_talker->token();<br>        m_widget->m_albumsCoB->clear();<br>





        m_talker->listAlbums(m_talker->getUserName());<br>    }<br>    else<br>    {<br>        KMessageBox::error(this, i18n("Picasaweb Call Failed: %1\n", errMsg));<br>    }<br>}<br><br><br>I have a question that the method loggedIn is not defined in picasawebtalker so is it to be implemented there.Also is it necessary since signallogindone signal in picasawebtalker is emitted after checking success to get username and password <br>





<br>Thanks<br>
<br></div></div>_______________________________________________<br>
Digikam-devel mailing list<br>
<a href="mailto:Digikam-devel@kde.org" target="_blank">Digikam-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/digikam-devel" target="_blank">https://mail.kde.org/mailman/listinfo/digikam-devel</a><br>
<br></blockquote></div><br></div>
<br>_______________________________________________<br>
Digikam-devel mailing list<br>
<a href="mailto:Digikam-devel@kde.org" target="_blank">Digikam-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/digikam-devel" target="_blank">https://mail.kde.org/mailman/listinfo/digikam-devel</a><br>
<br></blockquote></div><br>
</div></div><br>_______________________________________________<br>
Digikam-devel mailing list<br>
<a href="mailto:Digikam-devel@kde.org">Digikam-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/digikam-devel" target="_blank">https://mail.kde.org/mailman/listinfo/digikam-devel</a><br>
<br></blockquote></div><br></div>