24616639222

Committed 18 Apr 2026 11:50PM UTC coverage: 22.607%. Remained the same

Build # 24616639222

Build Type

push

github

Committed by

web-flow

Commit Message

Merge pull request #1052 from IJHack/fix/qt610-filter-change

Use beginFilterChange/endFilterChange for Qt 6.10+

Coverage Stats

1304 of 5768 relevant lines covered (22.61%)

8.52 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

51.61

/src/storemodel.cpp

// SPDX-FileCopyrightText: 2014 Anne Jan Brouwer
// SPDX-License-Identifier: GPL-3.0-or-later
#include "storemodel.h"
#include "qtpasssettings.h"

#include "util.h"
#include <QDebug>
#include <QFileSystemModel>
#include <QMessageBox>
#include <QMimeData>
#include <QRegularExpression>
#include <QtGlobal>
#include <utility>

auto operator<<(
    QDataStream &out,
    const dragAndDropInfoPasswordStore &dragAndDropInfoPasswordStore)
    -> QDataStream & {
  out << dragAndDropInfoPasswordStore.isDir
      << dragAndDropInfoPasswordStore.isFile
      << dragAndDropInfoPasswordStore.path;
  return out;
}

auto operator>>(QDataStream &in,
                dragAndDropInfoPasswordStore &dragAndDropInfoPasswordStore)
    -> QDataStream & {
  in >> dragAndDropInfoPasswordStore.isDir >>
      dragAndDropInfoPasswordStore.isFile >> dragAndDropInfoPasswordStore.path;
  return in;
}

/**
 * @brief StoreModel::StoreModel
 * SubClass of QSortFilterProxyModel via
 * http://www.qtcentre.org/threads/46471-QTreeView-Filter
 */
StoreModel::StoreModel() { fs = nullptr; }

/**
 * @brief StoreModel::filterAcceptsRow should row be shown, wrapper for
 * StoreModel::showThis method.
 * @param sourceRow
 * @param sourceParent
 * @return
 */
auto StoreModel::filterAcceptsRow(int sourceRow,
                                  const QModelIndex &sourceParent) const
    -> bool {
  QModelIndex index = sourceModel()->index(sourceRow, 0, sourceParent);
  return showThis(index);
}

/**
 * @brief StoreModel::showThis should a row be shown, based on our search
 * criteria.
 * @param index
 * @return
 */
auto StoreModel::showThis(const QModelIndex &index) const -> bool {
  bool retVal = false;
  if (fs == nullptr) {
    return retVal;
  }
  // Gives you the info for number of childs with a parent
  if (sourceModel()->rowCount(index) > 0) {
    for (int nChild = 0; nChild < sourceModel()->rowCount(index); ++nChild) {
      QModelIndex childIndex = sourceModel()->index(nChild, 0, index);
      if (!childIndex.isValid()) {
        break;
      }
      retVal = showThis(childIndex);
      if (retVal) {
        break;
      }
    }
  } else {
    QModelIndex useIndex = sourceModel()->index(index.row(), 0, index.parent());
    QString path = fs->filePath(useIndex);
    path = QDir(store).relativeFilePath(path);
    if (path.startsWith(".git")) {
      return false;
    }
    path.replace(Util::endsWithGpg(), "");
    retVal = path.contains(filterRegularExpression());
  }
  return retVal;
}

/**
 * @brief StoreModel::setModelAndStore update the source model and store.
 * @param sourceModel
 * @param passStore
 */
void StoreModel::setModelAndStore(QFileSystemModel *sourceModel,
                                  QString passStore) {
  setSourceModel(sourceModel);
  fs = sourceModel;
  store = std::move(passStore);
}

void StoreModel::setStore(const QString &passStore) {
#if QT_VERSION >= QT_VERSION_CHECK(6, 10, 0)
  beginFilterChange();
  store = passStore;
  endFilterChange();
#else
  store = passStore;
#if QT_VERSION >= QT_VERSION_CHECK(6, 5, 0)
  QSortFilterProxyModel::invalidateFilter();
#else
  invalidateFilter();
#endif
#endif
}

/**
 * @brief StoreModel::data don't show the .gpg at the end of a file.
 * @param index
 * @param role
 * @return
 */
