]> git.lyx.org Git - lyx.git/blobdiff - src/insets/insetbib.C
prepare for 1.1.6pre2
[lyx.git] / src / insets / insetbib.C
index 51ba1a6064dabf4fb0e42111695aeb58573c259a..e374b40c700840cd80d2707ecc47c4de36856011 100644 (file)
@@ -9,7 +9,6 @@
 
 #include FORMS_H_LOCATION  
 #include "insetbib.h"
-#include "combox.h"
 #include "buffer.h"
 #include "debug.h"
 #include "lyx_gui_misc.h"
@@ -19,6 +18,7 @@
 #include "lyxtext.h"
 #include "support/filetools.h"
 #include "support/path.h"
+#include "lyxrc.h"
 
 using std::ostream;
 using std::ifstream;
@@ -27,14 +27,17 @@ using std::endl;
 using std::vector;
 using std::pair;
 
-extern BufferView * current_view;
-
 FD_bibitem_form * bibitem_form = 0;
 
 FD_bibitem_form * create_form_bibitem_form(void);
 
+extern BufferView * current_view;
 
-extern "C" void bibitem_cb(FL_OBJECT *, long data)
+// This is foul!
+// called from both InsetBibKey and InsetBibtex dialogs yet cast off
+// only to InsetBibKey holder. Real problems can ensue.
+extern "C"
+void bibitem_cb(FL_OBJECT *, long data)
 {
        InsetBibKey::Holder * holder =
                static_cast<InsetBibKey::Holder*>
@@ -43,6 +46,7 @@ extern "C" void bibitem_cb(FL_OBJECT *, long data)
        holder->inset->callback( bibitem_form, data );
 }
 
+
 FD_bibitem_form * create_form_bibitem_form(void)
 {
        FL_OBJECT * obj;
@@ -71,43 +75,43 @@ FD_bibitem_form * create_form_bibitem_form(void)
 }
 
 
