Revert "NOISSUE rework of minecraft log"

This reverts commit fc198dd3085a2cd33fbaa7a3d2c95c2c8d3ee31a.

3 files changed, 179 insertions, 239 deletions

diff --git a/application/pages/LogPage.cpp b/application/pages/LogPage.cpp
index 0a76f8b3..de4ed4f3 100644
--- a/application/pages/LogPage.cpp
+++ b/application/pages/LogPage.cpp
@@ -12,113 +12,39 @@
 #include "GuiUtil.h"
 #include <ColorCache.h>
 
-class LogFormatProxyModel : public QIdentityProxyModel
+LogPage::LogPage(InstancePtr instance, QWidget *parent)
+	: QWidget(parent), ui(new Ui::LogPage), m_instance(instance)
 {
-public:
-	LogFormatProxyModel(QObject* parent = nullptr) : QIdentityProxyModel(parent)
-	{
-	}
-	QVariant data(const QModelIndex &index, int role) const override
-	{
-		switch(role)
-		{
-			case Qt::FontRole:
-				return m_font;
-			case Qt::TextColorRole:
-			{
-				MessageLevel::Enum level = (MessageLevel::Enum) QIdentityProxyModel::data(index, LogModel::LevelRole).toInt();
-				return m_colors->getFront(level);
-			}
-			case Qt::BackgroundRole:
-			{
-				MessageLevel::Enum level = (MessageLevel::Enum) QIdentityProxyModel::data(index, LogModel::LevelRole).toInt();
-				return m_colors->getBack(level);
-			}
-			default:
-				return QIdentityProxyModel::data(index, role);
-			}
-	}
-
-	void setFont(QFont font)
-	{
-		m_font = font;
-	}
+	ui->setupUi(this);
+	ui->tabWidget->tabBar()->hide();
 
-	void setColors(LogColorCache* colors)
+	// create the format and set its font
 	{
-		m_colors.reset(colors);
+		defaultFormat = new QTextCharFormat(ui->text->currentCharFormat());
+		QString fontFamily = MMC->settings()->get("ConsoleFont").toString();
+		bool conversionOk = false;
+		int fontSize = MMC->settings()->get("ConsoleFontSize").toInt(&conversionOk);
+		if(!conversionOk)
+		{
+			fontSize = 11;
+		}
+		defaultFormat->setFont(QFont(fontFamily, fontSize));
 	}
 
-	QModelIndex find(const QModelIndex &start, const QString &value, bool reverse) const
+	// ensure we don't eat all the RAM
 	{
-		QModelIndex parentIndex = parent(start);
-		auto compare = [&](int r) -> QModelIndex
-		{
-			QModelIndex idx = index(r, start.column(), parentIndex);
-			if (!idx.isValid() || idx == start)
-			{
-				return QModelIndex();
-			}
-			QVariant v = data(idx, Qt::DisplayRole);
-			QString t = v.toString();
-			if (t.contains(value, Qt::CaseInsensitive))
-				return idx;
-			return QModelIndex();
-		};
-		if(reverse)
+		auto lineSetting = MMC->settings()->getSetting("ConsoleMaxLines");
+		bool conversionOk = false;
+		int maxLines = lineSetting->get().toInt(&conversionOk);
+		if(!conversionOk)
 		{
-			int from = start.row();
-			int to = 0;
-
-			for (int i = 0; i < 2; ++i)
-			{
-				for (int r = from; (r >= to); --r)
-				{
-					auto idx = compare(r);
-					if(idx.isValid())
-						return idx;
-				}
-				// prepare for the next iteration
-				from = rowCount() - 1;
-				to = start.row();
-			}
+			maxLines = lineSetting->defValue().toInt();
+			qWarning() << "ConsoleMaxLines has nonsensical value, defaulting to" << maxLines;
 		}
-		else
-		{
-			int from = start.row();
-			int to = rowCount(parentIndex);
+		ui->text->setMaximumBlockCount(maxLines);
 
-			for (int i = 0; i < 2; ++i)
-			{
-				for (int r = from; (r < to); ++r)
-				{
-					auto idx = compare(r);
-					if(idx.isValid())
-						return idx;
-				}
-				// prepare for the next iteration
-				from = 0;
-				to = start.row();
-			}
-		}
-		return QModelIndex();
+		m_stopOnOverflow = MMC->settings()->get("ConsoleOverflowStop").toBool();
 	}
-private:
-	QFont m_font;
-	std::unique_ptr<LogColorCache> m_colors;
-};
-
-LogPage::LogPage(InstancePtr instance, QWidget *parent)
-	: QWidget(parent), ui(new Ui::LogPage), m_instance(instance)
-{
-	ui->setupUi(this);
-	ui->tabWidget->tabBar()->hide();
-
-	m_proxy = new LogFormatProxyModel(this);
-	connect(m_proxy, &QAbstractItemModel::rowsAboutToBeInserted, this, &LogPage::rowsAboutToBeInserted);
-	connect(m_proxy, &QAbstractItemModel::rowsInserted, this, &LogPage::rowsInserted);
-
-	ui->textView->setModel(m_proxy);
 
 	// set up instance and launch process recognition
 	{
@@ -131,28 +57,13 @@ LogPage::LogPage(InstancePtr instance, QWidget *parent)
 			this, &LogPage::on_InstanceLaunchTask_changed);
 	}
 
