this diff removes most of the get/set command function calls from the lyx code. I didn't touch cite stuff, as we decided.
On 10/19/06, Ozgur Ugras BARAN <[EMAIL PROTECTED]> wrote:
By the way, I am starting to cleanup of get/set options/contents stuff from Lyx code (excluding cite related files). However, tomorrow I am going to holidays for a week. I will send you the patch as much as I can finish. I can do the rest as I return back form holidays. ugras
Index: src/insets/insetindex.C =================================================================== --- src/insets/insetindex.C (revision 15365) +++ src/insets/insetindex.C (working copy) @@ -45,7 +45,7 @@ int InsetIndex::docbook(Buffer const &, ostream & os, OutputParams const &) const { - os << "<indexterm><primary>" << sgml::escapeString(getContents()) + os << "<indexterm><primary>" << sgml::escapeString(lyx::to_utf8(getParam("name"))) << "</primary></indexterm>"; return 0; } Index: src/insets/insetbibtex.C =================================================================== --- src/insets/insetbibtex.C (revision 15365) +++ src/insets/insetbibtex.C (working copy) @@ -153,7 +153,8 @@ typedef boost::tokenizer<Separator> Tokenizer; Separator const separator(","); - Tokenizer const tokens(getContents(), separator); + Tokenizer const tokens(lyx::to_utf8( + getParam("bibfiles")), separator); Tokenizer::const_iterator const begin = tokens.begin(); Tokenizer::const_iterator const end = tokens.end(); @@ -200,7 +201,7 @@ } // Style-Options - string style = getOptions(); // maybe empty! and with bibtotoc + string style = lyx::to_utf8(getParam("options")); // maybe empty! and with bibtotoc string bibtotoc; if (prefixIs(style, "bibtotoc")) { bibtotoc = "bibtotoc"; @@ -252,7 +253,7 @@ if (!db_out.empty() && buffer.params().use_bibtopic){ os << "\\begin{btSect}{" << db_out << "}\n"; - string btprint = getSecOptions(); + string btprint = lyx::to_utf8(getParam("btprint")); if (btprint.empty()) // default btprint = "btPrintCited"; @@ -302,7 +303,7 @@ vector<string> vec; string tmp; - string bibfiles = getContents(); + string bibfiles = lyx::to_utf8(getParam("bibfiles")); bibfiles = split(bibfiles, tmp, ','); while (!tmp.empty()) { string file = findtexfile(changeExtension(tmp, "bib"), "bib"); @@ -359,11 +360,11 @@ bool InsetBibtex::addDatabase(string const & db) { - string contents(getContents()); + string contents(lyx::to_utf8(getParam("bibfiles"))); if (tokenPos(contents, ',', db) == -1) { if (!contents.empty()) contents += ','; - setContents(contents + db); + setParam("bibfiles", lyx::from_utf8(contents + db)); return true; } return false; @@ -372,17 +373,17 @@ bool InsetBibtex::delDatabase(string const & db) { - string contents(getContents()); + string contents(lyx::to_utf8(getParam("bibfiles"))); if (contains(contents, db)) { int const n = tokenPos(contents, ',', db); string bd = db; if (n > 0) { // this is not the first database string tmp = ',' + bd; - setContents(subst(contents, tmp, "")); + setParam("bibfiles", lyx::from_utf8(subst(contents, tmp, ""))); } else if (n == 0) // this is the first (or only) database - setContents(split(contents, bd, ',')); + setParam("bibfiles", lyx::from_utf8(split(contents, bd, ','))); else return false; } Index: src/insets/insetbibitem.C =================================================================== --- src/insets/insetbibitem.C (revision 15365) +++ src/insets/insetbibitem.C (working copy) @@ -39,8 +39,8 @@ InsetBibitem::InsetBibitem(InsetCommandParams const & p) : InsetCommand(p, "bibitem"), counter(1) { - if (getContents().empty()) - setContents(key_prefix + convert<string>(++key_counter)); + if (getParam("key").empty()) + setParam("key", lyx::from_utf8(key_prefix + convert<string>(++key_counter))); } @@ -63,9 +63,9 @@ cur.noUpdate(); break; } - if (p.getContents() != params().getContents()) - cur.bv().buffer()->changeRefsIfUnique(params().getContents(), - p.getContents(), InsetBase::CITE_CODE); + if (p["key"] != params()["key"]) + cur.bv().buffer()->changeRefsIfUnique(lyx::to_utf8(params()["key"]), + lyx::to_utf8(p["key"]), InsetBase::CITE_CODE); setParams(p); } @@ -86,8 +86,8 @@ { InsetCommand::read(buf, lex); - if (prefixIs(getContents(), key_prefix)) { - int const key = convert<int>(getContents().substr(key_prefix.length())); + if (prefixIs(lyx::to_utf8(getParam("key")), key_prefix)) { + int const key = convert<int>(lyx::to_utf8(getParam("key")).substr(key_prefix.length())); key_counter = max(key_counter, key); } } @@ -96,16 +96,16 @@ docstring const InsetBibitem::getBibLabel() const { // FIXME UNICODE - return getOptions().empty() ? + return getParam("label").empty() ? convert<docstring>(counter) : - lyx::from_utf8(getOptions()); + getParam("label"); } docstring const InsetBibitem::getScreenLabel(Buffer const &) const { // FIXME UNICODE - return lyx::from_utf8(getContents()) + " [" + getBibLabel() + ']'; + return getParam("key") + " [" + getBibLabel() + ']'; } Index: src/insets/insetlabel.C =================================================================== --- src/insets/insetlabel.C (revision 15365) +++ src/insets/insetlabel.C (working copy) @@ -48,14 +48,14 @@ void InsetLabel::getLabelList(Buffer const &, std::vector<docstring> & list) const { // FIXME UNICODE - list.push_back(lyx::from_utf8(getContents())); + list.push_back(getParam("name")); } docstring const InsetLabel::getScreenLabel(Buffer const &) const { // FIXME UNICODE - return lyx::from_utf8(getContents()); + return getParam("name"); } @@ -70,9 +70,9 @@ cur.noUpdate(); break; } - if (p.getContents() != params().getContents()) - cur.bv().buffer()->changeRefsIfUnique(params().getContents(), - p.getContents(), InsetBase::REF_CODE); + if (p["name"] != params()["name"]) + cur.bv().buffer()->changeRefsIfUnique(lyx::to_utf8(params()["name"]), + lyx::to_utf8(p["name"]), InsetBase::REF_CODE); setParams(p); break; } @@ -96,7 +96,7 @@ OutputParams const &) const { // FIXME UNICODE - os << '<' << lyx::from_utf8(getContents()) << '>'; + os << '<' << getParam("name") << '>'; return 0; } @@ -104,6 +104,6 @@ int InsetLabel::docbook(Buffer const & buf, ostream & os, OutputParams const & runparams) const { - os << "<!-- anchor id=\"" << sgml::cleanID(buf, runparams, getContents()) << "\" -->"; + os << "<!-- anchor id=\"" << sgml::cleanID(buf, runparams, lyx::to_utf8(getParam("name"))) << "\" -->"; return 0; } Index: src/insets/insethfill.C =================================================================== --- src/insets/insethfill.C (revision 15365) +++ src/insets/insethfill.C (working copy) @@ -41,7 +41,7 @@ docstring const InsetHFill::getScreenLabel(Buffer const &) const { - return lyx::from_ascii(getContents()); + return getParam(""); } Index: src/insets/inseturl.C =================================================================== --- src/insets/inseturl.C (revision 15365) +++ src/insets/inseturl.C (working copy) @@ -39,12 +39,13 @@ docstring const temp = (getCmdName() == "url") ? _("Url: ") : _("HtmlUrl: "); + //TODO docstring string url; - if (!getOptions().empty()) - url += getOptions(); + if (!getParam("name").empty()) + url += lyx::to_utf8(getParam("name")); else - url += getContents(); + url += lyx::to_utf8(getParam("target")); // elide if long if (url.length() > 30) { @@ -59,11 +60,11 @@ int InsetUrl::latex(Buffer const &, ostream & os, OutputParams const & runparams) const { - if (!getOptions().empty()) - os << getOptions() + ' '; + if (!getParam("name").empty()) + os << lyx::to_utf8(getParam("name")) + ' '; if (runparams.moving_arg) os << "\\protect"; - os << "\\url{" << getContents() << '}'; + os << "\\url{" << lyx::to_utf8(getParam("target")) << '}'; return 0; } @@ -72,12 +73,12 @@ OutputParams const &) const { // FIXME UNICODE - os << '[' << lyx::from_utf8(getContents()); - if (getOptions().empty()) + os << '[' << getParam("target"); + if (getParam("name").empty()) os << ']'; else // FIXME UNICODE - os << "||" << lyx::from_utf8(getOptions()) << ']'; + os << "||" << getParam("name") << ']'; return 0; } @@ -85,8 +86,8 @@ int InsetUrl::docbook(Buffer const &, ostream & os, OutputParams const &) const { - os << "<ulink url=\"" << subst(getContents(),"&","&") - << "\">" << getOptions() << "</ulink>"; + os << "<ulink url=\"" << subst(lyx::to_utf8(getParam("target")),"&","&") + << "\">" << lyx::to_utf8(getParam("name")) << "</ulink>"; return 0; } Index: src/insets/insetref.C =================================================================== --- src/insets/insetref.C (revision 15365) +++ src/insets/insetref.C (working copy) @@ -47,7 +47,7 @@ case LFUN_MOUSE_PRESS: // Eventually trigger dialog with button 3 not 1 if (cmd.button() == mouse_button::button3) - lyx::dispatch(FuncRequest(LFUN_LABEL_GOTO, getContents())); + lyx::dispatch(FuncRequest(LFUN_LABEL_GOTO, getParam("reference"))); else { InsetCommandMailer("ref", *this).showDialog(&cur.bv()); cur.undispatched(); @@ -73,12 +73,12 @@ } } // FIXME UNICODE - temp += lyx::from_utf8(getContents()); + temp += getParam("reference"); - if (!isLatex && !getOptions().empty()) { + if (!isLatex && !getParam("name").empty()) { temp += "||"; // FIXME UNICODE - temp += lyx::from_utf8(getOptions()); + temp += getParam("name"); } return temp; } @@ -99,7 +99,7 @@ OutputParams const &) const { // FIXME UNICODE - os << '[' << lyx::from_utf8(getContents()) << ']'; + os << '[' << getParam("reference") << ']'; return 0; } @@ -107,13 +107,13 @@ int InsetRef::docbook(Buffer const & buf, ostream & os, OutputParams const & runparams) const { - if (getOptions().empty() && runparams.flavor == OutputParams::XML) { - os << "<xref linkend=\"" << sgml::cleanID(buf, runparams, getContents()) << "\" />"; - } else if (getOptions().empty()) { - os << "<xref linkend=\"" << sgml::cleanID(buf, runparams, getContents()) << "\">"; + if (getParam("name").empty() && runparams.flavor == OutputParams::XML) { + os << "<xref linkend=\"" << sgml::cleanID(buf, runparams, lyx::to_utf8(getParam("reference"))) << "\" />"; + } else if (getParam("name").empty()) { + os << "<xref linkend=\"" << sgml::cleanID(buf, runparams, lyx::to_utf8(getParam("reference"))) << "\">"; } else { - os << "<link linkend=\"" << sgml::cleanID(buf, runparams, getContents()) - << "\">" << getOptions() << "</link>"; + os << "<link linkend=\"" << sgml::cleanID(buf, runparams, lyx::to_utf8(getParam("reference"))) + << "\">" << lyx::to_utf8(getParam("name")) << "</link>"; } return 0; Index: src/insets/insetinclude.C =================================================================== --- src/insets/insetinclude.C (revision 15365) +++ src/insets/insetinclude.C (working copy) @@ -221,7 +221,7 @@ string const includedFilename(Buffer const & buffer, InsetCommandParams const & params) { - return makeAbsPath(params.getContents(), + return makeAbsPath(lyx::to_utf8(params["filename"]), onlyPath(parentFilename(buffer))); } @@ -296,11 +296,11 @@ temp += ": "; - if (params_.getContents().empty()) + if (params_["filename"].empty()) temp += "???"; else // FIXME: We don't know the encoding of the filename - temp += lyx::from_ascii(onlyFilename(params_.getContents())); + temp += lyx::from_ascii(onlyFilename(lyx::to_utf8(params_["filename"]))); return temp; } @@ -353,7 +353,7 @@ int InsetInclude::latex(Buffer const & buffer, ostream & os, OutputParams const & runparams) const { - string incfile(params_.getContents()); + string incfile(lyx::to_utf8(params_["filename"])); // Do nothing if no file name has been specified if (incfile.empty()) @@ -486,7 +486,7 @@ int InsetInclude::docbook(Buffer const & buffer, ostream & os, OutputParams const & runparams) const { - string incfile(params_.getContents()); + string incfile(lyx::to_utf8(params_["filename"])); // Do nothing if no file name has been specified if (incfile.empty()) @@ -532,7 +532,7 @@ void InsetInclude::validate(LaTeXFeatures & features) const { - string incfile(params_.getContents()); + string incfile(lyx::to_utf8(params_["filename"])); string writefile; Buffer const & buffer = features.buffer(); Index: src/mathed/InsetMathHull.C =================================================================== --- src/mathed/InsetMathHull.C (revision 15365) +++ src/mathed/InsetMathHull.C (working copy) @@ -55,6 +55,7 @@ #include "support/lyxlib.h" #include "support/lstrings.h" +#include "support/docstring.h" #include <boost/bind.hpp> @@ -1114,17 +1115,17 @@ if (name == "label") { InsetCommandParams p("label"); InsetCommandMailer::string2params(name, lyx::to_utf8(cmd.argument()), p); - string str = p.getContents(); + docstring str = p["name"]; recordUndoInset(cur); row_type const r = (type_ == hullMultline) ? nrows() - 1 : cur.row(); str = lyx::support::trim(str); if (!str.empty()) numbered(r, true); - string old = label(r); + docstring old = lyx::from_utf8(label(r)); if (str != old) { - cur.bv().buffer()->changeRefsIfUnique(old, str, + cur.bv().buffer()->changeRefsIfUnique(lyx::to_utf8(old), lyx::to_utf8(str), InsetBase::REF_CODE); - label(r, str); + label(r, lyx::to_utf8(str)); } break; } Index: src/factory.C =================================================================== --- src/factory.C (revision 15365) +++ src/factory.C (working copy) @@ -165,7 +165,7 @@ string const contents = cmd.argument().empty() ? bv->getLyXText()->getStringToIndex(bv->cursor()) : lyx::to_utf8(cmd.argument()); - icp.setContents(contents); + icp["name"] = lyx::from_utf8(contents); return new InsetIndex(icp); } @@ -380,8 +380,8 @@ || cmdName == "vref" || cmdName == "vpageref" || cmdName == "prettyref") { - if (!inscmd.getOptions().empty() - || !inscmd.getContents().empty()) { + if (!inscmd["name"].empty() + || !inscmd["reference"].empty()) { inset.reset(new InsetRef(inscmd, buf)); } } else if (cmdName == "tableofcontents") { Index: src/frontends/gtk/GBibItem.C =================================================================== --- src/frontends/gtk/GBibItem.C (revision 15365) +++ src/frontends/gtk/GBibItem.C (working copy) @@ -62,15 +62,17 @@ { bc().refreshReadOnly(); - keyentry_->set_text (controller().params().getContents()); - labelentry_->set_text (controller().params().getOptions()); + keyentry_->set_text (lyx::to_utf8(controller().params()["key"])); + labelentry_->set_text (lyx::to_utf8(controller().params()["label"])); } void GBibItem::apply() { - controller().params().setContents(keyentry_->get_text()); - controller().params().setOptions(labelentry_->get_text()); + controller().params()["key"] = lyx::from_utf8( + Glib::locale_to_utf8(keyentry_->get_text())); + controller().params()["label"] = lyx::from_utf8( + Glib::locale_to_utf8(labelentry_->get_text())); } void GBibItem::changed() Index: src/frontends/gtk/GBibtex.C =================================================================== --- src/frontends/gtk/GBibtex.C (revision 15365) +++ src/frontends/gtk/GBibtex.C (working copy) @@ -95,7 +95,7 @@ void GBibtex::update() { - string bibs(controller().params().getContents()); + string bibs(lyx::to_utf8(controller().params()["bibfiles"])); string bib; databasesstore_->clear(); @@ -118,7 +118,7 @@ toccheck_->set_sensitive(!bibtopic); - string btprint(controller().params().getSecOptions()); + string btprint(lyx::to_utf8(controller().params()["btprint"])); int btp = 0; if (btprint == "btPrintNotCited") btp = 1; @@ -169,7 +169,8 @@ dblist += ","; } - controller().params().setContents(dblist); + controller().params()["bibfiles"] = lyx::from_utf8( + Glib::locale_to_utf8(dblist)); string const bibstyle = stylecombo_.get_active_text(); bool const bibtotoc = toccheck_->get_active(); @@ -177,15 +178,18 @@ if (!bibtopic && bibtotoc && (!bibstyle.empty())) { // both bibtotoc and style - controller().params().setOptions("bibtotoc," + bibstyle); + controller().params()["options"] = lyx::from_utf8( + Glib::locale_to_utf8("bibtotoc," + bibstyle)); } else if (!bibtopic && bibtotoc) { // bibtotoc and no style - controller().params().setOptions("bibtotoc"); + controller().params()["options"] = lyx::from_utf8( + Glib::locale_to_utf8("bibtotoc")); } else { // only style. An empty one is valid, because some // documentclasses have an own \bibliographystyle{} // command! - controller().params().setOptions(bibstyle); + controller().params()["options"] = lyx::from_utf8( + Glib::locale_to_utf8(bibstyle)); } // bibtopic allows three kinds of sections: @@ -196,18 +200,25 @@ switch (btp) { case 0: - controller().params().setSecOptions("btPrintCited"); + controller().params()["btprint"] = lyx::from_utf8( + Glib::locale_to_utf8("btPrintCited")); break; case 1: - controller().params().setSecOptions("btPrintNotCited"); + + controller().params()["btprint"] = lyx::from_utf8( + Glib::locale_to_utf8("btPrintNotCited")); break; case 2: - controller().params().setSecOptions("btPrintAll"); + + controller().params()["btprint"] = lyx::from_utf8( + Glib::locale_to_utf8("btPrintAll")); break; } if (!bibtopic) - controller().params().setSecOptions(""); + + controller().params()["btprint"] = lyx::from_utf8( + Glib::locale_to_utf8("")); } Index: src/frontends/gtk/GInclude.C =================================================================== --- src/frontends/gtk/GInclude.C (revision 15365) +++ src/frontends/gtk/GInclude.C (working copy) @@ -72,7 +72,7 @@ void GInclude::update() { - string const filename = controller().params().getContents(); + string const filename = lyx::to_utf8(controller().params()["filename"]); fileentry_->set_text(filename); string const cmdname = controller().params().getCmdName(); @@ -113,7 +113,8 @@ InsetCommandParams params = controller().params(); params.preview(previewcheck_->get_active()); - params.setContents(fileentry_->get_text()); + params["filename"] = + lyx::from_utf8(Glib::locale_from_utf8(fileentry_->get_text())); if (includeradio_->get_active()) params.setCmdName("include"); Index: src/frontends/gtk/GRef.C =================================================================== --- src/frontends/gtk/GRef.C (revision 15365) +++ src/frontends/gtk/GRef.C (working copy) @@ -168,8 +168,10 @@ bc().refreshReadOnly(); jumptobutton_->set_sensitive(true); backbutton_->set_sensitive(false); - labelentry_->set_text(controller().params().getContents()); - nameentry_->set_text(controller().params().getOptions()); + labelentry_->set_text(lyx::to_utf8( + controller().params()["reference"])); + nameentry_->set_text(lyx::to_utf8( + controller().params()["name"])); // Name is irrelevant to LaTeX/Literate documents Kernel::DocType doctype = kernel().docType(); @@ -240,8 +242,10 @@ if (applylock_) return; - controller().params().setContents(labelentry_->get_text()); - controller().params().setOptions(nameentry_->get_text()); + controller().params()["reference"] = lyx::from_utf8( + Glib::locale_to_utf8(labelentry_->get_text())); + controller().params()["name"] = lyx::from_utf8( + Glib::locale_to_utf8(nameentry_->get_text())); int const type = formatcombo_->get_active_row_number(); controller().params().setCmdName(InsetRef::getName(type)); } Index: src/frontends/gtk/GUrl.C =================================================================== --- src/frontends/gtk/GUrl.C (revision 15365) +++ src/frontends/gtk/GUrl.C (working copy) @@ -77,10 +77,10 @@ void GUrl::update() { - url_->set_text(Glib::locale_to_utf8( - controller().params().getContents())); - name_->set_text(Glib::locale_to_utf8( - controller().params().getOptions())); + url_->set_text(lyx::to_utf8( + controller().params()["target"])); + name_->set_text(lyx::to_utf8( + controller().params()["name"])); if (controller().params().getCmdName() == "url") htmlType_->set_active(false); else @@ -90,9 +90,9 @@ void GUrl::apply() { - controller().params().setContents( + controller().params()["target"] = lyx::from_utf8( Glib::locale_to_utf8(url_->get_text())); - controller().params().setOptions( + controller().params()["name"] = lyx::from_utf8( Glib::locale_to_utf8(name_->get_text())); if (htmlType_->get_active()) controller().params().setCmdName("htmlurl"); Index: src/frontends/gtk/GText.C =================================================================== --- src/frontends/gtk/GText.C (revision 15365) +++ src/frontends/gtk/GText.C (working copy) @@ -44,14 +44,15 @@ void GText::apply() { - controller().params().setContents(entry_->get_text()); + controller().params()["name"] = lyx::from_utf8( + Glib::locale_to_utf8(entry_->get_text())); } void GText::update() { string const contents = support::trim( - controller().params().getContents()); + lyx::to_utf8(controller().params()["name"])); entry_->set_text(contents); } Index: src/frontends/qt3/QBibitem.C =================================================================== --- src/frontends/qt3/QBibitem.C (revision 15365) +++ src/frontends/qt3/QBibitem.C (working copy) @@ -46,15 +46,17 @@ void QBibitem::update_contents() { - dialog_->keyED->setText(toqstr(controller().params().getContents())); - dialog_->labelED->setText(toqstr(controller().params().getOptions())); + dialog_->keyED->setText(toqstr(controller().params()["key"])); + dialog_->labelED->setText(toqstr(controller().params()["label"])); } void QBibitem::apply() { - controller().params().setContents(fromqstr(dialog_->keyED->text())); - controller().params().setOptions(fromqstr(dialog_->labelED->text())); + controller().params()["key"] = + lyx::from_utf8(fromqstr(dialog_->keyED->text())); + controller().params()["label"] = + lyx::from_utf8(fromqstr(dialog_->labelED->text())); } Index: src/frontends/qt3/QBibtex.C =================================================================== --- src/frontends/qt3/QBibtex.C (revision 15365) +++ src/frontends/qt3/QBibtex.C (working copy) @@ -76,7 +76,7 @@ dialog_->databaseLB->clear(); - string bibs(controller().params().getContents()); + string bibs(lyx::to_utf8(controller().params()["bibfiles"])); string bib; while (!bibs.empty()) { @@ -99,7 +99,7 @@ dialog_->bibtocCB->setChecked(controller().bibtotoc() && !bibtopic); dialog_->bibtocCB->setEnabled(!bibtopic); - string btprint(controller().params().getSecOptions()); + string bibs(lyx::to_utf8(controller().params()["bibfiles"])); int btp = 0; if (btprint == "btPrintNotCited") btp = 1; @@ -146,22 +146,22 @@ dbs += fromqstr(dialog_->databaseLB->text(i)); } - controller().params().setContents(dbs); + controller().params()["bibfiles"] = lyx::from_utf8(dbs); string const bibstyle(fromqstr(dialog_->styleCB->currentText())); bool const bibtotoc(dialog_->bibtocCB->isChecked()); if (bibtotoc && (!bibstyle.empty())) { // both bibtotoc and style - controller().params().setOptions("bibtotoc," + bibstyle); + controller().params()["options"] = lyx::from_utf8("bibtotoc," + bibstyle); } else if (bibtotoc) { // bibtotoc and no style - controller().params().setOptions("bibtotoc"); + controller().params()["options"] = lyx::from_utf8("bibtotoc," ); } else { // only style. An empty one is valid, because some // documentclasses have an own \bibliographystyle{} // command! - controller().params().setOptions(bibstyle); + controller().params()["options"] = lyx::from_utf8(bibstyle); } // bibtopic allows three kinds of sections: @@ -172,18 +172,18 @@ switch (btp) { case 0: - controller().params().setSecOptions("btPrintCited"); + controller().params()["btprint"] = lyx::from_utf8("btPrintCited"); break; case 1: - controller().params().setSecOptions("btPrintNotCited"); + controller().params()["btprint"]= lyx::from_utf8("btPrintNotCited"); break; case 2: - controller().params().setSecOptions("btPrintAll"); + controller().params()["btprint"]= lyx::from_utf8("btPrintAll"); break; } if (!controller().usingBibtopic()) - controller().params().setSecOptions(""); + controller().params()["btprint"]= lyx::from_utf8(""); } Index: src/frontends/qt3/QURL.C =================================================================== --- src/frontends/qt3/QURL.C (revision 15365) +++ src/frontends/qt3/QURL.C (working copy) @@ -51,8 +51,8 @@ { InsetCommandParams const & params = controller().params(); - dialog_->urlED->setText(toqstr(params.getContents())); - dialog_->nameED->setText(toqstr(params.getOptions())); + dialog_->urlED->setText(toqstr(params["target"])); + dialog_->nameED->setText(toqstr(params["name"])); dialog_->hyperlinkCB->setChecked(params.getCmdName() != "url"); bc().valid(isValid()); @@ -63,8 +63,10 @@ { InsetCommandParams & params = controller().params(); - params.setContents(fromqstr(dialog_->urlED->text())); - params.setOptions(fromqstr(dialog_->nameED->text())); + params["target"] = + lyx::from_utf8(fromqstr(dialog_->urlED->text())); + params["name"] = + lyx::from_utf8(fromqstr(dialog_->nameED->text())); if (dialog_->hyperlinkCB->isChecked()) params.setCmdName("htmlurl"); Index: src/frontends/qt3/QRef.C =================================================================== --- src/frontends/qt3/QRef.C (revision 15365) +++ src/frontends/qt3/QRef.C (working copy) @@ -71,13 +71,13 @@ int orig_type = dialog_->typeCO->currentItem(); - dialog_->referenceED->setText(toqstr(params.getContents())); + dialog_->referenceED->setText(toqstr(params["reference"])); - dialog_->nameED->setText(toqstr(params.getOptions())); + dialog_->nameED->setText(toqstr(params["name"])); dialog_->nameED->setReadOnly(!nameAllowed() && !readOnly()); // restore type settings for new insets - if (params.getContents().empty()) + if (params["reference"].empty()) dialog_->typeCO->setCurrentItem(orig_type); else dialog_->typeCO->setCurrentItem(InsetRef::getType(params.getCmdName())); @@ -95,7 +95,7 @@ dialog_->bufferCO->insertItem(toqstr(*it)); } // restore the buffer combo setting for new insets - if (params.getContents().empty() && restored_buffer_ != -1 + if (params["reference"].empty() && restored_buffer_ != -1 && restored_buffer_ < dialog_->bufferCO->count()) dialog_->bufferCO->setCurrentItem(restored_buffer_); else @@ -111,8 +111,10 @@ InsetCommandParams & params = controller().params(); params.setCmdName(InsetRef::getName(dialog_->typeCO->currentItem())); - params.setContents(fromqstr(dialog_->referenceED->text())); - params.setOptions(fromqstr(dialog_->nameED->text())); + params["reference"] = + lyx::from_utf8(fromqstr(dialog_->referenceED->text())); + params["name"] = + lyx::from_utf8(fromqstr(dialog_->nameED->text())); restored_buffer_ = dialog_->bufferCO->currentItem(); } Index: src/frontends/qt3/QInclude.C =================================================================== --- src/frontends/qt3/QInclude.C (revision 15365) +++ src/frontends/qt3/QInclude.C (working copy) @@ -63,7 +63,7 @@ InsetCommandParams const & params = controller().params(); - dialog_->filenameED->setText(toqstr(params.getContents())); + dialog_->filenameED->setText(toqstr(params["filename"])); dialog_->visiblespaceCB->setChecked(false); dialog_->visiblespaceCB->setEnabled(false); @@ -100,7 +100,8 @@ { InsetCommandParams params = controller().params(); - params.setContents(fromqstr(dialog_->filenameED->text())); + params["filename"] = + lyx::from_utf8(fromqstr(dialog_->filenameED->text())); params.preview(dialog_->previewCB->isChecked()); int const item = dialog_->typeCO->currentItem(); Index: src/frontends/qt3/QIndex.C =================================================================== --- src/frontends/qt3/QIndex.C (revision 15365) +++ src/frontends/qt3/QIndex.C (working copy) @@ -50,7 +50,7 @@ void QIndex::update_contents() { - string const contents = controller().params().getContents(); + docstring const contents = controller().params()["name"]; dialog_->keywordED->setText(toqstr(contents)); bc().valid(!contents.empty()); @@ -59,7 +59,8 @@ void QIndex::apply() { - controller().params().setContents(fromqstr(dialog_->keywordED->text())); + controller().params()["name"] = + lyx::from_utf8(fromqstr(dialog_->keywordED->text())); } Index: src/frontends/qt4/QBibitem.C =================================================================== --- src/frontends/qt4/QBibitem.C (revision 15365) +++ src/frontends/qt4/QBibitem.C (working copy) @@ -46,15 +46,17 @@ void QBibitem::update_contents() { - dialog_->keyED->setText(toqstr(controller().params().getContents())); - dialog_->labelED->setText(toqstr(controller().params().getOptions())); + dialog_->keyED->setText(toqstr(controller().params()["key"])); + dialog_->labelED->setText(toqstr(controller().params()["label"])); } void QBibitem::apply() { - controller().params().setContents(fromqstr(dialog_->keyED->text())); - controller().params().setOptions(fromqstr(dialog_->labelED->text())); + controller().params()["key"] = + lyx::from_utf8(fromqstr(dialog_->keyED->text())); + controller().params()["label"] = + lyx::from_utf8(fromqstr(dialog_->labelED->text())); } Index: src/frontends/qt4/QBibtex.C =================================================================== --- src/frontends/qt4/QBibtex.C (revision 15365) +++ src/frontends/qt4/QBibtex.C (working copy) @@ -75,7 +75,7 @@ dialog_->databaseLW->clear(); - string bibs(controller().params().getContents()); + string bibs(lyx::to_utf8(controller().params()["bibfiles"])); string bib; while (!bibs.empty()) { @@ -100,7 +100,7 @@ dialog_->bibtocCB->setChecked(controller().bibtotoc() && !bibtopic); dialog_->bibtocCB->setEnabled(!bibtopic); - string btprint(controller().params().getSecOptions()); + string btprint(lyx::to_utf8(controller().params()["btprint"])); int btp = 0; if (btprint == "btPrintNotCited") btp = 1; @@ -146,22 +146,22 @@ dbs += fromqstr(dialog_->databaseLW->item(i)->text()); } - controller().params().setContents(dbs); + controller().params()["bibfiles"] = lyx::from_utf8(dbs); string const bibstyle(fromqstr(dialog_->styleCB->currentText())); bool const bibtotoc(dialog_->bibtocCB->isChecked()); if (bibtotoc && (!bibstyle.empty())) { // both bibtotoc and style - controller().params().setOptions("bibtotoc," + bibstyle); + controller().params()["options"] = lyx::from_utf8("bibtotoc," + bibstyle); } else if (bibtotoc) { // bibtotoc and no style - controller().params().setOptions("bibtotoc"); + controller().params()["options"] = lyx::from_utf8("bibtotoc"); } else { // only style. An empty one is valid, because some // documentclasses have an own \bibliographystyle{} // command! - controller().params().setOptions(bibstyle); + controller().params()["options"] = lyx::from_utf8(bibstyle); } // bibtopic allows three kinds of sections: @@ -172,18 +172,18 @@ switch (btp) { case 0: - controller().params().setSecOptions("btPrintCited"); + controller().params()["btprint"]= lyx::from_utf8("btPrintCited"); break; case 1: - controller().params().setSecOptions("btPrintNotCited"); + controller().params()["btprint"]= lyx::from_utf8("btPrintNotCited"); break; case 2: - controller().params().setSecOptions("btPrintAll"); + controller().params()["btprint"]= lyx::from_utf8("btPrintAll"); break; } if (!controller().usingBibtopic()) - controller().params().setSecOptions(""); + controller().params()["btprint"]= lyx::from_utf8(""); } Index: src/frontends/qt4/QRef.C =================================================================== --- src/frontends/qt4/QRef.C (revision 15365) +++ src/frontends/qt4/QRef.C (working copy) @@ -72,13 +72,13 @@ int orig_type = dialog_->typeCO->currentIndex(); - dialog_->referenceED->setText(toqstr(params.getContents())); + dialog_->referenceED->setText(toqstr(params["reference"])); - dialog_->nameED->setText(toqstr(params.getOptions())); + dialog_->nameED->setText(toqstr(params["name"])); dialog_->nameED->setReadOnly(!nameAllowed() && !readOnly()); // restore type settings for new insets - if (params.getContents().empty()) + if (params["reference"].empty()) dialog_->typeCO->setCurrentIndex(orig_type); else dialog_->typeCO->setCurrentIndex(InsetRef::getType(params.getCmdName())); @@ -96,7 +96,7 @@ dialog_->bufferCO->addItem(toqstr(*it)); } // restore the buffer combo setting for new insets - if (params.getContents().empty() && restored_buffer_ != -1 + if (params["reference"].empty() && restored_buffer_ != -1 && restored_buffer_ < dialog_->bufferCO->count()) dialog_->bufferCO->setCurrentIndex(restored_buffer_); else @@ -112,8 +112,10 @@ InsetCommandParams & params = controller().params(); params.setCmdName(InsetRef::getName(dialog_->typeCO->currentIndex())); - params.setContents(fromqstr(dialog_->referenceED->text())); - params.setOptions(fromqstr(dialog_->nameED->text())); + params["reference"] = + lyx::from_utf8(fromqstr(dialog_->referenceED->text())); + params["name"] = + lyx::from_utf8(fromqstr(dialog_->nameED->text())); restored_buffer_ = dialog_->bufferCO->currentIndex(); } Index: src/frontends/qt4/UrlView.C =================================================================== --- src/frontends/qt4/UrlView.C (revision 15365) +++ src/frontends/qt4/UrlView.C (working copy) @@ -51,8 +51,8 @@ { InsetCommandParams const & params = controller().params(); - dialog_->urlED->setText(toqstr(params.getContents())); - dialog_->nameED->setText(toqstr(params.getOptions())); + dialog_->urlED->setText(toqstr(params["target"])); + dialog_->nameED->setText(toqstr(params["name"])); dialog_->hyperlinkCB->setChecked(params.getCmdName() != "url"); bc().valid(isValid()); @@ -63,8 +63,10 @@ { InsetCommandParams & params = controller().params(); - params.setContents(fromqstr(dialog_->urlED->text())); - params.setOptions(fromqstr(dialog_->nameED->text())); + params["target"] = + lyx::from_utf8(fromqstr(dialog_->urlED->text())); + params["name"] = + lyx::from_utf8(fromqstr(dialog_->nameED->text())); if (dialog_->hyperlinkCB->isChecked()) params.setCmdName("htmlurl"); Index: src/frontends/qt4/QInclude.C =================================================================== --- src/frontends/qt4/QInclude.C (revision 15365) +++ src/frontends/qt4/QInclude.C (working copy) @@ -62,7 +62,7 @@ InsetCommandParams const & params = controller().params(); - dialog_->filenameED->setText(toqstr(params.getContents())); + dialog_->filenameED->setText(toqstr(params["filename"])); dialog_->visiblespaceCB->setChecked(false); dialog_->visiblespaceCB->setEnabled(false); @@ -99,7 +99,8 @@ { InsetCommandParams params = controller().params(); - params.setContents(fromqstr(dialog_->filenameED->text())); + params["filename"] = + lyx::from_utf8(fromqstr(dialog_->filenameED->text())); params.preview(dialog_->previewCB->isChecked()); int const item = dialog_->typeCO->currentIndex(); Index: src/frontends/qt4/QIndex.C =================================================================== --- src/frontends/qt4/QIndex.C (revision 15365) +++ src/frontends/qt4/QIndex.C (working copy) @@ -50,7 +50,7 @@ void QIndex::update_contents() { - string const contents = controller().params().getContents(); + docstring const contents = controller().params()["name"]; dialog_->keywordED->setText(toqstr(contents)); bc().valid(!contents.empty()); @@ -59,7 +59,8 @@ void QIndex::apply() { - controller().params().setContents(fromqstr(dialog_->keywordED->text())); + controller().params()["name"] = + lyx::from_utf8(fromqstr(dialog_->keywordED->text())); } Index: src/frontends/controllers/ControlBibtex.C =================================================================== --- src/frontends/controllers/ControlBibtex.C (revision 15365) +++ src/frontends/controllers/ControlBibtex.C (working copy) @@ -125,7 +125,7 @@ bool ControlBibtex::bibtotoc() const { - return prefixIs(params().getOptions(), "bibtotoc"); + return prefixIs(lyx::to_utf8(params()["options"]), "bibtotoc"); } @@ -151,7 +151,7 @@ break; } - string bst = params().getOptions(); + string bst = lyx::to_utf8(params()["btprint"]); if (bibtotoc()){ // bibstyle exists? if (contains(bst,',')) { @@ -164,7 +164,7 @@ // propose default style file for new insets // existing insets might have (legally) no bst files // (if the class already provides a style) - if (bst.empty() && params().getContents().empty()) + if (bst.empty() && params()["bibfiles"].empty()) bst = defaultstyle; return bst; Index: src/BufferView.C =================================================================== --- src/BufferView.C (revision 15365) +++ src/BufferView.C (working copy) @@ -772,7 +772,7 @@ getInsetByCode<InsetRef>(cursor_, InsetBase::REF_CODE); if (inset) { - label = lyx::from_utf8(inset->getContents()); + label = inset->getParam("reference"); savePosition(0); } }