> On Jan. 22, 2014, 11:42 p.m., David Edmundson wrote: > > src/plugins/skype/skypedatasource.cpp, line 187 > > <https://git.reviewboard.kde.org/r/115233/diff/1/?file=235301#file235301line187> > > > > it's better to use the SQL query API for doing this with .prepare and > > .bindValue() as that does automatic escaping. > > > > > >
Can I insert column names as-is, or I should perform QSqlDriver::escapeIdentifier(QSqlDriver::FieldName) as well? - Alexandr ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115233/#review48065 ----------------------------------------------------------- On Jan. 22, 2014, 11:19 p.m., Alexandr Akulich wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/115233/ > ----------------------------------------------------------- > > (Updated Jan. 22, 2014, 11:19 p.m.) > > > Review request for Telepathy, David Edmundson and Martin Klapetek. > > > Repository: libkpeople > > > Description > ------- > > Initial version of SkypeDataSource plugin. > UI for account selection is not yet implemented, so in current state plugin > doesn't works. (You can edit skypedatasource.cpp:67 and manualy set your > account) > > > Diffs > ----- > > src/plugins/CMakeLists.txt 3633676 > src/plugins/skype/CMakeLists.txt PRE-CREATION > src/plugins/skype/skype_kpeople_plugin.desktop PRE-CREATION > src/plugins/skype/skypedatasource.h PRE-CREATION > src/plugins/skype/skypedatasource.cpp PRE-CREATION > > Diff: https://git.reviewboard.kde.org/r/115233/diff/ > > > Testing > ------- > > > Thanks, > > Alexandr Akulich > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
