X-Git-Url: https://git.lyx.org/gitweb/?a=blobdiff_plain;f=src%2Ffrontends%2Fqt4%2FGuiCitation.cpp;h=9da6f1016d3d3da83e432006114a2d5dbcba5e7a;hb=72f91cb394e180419cd3ab85f710b8ad538208f9;hp=7b355ef9bb6dfc5bdcbdeca4ecc52cdd94fea784;hpb=a17926e67300c3fc76d3222cd6cc76538a15c33b;p=lyx.git

diff --git a/src/frontends/qt4/GuiCitation.cpp b/src/frontends/qt4/GuiCitation.cpp
index 7b355ef9bb..9da6f1016d 100644
--- a/src/frontends/qt4/GuiCitation.cpp
+++ b/src/frontends/qt4/GuiCitation.cpp
@@ -1,3 +1,4 @@
+
 /**
  * \file GuiCitation.cpp
  * This file is part of LyX, the document processor.
@@ -15,32 +16,46 @@
 
 #include "GuiCitation.h"
 
-#include "debug.h"
-#include "gettext.h"
-#include "frontend_helpers.h"
+#include "GuiSelectionManager.h"
 #include "qt_helpers.h"
 
-#include "support/lstrings.h"
+#include "Buffer.h"
+#include "BiblioInfo.h"
+#include "BufferParams.h"
+#include "FuncRequest.h"
+
+#include "insets/InsetCommand.h"
+
+#include "support/debug.h"
 #include "support/docstring.h"
+#include "support/gettext.h"
+#include "support/lstrings.h"
+
+#include <QCloseEvent>
+#include <QSettings>
+#include <QShowEvent>
+#include <QVariant>
 
 #include <vector>
 #include <string>
 
-#include <QCloseEvent>
-
 #undef KeyPress
 
+#include "support/regex.h"
+
 #include <algorithm>
 #include <string>
 #include <vector>
 
-using std::vector;
-using std::string;
-
+using namespace std;
+using namespace lyx::support;
 
 namespace lyx {
 namespace frontend {
 
+static vector<CiteStyle> citeStyles_;
+
+
 template<typename String>
 static QStringList to_qstring_list(vector<String> const & v)
 {
@@ -67,66 +82,61 @@ static vector<lyx::docstring> to_docstring_vector(QStringList const & qlist)
 }
 
 
-GuiCitationDialog::GuiCitationDialog(LyXView & lv)
-	: GuiDialog(lv, "citation")
+GuiCitation::GuiCitation(GuiView & lv)
+	: DialogView(lv, "citation", qt_("Citation")),
+	  params_(insetCode("citation"))
 {
 	setupUi(this);
-	setViewTitle(_("Citation"));
-	setController(new ControlCitation(*this));
 
 	connect(citationStyleCO, SIGNAL(activated(int)),
-		this, SLOT(changed()));
+		this, SLOT(on_citationStyleCO_currentIndexChanged(int)));
 	connect(fulllistCB, SIGNAL(clicked()),
 		this, SLOT(changed()));
 	connect(forceuppercaseCB, SIGNAL(clicked()),
 		this, SLOT(changed()));
-	connect(textBeforeED, SIGNAL(textChanged(const QString&)),
+	connect(textBeforeED, SIGNAL(textChanged(QString)),
 		this, SLOT(changed()));
-	connect(textAfterED, SIGNAL(textChanged(const QString&)),
+	connect(textAfterED, SIGNAL(textChanged(QString)),
 		this, SLOT(changed()));
-	connect(clearPB, SIGNAL(clicked()),
-		findLE, SLOT(clear()));
-	connect(this, SIGNAL(rejected()), this, SLOT(cleanUp()));
-
-	selectionManager = 
-		new GuiSelectionManager(availableLV, selectedLV, 
-			addPB, deletePB, upPB, downPB, available(), selected());
+	connect(findLE, SIGNAL(returnPressed()), 
+		this, SLOT(on_searchPB_clicked()));
+	connect(textBeforeED, SIGNAL(returnPressed()),
+		this, SLOT(on_okPB_clicked()));
+	connect(textAfterED, SIGNAL(returnPressed()),
+		this, SLOT(on_okPB_clicked()));
+
+	selectionManager = new GuiSelectionManager(availableLV, selectedLV, 
+			addPB, deletePB, upPB, downPB, &available_model_, &selected_model_);
 	connect(selectionManager, SIGNAL(selectionChanged()),
 		this, SLOT(setCitedKeys()));
 	connect(selectionManager, SIGNAL(updateHook()),
-		this, SLOT(updateDialog()));
+		this, SLOT(updateControls()));
 	connect(selectionManager, SIGNAL(okHook()),
-					this, SLOT(on_okPB_clicked()));
-
-	bc().setPolicy(ButtonPolicy::NoRepeatedApplyReadOnlyPolicy);
-}
+		this, SLOT(on_okPB_clicked()));
 
+	setFocusProxy(availableLV);
 
-ControlCitation & GuiCitationDialog::controller() const
-{
-	return static_cast<ControlCitation &>(Dialog::controller());
+	// FIXME: the sizeHint() for this is _way_ too high
+	infoML->setFixedHeight(60);
 }
 
 
-void GuiCitationDialog::cleanUp() 
+GuiCitation::~GuiCitation()
 {
-	clearSelection();
-	controller().clearParams();
-	close();
+	delete selectionManager;
 }
 
 
-void GuiCitationDialog::closeEvent(QCloseEvent * e)
+void GuiCitation::closeEvent(QCloseEvent * e)
 {
 	clearSelection();
-	controller().clearParams();
-	e->accept();
+	DialogView::closeEvent(e);
 }
 
 
-void GuiCitationDialog::applyView()
+void GuiCitation::applyView()
 {
-	int  const choice = std::max(0, citationStyleCO->currentIndex());
+	int const choice = max(0, citationStyleCO->currentIndex());
 	style_ = choice;
 	bool const full  = fulllistCB->isChecked();
 	bool const force = forceuppercaseCB->isChecked();
@@ -138,62 +148,45 @@ void GuiCitationDialog::applyView()
 }
 
 
-void GuiCitationDialog::hideView()
-{
-	controller().clearParams();
-	accept();
-}
-
-
-void GuiCitationDialog::showView()
+void GuiCitation::showEvent(QShowEvent * e)
 {
 	findLE->clear();
 	availableLV->setFocus();
-	QDialog::show();
-	raise();
-	activateWindow();
-}
-
-
-bool GuiCitationDialog::isVisibleView() const
-{
-	return QDialog::isVisible();
+	DialogView::showEvent(e);
 }
 
 
-void GuiCitationDialog::on_okPB_clicked()
+void GuiCitation::on_okPB_clicked()
 {
 	applyView();
 	clearSelection();
-	hideView();
+	hide();
 }
 
 
-void GuiCitationDialog::on_cancelPB_clicked()
+void GuiCitation::on_cancelPB_clicked()
 {
 	clearSelection();
-	hideView();
+	hide();
 }
 
 
-void GuiCitationDialog::on_applyPB_clicked()
+void GuiCitation::on_applyPB_clicked()
 {
 	applyView();
 }
 
 
-void GuiCitationDialog::on_restorePB_clicked()
+void GuiCitation::on_restorePB_clicked()
 {
 	init();
-	updateView();
 }
 
 
-void GuiCitationDialog::updateView()
+void GuiCitation::updateControls()
 {
-	fillFields();
-	fillEntries();
-	updateDialog();
+	BiblioInfo const & bi = bibInfo();
+	updateControls(bi);
 }
 
 
@@ -201,125 +194,132 @@ void GuiCitationDialog::updateView()
 // called in update() do not need to be called for INTERNAL updates,
 // such as when addPB is pressed, as the list of fields, entries, etc,
 // will not have changed. At the moment, however, the division between
-// fillStyles() and updateStyles() doesn't lend itself to dividing the
-// two methods, though they should be divisible.
-void GuiCitationDialog::updateDialog()
-{
-	if (selectionManager->selectedFocused()) { 
-		if (selectedLV->selectionModel()->selectedIndexes().isEmpty())
-			updateInfo(availableLV->currentIndex());
-		else
-			updateInfo(selectedLV->currentIndex());
-	} else {
-		if (availableLV->selectionModel()->selectedIndexes().isEmpty())
-			updateInfo(QModelIndex());
-		else
-			updateInfo(availableLV->currentIndex());
-	}
+// fillStyles() and updateStyle() doesn't lend itself to dividing the
+// two methods, though they should be divisible. That is, we should
+// not have to call fillStyles() every time through here.
+void GuiCitation::updateControls(BiblioInfo const & bi)
+{
+	QModelIndex idx = selectionManager->getSelectedIndex();
+	updateInfo(bi, idx);
 	setButtons();
 
-	textBeforeED->setText(textBefore());
-	textAfterED->setText(textAfter());
-	fillStyles();
+	textBeforeED->setText(toqstr(params_["before"]));
+	textAfterED->setText(toqstr(params_["after"]));
+	fillStyles(bi);
 	updateStyle();
 }
 
 
-void GuiCitationDialog::updateStyle()
+void GuiCitation::updateFormatting(CiteStyle currentStyle)
 {
-	biblio::CiteEngine const engine = controller().getEngine();
+	CiteEngine const engine = citeEngine();
 	bool const natbib_engine =
-		engine == biblio::ENGINE_NATBIB_AUTHORYEAR ||
-		engine == biblio::ENGINE_NATBIB_NUMERICAL;
-	bool const basic_engine = engine == biblio::ENGINE_BASIC;
+		engine == ENGINE_NATBIB_AUTHORYEAR ||
+		engine == ENGINE_NATBIB_NUMERICAL;
+	bool const basic_engine = engine == ENGINE_BASIC;
 
 	bool const haveSelection = 
 		selectedLV->model()->rowCount() > 0;
-	fulllistCB->setEnabled(natbib_engine && haveSelection);
-	forceuppercaseCB->setEnabled(natbib_engine && haveSelection);
-	textBeforeED->setEnabled(!basic_engine && haveSelection);
-	textBeforeLA->setEnabled(!basic_engine && haveSelection);
-	textAfterED->setEnabled(haveSelection);
-	textAfterLA->setEnabled(haveSelection);
-	citationStyleCO->setEnabled(!basic_engine && haveSelection);
-	citationStyleLA->setEnabled(!basic_engine && haveSelection);
 
-	string const & command = controller().params().getCmdName();
+	bool const isNocite = currentStyle == NOCITE;
+
+	bool const isCiteyear =
+		currentStyle == CITEYEAR ||
+		currentStyle == CITEYEARPAR;
+
+	fulllistCB->setEnabled(natbib_engine && haveSelection && !isNocite
+		&& !isCiteyear);
+	forceuppercaseCB->setEnabled(natbib_engine && haveSelection
+		&& !isNocite && !isCiteyear);
+	textBeforeED->setEnabled(!basic_engine && haveSelection && !isNocite);
+	textBeforeLA->setEnabled(!basic_engine && haveSelection && !isNocite);
+	textAfterED->setEnabled(haveSelection && !isNocite);
+	textAfterLA->setEnabled(haveSelection && !isNocite);
+	citationStyleCO->setEnabled(haveSelection);
+	citationStyleLA->setEnabled(haveSelection);
+}
+
+
+void GuiCitation::updateStyle()
+{
+	string const & command = params_.getCmdName();
 
 	// Find the style of the citekeys
-	vector<biblio::CiteStyle> const & styles =
-		ControlCitation::getCiteStyles();
-	biblio::CitationStyle const cs(command);
+	vector<CiteStyle> const & styles = citeStyles_;
+	CitationStyle const cs = citationStyleFromString(command);
 
-	vector<biblio::CiteStyle>::const_iterator cit =
+	vector<CiteStyle>::const_iterator cit =
 		std::find(styles.begin(), styles.end(), cs.style);
 
-	// restore the latest natbib style
-	if (style_ >= 0 && style_ < citationStyleCO->count())
-		citationStyleCO->setCurrentIndex(style_);
-	else
-		citationStyleCO->setCurrentIndex(0);
-
 	if (cit != styles.end()) {
-		int const i = int(cit - styles.begin());
-		citationStyleCO->setCurrentIndex(i);
 		fulllistCB->setChecked(cs.full);
-		forceuppercaseCB->setChecked(cs.forceUCase);
+		forceuppercaseCB->setChecked(cs.forceUpperCase);
 	} else {
+		// restore the last used natbib style
+		if (style_ >= 0 && style_ < citationStyleCO->count()) {
+			// the necessary update will be performed later
+			citationStyleCO->blockSignals(true);
+			citationStyleCO->setCurrentIndex(style_);
+			citationStyleCO->blockSignals(false);
+		}
 		fulllistCB->setChecked(false);
 		forceuppercaseCB->setChecked(false);
 	}
+	updateFormatting(cs.style);
 }
 
 
-//This one needs to be called whenever citationStyleCO needs
-//to be updated---and this would be on anything that changes the
-//selection in selectedLV, or on a general update.
-void GuiCitationDialog::fillStyles()
+// This one needs to be called whenever citationStyleCO needs
+// to be updated---and this would be on anything that changes the
+// selection in selectedLV, or on a general update.
+void GuiCitation::fillStyles(BiblioInfo const & bi)
 {
-	int const oldIndex = citationStyleCO->currentIndex();
-
-	citationStyleCO->clear();
+	QStringList selected_keys = selected_model_.stringList();
+	int curr = selectedLV->model()->rowCount() - 1;
 
-	QStringList selected_keys = selected()->stringList();
-	if (selected_keys.empty()) {
+	if (curr < 0 || selected_keys.empty()) {
+		citationStyleCO->clear();
 		citationStyleCO->setEnabled(false);
 		citationStyleLA->setEnabled(false);
 		return;
 	}
 
-	int curr = selectedLV->model()->rowCount() - 1;
-	if (curr < 0)
-		return;
-
 	if (!selectedLV->selectionModel()->selectedIndexes().empty())
 		curr = selectedLV->selectionModel()->selectedIndexes()[0].row();
 
-	QStringList sty = citationStyles(curr);
+	QStringList sty = citationStyles(bi, curr);
 
-	bool const basic_engine =
-			(controller().getEngine() == biblio::ENGINE_BASIC);
-
-	citationStyleCO->setEnabled(!sty.isEmpty() && !basic_engine);
-	citationStyleLA->setEnabled(!sty.isEmpty() && !basic_engine);
-
-	if (sty.isEmpty() || basic_engine)
+	if (sty.isEmpty()) { 
+		// some error
+		citationStyleCO->setEnabled(false);
+		citationStyleLA->setEnabled(false);
+		citationStyleCO->clear();
 		return;
-
+	}
+	
+	citationStyleCO->blockSignals(true);
+	
+	// save old index
+	int const oldIndex = citationStyleCO->currentIndex();
+	citationStyleCO->clear();
 	citationStyleCO->insertItems(0, sty);
-
+	citationStyleCO->setEnabled(true);
+	citationStyleLA->setEnabled(true);
+	// restore old index
 	if (oldIndex != -1 && oldIndex < citationStyleCO->count())
 		citationStyleCO->setCurrentIndex(oldIndex);
+
+	citationStyleCO->blockSignals(false);
 }
 
 
-void GuiCitationDialog::fillFields()
+void GuiCitation::fillFields(BiblioInfo const & bi)
 {
 	fieldsCO->blockSignals(true);
 	int const oldIndex = fieldsCO->currentIndex();
 	fieldsCO->clear();
-	QStringList const & fields = getFieldsAsQStringList();
-	fieldsCO->insertItem(0, qt_("All Fields"));
+	QStringList const fields = to_qstring_list(bi.getFields());
+	fieldsCO->insertItem(0, qt_("All fields"));
 	fieldsCO->insertItem(1, qt_("Keys"));
 	fieldsCO->insertItems(2, fields);
 	if (oldIndex != -1 && oldIndex < fieldsCO->count())
@@ -328,13 +328,13 @@ void GuiCitationDialog::fillFields()
 }
 
 
-void GuiCitationDialog::fillEntries()
+void GuiCitation::fillEntries(BiblioInfo const & bi)
 {
 	entriesCO->blockSignals(true);
 	int const oldIndex = entriesCO->currentIndex();
 	entriesCO->clear();
-	QStringList const & entries = getEntriesAsQStringList();
-	entriesCO->insertItem(0, qt_("All Entry Types"));
+	QStringList const entries = to_qstring_list(bi.getEntries());
+	entriesCO->insertItem(0, qt_("All entry types"));
 	entriesCO->insertItems(1, entries);
 	if (oldIndex != -1 && oldIndex < entriesCO->count())
 		entriesCO->setCurrentIndex(oldIndex);
@@ -342,37 +342,41 @@ void GuiCitationDialog::fillEntries()
 }
 
 
-bool GuiCitationDialog::isSelected(const QModelIndex & idx)
+bool GuiCitation::isSelected(QModelIndex const & idx)
 {
 	QString const str = idx.data().toString();
-	return selected()->stringList().contains(str);
+	return selected_model_.stringList().contains(str);
 }
 
 
-void GuiCitationDialog::setButtons()
+void GuiCitation::setButtons()
 {
-	selectionManager->updateView();
+	selectionManager->update();
 	int const srows = selectedLV->model()->rowCount();
 	applyPB->setEnabled(srows > 0);
 	okPB->setEnabled(srows > 0);
 }
 
 
-void GuiCitationDialog::updateInfo(QModelIndex const & idx)
+void GuiCitation::updateInfo(BiblioInfo const & bi, QModelIndex const & idx)
 {
-	if (idx.isValid()) {
-		QString const keytxt = getKeyInfo(idx.data().toString());
-		infoML->document()->setPlainText(keytxt);
-	} else
+	if (!idx.isValid() || bi.empty()) {
 		infoML->document()->clear();
+		return;
+	}
+
+	QString const keytxt = toqstr(
+		bi.getInfo(qstring_to_ucs4(idx.data().toString()), buffer(), true));
+	infoML->document()->setHtml(keytxt);
 }
 
 
-void GuiCitationDialog::findText(QString const & text, bool reset)
+void GuiCitation::findText(QString const & text, bool reset)
 {
 	//"All Fields" and "Keys" are the first two
 	int index = fieldsCO->currentIndex() - 2; 
-	vector<docstring> const & fields = controller().availableFields();
+	BiblioInfo const & bi = bibInfo();
+	vector<docstring> const & fields = bi.getFields();
 	docstring field;
 	
 	if (index <= -1 || index >= int(fields.size()))
@@ -386,134 +390,175 @@ void GuiCitationDialog::findText(QString const & text, bool reset)
 	
 	//"All Entry Types" is first.
 	index = entriesCO->currentIndex() - 1; 
-	vector<docstring> const & entries = controller().availableEntries();
-	docstring entryType;
+	vector<docstring> const & entries = bi.getEntries();
+	docstring entry_type;
 	if (index < 0 || index >= int(entries.size()))
-		entryType = from_ascii("");
+		entry_type = from_ascii("");
 	else 
-		entryType = entries[index];
+		entry_type = entries[index];
 	
 	bool const case_sentitive = caseCB->checkState();
 	bool const reg_exp = regexCB->checkState();
-	findKey(text, onlyKeys, field, entryType, 
+	findKey(bi, text, onlyKeys, field, entry_type, 
 	               case_sentitive, reg_exp, reset);
 	//FIXME
 	//It'd be nice to save and restore the current selection in 
 	//availableLV. Currently, we get an automatic reset, since the
 	//model is reset.
 	
-	updateDialog();
+	updateControls(bi);
 }
 
 
-void GuiCitationDialog::on_fieldsCO_currentIndexChanged(int /*index*/)
+void GuiCitation::on_fieldsCO_currentIndexChanged(int /*index*/)
 {
 	findText(findLE->text(), true);
 }
 
 