-	// set up text colors in the log proxy and adapt them to the current theme foreground and background
-	{
-		auto origForeground = ui->textView->palette().color(ui->textView->foregroundRole());
-		auto origBackground = ui->textView->palette().color(ui->textView->backgroundRole());
-		m_proxy->setColors(new LogColorCache(origForeground, origBackground));
-	}
-
-	// set up fonts in the log proxy
+	// set up text colors and adapt them to the current theme foreground and background
 	{
-		QString fontFamily = MMC->settings()->get("ConsoleFont").toString();
-		bool conversionOk = false;
-		int fontSize = MMC->settings()->get("ConsoleFontSize").toInt(&conversionOk);
-		if(!conversionOk)
-		{
-			fontSize = 11;
-		}
-		m_proxy->setFont(QFont(fontFamily, fontSize));
+		auto origForeground = ui->text->palette().color(ui->text->foregroundRole());
+		auto origBackground = ui->text->palette().color(ui->text->backgroundRole());
+		m_colors.reset(new LogColorCache(origForeground, origBackground));
 	}
 
-	ui->textView->setWordWrap(true);
-	ui->textView->setHorizontalScrollBarPolicy(Qt::ScrollBarAlwaysOff);
-
 	auto findShortcut = new QShortcut(QKeySequence(QKeySequence::Find), this);
 	connect(findShortcut, SIGNAL(activated()), SLOT(findActivated()));
 	auto findNextShortcut = new QShortcut(QKeySequence(QKeySequence::FindNext), this);
@@ -165,33 +76,20 @@ LogPage::LogPage(InstancePtr instance, QWidget *parent)
 LogPage::~LogPage()
 {
 	delete ui;
+	delete defaultFormat;
 }
 
 void LogPage::on_InstanceLaunchTask_changed(std::shared_ptr<LaunchTask> proc)
 {
-	m_process = proc;
 	if(m_process)
 	{
-		m_model = proc->getLogModel();
-		auto lineSetting = MMC->settings()->getSetting("ConsoleMaxLines");
-		bool conversionOk = false;
-		int maxLines = lineSetting->get().toInt(&conversionOk);
-		if(!conversionOk)
-		{
-			maxLines = lineSetting->defValue().toInt();
-			qWarning() << "ConsoleMaxLines has nonsensical value, defaulting to" << maxLines;
-		}
-		m_model->setMaxLines(maxLines);
-		m_model->setStopOnOverflow(MMC->settings()->get("ConsoleOverflowStop").toBool());
-		m_model->setOverflowMessage(tr("MultiMC stopped watching the game log because the log length surpassed %1 lines.\n"
-			"You may have to fix your mods because the game is still loggging to files and"
-			" likely wasting harddrive space at an alarming rate!").arg(maxLines));
-		m_proxy->setSourceModel(m_model.get());
+		disconnect(m_process.get(), &LaunchTask::log, this, &LogPage::write);
 	}
-	else
+	m_process = proc;
+	if(m_process)
 	{
-		m_proxy->setSourceModel(nullptr);
-		m_model.reset();
+		ui->text->clear();
+		connect(m_process.get(), &LaunchTask::log, this, &LogPage::write);
 	}
 }
 
