X-Git-Url: https://git.lyx.org/gitweb/?a=blobdiff_plain;f=src%2Ffrontends%2Fqt4%2FGuiErrorList.cpp;h=f104cd2ffba1461ff9213a1ff6d639fe4b6aafc4;hb=425d092204118ea6c24c28e85fdf03fcf2bb51a4;hp=9fe844cc6cef7086582e29d9a82f2ad772e60f13;hpb=ba76bf5eb85db5a10839fccee3430d906d3f7b70;p=lyx.git

diff --git a/src/frontends/qt4/GuiErrorList.cpp b/src/frontends/qt4/GuiErrorList.cpp
index 9fe844cc6c..f104cd2ffb 100644
--- a/src/frontends/qt4/GuiErrorList.cpp
+++ b/src/frontends/qt4/GuiErrorList.cpp
@@ -12,20 +12,22 @@
 
 #include "GuiErrorList.h"
 
+#include "qt_helpers.h"
+
 #include "Buffer.h"
 #include "BufferView.h"
-#include "support/debug.h"
-#include "support/gettext.h"
-#include "Text.h"
+#include "FuncRequest.h"
 #include "ParIterator.h"
+#include "Text.h"
 
-#include "qt_helpers.h"
-
+#include "support/debug.h"
+#include "support/gettext.h"
 #include "support/lstrings.h"
 
 #include <QListWidget>
-#include <QTextBrowser>
 #include <QPushButton>
+#include <QShowEvent>
+#include <QTextBrowser>
 
 using namespace std;
 using namespace lyx::support;
@@ -40,8 +42,10 @@ GuiErrorList::GuiErrorList(GuiView & lv)
 
 	connect(closePB, SIGNAL(clicked()),
 		this, SLOT(slotClose()));
-	connect(errorsLW, SIGNAL(itemClicked(QListWidgetItem *)),
-		this, SLOT(select(QListWidgetItem *)));
+	connect(viewLogPB, SIGNAL(clicked()),
+		this, SLOT(viewLog()));
+	connect(errorsLW, SIGNAL(currentRowChanged(int)),
+		this, SLOT(select()));
 
 	bc().setPolicy(ButtonPolicy::OkCancelPolicy);
 	bc().setCancel(closePB);
@@ -50,21 +54,30 @@ GuiErrorList::GuiErrorList(GuiView & lv)
 
 void GuiErrorList::showEvent(QShowEvent * e)
 {
-	errorsLW->setCurrentRow(0);
-	select(errorsLW->item(0));
+	select();
+	paramsToDialog();
 	e->accept();
 }
 
 
-void GuiErrorList::select(QListWidgetItem * wi)
+void GuiErrorList::select()
 {
-	int const item = errorsLW->row(wi);
+	int const item = errorsLW->row(errorsLW->currentItem());
+	if (item == -1)
+		return;
 	goTo(item);
-	descriptionTB->setPlainText(toqstr(errorList()[item].description));
+	descriptionTB->setPlainText(
+		toqstr(errorList()[item].description));
+}
+
+
+void GuiErrorList::viewLog()
+{
+	dispatch(FuncRequest(LFUN_DIALOG_SHOW, "latexlog"));
 }
 
 
-void GuiErrorList::updateContents()
+void GuiErrorList::paramsToDialog()
 {
 	setTitle(toqstr(name_));
 	errorsLW->clear();
@@ -74,38 +87,54 @@ void GuiErrorList::updateContents()
 	ErrorList::const_iterator end = errorList().end();
 	for (; it != end; ++it)
 		errorsLW->addItem(toqstr(it->error));
+	errorsLW->setCurrentRow(0);
 }
 
 
 ErrorList const & GuiErrorList::errorList() const
 {
-	return bufferview()->buffer().errorList(error_type_);
+	return from_master_ ? 
+		bufferview()->buffer().masterBuffer()->errorList(error_type_)
+		: bufferview()->buffer().errorList(error_type_);
 }
 
 
-bool GuiErrorList::initialiseParams(string const & error_type)
+bool GuiErrorList::initialiseParams(string const & data)
 {
+	from_master_ = prefixIs(data, "from_master|");
+	string error_type = data;
+	if (from_master_)
+		error_type = split(data, '|');
 	error_type_ = error_type;
-	Buffer const & buf = bufferview()->buffer();
+	Buffer const * buf = from_master_ ?
+		bufferview()->buffer().masterBuffer()
+		: &bufferview()->buffer();
 	name_ = bformat(_("%1$s Errors (%2$s)"), _(error_type),
-				     from_utf8(buf.absFileName()));
+				     from_utf8(buf->absFileName()));
+	paramsToDialog();
 	return true;
 }
 
 
-void GuiErrorList::goTo(int item)
+bool GuiErrorList::goTo(int item)
 {
 	ErrorItem const & err = errorList()[item];
 
 	if (err.par_id == -1)
-		return;
+		return false;
+
+	if (from_master_)
+		// FIXME: implement
+		return false;
 
-	Buffer & buf = buffer();
+	Buffer const & buf = buffer();
 	DocIterator dit = buf.getParFromID(err.par_id);
 
-	if (dit == doc_iterator_end(buf.inset())) {
+	if (dit == doc_iterator_end(&buf)) {
+        // FIXME: Happens when loading a read-only doc with 
+        // unknown layout. Should this be the case?
 		LYXERR0("par id " << err.par_id << " not found");
-		return;
+		return false;
 	}
 
 	// Now make the selection.
@@ -115,9 +144,11 @@ void GuiErrorList::goTo(int item)
 	pos_type const start = min(err.pos_start, end);
 	pos_type const range = end - start;
 	dit.pos() = start;
-	bufferview()->putSelectionAt(dit, range, false);
+	BufferView * bv = const_cast<BufferView *>(bufferview());
 	// FIXME: If we used an LFUN, we would not need this line:
-	bufferview()->processUpdateFlags(Update::Force | Update::FitCursor);
+	bv->putSelectionAt(dit, range, false);
+	bv->processUpdateFlags(Update::Force | Update::FitCursor);
+	return true;
 }
 
 
@@ -127,4 +158,4 @@ Dialog * createGuiErrorList(GuiView & lv) { return new GuiErrorList(lv); }
 } // namespace lyx
 
 
-#include "GuiErrorList_moc.cpp"
+#include "moc_GuiErrorList.cpp"