]> git.lyx.org Git - lyx.git/blobdiff - src/frontends/xforms/FormCitation.C
fix crash with "save as"
[lyx.git] / src / frontends / xforms / FormCitation.C
index 91cf7c2f526ebc498cfaa7b31194b16535b7399b..fafc6375c9140de316a3bf4c0c59e65084d4a1d9 100644 (file)
@@ -3,7 +3,8 @@
  * This file is part of LyX, the document processor.
  * Licence details can be found in the file COPYING.
  *
- * \author Angus Leeming 
+ * \author Angus Leeming
+ * \author Rob Lahaye
  *
  * Full author contact details are available in file CREDITS
  */
@@ -57,34 +58,12 @@ void fillChoice(FD_citation * dialog, vector<string> vec)
                return;
 
        // They will be changed. Proceed
-       string str = " ";
-       if (!vec.empty())
-               str += getStringFromVector(vec, " | ") + " ";
+       bool const noVec = vec.empty();
+       string const str = noVec ? string() : getStringFromVector(vec, "|");
 
        fl_clear_choice(dialog->choice_style);
        fl_addto_choice(dialog->choice_style, str.c_str());
-
-       setEnabled(dialog->choice_style, !vec.empty());
-       if (vec.empty())
-               return;
-
-       // The width of the choice varies with the contents.
-       // Ensure that it is centred in the frame.
-
-       int width = 0;
-       for (vector<string>::const_iterator it = vec.begin();
-            it != vec.end(); ++it) {
-               width = max(width, string_width(*it));
-       }
-
-       int const dx =
-               max(5, int(0.5 * (dialog->frame_style->w - width)));
-
-       fl_set_object_geometry(dialog->choice_style,
-                              dialog->frame_style->x + dx,
-                              dialog->choice_style->y,
-                              width,
-                              dialog->choice_style->h);
+       setEnabled(dialog->choice_style, !noVec);
 }
 
 
@@ -99,16 +78,13 @@ void updateStyle(FD_citation * dialog, string command)
                find(styles.begin(), styles.end(), cs.style);
 
        // Use this to initialise the GUI
-       if (cit == styles.end()) {
-               fl_set_choice(dialog->choice_style, 1);
-               fl_set_button(dialog->check_full_author_list, 0);
-               fl_set_button(dialog->check_force_uppercase, 0);
-       } else {
-               int const i = int(cit - styles.begin());
-               fl_set_choice(dialog->choice_style, i+1);
-               fl_set_button(dialog->check_full_author_list,  cs.full);
-               fl_set_button(dialog->check_force_uppercase, cs.forceUCase);
-       }
+       bool const noStyles = cit == styles.end();
+       int const index = 1 + ( noStyles ? 0 : int(cit - styles.begin()) );
+       fl_set_choice(dialog->choice_style, index);
+       
+       // Disable if there are no styles, otherwise use cs member settings.
+       fl_set_button(dialog->check_full_author_list, !noStyles && cs.full);
+       fl_set_button(dialog->check_force_uppercase, !noStyles && cs.forceUCase);
 }
 
 } // namespace anon
@@ -118,14 +94,14 @@ typedef FormCB<ControlCitation, FormDB<FD_citation> > base_class;
 
 
 FormCitation::FormCitation()
