/**
* \file insetcite.C
- * Copyright 2001 the LyX Team
- * Read the file COPYING
+ * This file is part of LyX, the document processor.
+ * Licence details can be found in the file COPYING.
*
- * \author Angus Leeming, a.leeming@ic.ac.uk
- * \author Herbert Voss, voss@lyx.org 2002-03-17
+ * \author Angus Leeming
+ * \author Herbert Voss
+ *
+ * Full author contact details are available in file CREDITS
*/
#include <config.h>
-#ifdef __GNUG__
-#pragma implementation
-#endif
-
#include "insetcite.h"
+#include "funcrequest.h"
#include "buffer.h"
#include "BufferView.h"
#include "LaTeXFeatures.h"
-#include "frontends/LyXView.h"
-#include "debug.h"
-#include "gettext.h"
#include "frontends/controllers/biblio.h"
-#include "frontends/Dialogs.h"
-#include "support/filetools.h"
#include "support/lstrings.h"
-#include "support/path.h"
-#include "support/os.h"
-#include "support/lstrings.h"
-#include "support/LAssert.h"
#include <map>
+using namespace lyx::support;
+
using std::ostream;
using std::vector;
using std::map;
namespace {
// An optimisation. We assume that until the first InsetCitation::edit is
-// called, we're loding the buffer and that, therefore, we don't need to
+// called, we're loading the buffer and that, therefore, we don't need to
// reload the bibkey list
std::map<Buffer const *, bool> loading_buffer;
string const & before, string const & after,
bool numerical)
{
- // Only reload the bibkeys if we have to...
- map<Buffer const *, bool>::iterator lit = loading_buffer.find(buffer);
- if (lit != loading_buffer.end())
- loading_buffer[buffer] = true;
-
typedef std::map<Buffer const *, biblio::InfoMap> CachedMap;
static CachedMap cached_keys;
- CachedMap::iterator kit = cached_keys.find(buffer);
+ // Only load the bibkeys once if we're loading up the buffer,
+ // else load them afresh each time.
+ map<Buffer const *, bool>::iterator lit = loading_buffer.find(buffer);
+ if (lit == loading_buffer.end())
+ loading_buffer[buffer] = true;
+
+ bool loadkeys = !loading_buffer[buffer];
+ if (!loadkeys) {
+ CachedMap::iterator kit = cached_keys.find(buffer);
+ loadkeys = kit == cached_keys.end();
+ }
- if (!loading_buffer[buffer] || kit == cached_keys.end()) {
+ if (loadkeys) {
// build the keylist
typedef vector<std::pair<string, string> > InfoType;
- InfoType bibkeys = buffer->getBibkeyList();
+ InfoType bibkeys;
+ buffer->fillWithBibKeys(bibkeys);
InfoType::const_iterator bit = bibkeys.begin();
InfoType::const_iterator bend = bibkeys.end();
infomap[bit->first] = bit->second;
}
if (infomap.empty())
- return string();
+ return string();
cached_keys[buffer] = infomap;
}
// puctuation mark separating citation entries.
char const * const sep = ";";
- string const op_str(string(1, ' ') + string(1, op));
- string const cp_str(string(1, cp) + string(1, ' '));
- string const sep_str(string(sep) + " ");
+ string const op_str(' ' + string(1, op));
+ string const cp_str(string(1, cp) + ' ');
+ string const sep_str(string(sep) + ' ');
string label;
vector<string> keys = getVectorFromString(keyList);
label += year + sep_str;
}
}
- label = strip(strip(label), sep);
+ label = rtrim(rtrim(label), sep);
if (!after_str.empty()) {
if (cite_type == "citet") {
if (contains(keys, ",")) {
// Final comma allows while loop to cover all keys
- keys = frontStrip(split(keys, label, ',')) + ",";
+ keys = ltrim(split(keys, label, ',')) + ',';
while (contains(keys, ",")) {
string key;
- keys = frontStrip(split(keys, key, ','));
+ keys = ltrim(split(keys, key, ','));
label += ", " + key;
}
} else
if (!after.empty())
label += ", " + after;
- return "[" + label + "]";
+ return '[' + label + ']';
}
} // anon namespace
-InsetCitation::InsetCitation(InsetCommandParams const & p, bool)
+InsetCitation::InsetCitation(InsetCommandParams const & p)
: InsetCommand(p)
{}
+// InsetCitation::InsetCitation(InsetCommandParams const & p, bool)
+// : InsetCommand(p, false)
+// {}
+
+
+InsetCitation::~InsetCitation()
+{
+ InsetCommandMailer mailer("citation", *this);
+ mailer.hideDialog();
+}
+
+
string const InsetCitation::generateLabel(Buffer const * buffer) const
{
string const before = string();
}
-void InsetCitation::edit(BufferView * bv, int, int, mouse_button::state)
+void InsetCitation::setLoadingBuffer(Buffer const * buffer, bool state) const
{
- // A call to edit() indicates that we're no longer loading the
- // buffer but doing some real work.
// Doesn't matter if there is no bv->buffer() entry in the map.
- loading_buffer[bv->buffer()] = false;
-
- bv->owner()->getDialogs()->showCitation(this);
+ loading_buffer[buffer] = state;
}
-void InsetCitation::edit(BufferView * bv, bool)
+dispatch_result InsetCitation::localDispatch(FuncRequest const & cmd)
{
- edit(bv, 0, 0, mouse_button::none);
+ switch (cmd.action) {
+ case LFUN_INSET_EDIT:
+ // A call to edit indicates that we're no longer loading the
+ // buffer but doing some real work.
+ setLoadingBuffer(cmd.view()->buffer(), false);
+ InsetCommandMailer("citation", *this).showDialog(cmd.view());
+ return DISPATCHED;
+
+ default:
+ return InsetCommand::localDispatch(cmd);
+ }
}
int InsetCitation::ascii(Buffer const * buffer, ostream & os, int) const
{
- string label;
-
if (cache.params == params() && cache.style == getStyle(buffer))
- label = cache.generated_label;
+ os << cache.generated_label;
else
- label = generateLabel(buffer);
-
- os << label;
+ os << generateLabel(buffer);
return 0;
}
// citations and then changes his mind, turning natbib support off. The output
// should revert to \cite[]{}
int InsetCitation::latex(Buffer const * buffer, ostream & os,
- bool /*fragile*/, bool/*fs*/) const
+ LatexRunParams const &) const
{
os << "\\";
if (buffer->params.use_natbib)
os << "cite";
#warning What is this code supposed to do? (Lgb)
+// my guess is that this is just waiting for when we support before,
+// so it's a oneliner. But this is very silly ! - jbl
#if 1
// The current strange code
string const before = string();
string const after = getOptions();
if (!before.empty() && buffer->params.use_natbib)
- os << "[" << before << "][" << after << "]";
+ os << '[' << before << "][" << after << ']';
else if (!after.empty())
- os << "[" << after << "]";
+ os << '[' << after << ']';
#else
// and the cleaned up equvalent, should it just be changed? (Lgb)
string const after = getOptions();
if (!after.empty())
- os << "[" << after << "]";
+ os << '[' << after << ']';
#endif
string::const_iterator it = getContents().begin();
string::const_iterator end = getContents().end();
// Paranoia check: make sure that there is no whitespace in here
string content;
+ char last = ',';
for (; it != end; ++it) {
- if (*it != ' ') content += *it;
+ if (*it != ' ')
+ last = *it;
+ if (*it != ' ' || last != ',')
+ content += *it;
}
- os << "{" << content << "}";
+ os << '{' << content << '}';
return 0;
}