Stupid bug fix.

[lyx.git] / src / insets / InsetInfo.cpp

diff --git a/src/insets/InsetInfo.cpp b/src/insets/InsetInfo.cpp
index 98b3a6167b727abfbaba15f2c512da3e6b840c04..8cf80bbbc2de0f7b889482466881d8ac56cd7003 100644 (file)
--- a/src/insets/InsetInfo.cpp
+++ b/src/insets/InsetInfo.cpp
@@ -11,7 +11,6 @@
 
 #include "InsetInfo.h"
 
-#include "LayoutFile.h"
 #include "Buffer.h"
 #include "BufferParams.h"
 #include "BufferView.h"
@@ -21,6 +20,7 @@
 #include "InsetSpecialChar.h"
 #include "KeyMap.h"
 #include "LaTeXFeatures.h"
+#include "LayoutFile.h"
 #include "LyXAction.h"
 #include "LyXRC.h"
 #include "Lexer.h"
@@ -32,11 +32,11 @@
 #include "support/debug.h"
 #include "support/docstream.h"
 #include "support/docstring_list.h"
+#include "support/ExceptionMessage.h"
 #include "support/FileName.h"
 #include "support/filetools.h"
 #include "support/gettext.h"
 #include "support/lstrings.h"
-#include "support/ExceptionMessage.h"
 
 #include <sstream>
 
@@ -56,9 +56,10 @@ InsetInfo::InsetInfo(Buffer const & buf, string const & name)
 }
 
 
-Inset * InsetInfo::editXY(Cursor &, int, int)
+Inset * InsetInfo::editXY(Cursor & cur, int x, int y)
 {
-       return this;
+       cur.push(*this);
+       return InsetText::editXY(cur, x, y);
 }
 
 
@@ -68,6 +69,12 @@ void InsetInfo::draw(PainterInfo & pi, int x, int y) const
 }
 
 
+string InsetInfo::infoType() const
+{
+       return nameTranslator().find(type_);
+}
+
+
 docstring InsetInfo::toolTip(BufferView const &, int, int) const
 {
        odocstringstream os;
@@ -140,6 +147,13 @@ void InsetInfo::write(ostream & os) const
 }
 
 