auto StoreModel::data(const QModelIndex &index, int role) const -> QVariant {
  if (!index.isValid()) {
    return {};
  }

  QVariant initial_value;
  initial_value = QSortFilterProxyModel::data(index, role);

  if (role == Qt::DisplayRole) {
    QString name = initial_value.toString();
    name.replace(Util::endsWithGpg(), "");
    initial_value.setValue(name);
  }

  return initial_value;
}

/**
 * @brief StoreModel::supportedDropActions enable drop.
 * @return
 */
auto StoreModel::supportedDropActions() const -> Qt::DropActions {
  return Qt::CopyAction | Qt::MoveAction;
}

/**
 * @brief StoreModel::supportedDragActions enable drag.
 * @return
 */
auto StoreModel::supportedDragActions() const -> Qt::DropActions {
  return Qt::CopyAction | Qt::MoveAction;
}

/**
 * @brief StoreModel::flags
 * @param index
 * @return
 */
auto StoreModel::flags(const QModelIndex &index) const -> Qt::ItemFlags {
  Qt::ItemFlags defaultFlags = QSortFilterProxyModel::flags(index);

  if (index.isValid()) {
    return Qt::ItemIsDragEnabled | Qt::ItemIsDropEnabled | defaultFlags;
  }
  return Qt::ItemIsDropEnabled | defaultFlags;
}

/**
 * @brief StoreModel::mimeTypes
 * @return
 */
auto StoreModel::mimeTypes() const -> QStringList {
  QStringList types;
  types << "application/vnd+qtpass.dragAndDropInfoPasswordStore";
  return types;
}

/**
 * @brief StoreModel::mimeData
 * @param indexes
 * @return
 */
auto StoreModel::mimeData(const QModelIndexList &indexes) const -> QMimeData * {
  dragAndDropInfoPasswordStore info;

  QByteArray encodedData;
  // only use the first, otherwise we should enable multiselection
  QModelIndex index = indexes.at(0);
  if (index.isValid()) {
    QModelIndex useIndex = mapToSource(index);

    info.isDir = fs->fileInfo(useIndex).isDir();
    info.isFile = fs->fileInfo(useIndex).isFile();
    info.path = fs->fileInfo(useIndex).absoluteFilePath();
    QDataStream stream(&encodedData, QIODevice::WriteOnly);
    stream << info;
  }

  auto *mimeData = new QMimeData();
  mimeData->setData("application/vnd+qtpass.dragAndDropInfoPasswordStore",
                    encodedData);
  return mimeData;
}

/**
 * @brief StoreModel::canDropMimeData
 * @param data
 * @param action
 * @param row
 * @param column
 * @param parent
 * @return
 */
auto StoreModel::canDropMimeData(const QMimeData *data, Qt::DropAction action,
                                 int row, int column,
                                 const QModelIndex &parent) const -> bool {
#ifdef QT_DEBUG
  qDebug() << action << row;
#else
  Q_UNUSED(action)
  Q_UNUSED(row)
#endif

  if (data == nullptr ||
      !data->hasFormat("application/vnd+qtpass.dragAndDropInfoPasswordStore")) {
    return false;
  }

  QByteArray encodedData =
      data->data("application/vnd+qtpass.dragAndDropInfoPasswordStore");
  if (encodedData.isEmpty()) {
    return false;
  }
  QDataStream stream(&encodedData, QIODevice::ReadOnly);
  dragAndDropInfoPasswordStore info;
  stream >> info;
  if (stream.status() != QDataStream::Ok) {
    return false;
  }

  QModelIndex useIndex =
      this->index(parent.row(), parent.column(), parent.parent());

  if (column > 0) {
    return false;
  }

  // you can drop a folder on a folder
  if (fs->fileInfo(mapToSource(useIndex)).isDir() && info.isDir) {
    return true;
  }
  // you can drop a file on a folder
  if (fs->fileInfo(mapToSource(useIndex)).isDir() && info.isFile) {
    return true;
  }
  // you can drop a file on a file
  if (fs->fileInfo(mapToSource(useIndex)).isFile() && info.isFile) {
    return true;
  }

  return false;
}

/**
 * @brief StoreModel::dropMimeData
 * @param data
 * @param action
 * @param row
 * @param column
 * @param parent
 * @return
 */