-void GuiCitationDialog::on_entriesCO_currentIndexChanged(int /*index*/)
+void GuiCitation::on_entriesCO_currentIndexChanged(int /*index*/)
 {
 	findText(findLE->text(), true);
 }
 
 
-void GuiCitationDialog::on_findLE_textChanged(const QString & text)
+void GuiCitation::on_citationStyleCO_currentIndexChanged(int index)
 {
-	clearPB->setDisabled(text.isEmpty());
-	if (text.isEmpty())
-		findLE->setFocus();
-	findText(text);
+	if (index >= 0 && index < citationStyleCO->count()) {
+		vector<CiteStyle> const & styles = citeStyles_;
+		updateFormatting(styles[index]);
+	}
 }
 
 
-void GuiCitationDialog::on_caseCB_stateChanged(int)
+void GuiCitation::on_findLE_textChanged(const QString & text)
 {
+	bool const searchAsWeGo = (asTypeCB->checkState() == Qt::Checked);
+	searchPB->setDisabled(text.isEmpty() || searchAsWeGo);
+	if (!text.isEmpty()) {
+		if (searchAsWeGo)
+			findText(findLE->text());
+		return;
+	}
 	findText(findLE->text());
+	findLE->setFocus();
 }
 
-
-void GuiCitationDialog::on_regexCB_stateChanged(int)
+void GuiCitation::on_searchPB_clicked()
 {
-	findText(findLE->text());
+	findText(findLE->text(), true);
 }
 
 