@@ -202,50 +100,38 @@ bool LogPage::apply()
 
 bool LogPage::shouldDisplay() const
 {
-	return m_instance->isRunning() || m_proxy->rowCount() > 0;
+	return m_instance->isRunning() || ui->text->blockCount() > 1;
 }
 
 void LogPage::on_btnPaste_clicked()
 {
-	if(!m_model)
-		return;
-
 	//FIXME: turn this into a proper task and move the upload logic out of GuiUtil!
-	m_model->append(MessageLevel::MultiMC, tr("MultiMC: Log upload triggered at: %1").arg(QDateTime::currentDateTime().toString(Qt::RFC2822Date)));
-	auto url = GuiUtil::uploadPaste(m_model->toPlainText(), this);
+	write(tr("MultiMC: Log upload triggered at: %1").arg(QDateTime::currentDateTime().toString(Qt::RFC2822Date)), MessageLevel::MultiMC);
+	auto url = GuiUtil::uploadPaste(ui->text->toPlainText(), this);
 	if(!url.isEmpty())
 	{
-		m_model->append(MessageLevel::MultiMC, tr("MultiMC: Log uploaded to: %1").arg(url));
+		write(tr("MultiMC: Log uploaded to: %1").arg(url), MessageLevel::MultiMC);
 	}
 	else
 	{
-		m_model->append(MessageLevel::Error, tr("MultiMC: Log upload failed!"));
+		write(tr("MultiMC: Log upload failed!"), MessageLevel::Error);
 	}
 }
 
 void LogPage::on_btnCopy_clicked()
 {
-	if(!m_model)
-		return;
-	m_model->append(MessageLevel::MultiMC, QString("Clipboard copy at: %1").arg(QDateTime::currentDateTime().toString(Qt::RFC2822Date)));
-	GuiUtil::setClipboardText(m_model->toPlainText());
+	write(QString("Clipboard copy at: %1").arg(QDateTime::currentDateTime().toString(Qt::RFC2822Date)), MessageLevel::MultiMC);
+	GuiUtil::setClipboardText(ui->text->toPlainText());
 }
 
 void LogPage::on_btnClear_clicked()
 {
-	if(!m_model)
-		return;
-	m_model->clear();
+	ui->text->clear();
 }
 
 void LogPage::on_btnBottom_clicked()
 {
-	/*
-	ui->textView->verticalScrollBar()->setSliderPosition(ui->textView->verticalScrollBar()->maximum());
-	*/
-	auto numRows = m_proxy->rowCount(QModelIndex());
-	auto lastIndex = m_proxy->index(numRows - 1, 0 , QModelIndex());
-	ui->textView->scrollTo(lastIndex, QAbstractItemView::ScrollHint::EnsureVisible);
+	ui->text->verticalScrollBar()->setSliderPosition(ui->text->verticalScrollBar()->maximum());
 }
 
 void LogPage::on_trackLogCheckbox_clicked(bool checked)
@@ -257,61 +143,58 @@ void LogPage::on_wrapCheckbox_clicked(bool checked)
 {
 	if(checked)
 	{
-		ui->textView->setWordWrap(true);
-		ui->textView->setHorizontalScrollBarPolicy(Qt::ScrollBarAlwaysOff);
+		ui->text->setWordWrapMode(QTextOption::WrapAtWordBoundaryOrAnywhere);
 	}
 	else
 	{
-		ui->textView->setWordWrap(false);
-		ui->textView->setHorizontalScrollBarPolicy(Qt::ScrollBarAsNeeded);
+		ui->text->setWordWrapMode(QTextOption::WrapMode::NoWrap);
 	}
 }
 
-void LogPage::findImpl(bool reverse)
+void LogPage::on_findButton_clicked()
 {
-	auto toSearch = ui->searchBar->text();
-	if (toSearch.size())
+	auto modifiers = QApplication::keyboardModifiers();
+	if (modifiers & Qt::ShiftModifier)
 	{
-		auto index = ui->textView->currentIndex();
-		if(!index.isValid())
-		{
-			index = m_proxy->index(0,0);
-		}
-		if(!index.isValid())
-		{
-			// just give up
-			return;
-		}
-		auto found = m_proxy->find(index, toSearch, reverse);
-		if(found.isValid())
-			ui->textView->setCurrentIndex(found);
+		findPreviousActivated();
+	}
+	else
+	{
+		findNextActivated();
 	}
 }
 
