diff --git a/src/ExportData.cpp b/src/ExportData.cpp index 23784c5..c5a1044 100644 --- a/src/ExportData.cpp +++ b/src/ExportData.cpp @@ -39,7 +39,8 @@ bool ExportData::rowShouldBeSkipped(const QAbstractItemView& view, int row) (QAbstractItemView::MultiSelection == view.selectionMode())}; const QItemSelectionModel* selectionModel{view.selectionModel()}; - return multiSelection && !selectionModel->isSelected(model->index(row, 0)); + return multiSelection && + (!selectionModel->isSelected(model->index(row, 0))); } QByteArray ExportData::getContentEnding() { return {}; } diff --git a/src/ExportXlsx.cpp b/src/ExportXlsx.cpp index 504ffae..6d8806c 100644 --- a/src/ExportXlsx.cpp +++ b/src/ExportXlsx.cpp @@ -111,8 +111,8 @@ QByteArray ExportXlsx::generateRowContent(const QAbstractItemModel& model, rowContent.append(ROW_NUMBER_CLOSE); rowContent.append(getCellTypeTag(cell)); rowContent.append(VALUE_START); - if (cell.typeId() == QMetaType::QDate || - cell.typeId() == QMetaType::QDateTime) + if ((cell.typeId() == QMetaType::QDate) || + (cell.typeId() == QMetaType::QDateTime)) rowContent.append(QByteArray::number( -1 * cell.toDate().daysTo(utilities::getStartOfExcelWorld()))); else diff --git a/src/ImportOds.cpp b/src/ImportOds.cpp index 4508703..e845612 100644 --- a/src/ImportOds.cpp +++ b/src/ImportOds.cpp @@ -219,20 +219,22 @@ std::pair ImportOds::retrieveColumnNames( xmlStreamReader.setDevice(&zipFile); skipToSheet(xmlStreamReader, sheetName); - while (!xmlStreamReader.atEnd() && xmlStreamReader.name() != TABLE_ROW_TAG) + while ((!xmlStreamReader.atEnd()) && + (xmlStreamReader.name() != TABLE_ROW_TAG)) xmlStreamReader.readNext(); xmlStreamReader.readNext(); QXmlStreamReader::TokenType lastToken{xmlStreamReader.tokenType()}; QStringList columnNames; - while (!xmlStreamReader.atEnd() && xmlStreamReader.name() != TABLE_ROW_TAG) + while (!xmlStreamReader.atEnd() && + (xmlStreamReader.name() != TABLE_ROW_TAG)) { if (isCellStart(xmlStreamReader) && - getColumnRepeatCount(xmlStreamReader.attributes()) > 1) + (getColumnRepeatCount(xmlStreamReader.attributes()) > 1)) break; - if (xmlStreamReader.name().toString() == P_TAG && + if ((xmlStreamReader.name().toString() == P_TAG) && xmlStreamReader.isStartElement()) { while (xmlStreamReader.tokenType() != QXmlStreamReader::Characters) @@ -241,7 +243,7 @@ std::pair ImportOds::retrieveColumnNames( } if (isCellEnd(xmlStreamReader) && - lastToken == QXmlStreamReader::StartElement) + (lastToken == QXmlStreamReader::StartElement)) columnNames << emptyColName_; lastToken = xmlStreamReader.tokenType(); @@ -355,12 +357,12 @@ bool ImportOds::isRecognizedColumnType( const QString columnType{ attributes.value(OFFICE_VALUE_TYPE_TAG).toString()}; - return 0 == columnType.compare(STRING_TAG) || - 0 == columnType.compare(DATE_TAG) || - 0 == columnType.compare(FLOAT_TAG) || - 0 == columnType.compare(PERCENTAGE_TAG) || - 0 == columnType.compare(CURRENCY_TAG) || - 0 == columnType.compare(TIME_TAG); + return (0 == columnType.compare(STRING_TAG)) || + (0 == columnType.compare(DATE_TAG)) || + (0 == columnType.compare(FLOAT_TAG)) || + (0 == columnType.compare(PERCENTAGE_TAG)) || + (0 == columnType.compare(CURRENCY_TAG)) || + (0 == columnType.compare(TIME_TAG)); } unsigned int ImportOds::getColumnRepeatCount( @@ -372,25 +374,25 @@ unsigned int ImportOds::getColumnRepeatCount( bool ImportOds::isRowStart(const QXmlStreamReader& xmlStreamReader) const { - return 0 == xmlStreamReader.name().compare(TABLE_ROW_TAG) && + return (0 == xmlStreamReader.name().compare(TABLE_ROW_TAG)) && xmlStreamReader.isStartElement(); } bool ImportOds::isRowEnd(const QXmlStreamReader& xmlStreamReader) const { - return 0 == xmlStreamReader.name().compare(TABLE_ROW_TAG) && + return (0 == xmlStreamReader.name().compare(TABLE_ROW_TAG)) && xmlStreamReader.isEndElement(); } bool ImportOds::isCellStart(const QXmlStreamReader& xmlStreamReader) const { - return 0 == xmlStreamReader.name().compare(TABLE_CELL_TAG) && + return (0 == xmlStreamReader.name().compare(TABLE_CELL_TAG)) && xmlStreamReader.isStartElement(); } bool ImportOds::isCellEnd(const QXmlStreamReader& xmlStreamReader) const { - return xmlStreamReader.name().toString() == TABLE_CELL_TAG && + return (xmlStreamReader.name().toString() == TABLE_CELL_TAG) && xmlStreamReader.isEndElement(); } @@ -413,10 +415,10 @@ ColumnType ImportOds::recognizeColumnType(ColumnType currentType, return ColumnType::STRING; } - if (0 == xmlColTypeValue.compare(FLOAT_TAG) || - 0 == xmlColTypeValue.compare(PERCENTAGE_TAG) || - 0 == xmlColTypeValue.compare(CURRENCY_TAG) || - 0 == xmlColTypeValue.compare(TIME_TAG)) + if ((0 == xmlColTypeValue.compare(FLOAT_TAG)) || + (0 == xmlColTypeValue.compare(PERCENTAGE_TAG)) || + (0 == xmlColTypeValue.compare(CURRENCY_TAG)) || + (0 == xmlColTypeValue.compare(TIME_TAG))) { if (currentType == ColumnType::UNKNOWN) return ColumnType::NUMBER;