-void GuiCitationDialog::changed()
+void GuiCitation::on_caseCB_stateChanged(int)
 {
-	fillStyles();
-	setButtons();
+	findText(findLE->text());
 }
 
 
-void GuiCitationDialog::apply(int const choice,
-	bool const full, bool const force,
-	QString before, QString after)
+void GuiCitation::on_regexCB_stateChanged(int)
 {
-	if (cited_keys_.isEmpty())
-		return;
-
-	vector<biblio::CiteStyle> const & styles =
-		ControlCitation::getCiteStyles();
-
-	string const command =
-		biblio::CitationStyle(styles[choice], full, force)
-		.asLatexStr();
-
-	controller().params().setCmdName(command);
-	controller().params()["key"] = qstring_to_ucs4(cited_keys_.join(","));
-	controller().params()["before"] = qstring_to_ucs4(before);
-	controller().params()["after"] = qstring_to_ucs4(after);
-	controller().dispatchParams();
+	findText(findLE->text());
 }
 
 
-void GuiCitationDialog::clearSelection()
+void GuiCitation::on_asTypeCB_stateChanged(int)
 {
-	cited_keys_.clear();
-	selected_model_.setStringList(cited_keys_);
+	bool const searchAsWeGo = (asTypeCB->checkState() == Qt::Checked);
+	searchPB->setDisabled(findLE->text().isEmpty() || searchAsWeGo);
+	if (searchAsWeGo)
+		findText(findLE->text(), true);
 }
 
 