-void LogPage::on_findButton_clicked()
+void LogPage::findActivated()
 {
-	auto modifiers = QApplication::keyboardModifiers();
-	bool reverse = modifiers & Qt::ShiftModifier;
-	findImpl(reverse);
+	// focus the search bar if it doesn't have focus
+	if (!ui->searchBar->hasFocus())
+	{
+		auto searchForCursor = ui->text->textCursor();
+		auto searchForString = searchForCursor.selectedText();
+		if (searchForString.size())
+		{
+			ui->searchBar->setText(searchForString);
+		}
+		ui->searchBar->setFocus();
+		ui->searchBar->selectAll();
+	}
 }
 
 void LogPage::findNextActivated()
 {
-	findImpl(false);
+	auto toSearch = ui->searchBar->text();
+	if (toSearch.size())
+	{
+		ui->text->find(toSearch);
+	}
 }
 
 void LogPage::findPreviousActivated()
 {
-	findImpl(true);
-}
-
-void LogPage::findActivated()
-{
-	// focus the search bar if it doesn't have focus
-	if (!ui->searchBar->hasFocus())
+	auto toSearch = ui->searchBar->text();
+	if (toSearch.size())
 	{
-		ui->searchBar->setFocus();
-		ui->searchBar->selectAll();
+		ui->text->find(toSearch, QTextDocument::FindBackward);
 	}
 }
 
@@ -320,19 +203,84 @@ void LogPage::setParentContainer(BasePageContainer * container)
 	m_parentContainer = container;
 }
 
-void LogPage::rowsAboutToBeInserted(const QModelIndex& parent, int first, int last)
+void LogPage::write(QString data, MessageLevel::Enum mode)
 {
-	auto numRows = m_proxy->rowCount(QModelIndex());
-	auto lastIndex = m_proxy->index(numRows - 1, 0 , QModelIndex());
-	auto rect = ui->textView->visualRect(lastIndex);
-	auto viewPortRect = ui->textView->viewport()->rect();
-	m_autoScroll = rect.intersects(viewPortRect);
-}
+	if (!m_write_active)
+	{
+		if (mode != MessageLevel::MultiMC)
+		{
+			return;
+		}
+	}
+	if(m_stopOnOverflow && m_write_active)
+	{
+		if(mode != MessageLevel::MultiMC)
+		{
+			if(ui->text->blockCount() >= ui->text->maximumBlockCount())
+			{
+				m_write_active = false;
+				data = tr("MultiMC stopped watching the game log because the log length surpassed %1 lines.\n"
+					"You may have to fix your mods because the game is still loggging to files and"
+						" likely wasting harddrive space at an alarming rate!")
+							.arg(ui->text->maximumBlockCount());
+				mode = MessageLevel::Fatal;
+				ui->trackLogCheckbox->setCheckState(Qt::Unchecked);
+				if(!isVisible())
+				{
+					m_parentContainer->selectPage(id());
+				}
+			}
+		}
+	}
 
-void LogPage::rowsInserted(const QModelIndex& parent, int first, int last)
-{
-	if(m_autoScroll)
+	// save the cursor so it can be restored.
+	auto savedCursor = ui->text->cursor();
+
+	QScrollBar *bar = ui->text->verticalScrollBar();
+	int max_bar = bar->maximum();
+	int val_bar = bar->value();
+	if (isVisible())
+	{
+		if (m_scroll_active)
+		{
+			m_scroll_active = (max_bar - val_bar) <= 1;
+		}
+		else
+		{
+			m_scroll_active = val_bar == max_bar;
+		}
+	}
+	if (data.endsWith('\n'))
+		data = data.left(data.length() - 1);
+	QStringList paragraphs = data.split('\n');
+	QStringList filtered;
+	for (QString &paragraph : paragraphs)
 	{
-		QMetaObject::invokeMethod(this, "on_btnBottom_clicked", Qt::QueuedConnection);
+		//TODO: implement filtering here.
+		filtered.append(paragraph);
+	}
+	QListIterator<QString> iter(filtered);
+	QTextCharFormat format(*defaultFormat);
+
+	format.setForeground(m_colors->getFront(mode));
+	format.setBackground(m_colors->getBack(mode));
+
+	while (iter.hasNext())
+	{
+		// append a paragraph/line
+		auto workCursor = ui->text->textCursor();
+		workCursor.movePosition(QTextCursor::End);
+		workCursor.insertText(iter.next(), format);
+		workCursor.insertBlock();
+	}
+
+	if (isVisible())
+	{
+		if (m_scroll_active)
+		{
+			bar->setValue(bar->maximum());
+		}
+		m_last_scroll_value = bar->value();
 	}
+	ui->text->setCursor(savedCursor);
 }
diff --git a/application/pages/LogPage.h b/application/pages/LogPage.h
index 71e23977..e902ad13 100644
--- a/application/pages/LogPage.h
+++ b/application/pages/LogPage.h
@@ -21,13 +21,13 @@
 #include "launch/LaunchTask.h"
 #include "BasePage.h"
 #include <MultiMC.h>
