if (book->creators.size()) {
QLabel *creators = new QLabel(this);
creators->setWordWrap(true);
- QString c = "By " + book->creators[0];
- for (int i = 1; i < book->creators.size(); i++) {
- c += ", " + book->creators[i];
- }
- creators->setText(c);
+ creators->setText(book->creators.join(", "));
addWidget(creators);
}
QLabel *path = new QLabel("File: " + book->path(), this);
#include "progressdialog.h"
#include "settings.h"
-LibraryDialog::LibraryDialog(QWidget *parent): ListWindow(tr("(No books)"), parent)
+LibraryDialog::LibraryDialog(QWidget *parent):
+ ListWindow(tr("(No books)"), parent)
{
TRACE;
setWindowTitle(tr("Library"));
setAttribute(Qt::WA_DeleteOnClose, true);
// Add menu actions
- sortByTitle = addMenuAction(tr("Sort by title"), this, SLOT(onSortByTitle()));
+ sortByTitle =
+ addMenuAction(tr("Sort by title"), this, SLOT(onSortByTitle()));
sortByAuthor =
addMenuAction(tr("Sort by author"), this, SLOT(onSortByAuthor()));
if (title.size()) {
QString ret = title;
if (creators.length()) {
- ret += "\nBy " + creators[0];
- for (int i = 1; i < creators.length(); i++) {
- ret += ", " + creators[i];
- }
+ ret += "\nBy " + creators.join(", ");
}
return ret;
}
class OpsHandler: public XmlHandler
{
public:
- OpsHandler(Book &b): book(b), partCount(0) {}
+ OpsHandler(Book &b): book(b), partCount(0) {
+ book.creators.clear();
+ }
bool endElement(const QString &namespaceUri, const QString &name,
const QString &qName) {
return QString::localeAwareCompare(left->creators.join(" "),
right->creators.join(" "));
default:
- return QString::localeAwareCompare(left->shortName(), right->shortName());
+ return QString::localeAwareCompare(left->shortName(),
+ right->shortName());
}
}
* Have an overlap between current/next page
* Add tool bars to list windows on Symbian
+ * Fix author name in libary and book info
-- Akos Polster <akos@pipacs.com> Sun, 5 Dec 2010 02:00:00 +0100