-QString GuiCitationDialog::textBefore()
+void GuiCitation::changed()
 {
-	return toqstr(controller().params()["before"]);
+	setButtons();
 }
 
 
-QString GuiCitationDialog::textAfter()
+void GuiCitation::apply(int const choice, bool full, bool force,
+	QString before, QString after)
 {
-	return toqstr(controller().params()["after"]);
+	if (cited_keys_.isEmpty())
+		return;
+
+	vector<CiteStyle> const & styles = citeStyles_;
+	if (styles[choice] == NOCITE) {
+		full = false;
+		force = false;
+		before.clear();
+		after.clear();
+	}
+	
+	CitationStyle s;
+	s.style = styles[choice];
+	s.full = full;
+	s.forceUpperCase = force;
+	string const command = citationStyleToString(s);
+
+	params_.setCmdName(command);
+	params_["key"] = qstring_to_ucs4(cited_keys_.join(","));
+	params_["before"] = qstring_to_ucs4(before);
+	params_["after"] = qstring_to_ucs4(after);
+	dispatchParams();
 }
 
 
-bool GuiCitationDialog::initialiseParams(std::string const & data)
+void GuiCitation::clearSelection()
 {
-	if (!controller().initialiseParams(data))
-		return false;
-	init();
-	return true;
+	cited_keys_.clear();
+	selected_model_.setStringList(cited_keys_);
 }
 
 