+#include <ColorCache.h>
 
 namespace Ui
 {
 class LogPage;
 }
 class QTextCharFormat;
-class LogFormatProxyModel;
 
 class LogPage : public QWidget, public BasePage
 {
@@ -57,6 +57,13 @@ public:
 	virtual void setParentContainer(BasePageContainer *) override;
 
 private slots:
+	/**
+	 * @brief write a string
+	 * @param data the string
+	 * @param level the @MessageLevel the string should be written under
+	 * lines have to be put through this as a whole!
+	 */
+	void write(QString data, MessageLevel::Enum level = MessageLevel::MultiMC);
 	void on_btnPaste_clicked();
 	void on_btnCopy_clicked();
 	void on_btnClear_clicked();
@@ -72,12 +79,6 @@ private slots:
 
 	void on_InstanceLaunchTask_changed(std::shared_ptr<LaunchTask> proc);
 
-	void rowsAboutToBeInserted(const QModelIndex &parent, int first, int last);
-	void rowsInserted(const QModelIndex &parent, int first, int last);
-
-private: /* methods */
-	void findImpl(bool reverse);
-
 private:
 	Ui::LogPage *ui;
 	InstancePtr m_instance;
@@ -87,9 +88,8 @@ private:
 	int m_saved_offset = 0;
 	bool m_write_active = true;
 	bool m_stopOnOverflow = true;
-	bool m_autoScroll = false;
 
+	QTextCharFormat * defaultFormat;
 	BasePageContainer * m_parentContainer;
-	LogFormatProxyModel * m_proxy;
-	shared_qobject_ptr <LogModel> m_model;
+	std::unique_ptr<LogColorCache> m_colors;
 };
diff --git a/application/pages/LogPage.ui b/application/pages/LogPage.ui
index 18fb654b..bf5ec004 100644
--- a/application/pages/LogPage.ui
+++ b/application/pages/LogPage.ui
@@ -33,6 +33,25 @@
        <string notr="true">Tab 1</string>
       </attribute>
       <layout class="QGridLayout" name="gridLayout">
+       <item row="1" column="0" colspan="5">
+        <widget class="QPlainTextEdit" name="text">
+         <property name="undoRedoEnabled">
+          <bool>false</bool>
+         </property>
+         <property name="readOnly">
+          <bool>true</bool>
+         </property>
+         <property name="plainText">
+          <string notr="true"/>
+         </property>
+         <property name="textInteractionFlags">
+          <set>Qt::LinksAccessibleByKeyboard|Qt::LinksAccessibleByMouse|Qt::TextBrowserInteraction|Qt::TextSelectableByKeyboard|Qt::TextSelectableByMouse</set>
+         </property>
+         <property name="centerOnScroll">
+          <bool>false</bool>
+         </property>
+        </widget>
+       </item>
        <item row="0" column="0" colspan="5">
         <layout class="QHBoxLayout" name="horizontalLayout">
          <item>
@@ -134,34 +153,6 @@
          </property>
         </widget>
        </item>
-       <item row="1" column="0" colspan="5">
-        <widget class="QListView" name="textView">
-         <property name="verticalScrollBarPolicy">
-          <enum>Qt::ScrollBarAlwaysOn</enum>
-         </property>
-         <property name="horizontalScrollBarPolicy">
-          <enum>Qt::ScrollBarAlwaysOff</enum>
-         </property>
-         <property name="showDropIndicator" stdset="0">
-          <bool>false</bool>
-         </property>
-         <property name="alternatingRowColors">
-          <bool>true</bool>
-         </property>
-         <property name="selectionMode">
-          <enum>QAbstractItemView::ExtendedSelection</enum>
-         </property>
-         <property name="selectionBehavior">
-          <enum>QAbstractItemView::SelectRows</enum>
-         </property>
-         <property name="verticalScrollMode">
-          <enum>QAbstractItemView::ScrollPerPixel</enum>
-         </property>
-         <property name="wordWrap">
-          <bool>true</bool>
-         </property>
-        </widget>
-       </item>
       </layout>
      </widget>
     </widget>
@@ -175,6 +166,7 @@
   <tabstop>btnCopy</tabstop>
   <tabstop>btnPaste</tabstop>
   <tabstop>btnClear</tabstop>
+  <tabstop>text</tabstop>
   <tabstop>searchBar</tabstop>
   <tabstop>findButton</tabstop>
  </tabstops>