auto StoreModel::dropMimeData(const QMimeData *data, Qt::DropAction action,
                              int row, int column, const QModelIndex &parent)
    -> bool {
  if (!canDropMimeData(data, action, row, column, parent)) {
    return false;
  }

  if (action == Qt::IgnoreAction) {
    return true;
  }
  QByteArray encodedData =
      data->data("application/vnd+qtpass.dragAndDropInfoPasswordStore");
  if (encodedData.isEmpty()) {
    return false;
  }
  QDataStream stream(&encodedData, QIODevice::ReadOnly);
  dragAndDropInfoPasswordStore info;
  stream >> info;
  if (stream.status() != QDataStream::Ok) {
    return false;
  }
  QModelIndex destIndex =
      this->index(parent.row(), parent.column(), parent.parent());
  QFileInfo destFileinfo = fs->fileInfo(mapToSource(destIndex));
  QFileInfo srcFileInfo = QFileInfo(info.path);
  QString cleanedSrc = QDir::cleanPath(srcFileInfo.absoluteFilePath());
  QString cleanedDest = QDir::cleanPath(destFileinfo.absoluteFilePath());
  if (info.isDir) {
    // dropped dir onto dir
    if (destFileinfo.isDir()) {
      QDir destDir = QDir(cleanedDest).filePath(srcFileInfo.fileName());
      QString cleanedDestDir = QDir::cleanPath(destDir.absolutePath());
      if (action == Qt::MoveAction) {
        QtPassSettings::getPass()->Move(cleanedSrc, cleanedDestDir);
      } else if (action == Qt::CopyAction) {
        QtPassSettings::getPass()->Copy(cleanedSrc, cleanedDestDir);
      }
    }
  } else if (info.isFile) {
    // dropped file onto a directory
    if (destFileinfo.isDir()) {
      if (action == Qt::MoveAction) {
        QtPassSettings::getPass()->Move(cleanedSrc, cleanedDest);
      } else if (action == Qt::CopyAction) {
        QtPassSettings::getPass()->Copy(cleanedSrc, cleanedDest);
      }
    } else if (destFileinfo.isFile()) {
      // dropped file onto a file
      int answer = QMessageBox::question(
          nullptr, tr("force overwrite?"),
          tr("overwrite %1 with %2?").arg(cleanedDest, cleanedSrc),
          QMessageBox::Yes | QMessageBox::No);
      bool force = answer == QMessageBox::Yes;
      if (action == Qt::MoveAction) {
        QtPassSettings::getPass()->Move(cleanedSrc, cleanedDest, force);
      } else if (action == Qt::CopyAction) {
        QtPassSettings::getPass()->Copy(cleanedSrc, cleanedDest, force);
      }
    }
  }
  return true;
}

/**
 * @brief StoreModel::lessThan
 * @param source_left
 * @param source_right
 * @return
 */
auto StoreModel::lessThan(const QModelIndex &source_left,
                          const QModelIndex &source_right) const -> bool {
/* matches logic in QFileSystemModelSorter::compareNodes() */
#ifndef Q_OS_MAC
  if (fs && (source_left.column() == 0 || source_left.column() == 1)) {
    bool leftD = fs->isDir(source_left);
    bool rightD = fs->isDir(source_right);

    if (leftD ^ rightD) {
      return leftD;
    }
  }
#endif

  return QSortFilterProxyModel::lessThan(source_left, source_right);
}

