D12811: Update to display the name of the session in the "Delete Session" confirmation dialogue box

[Aleix Pol Gonzalez]

apol requested changes to this revision.
apol added inline comments.
This revision now requires changes to proceed.

INLINE COMMENTS

> sessionchooserdialog.cpp:201
>      const QString uuid = m_model->data(index, Qt::DisplayRole).toString();
> -
> +    const Qstring sessionName = m_model->data(1, Qt::DisplayRole).toString();
> +    

You can't pass an index as a number. Does that even compile?

REPOSITORY
  R32 KDevelop

REVISION DETAIL
  https://phabricator.kde.org/D12811

To: sagnikchaudhuri, apol
Cc: apol, kdevelop-devel, antismap, iodelay, vbspam, njensen, geetamc, Pilzschaf, akshaydeo, surgenight, arrowdodger
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20180510/db7074a2/attachment.html>