-void GuiCitationDialog::init()
+void GuiCitation::init()
 {
 	// Make the list of all available bibliography keys
-	all_keys_ = to_qstring_list(controller().availableKeys());
+	BiblioInfo const & bi = bibInfo();
+	all_keys_ = to_qstring_list(bi.getKeys());
 	available_model_.setStringList(all_keys_);
 
 	// Ditto for the keys cited in this inset
-	QString str = toqstr(controller().params()["key"]);
+	QString str = toqstr(params_["key"]);
 	if (str.isEmpty())
 		cited_keys_.clear();
 	else
 		cited_keys_ = str.split(",");
 	selected_model_.setStringList(cited_keys_);
+	if (selected_model_.rowCount()) {
+		selectedLV->blockSignals(true);
+		selectedLV->setFocus();
+		QModelIndex idx = selected_model_.index(0, 0);
+		selectedLV->selectionModel()->select(idx, 
+				QItemSelectionModel::ClearAndSelect);
+		selectedLV->blockSignals(false);
+		
+		// set the style combo appropriately
+		string const & command = params_.getCmdName();
+		vector<CiteStyle> const & styles = citeStyles_;
+		CitationStyle const cs = citationStyleFromString(command);
+	
+		vector<CiteStyle>::const_iterator cit =
+			std::find(styles.begin(), styles.end(), cs.style);
+		if (cit != styles.end()) {
+			int const i = int(cit - styles.begin());
+			// the necessary update will be performed later
+			citationStyleCO->blockSignals(true);
+			citationStyleCO->setCurrentIndex(i);
+			citationStyleCO->blockSignals(false);
+		}
+	} else
+		availableLV->setFocus();
+	fillFields(bi);
+	fillEntries(bi);
+	updateControls(bi);
 }
 
 