-InsetBibKey::InsetBibKey(string const & key, string const & label):
-       InsetCommand("bibitem", key, label)
+InsetBibKey::InsetBibKey(InsetCommandParams const & p)
+       : InsetCommand(p)
 {
        counter = 1;
-       if (key.empty())
-               setCmdName(" ");
-}
-
-
-InsetBibKey::InsetBibKey(InsetBibKey const * b):
-       InsetCommand("bibitem", b->getContents(), b->getOptions())
-{
-       counter = b->counter;
 }
 
 
 InsetBibKey::~InsetBibKey()
 {
-       if(bibitem_form && bibitem_form->bibitem_form
+       if (bibitem_form && bibitem_form->bibitem_form
           && bibitem_form->bibitem_form->visible
           && bibitem_form->bibitem_form->u_vdata == &holder)
                fl_hide_form(bibitem_form->bibitem_form);
 }
 
 
+Inset * InsetBibKey::Clone(Buffer const &) const
+{
+       InsetBibKey * b = new InsetBibKey(params());
+       b->setCounter(counter);
+       return b;
+}
+
+
 void InsetBibKey::callback( FD_bibitem_form * form, long data )
 {
-       switch (data)
-       {
+       switch (data) {
        case 1:
-               if(!holder.view->buffer()->isReadonly())
-               {
+               // Do NOT change this to
+               // holder.view->buffer() as this code is used by both
+               // InsetBibKey and InsetBibtex! Ughhhhhhh!!!!
+               if (!current_view->buffer()->isReadonly()) {
                        setContents(fl_get_input(form->key));
                        setOptions(fl_get_input(form->label));
                        // shouldn't mark the buffer dirty unless
                        // something was actually altered
-                       holder.view->updateInset( this, true );
+                       current_view->updateInset( this, true );
                } // fall through to Cancel
        case 0:
                fl_hide_form(form->bibitem_form);
@@ -141,7 +145,21 @@ void InsetBibKey::Write(Buffer const *, ostream & os) const
 }
 
 
-string InsetBibKey::getScreenLabel() const
+// This is necessary here because this is written without begin_inset
+// This should be changed!!! (Jug)
+void InsetBibKey::Read(Buffer const *, LyXLex & lex)
+{    
+       string token;
+
+       if (lex.EatLine()) {
+               token = lex.GetString();
+               scanCommand(token);
+       } else
+               lex.printError("InsetCommand: Parse error: `$$Token'");
+}
+
+
+string const InsetBibKey::getScreenLabel() const
 {
        if (! getOptions().empty())
                return getOptions();
@@ -150,7 +168,7 @@ string InsetBibKey::getScreenLabel() const
 }
 
 
-/*
+/**
   The value in "Key:" isn't allways set right after a few bibkey insets have
   been added/removed.  Perhaps the wrong object is deleted/used somewhere
   upwards?
@@ -158,7 +176,7 @@ string InsetBibKey::getScreenLabel() const
 */
 void InsetBibKey::Edit(BufferView * bv, int, int, unsigned int)
 {
-       if(bv->buffer()->isReadonly())
+       if (bv->buffer()->isReadonly())
                WarnReadonly(bv->buffer()->fileName());
        
        if (!bibitem_form) {
@@ -181,44 +199,36 @@ void InsetBibKey::Edit(BufferView * bv, int, int, unsigned int)
        if (bibitem_form->bibitem_form->visible) {
                fl_raise_form(bibitem_form->bibitem_form);
        } else {
-               fl_show_form(bibitem_form->bibitem_form, FL_PLACE_MOUSE,
-                            FL_FULLBORDER,
+               fl_show_form(bibitem_form->bibitem_form,
+                            FL_PLACE_MOUSE | FL_FREE_SIZE, FL_TRANSIENT,
                             _("Bibliography item"));
        }   
 }
 
 
-InsetBibtex::InsetBibtex(string const & dbase, string const & style,
-                        Buffer * o)
-       : InsetCommand("BibTeX", dbase, style), owner(o)
-{
-}
+InsetBibtex::InsetBibtex(InsetCommandParams const & p)
+       : InsetCommand(p)
+{}
+
 
 InsetBibtex::~InsetBibtex()
 {
-       if(bibitem_form && bibitem_form->bibitem_form
+       if (bibitem_form && bibitem_form->bibitem_form
           && bibitem_form->bibitem_form->visible
           && bibitem_form->bibitem_form->u_vdata == &holder)
                fl_hide_form(bibitem_form->bibitem_form);
 }
 
 
-string InsetBibtex::getScreenLabel() const
+string const InsetBibtex::getScreenLabel() const
 {
        return _("BibTeX Generated References");
 }
 
 
-int InsetBibtex::Latex(Buffer const *, ostream & os,
+int InsetBibtex::Latex(Buffer const * buffer, ostream & os,
                       bool /*fragile*/, bool/*fs*/) const
 {
-       // this looks like an horrible hack and it is :) The problem
-       // is that owner is not initialized correctly when the bib
-       // inset is cut and pasted. Such hacks will not be needed
-       // later (JMarc)
-       if (!owner) {
-               owner = current_view->buffer();
-       }
        // If we generate in a temp dir, we might need to give an
        // absolute path there. This is a bit complicated since we can
        // have a comma-separated list of bibliographies
@@ -226,9 +236,9 @@ int InsetBibtex::Latex(Buffer const *, ostream & os,
        string db_in = getContents();
        db_in = split(db_in, adb, ',');
        while(!adb.empty()) {
-               if (!owner->niceFile &&
-                   IsFileReadable(MakeAbsPath(adb, owner->filepath)+".bib")) 
-                         adb = MakeAbsPath(adb, owner->filepath);
+               if (!buffer->niceFile &&
+                   IsFileReadable(MakeAbsPath(adb, buffer->filepath)+".bib")) 
+                         adb = MakeAbsPath(adb, buffer->filepath);
 
                db_out += adb;
                db_out += ',';
@@ -237,10 +247,10 @@ int InsetBibtex::Latex(Buffer const *, ostream & os,
        db_out = strip(db_out, ',');
        // Idem, but simpler
        string style;
-       if (!owner->niceFile 
-           && IsFileReadable(MakeAbsPath(getOptions(), owner->filepath)
+       if (!buffer->niceFile 
+           && IsFileReadable(MakeAbsPath(getOptions(), buffer->filepath)
                              + ".bst")) 
-               style = MakeAbsPath(getOptions(), owner->filepath);
+               style = MakeAbsPath(getOptions(), buffer->filepath);
        else
                style = getOptions();
 
@@ -251,19 +261,9 @@ int InsetBibtex::Latex(Buffer const *, ostream & os,
 
 
 // This method returns a comma separated list of Bibtex entries
-vector<pair<string,string> > InsetBibtex::getKeys() const
+vector<pair<string, string> > const InsetBibtex::getKeys(Buffer const * buffer) const
 {
-       // This hack is copied from InsetBibtex::Latex.
-       // Is it still needed? Probably yes.
-       // Why is this needed here when it already is in Latex?
-       // Anyway we need a different way to get to the
-       // buffer the inset is in. (Lgb)
-       
-       //if (!owner) {
-       //      owner = current_view->buffer();
-       //}
-       
-       Path p(owner->filepath);
+       Path p(buffer->filepath);
 
        vector<pair<string,string> > keys;
        string tmp;
@@ -283,7 +283,7 @@ vector<pair<string,string> > InsetBibtex::getKeys() const
                        string linebuf0;
                        while (getline(ifs, linebuf0)) {
                                string linebuf = frontStrip(strip(linebuf0));
-                               iflinebuf.empty() ) continue;
+                               if (linebuf.empty() ) continue;
                                if (prefixIs(linebuf, "@")) {
                                        linebuf = subst(linebuf, '{', '(');
                                        linebuf = split(linebuf, tmp, '(');
@@ -296,7 +296,7 @@ vector<pair<string,string> > InsetBibtex::getKeys() const
                                                        keys.push_back(pair<string,string>(tmp,string()));
                                                }
                                        }
-                               } else if( !keys.empty() ) {
+                               } else if (!keys.empty()) {
                                        keys.back().second += linebuf + "\n";
                                }
                        }
@@ -329,7 +329,7 @@ void InsetBibtex::Edit(BufferView * bv, int, int, unsigned int)
                fl_raise_form(bibitem_form->bibitem_form);
        } else {
                fl_show_form(bibitem_form->bibitem_form,
-                            FL_PLACE_MOUSE, FL_FULLBORDER,
+                            FL_PLACE_MOUSE | FL_FREE_SIZE, FL_TRANSIENT,
                             _("BibTeX"));
        }   
 }
@@ -337,10 +337,11 @@ void InsetBibtex::Edit(BufferView * bv, int, int, unsigned int)
 
 bool InsetBibtex::addDatabase(string const & db)
 {
-       if (!contains(getContents(), db.c_str())) {
-               if (!getContents().empty()) 
-                       addContents(",");
-               addContents(db);
+       string contents(getContents());
+       if (!contains(contents, db)) {
+               if (!contents.empty()) 
+                       contents += ",";
+               setContents(contents + db);
                return true;
        }
        return false;
@@ -349,14 +350,14 @@ bool InsetBibtex::addDatabase(string const & db)
 
 bool InsetBibtex::delDatabase(string const & db)
 {
-       if (contains(getContents(), db.c_str())) {
+       if (contains(getContents(), db)) {
                string bd = db;
                int n = tokenPos(getContents(), ',', bd);
                if (n > 0) {
                        // Weird code, would someone care to explain this?(Lgb)
                        string tmp(", ");
                        tmp += bd;
-                       setContents(subst(getContents(), tmp.c_str(), ", "));
+                       setContents(subst(getContents(), tmp, ", "));
                } else if (n == 0)
                        setContents(split(getContents(), bd, ','));
                else 
@@ -371,7 +372,7 @@ int bibitemMaxWidth(BufferView * bv, LyXFont const & font)
 {
        int w = 0;
        // Does look like a hack? It is! (but will change at 0.13)
-       LyXParagraph * par = current_view->buffer()->paragraph;
+       LyXParagraph * par = bv->buffer()->paragraph;
     
        while (par) {
                if (par->bibkey) {
@@ -385,11 +386,12 @@ int bibitemMaxWidth(BufferView * bv, LyXFont const & font)
 
 
 // ale070405
-string bibitemWidest(BufferView * bv)
+string const bibitemWidest(Buffer const * buffer)
 {
        int w = 0;
        // Does look like a hack? It is! (but will change at 0.13)
-       LyXParagraph * par = current_view->buffer()->paragraph;
+       LyXParagraph * par = buffer->paragraph;
+       BufferView * bv = buffer->getUser();
        InsetBibKey * bkey = 0;
        LyXFont font;