+bool InsetInfo::validate(docstring const & /* argument */) const
+{
+       // FIXME!
+       return true;
+}
+
+
 bool InsetInfo::showInsetDialog(BufferView * bv) const
 {
        bv->showDialog("info");
@@ -151,13 +165,18 @@ bool InsetInfo::getStatus(Cursor & cur, FuncRequest const & cmd,
                FuncStatus & flag) const
 {
        switch (cmd.action) {
+       case LFUN_MOUSE_PRESS:
+       case LFUN_MOUSE_RELEASE:
+       case LFUN_MOUSE_MOTION:
+       case LFUN_MOUSE_DOUBLE:
+       case LFUN_MOUSE_TRIPLE:
+       case LFUN_COPY:
+       case LFUN_INSET_SETTINGS:
+               return InsetText::getStatus(cur, cmd, flag);
 
        case LFUN_INSET_MODIFY:
                flag.setEnabled(true);
                break;
-       //FIXME: do something.
-       /*
-       */
 
        default:
                return false;
@@ -168,19 +187,24 @@ bool InsetInfo::getStatus(Cursor & cur, FuncRequest const & cmd,
 
 void InsetInfo::doDispatch(Cursor & cur, FuncRequest & cmd)
 {
-       // FIXME: we should allow selection, copy etc...
+       // allow selection, copy but not cut, delete etc
        switch (cmd.action) {
        case LFUN_MOUSE_PRESS:
        case LFUN_MOUSE_RELEASE:
        case LFUN_MOUSE_MOTION:
        case LFUN_MOUSE_DOUBLE:
        case LFUN_MOUSE_TRIPLE:
-               // do not dispatch to InsetText
-               cur.dispatched();
+       case LFUN_COPY:
+       case LFUN_INSET_SETTINGS:
+               InsetText::doDispatch(cur, cmd);
+               break;
+
+       case LFUN_INSET_MODIFY:
+               setInfo(to_utf8(cmd.argument()));
+               cur.pos() = 0;
                break;
 
        default:
-               InsetText::doDispatch(cur, cmd);
                break;
        }
 }
@@ -194,6 +218,20 @@ void InsetInfo::setInfo(string const & name)
        string type;
        name_ = trim(split(name, type, ' '));
        type_ = nameTranslator().find(type);
+       updateInfo();
+}
+
+
+void InsetInfo::error(string const & err)
+{
+       InsetText::setText(bformat(_(err), from_utf8(name_)), buffer().params().getFont(),
+               false);
+}
+
+
+void InsetInfo::setText(docstring const & str)
+{
+       InsetText::setText(str, buffer().params().getFont(), false);
 }
 
 
@@ -205,24 +243,24 @@ void InsetInfo::updateInfo()
 
        switch (type_) {
        case UNKNOWN_INFO:
-               setText(_("Unknown Info: ") + from_utf8(name_),
-                       bp.getFont(), false);
+               error("Unknown Info: %1$s");
                break;
+       case SHORTCUT_INFO:
        case SHORTCUTS_INFO: {
                FuncRequest func = lyxaction.lookupFunc(name_);
-               if (func.action != LFUN_UNKNOWN_ACTION)
-                       setText(theTopLevelKeymap().printBindings(func),
-                               bp.getFont(), false);
-               break;
-       }
-       case SHORTCUT_INFO: {
-               FuncRequest func = lyxaction.lookupFunc(name_);
-               if (func.action != LFUN_UNKNOWN_ACTION) {
-                       KeyMap::Bindings bindings = theTopLevelKeymap().findBindings(func);
-                       if (!bindings.empty())
-                               setText(bindings.rbegin()->print(KeySequence::ForGui),
-                                       bp.getFont(), false);
+               if (func.action == LFUN_UNKNOWN_ACTION) {
+                       error("Unknown action %1$s");
+                       break;
                }
+               KeyMap::Bindings bindings = theTopLevelKeymap().findBindings(func);
+               if (bindings.empty()) {
+                       error("No binding for action %1$s");
+                       break;
+               }
+               if (type_ == SHORTCUT_INFO)
+                       setText(bindings.rbegin()->print(KeySequence::ForGui));
+               else
+                       setText(theTopLevelKeymap().printBindings(func));
                break;
        }
        case LYXRC_INFO: {
@@ -234,31 +272,28 @@ void InsetInfo::updateInfo()
                // remove \n and ""
                result = rtrim(result, "\n");
                result = trim(result, "\"");
-               setText(from_utf8(result), bp.getFont(), false);
+               setText(from_utf8(result));
                break;
        }
        case PACKAGE_INFO:
                // check in packages.lst
-               setText(LaTeXFeatures::isAvailable(name_) ? _("yes") : _("no"),
-                       bp.getFont(), false);
+               setText(LaTeXFeatures::isAvailable(name_) ? _("yes") : _("no"));
                break;
        case TEXTCLASS_INFO: {
                // name_ is the class name
-               setText(LayoutFileList::get().haveClass(name_) ? _("yes") : _("no"),
-               bp.getFont(), false);
+               setText(LayoutFileList::get().haveClass(name_) ? _("yes") : _("no"));
                break;
        }
        case MENU_INFO: {
                docstring_list names;
                FuncRequest func = lyxaction.lookupFunc(name_);
                if (func.action == LFUN_UNKNOWN_ACTION) {
-                       setText(bformat(_("Unknown action %1$s"), from_utf8(name_)), bp.getFont(), false);
+                       error("Unknown action %1$s");
                        break;
                }
                // iterate through the menubackend to find it
                if (!theApp()->searchMenu(func, names)) {
-                       setText(bformat(_("No menu entry for action %1$s"), from_utf8(name_)),
-                               bp.getFont(), false);
+                       error("No menu entry for action %1$s");
                        break;
                }
                // if find, return its path.
@@ -294,14 +329,13 @@ void InsetInfo::updateInfo()
        }
        case BUFFER_INFO: {
                if (name_ == "name")
-                       setText(from_utf8(buffer().fileName().onlyFileName()),
-                               bp.getFont(), false);
+                       setText(from_utf8(buffer().fileName().onlyFileName()));
                else if (name_ == "path")
-                       setText(from_utf8(buffer().filePath()), bp.getFont(), false);
+                       setText(from_utf8(buffer().filePath()));
                else if (name_ == "class")
-                       setText(from_utf8(bp.documentClass().name()), bp.getFont(), false);
+                       setText(from_utf8(bp.documentClass().name()));
                else
-                       setText(_("Unknown buffer info"), bp.getFont(), false);
+                       setText(_("Unknown buffer info"));
                break;
        }
        }