-void GuiCitationDialog::findKey(QString const & str, bool only_keys,
-	docstring field, docstring entryType,
+void GuiCitation::findKey(BiblioInfo const & bi,
+	QString const & str, bool only_keys,
+	docstring field, docstring entry_type,
 	bool case_sensitive, bool reg_exp, bool reset)
 {
 	// Used for optimisation: store last searched string.
@@ -524,14 +569,14 @@ void GuiCitationDialog::findKey(QString const & str, bool only_keys,
 	// Reset last_searched_string in case of changed option.
 	if (last_case_sensitive != case_sensitive
 		|| last_reg_exp != reg_exp) {
-			LYXERR(Debug::GUI) << "GuiCitation::findKey: optimisation disabled!" << std::endl;
+			LYXERR(Debug::GUI, "GuiCitation::findKey: optimisation disabled!");
 		last_searched_string.clear();
 	}
 	// save option for next search.
 	last_case_sensitive = case_sensitive;
 	last_reg_exp = reg_exp;
 
-	Qt::CaseSensitivity qtcase = case_sensitive?
+	Qt::CaseSensitivity qtcase = case_sensitive ?
 			Qt::CaseSensitive: Qt::CaseInsensitive;
 	QStringList keys;
 	// If new string (str) contains the last searched one...
@@ -549,53 +594,206 @@ void GuiCitationDialog::findKey(QString const & str, bool only_keys,
 
 	QStringList result;
 	
-	// First, filter by entryType, which will be faster than 
+	// First, filter by entry_type, which will be faster than 
 	// what follows, so we may get to do that on less.
 	vector<docstring> keyVector = to_docstring_vector(keys);
-	controller().filterByEntryType(keyVector, entryType);
+	filterByEntryType(bi, keyVector, entry_type);
 	
 	if (str.isEmpty())
 		result = to_qstring_list(keyVector);
 	else
-		result = to_qstring_list(controller().searchKeys(keyVector, only_keys, 
+		result = to_qstring_list(searchKeys(bi, keyVector, only_keys, 
 			qstring_to_ucs4(str), field, case_sensitive, reg_exp));
 	
 	available_model_.setStringList(result);
 }
 
 
-QStringList GuiCitationDialog::getFieldsAsQStringList()
+QStringList GuiCitation::citationStyles(BiblioInfo const & bi, int sel)
 {
-	return to_qstring_list(controller().availableFields());
+	docstring const key = qstring_to_ucs4(cited_keys_[sel]);
+	return to_qstring_list(bi.getCiteStrings(key, buffer()));
 }
 
 
-QStringList GuiCitationDialog::getEntriesAsQStringList()
+void GuiCitation::setCitedKeys() 
 {
-	return to_qstring_list(controller().availableEntries());
+	cited_keys_ = selected_model_.stringList();
 }
 
 
-QStringList GuiCitationDialog::citationStyles(int sel)
+bool GuiCitation::initialiseParams(string const & data)
 {
-	docstring const key = qstring_to_ucs4(cited_keys_[sel]);
-	return to_qstring_list(controller().getCiteStrings(key));
+	InsetCommand::string2params(data, params_);
+	CiteEngine const engine = buffer().params().citeEngine();
+	citeStyles_ = citeStyles(engine);
+	init();
+	return true;
 }
 
 
-QString GuiCitationDialog::getKeyInfo(QString const & sel)
+void GuiCitation::clearParams()
 {
-	return toqstr(controller().getInfo(qstring_to_ucs4(sel)));
+	params_.clear();
 }
 
 
-void GuiCitationDialog::setCitedKeys() 
+void GuiCitation::filterByEntryType(BiblioInfo const & bi,
+	vector<docstring> & keyVector, docstring entry_type) 
 {
-	cited_keys_ = selected_model_.stringList();
+	if (entry_type.empty())
+		return;
+	
+	vector<docstring>::iterator it = keyVector.begin();
+	vector<docstring>::iterator end = keyVector.end();
+
+	vector<docstring> result;
+	for (; it != end; ++it) {
+		docstring const key = *it;
+		BiblioInfo::const_iterator cit = bi.find(key);
+		if (cit == bi.end())
+			continue;
+		if (cit->second.entryType() == entry_type)
+			result.push_back(key);
+	}
+	keyVector = result;
 }
 
+
+CiteEngine GuiCitation::citeEngine() const
+{
+	return buffer().params().citeEngine();
+}
+
+
+// Escape special chars.
+// All characters are literals except: '.|*?+(){}[]^$\'
+// These characters are literals when preceded by a "\", which is done here
+// @todo: This function should be moved to support, and then the test in tests
+//        should be moved there as well.
+static docstring escape_special_chars(docstring const & expr)
+{
+	// Search for all chars '.|*?+(){}[^$]\'
+	// Note that '[' and '\' must be escaped.
+	// This is a limitation of lyx::regex, but all other chars in BREs
+	// are assumed literal.
+	static const lyx::regex reg("[].|*?+(){}^$\\[\\\\]");
+
+	// $& is a perl-like expression that expands to all
+	// of the current match
+	// The '$' must be prefixed with the escape character '\' for
+	// boost to treat it as a literal.
+	// Thus, to prefix a matched expression with '\', we use:
+	// FIXME: UNICODE
+	return from_utf8(lyx::regex_replace(to_utf8(expr), reg, string("\\\\$&")));
+}
+
+
+vector<docstring> GuiCitation::searchKeys(BiblioInfo const & bi,	
+	vector<docstring> const & keys_to_search, bool only_keys,
+ 	docstring const & search_expression, docstring field,
+	bool case_sensitive, bool regex)
+{
+	vector<docstring> foundKeys;
+
+	docstring expr = trim(search_expression);
+	if (expr.empty())
+		return foundKeys;
+
+	if (!regex)
+		// We must escape special chars in the search_expr so that
+		// it is treated as a simple string by lyx::regex.
+		expr = escape_special_chars(expr);
+
+	lyx::regex reg_exp;
+	try {
+		reg_exp.assign(to_utf8(expr), case_sensitive ?
+			lyx::regex_constants::ECMAScript : lyx::regex_constants::icase);
+	} catch (lyx::regex_error & e) {
+		// lyx::regex throws an exception if the regular expression is not
+		// valid.
+		LYXERR(Debug::GUI, e.what());
+		return vector<docstring>();
+	}
+
+	vector<docstring>::const_iterator it = keys_to_search.begin();
+	vector<docstring>::const_iterator end = keys_to_search.end();
+	for (; it != end; ++it ) {
+		BiblioInfo::const_iterator info = bi.find(*it);
+		if (info == bi.end())
+			continue;
+		
+		BibTeXInfo const & kvm = info->second;
+		string data;
+		if (only_keys)
+			data = to_utf8(*it);
+		else if (field.empty())
+			data = to_utf8(*it) + ' ' + to_utf8(kvm.allData());
+		else 
+			data = to_utf8(kvm[field]);
+		
+		if (data.empty())
+			continue;
+
+		try {
+			if (lyx::regex_search(data, reg_exp))
+				foundKeys.push_back(*it);
+		}
+		catch (lyx::regex_error & e) {
+			LYXERR(Debug::GUI, e.what());
+			return vector<docstring>();
+		}
+	}
+	return foundKeys;
+}
+
+
+void GuiCitation::dispatchParams()
+{
+	std::string const lfun = InsetCommand::params2string(params_);
+	dispatch(FuncRequest(getLfun(), lfun));
+}
+
+
+BiblioInfo const & GuiCitation::bibInfo() const
+{
+	Buffer const & buf = buffer();
+	buf.reloadBibInfoCache();
+	return buf.masterBibInfo();
+}
+
+
+void GuiCitation::saveSession() const
+{
+	Dialog::saveSession();
+	QSettings settings;
+	settings.setValue(
+		sessionKey() + "/regex", regexCB->isChecked());
+	settings.setValue(
+		sessionKey() + "/casesensitive", caseCB->isChecked());
+	settings.setValue(
+		sessionKey() + "/autofind", asTypeCB->isChecked());
+}
+
+
+void GuiCitation::restoreSession()
+{
+	Dialog::restoreSession();
+	QSettings settings;
+	regexCB->setChecked(
+		settings.value(sessionKey() + "/regex").toBool());
+	caseCB->setChecked(
+		settings.value(sessionKey() + "/casesensitive").toBool());
+	asTypeCB->setChecked(
+		settings.value(sessionKey() + "/autofind").toBool());
+}
+
+
+Dialog * createGuiCitation(GuiView & lv) { return new GuiCitation(lv); }
+
+
 } // namespace frontend
 } // namespace lyx
 
-#include "GuiCitation_moc.cpp"
+#include "moc_GuiCitation.cpp"