-       : base_class(_("Citation"), false)
+       : base_class(_("Citation"))
 {}
 
 
 void FormCitation::apply()
 {
        string command = "cite";
-       if (dialog_->choice_style->active != 0) {
+       if (isActive(dialog_->choice_style)) {
                vector<biblio::CiteStyle> const & styles =
                        ControlCitation::getCiteStyles();
 
@@ -142,7 +118,7 @@ void FormCitation::apply()
        controller().params().setCmdName(command);
        controller().params().setContents(getStringFromVector(citekeys));
 
-       string const after  = fl_get_input(dialog_->input_after);
+       string const after  = getString(dialog_->input_after);
        controller().params().setOptions(after);
 }
 
@@ -160,23 +136,13 @@ void FormCitation::build()
 {
        dialog_.reset(build_citation(this));
 
-       fl_set_input_return(dialog_->input_after,  FL_RETURN_CHANGED);
-       fl_set_input_return(dialog_->input_before, FL_RETURN_CHANGED);
-       fl_set_input_return(dialog_->input_search, FL_RETURN_END);
-
-       fl_set_button(dialog_->check_search_case, 0);
-       fl_set_button(dialog_->check_search_type, 0);
-
-       setPrehandler(dialog_->input_search);
-       setPrehandler(dialog_->input_before);
-       setPrehandler(dialog_->input_after);
-
        // Manage the ok, apply, restore and cancel/close buttons
        bc().setOK(dialog_->button_ok);
        bc().setApply(dialog_->button_apply);
        bc().setCancel(dialog_->button_close);
        bc().setRestore(dialog_->button_restore);
 
+       // disable for read-only documents
        bc().addReadOnly(dialog_->button_add);
        bc().addReadOnly(dialog_->button_del);
        bc().addReadOnly(dialog_->button_up);
@@ -187,6 +153,15 @@ void FormCitation::build()
        bc().addReadOnly(dialog_->check_full_author_list);
        bc().addReadOnly(dialog_->check_force_uppercase);
 
+       // trigger an input event for cut&paste with middle mouse button.
+       setPrehandler(dialog_->input_search);
+       setPrehandler(dialog_->input_before);
+       setPrehandler(dialog_->input_after);
+
+       fl_set_input_return(dialog_->input_after,  FL_RETURN_CHANGED);
+       fl_set_input_return(dialog_->input_before, FL_RETURN_CHANGED);
+       fl_set_input_return(dialog_->input_search, FL_RETURN_END);
+
        //set up the tooltip mechanism
        string str = _("Add the selected entry to the current citation reference.");
        tooltips().init(dialog_->button_add, str);
@@ -202,12 +177,18 @@ void FormCitation::build()
 
        str = _("The entries which will be cited. Select them with the arrow buttons from the right browser window.");
        tooltips().init(dialog_->browser_cite, str);
+       // Work-around xforms' bug; enable tooltips for browser widgets.
+       setPrehandler(dialog_->browser_cite);
 
        str = _("All entries in the database you have loaded (via \"Insert->Lists&TOC->BibTex Reference\"). Move the ones you want to cite with the arrow buttons into the left browser window.");
        tooltips().init(dialog_->browser_bib, str);
+       // Work-around xforms' bug; enable tooltips for browser widgets.
+       setPrehandler(dialog_->browser_bib);
 
        str = _("Information about the selected entry");
        tooltips().init(dialog_->browser_info, str);
+       // Work-around xforms' bug; enable tooltips for browser widgets.
+       setPrehandler(dialog_->browser_info);
 
        str = _("Here you may select how the citation label should look inside the text (Natbib).");
        tooltips().init(dialog_->choice_style, str);
@@ -237,7 +218,7 @@ void FormCitation::build()
 
 void FormCitation::findBiblio(biblio::Direction const dir)
 {
-       string const str = fl_get_input(dialog_->input_search);
+       string const str = getString(dialog_->input_search);
        biblio::InfoMap const & theMap = controller().bibkeysInfo();
        bool const caseSensitive =
                fl_get_button(dialog_->check_search_case);
@@ -489,8 +470,11 @@ void FormCitation::updateBrowser(FL_OBJECT * browser,
        // changed by loading the contents of the vec...
        vector<string> browser_keys = getVector(browser);
 
-       if (browser_keys == keys)
+       if (browser_keys == keys) {
+               fl_deselect_browser(browser);
+               fl_set_browser_topline(browser, 1);
                return;
+       }
 
        // They will be changed. Proceed.
        fl_clear_browser(browser);