TextManager should not know anything about the SettingsManager

Author: MinyazevR

qrgui/mainWindow/mainWindow.cpp

@@ -342,6 +342,7 @@ void MainWindow::connectActions()
 	connect(mUi->tabs, &QTabWidget::currentChanged, this, &MainWindow::sceneSelectionChanged);
 	connect(&*mTextManager, &text::TextManager::textChanged, this, &MainWindow::setTextChanged);
 	connect(&*mTextManager, &text::TextManager::textChanged, mUi->actionUndo, &QAction::setEnabled);
+	connect(&*mTextManager, &text::TextManager::needRefreshRecentFileList, this, &MainWindow::refreshRecentFilesList);
 
 	connect(&*mProjectManager, &ProjectManager::afterOpen, mUi->paletteTree, &PaletteTree::refreshUserPalettes);
 	connect(&*mProjectManager, &ProjectManager::closed, mUi->paletteTree, &PaletteTree::refreshUserPalettes);
@@ -597,6 +598,31 @@ void MainWindow::refreshRecentProjectsList(const QString &fileName)
 	SettingsManager::setValue("recentProjects", previousString);
 }
 
+void MainWindow::refreshRecentFilesList(const QString &fileName) {
+	auto previousString = SettingsManager::value("recentFiles").toString();
+	auto previousList = previousString.split(";", QString::SkipEmptyParts);
+	previousList.removeOne(fileName);
+
+	if (!previousList.isEmpty() && (previousList.size() == mRecentFilesLimit)) {
+		previousList.removeLast();
+	}
+
+	previousString.clear();
+	if (mRecentFilesLimit > 0) {
+		previousList.push_front(fileName);
+		QStringListIterator iterator(previousList);
+		while (iterator.hasNext()) {
+			const auto recentFileName = iterator.next();
+			const QFileInfo fileInfo(recentFileName);
+			if (fileInfo.exists() && fileInfo.isFile()) {
+				previousString = previousString + recentFileName + ";";
+			}
+		}
+	}
+
+	SettingsManager::setValue("recentFiles", previousString);
+}
+
 void MainWindow::openRecentProjectsMenu()
 {
 	mRecentProjectsMenu->clear();

qrgui/mainWindow/mainWindow.h

@@ -195,6 +195,7 @@ public slots:
 	void closeStartTab();
 	void closeAllTabs();
 	void refreshRecentProjectsList(const QString &fileName);
+	void refreshRecentFilesList(const QString &fileName);
 	void createDiagram(const QString &idString);
 	/// Creates project with specified root diagram
 	bool createProject(const QString &diagramIdString);

qrgui/textEditor/textManager.cpp

@@ -30,8 +30,7 @@ using namespace qReal;
 using namespace text;
 
 TextManager::TextManager(SystemEvents &systemEvents, gui::MainWindowInterpretersInterface &mainWindow)
-	: mRecentFilesLimit(SettingsManager::value("recentFilesLimit").toInt())
-	, mMainWindow(mainWindow)
+	: mMainWindow(mainWindow)
 	, mSystemEvents(systemEvents)
 {
 	connect(&mSystemEvents, &SystemEvents::codeTabClosed, this, &TextManager::onTabClosed);
@@ -239,31 +238,6 @@ void TextManager::showInTextEditor(const QFileInfo &fileInfo
 	}
 }
 
-void TextManager::refreshRecentFilesList(const QString &fileName) {
-	auto previousString = SettingsManager::value("recentFiles").toString();
-	auto previousList = previousString.split(";", QString::SkipEmptyParts);
-	previousList.removeOne(fileName);
-
-	if (!previousList.isEmpty() && (previousList.size() == mRecentFilesLimit)) {
-		previousList.removeLast();
-	}
-
-	previousString.clear();
-	if (mRecentFilesLimit > 0) {
-		previousList.push_front(fileName);
-		QStringListIterator iterator(previousList);
-		while (iterator.hasNext()) {
-			const auto recentFileName = iterator.next();
-			const QFileInfo fileInfo(recentFileName);
-			if (fileInfo.exists() && fileInfo.isFile()) {
-				previousString = previousString + recentFileName + ";";
-			}
-		}
-	}
-
-	SettingsManager::setValue("recentFiles", previousString);
-}
-
 void TextManager::showInTextEditor(const QFileInfo &fileInfo, const text::LanguageInfo &language)
 {
 	Q_ASSERT(!fileInfo.completeBaseName().isEmpty());
@@ -280,7 +254,7 @@ void TextManager::showInTextEditor(const QFileInfo &fileInfo, const text::Langua
 		return;
 	}
 
-	refreshRecentFilesList(filePath);
+	emit needRefreshRecentFileList(filePath);
 	area->show();
 
 	// Need to bind diagram and code file only if code is just generated
@@ -331,7 +305,7 @@ bool TextManager::saveText(bool saveAs)
 			emit mSystemEvents.codePathChanged(diagram, path(area), fileInfo);
 		}
 
-		refreshRecentFilesList(fileInfo.absoluteFilePath());
+		emit needRefreshRecentFileList(fileInfo.absoluteFilePath());
 	}
 
 	return true;

qrgui/textEditor/textManager.h

@@ -17,7 +17,6 @@
 #include <QtCore/QObject>
 #include <QtCore/QMap>
 #include <QtCore/QMultiHash>
-#include <qrkernel/settingsManager.h>
 #include "qrgui/textEditor/textEditorDeclSpec.h"
 #include "qrgui/textEditor/textManagerInterface.h"
 #include "qrgui/textEditor/codeBlockManager.h"
@@ -82,14 +81,12 @@ private slots:
 	void onTabClosed(const QFileInfo &file);
 
 private:
-
-	void refreshRecentFilesList(const QString &fileName);
 	QMap<QString, text::QScintillaTextEdit*> mText; // No ownership
 	QMap<text::QScintillaTextEdit*, QString> mPath; // No ownership
 
 	/// If default path - true.
 	QMap<QString, bool> mPathType;
-	int mRecentFilesLimit {};
+
 	/// Contains names of generator, which generate each file
 	QMap<QString, QString> mGeneratorName;
 

qrgui/textEditor/textManagerInterface.h

@@ -56,6 +56,7 @@ class QRGUI_TEXT_EDITOR_EXPORT TextManagerInterface : public QObject
 
 signals:
 	void textChanged(text::QScintillaTextEdit *editor, bool changed);
+	void needRefreshRecentFileList(const QString &fileName);
 };
 
 }