Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AutoSave Feature #490

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 8 additions & 4 deletions .github/workflows/codacy.yml
Original file line number Diff line number Diff line change
Expand Up @@ -53,13 +53,17 @@ jobs:
# This will handover control about PR rejection to the GitHub side
max-allowed-issues: 2147483647

- name: Filter out MISRA
# Filter out MISRA and potentially reduce the number of runs in the SARIF output
- name: Filter and Reduce SARIF
run: |
pip install globber
python3 scripts/filter_sarif.py --input results.sarif --output filtered.sarif --split-lines -- "-**/*.*:cppcheck_misra*"
python3 scripts/filter_and_reduce_sarif.py --input results.sarif --output filtered_and_reduced.sarif
env:
PYTHONUNBUFFERED: 1

# Upload the SARIF file generated in the previous step

# Upload the SARIF file generated in the previous step
- name: Upload SARIF results file
uses: github/codeql-action/upload-sarif@v3
with:
sarif_file: filtered.sarif
sarif_file: filtered_and_reduced.sarif
42 changes: 41 additions & 1 deletion avogadro/mainwindow.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1979,6 +1979,46 @@ void MainWindow::buildMenu()
m_menuBuilder->addAction(path, action, 960);
m_fileToolBar->addAction(action);
connect(action, SIGNAL(triggered()), SLOT(saveFileAs()));
// Initialize autosave feature
m_autosaveInterval = 5; // Autosave interval in minutes
m_autosaveTimer = new QTimer(this);
connect(m_autosaveTimer, &QTimer::timeout, this,
&MainWindow::autosaveDocument);
m_autosaveTimer->start(m_autosaveInterval *
60000); // Convert minutes to milliseconds

void MainWindow::autosaveDocument()
{
if (!m_molecule || !m_moleculeDirty) {
return; // No molecule loaded or no changes made since the last save.
}

QString autosaveDirPath =
QStandardPaths::writableLocation(QStandardPaths::AppDataLocation) +
"/autosave";
QDir autosaveDir(autosaveDirPath);
if (!autosaveDir.exists()) {
autosaveDir.mkpath(".");
}

// Construct autosave file name
QString autosaveFilename;
if (m_molecule->hasData("fileName")) {
QFileInfo fileInfo(m_molecule->data("fileName").toString().c_str());
autosaveFilename = fileInfo.baseName() + "_autosave.cjson";
} else {
autosaveFilename = "unsaved_autosave.cjson";
}
QString autosaveFilePath = autosaveDirPath + "/" + autosaveFilename;

// Use CJSON format for autosaving
Io::CjsonFormat writer;
if (!writer.writeFile(autosaveFilePath, *m_molecule)) {
qWarning() << "Failed to autosave the document to" << autosaveFilePath;
} else {
qDebug() << "Document autosaved to" << autosaveFilePath;
}
}

// Export action for menu
QStringList exportPath = path;
Expand Down Expand Up @@ -2635,4 +2675,4 @@ bool MainWindow::handleCommand(const QString& command,
return false;
}

} // End of Avogadro namespace
} // End of Avogadro namespace
7 changes: 4 additions & 3 deletions avogadro/mainwindow.h
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@ class MainWindow : public QMainWindow

public slots:
void setMolecule(Avogadro::QtGui::Molecule* molecule);

void autosaveDocument(); // line to declare the autosave slot
/**
* Update internal state to reflect that the molecule has been modified.
*/
Expand Down Expand Up @@ -398,7 +398,8 @@ private slots:
QtGui::ScenePlugin* m_activeScenePlugin;
bool m_queuedFilesStarted;
QStringList m_queuedFiles;

QTimer* m_autosaveTimer; // for the autosave timer
int m_autosaveInterval; // for autosave interval in minutes
QStringList m_recentFiles;
QList<QAction*> m_actionRecentFiles;

Expand Down Expand Up @@ -502,4 +503,4 @@ private slots:

} // End Avogadro namespace

#endif
#endif