1	// SPDX-FileCopyrightText: 2014 Anne Jan Brouwer
2	// SPDX-License-Identifier: GPL-3.0-or-later
3	#include "storemodel.h"
4	#include "qtpasssettings.h"
5
6	#include "util.h"
7	#include <QDebug>
8	#include <QFileSystemModel>
9	#include <QMessageBox>
10	#include <QMimeData>
11	#include <QRegularExpression>
12	#include <QtGlobal>
13	#include <utility>
14
15	auto operator<<(	1✔
16	QDataStream &out,
17	const dragAndDropInfoPasswordStore &dragAndDropInfoPasswordStore)
18	-> QDataStream & {
19	out << dragAndDropInfoPasswordStore.isDir	1✔
20	<< dragAndDropInfoPasswordStore.isFile	1✔
21	<< dragAndDropInfoPasswordStore.path;	1✔
22	return out;	1✔
23	}
24
25	auto operator>>(QDataStream &in,	×
26	dragAndDropInfoPasswordStore &dragAndDropInfoPasswordStore)
27	-> QDataStream & {
28	in >> dragAndDropInfoPasswordStore.isDir >>	×
29	dragAndDropInfoPasswordStore.isFile >> dragAndDropInfoPasswordStore.path;	×
30	return in;	×
31	}
32
33	/**
34	* @brief StoreModel::StoreModel
35	* SubClass of QSortFilterProxyModel via
36	* http://www.qtcentre.org/threads/46471-QTreeView-Filter
37	*/
38	StoreModel::StoreModel() { fs = nullptr; }	19✔
39
40	/**
41	* @brief StoreModel::filterAcceptsRow should row be shown, wrapper for
42	* StoreModel::showThis method.
43	* @param sourceRow
44	* @param sourceParent
45	* @return
46	*/
47	auto StoreModel::filterAcceptsRow(int sourceRow,	19✔
48	const QModelIndex &sourceParent) const
49	-> bool {
50	QModelIndex index = sourceModel()->index(sourceRow, 0, sourceParent);	19✔
51	return showThis(index);	19✔
52	}
53
54	/**
55	* @brief StoreModel::showThis should a row be shown, based on our search
56	* criteria.
57	* @param index
58	* @return
59	*/
60	auto StoreModel::showThis(const QModelIndex &index) const -> bool {	44✔
61	bool retVal = false;
62	if (fs == nullptr) {	44✔
63	return retVal;
64	}
65	// Gives you the info for number of childs with a parent
66	if (sourceModel()->rowCount(index) > 0) {	43✔
67	for (int nChild = 0; nChild < sourceModel()->rowCount(index); ++nChild) {	24✔
68	QModelIndex childIndex = sourceModel()->index(nChild, 0, index);	24✔
69	if (!childIndex.isValid()) {
70	break;
71	}
72	retVal = showThis(childIndex);	24✔
73	if (retVal) {	24✔
74	break;
75	}
76	}
77	} else {
78	QModelIndex useIndex = sourceModel()->index(index.row(), 0, index.parent());	19✔
79	QString path = fs->filePath(useIndex);	19✔
80	path = QDir(store).relativeFilePath(path);	38✔
81	if (path.startsWith(".git")) {	38✔
82	return false;
83	}
84	path.replace(Util::endsWithGpg(), "");	18✔
85	retVal = path.contains(filterRegularExpression());	18✔
86	}
87	return retVal;
88	}
89
90	/**
91	* @brief StoreModel::setModelAndStore update the source model and store.
92	* @param sourceModel
93	* @param passStore
94	*/
95	void StoreModel::setModelAndStore(QFileSystemModel *sourceModel,	14✔
96	QString passStore) {
97	setSourceModel(sourceModel);	14✔
98	fs = sourceModel;	14✔
99	store = std::move(passStore);
100	}	14✔
101
102	void StoreModel::setStore(const QString &passStore) {	×
103	#if QT_VERSION >= QT_VERSION_CHECK(6, 10, 0)
104	beginFilterChange();
105	store = passStore;
106	endFilterChange();
107	#else
108	store = passStore;	×
109	#if QT_VERSION >= QT_VERSION_CHECK(6, 5, 0)
110	QSortFilterProxyModel::invalidateFilter();	×
111	#else
112	invalidateFilter();
113	#endif
114	#endif
115	}	×
116
117	/**
118	* @brief StoreModel::data don't show the .gpg at the end of a file.
119	* @param index
120	* @param role
121	* @return
122	*/
123	auto StoreModel::data(const QModelIndex &index, int role) const -> QVariant {	2✔
124	if (!index.isValid()) {
125	return {};
126	}
127
128	QVariant initial_value;
129	initial_value = QSortFilterProxyModel::data(index, role);	1✔
130
131	if (role == Qt::DisplayRole) {	1✔
132	QString name = initial_value.toString();	1✔
133	name.replace(Util::endsWithGpg(), "");	1✔
134	initial_value.setValue(name);	1✔
135	}
136
137	return initial_value;	1✔
138	}	1✔
139
140	/**
141	* @brief StoreModel::supportedDropActions enable drop.
142	* @return
143	*/
144	auto StoreModel::supportedDropActions() const -> Qt::DropActions {	1✔
145	return Qt::CopyAction \| Qt::MoveAction;	1✔
146	}
147
148	/**
149	* @brief StoreModel::supportedDragActions enable drag.
150	* @return
151	*/
152	auto StoreModel::supportedDragActions() const -> Qt::DropActions {	1✔
153	return Qt::CopyAction \| Qt::MoveAction;	1✔
154	}
155
156	/**
157	* @brief StoreModel::flags
158	* @param index
159	* @return
160	*/
161	auto StoreModel::flags(const QModelIndex &index) const -> Qt::ItemFlags {	2✔
162	Qt::ItemFlags defaultFlags = QSortFilterProxyModel::flags(index);	2✔
163
164	if (index.isValid()) {
165	return Qt::ItemIsDragEnabled \| Qt::ItemIsDropEnabled \| defaultFlags;
166	}
167	return Qt::ItemIsDropEnabled \| defaultFlags;	1✔
168	}
169
170	/**
171	* @brief StoreModel::mimeTypes
172	* @return
173	*/
174	auto StoreModel::mimeTypes() const -> QStringList {	1✔
175	QStringList types;	1✔
176	types << "application/vnd+qtpass.dragAndDropInfoPasswordStore";	1✔
177	return types;	1✔
178	}
179
180	/**
181	* @brief StoreModel::mimeData
182	* @param indexes
183	* @return
184	*/
185	auto StoreModel::mimeData(const QModelIndexList &indexes) const -> QMimeData * {	1✔
186	dragAndDropInfoPasswordStore info;
187
188	QByteArray encodedData;	1✔
189	// only use the first, otherwise we should enable multiselection
190	QModelIndex index = indexes.at(0);	1✔
191	if (index.isValid()) {
192	QModelIndex useIndex = mapToSource(index);	1✔
193
194	info.isDir = fs->fileInfo(useIndex).isDir();	1✔
195	info.isFile = fs->fileInfo(useIndex).isFile();	1✔
196	info.path = fs->fileInfo(useIndex).absoluteFilePath();	2✔
197	QDataStream stream(&encodedData, QIODevice::WriteOnly);	1✔
198	stream << info;	1✔
199	}	1✔
200
201	auto *mimeData = new QMimeData();	1✔
202	mimeData->setData("application/vnd+qtpass.dragAndDropInfoPasswordStore",	2✔
203	encodedData);
204	return mimeData;	1✔
205	}
206
207	/**
208	* @brief StoreModel::canDropMimeData
209	* @param data
210	* @param action
211	* @param row
212	* @param column
213	* @param parent
214	* @return
215	*/
216	auto StoreModel::canDropMimeData(const QMimeData *data, Qt::DropAction action,	×
217	int row, int column,
218	const QModelIndex &parent) const -> bool {
219	#ifdef QT_DEBUG
220	qDebug() << action << row;
221	#else
222	Q_UNUSED(action)
223	Q_UNUSED(row)
224	#endif
225
226	if (data == nullptr \|\|	×
227	!data->hasFormat("application/vnd+qtpass.dragAndDropInfoPasswordStore")) {	×
228	return false;
229	}
230
231	QByteArray encodedData =
232	data->data("application/vnd+qtpass.dragAndDropInfoPasswordStore");	×
233	if (encodedData.isEmpty()) {	×
234	return false;
235	}
236	QDataStream stream(&encodedData, QIODevice::ReadOnly);	×
237	dragAndDropInfoPasswordStore info;
238	stream >> info;	×
239	if (stream.status() != QDataStream::Ok) {	×
240	return false;
241	}
242
243	QModelIndex useIndex =
244	this->index(parent.row(), parent.column(), parent.parent());	×
245
246	if (column > 0) {	×
247	return false;
248	}
249
250	// you can drop a folder on a folder
251	if (fs->fileInfo(mapToSource(useIndex)).isDir() && info.isDir) {	×
252	return true;
253	}
254	// you can drop a file on a folder
255	if (fs->fileInfo(mapToSource(useIndex)).isDir() && info.isFile) {	×
256	return true;
257	}
258	// you can drop a file on a file
259	if (fs->fileInfo(mapToSource(useIndex)).isFile() && info.isFile) {	×
260	return true;
261	}
262
263	return false;
264	}	×
265
266	/**
267	* @brief StoreModel::dropMimeData
268	* @param data
269	* @param action
270	* @param row
271	* @param column
272	* @param parent
273	* @return
274	*/
275	auto StoreModel::dropMimeData(const QMimeData *data, Qt::DropAction action,	×
276	int row, int column, const QModelIndex &parent)
277	-> bool {
278	if (!canDropMimeData(data, action, row, column, parent)) {	×
279	return false;
280	}
281
282	if (action == Qt::IgnoreAction) {	×
283	return true;
284	}
285	QByteArray encodedData =
286	data->data("application/vnd+qtpass.dragAndDropInfoPasswordStore");	×
287	if (encodedData.isEmpty()) {	×
288	return false;
289	}
290	QDataStream stream(&encodedData, QIODevice::ReadOnly);	×
291	dragAndDropInfoPasswordStore info;
292	stream >> info;	×
293	if (stream.status() != QDataStream::Ok) {	×
294	return false;
295	}
296	QModelIndex destIndex =
297	this->index(parent.row(), parent.column(), parent.parent());	×
298	QFileInfo destFileinfo = fs->fileInfo(mapToSource(destIndex));	×
299	QFileInfo srcFileInfo = QFileInfo(info.path);	×
300	QString cleanedSrc = QDir::cleanPath(srcFileInfo.absoluteFilePath());	×
301	QString cleanedDest = QDir::cleanPath(destFileinfo.absoluteFilePath());	×
302	if (info.isDir) {	×
303	// dropped dir onto dir
304	if (destFileinfo.isDir()) {	×
305	QDir destDir = QDir(cleanedDest).filePath(srcFileInfo.fileName());	×
306	QString cleanedDestDir = QDir::cleanPath(destDir.absolutePath());	×
307	if (action == Qt::MoveAction) {	×
308	QtPassSettings::getPass()->Move(cleanedSrc, cleanedDestDir);	×
309	} else if (action == Qt::CopyAction) {	×
310	QtPassSettings::getPass()->Copy(cleanedSrc, cleanedDestDir);	×
311	}
312	}	×
313	} else if (info.isFile) {	×
314	// dropped file onto a directory
315	if (destFileinfo.isDir()) {	×
316	if (action == Qt::MoveAction) {	×
317	QtPassSettings::getPass()->Move(cleanedSrc, cleanedDest);	×
318	} else if (action == Qt::CopyAction) {	×
319	QtPassSettings::getPass()->Copy(cleanedSrc, cleanedDest);	×
320	}
321	} else if (destFileinfo.isFile()) {	×
322	// dropped file onto a file
323	int answer = QMessageBox::question(	×
324	nullptr, tr("force overwrite?"),	×
325	tr("overwrite %1 with %2?").arg(cleanedDest, cleanedSrc),	×
326	QMessageBox::Yes \| QMessageBox::No);
327	bool force = answer == QMessageBox::Yes;	×
328	if (action == Qt::MoveAction) {	×
329	QtPassSettings::getPass()->Move(cleanedSrc, cleanedDest, force);	×
330	} else if (action == Qt::CopyAction) {	×
331	QtPassSettings::getPass()->Copy(cleanedSrc, cleanedDest, force);	×
332	}
333	}
334	}
335	return true;
336	}	×
337
338	/**
339	* @brief StoreModel::lessThan
340	* @param source_left
341	* @param source_right
342	* @return
343	*/
344	auto StoreModel::lessThan(const QModelIndex &source_left,	2✔
345	const QModelIndex &source_right) const -> bool {
346	/* matches logic in QFileSystemModelSorter::compareNodes() */
347	#ifndef Q_OS_MAC
348	if (fs && (source_left.column() == 0 \|\| source_left.column() == 1)) {	2✔
349	bool leftD = fs->isDir(source_left);	2✔
350	bool rightD = fs->isDir(source_right);	2✔
351
352	if (leftD ^ rightD) {	2✔
353	return leftD;
354	}
355	}
356	#endif
357
358	return QSortFilterProxyModel::lessThan(source_left, source_right);	1✔
359	}

IJHack / QtPass / 24616639222

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous