struct SpellcheckerWidget::Private
{
- Private(SpellcheckerWidget * parent, DockView * dv)
- : p(parent), dv_(dv), incheck_(false), wrap_around_(false) {}
+ Private(SpellcheckerWidget * parent, DockView * dv, GuiView * gv)
+ : p(parent), dv_(dv), gv_(gv), incheck_(false), wrap_around_(false) {}
/// update from controller
void updateSuggestions(docstring_list & words);
/// move to next position after current word
void forward();
/// check text until next misspelled/unknown word
void check();
- ///
+ /// close the spell checker dialog
void hide() const;
- ///
+ /// make/restore a selection between from and to
void setSelection(DocIterator const & from, DocIterator const & to) const;
- ///
+ /// if no selection was checked:
+ /// ask the user if the check should start over
bool continueFromBeginning();
- ///
+ /// set the given language in language chooser
void setLanguage(Language const * lang);
/// test and set guard flag
bool inCheck() {
end_ = start_;
}
}
- bool isWrapAround(DocIterator cursor) const;
+ /// test for existing association with a document buffer
+ /// and test for already active check
+ bool disabled() {
+ return gv_->documentBufferView() == 0 || inCheck();
+ }
+ /// the cursor position of the buffer view
+ DocIterator const cursor() const;
+ /// status checks
+ bool isCurrentBuffer(DocIterator const & cursor) const;
+ bool isWrapAround(DocIterator const & cursor) const;
bool isWrapAround() const { return wrap_around_; }
- ///
- bool atLastPos(DocIterator cursor) const;
+ bool atLastPos(DocIterator const & cursor) const;
+ /// validate the cached doc iterators
+ /// The spell checker dialog is not modal.
+ /// The user may change the buffer being checked and break the iterators.
+ void fixPositionsIfBroken();
///
Ui::SpellcheckerUi ui;
///
SpellcheckerWidget * p;
///
- GuiView * gv_;
- ///
DockView * dv_;
+ ///
+ GuiView * gv_;
/// current word being checked and lang code
WordLangTuple word_;
- /// cursor position when spell checking starts
+ /// cursor position where spell checking starts
DocIterator start_;
/// range to spell check
/// for selection both are non-empty
- /// after wrap around the start becomes the end
+ /// after wrap around the start position becomes the end
DocIterator begin_;
DocIterator end_;
///
SpellcheckerWidget::SpellcheckerWidget(GuiView * gv, DockView * dv, QWidget * parent)
- : QTabWidget(parent), d(new Private(this, dv))
+ : QTabWidget(parent), d(new Private(this, dv, gv))
{
d->ui.setupUi(this);
- d->gv_ = gv;
connect(d->ui.suggestionsLW, SIGNAL(itemDoubleClicked(QListWidgetItem*)),
this, SLOT(on_replacePB_clicked()));
setEnabled(enabled);
if (enabled && hasFocus()) {
Cursor const & cursor = bv->cursor();
- if (d->start_.empty() || d->start_.buffer() != cursor.buffer()) {
+ if (d->start_.empty() || !d->isCurrentBuffer(cursor)) {
if (cursor.selection()) {
d->begin_ = cursor.selectionBegin();
d->end_ = cursor.selectionEnd();
}
}
+DocIterator const SpellcheckerWidget::Private::cursor() const
+{
+ BufferView * bv = gv_->documentBufferView();
+ return bv ? bv->cursor() : DocIterator();
+}
bool SpellcheckerWidget::Private::continueFromBeginning()
{
- BufferView * bv = gv_->documentBufferView();
- if (!begin_.empty()) {
+ DocIterator const current_ = cursor();
+ if (isCurrentBuffer(current_) && !begin_.empty()) {
// selection was checked
// start over from beginning makes no sense
- DocIterator current_ = bv->cursor();
+ fixPositionsIfBroken();
hide();
if (current_ == start_) {
// no errors found... tell the user the good news
"continue from the beginning?"),
QMessageBox::Yes | QMessageBox::No, QMessageBox::No);
if (answer == QMessageBox::No) {
+ fixPositionsIfBroken();
hide();
return false;
}
return true;
}
-bool SpellcheckerWidget::Private::atLastPos(DocIterator cursor) const
+bool SpellcheckerWidget::Private::isCurrentBuffer(DocIterator const & cursor) const
+{
+ return start_.buffer() == cursor.buffer();
+}
+
+bool SpellcheckerWidget::Private::atLastPos(DocIterator const & cursor) const
{
bool const valid_end = !end_.empty();
return cursor.depth() <= 1 && (
cursor.atEnd() ||
- valid_end && cursor >= end_);
+ (valid_end && isCurrentBuffer(cursor) && cursor >= end_));
+}
+
+bool SpellcheckerWidget::Private::isWrapAround(DocIterator const & cursor) const
+{
+ return wrap_around_ && isCurrentBuffer(cursor) && start_ < cursor;
}
-bool SpellcheckerWidget::Private::isWrapAround(DocIterator cursor) const
+void SpellcheckerWidget::Private::fixPositionsIfBroken()
{
- return wrap_around_ && start_ < cursor;
+ DocIterator const current_ = cursor();
+ if (!isCurrentBuffer(current_)) {
+ LYXERR(Debug::GUI, "wrong document of current cursor position " << start_);
+ start_ = current_;
+ begin_ = DocIterator();
+ end_ = DocIterator();
+ }
+ if (start_.fixIfBroken())
+ LYXERR(Debug::GUI, "broken start position fixed " << start_);
+ if (begin_.fixIfBroken()) {
+ LYXERR(Debug::GUI, "broken selection begin position fixed " << begin_);
+ begin_ = DocIterator();
+ end_ = DocIterator();
+ }
+ if (end_.fixIfBroken())
+ LYXERR(Debug::GUI, "broken selection end position fixed " << end_);
}
void SpellcheckerWidget::Private::hide() const
{
+ BufferView * bv = gv_->documentBufferView();
+ Cursor & bvcur = bv->cursor();
dv_->hide();
- if (!begin_.empty() && !end_.empty()) {
- // restore previous selection
- setSelection(begin_, end_);
- } else {
- // restore cursor position
- BufferView * bv = gv_->documentBufferView();
- Cursor & bvcur = bv->cursor();
- bvcur.setCursor(start_);
- bvcur.clearSelection();
- bv->processUpdateFlags(Update::Force | Update::FitCursor);
+ if (isCurrentBuffer(bvcur)) {
+ if (!begin_.empty() && !end_.empty()) {
+ // restore previous selection
+ setSelection(begin_, end_);
+ } else {
+ // restore cursor position
+ bvcur.setCursor(start_);
+ bvcur.clearSelection();
+ bv->processUpdateFlags(Update::Force | Update::FitCursor);
+ }
}
}
BufferView * bv = gv_->documentBufferView();
DocIterator end = to;
- // spell checker corrections may have invalidated the end
- end.fixIfBroken();
-
if (from.pit() != end.pit()) {
// there are multiple paragraphs in selection
Cursor & bvcur = bv->cursor();
void SpellcheckerWidget::Private::forward()
{
- BufferView * bv = gv_->documentBufferView();
- DocIterator from = bv->cursor();
+ DocIterator const from = cursor();
dispatch(FuncRequest(LFUN_ESCAPE));
- if (!atLastPos(bv->cursor())) {
+ fixPositionsIfBroken();
+ if (!atLastPos(cursor())) {
dispatch(FuncRequest(LFUN_CHAR_FORWARD));
}
- if (atLastPos(bv->cursor())) {
+ if (atLastPos(cursor())) {
return;
}
- if (from == bv->cursor()) {
+ if (from == cursor()) {
//FIXME we must be at the end of a cell
dispatch(FuncRequest(LFUN_CHAR_FORWARD));
}
- if (isWrapAround(bv->cursor())) {
+ if (isWrapAround(cursor())) {
hide();
}
}
void SpellcheckerWidget::on_ignoreAllPB_clicked()
{
/// ignore all occurrences of word
- if (d->inCheck())
+ if (d->disabled())
return;
LYXERR(Debug::GUI, "Spellchecker: ignore all button");
if (d->word_.lang() && !d->word_.word().empty())
void SpellcheckerWidget::on_addPB_clicked()
{
/// insert word in personal dictionary
- if (d->inCheck())
+ if (d->disabled())
return;
LYXERR(Debug::GUI, "Spellchecker: add word button");
theSpellChecker()->insert(d->word_);
void SpellcheckerWidget::on_ignorePB_clicked()
{
/// ignore this occurrence of word
- if (d->inCheck())
+ if (d->disabled())
return;
LYXERR(Debug::GUI, "Spellchecker: ignore button");
d->forward();
void SpellcheckerWidget::on_findNextPB_clicked()
{
- if (d->inCheck())
+ if (d->disabled())
return;
docstring const textfield = qstring_to_ucs4(d->ui.wordED->text());
docstring const datastring = find2string(textfield,
void SpellcheckerWidget::on_replacePB_clicked()
{
- if (d->inCheck())
+ if (d->disabled())
return;
docstring const textfield = qstring_to_ucs4(d->ui.wordED->text());
docstring const replacement = qstring_to_ucs4(d->ui.replaceCO->currentText());
void SpellcheckerWidget::on_replaceAllPB_clicked()
{
- if (d->inCheck())
+ if (d->disabled())
return;
docstring const textfield = qstring_to_ucs4(d->ui.wordED->text());
docstring const replacement = qstring_to_ucs4(d->ui.replaceCO->currentText());
if (!bv || bv->buffer().text().empty())
return;
+ fixPositionsIfBroken();
+
+ SpellChecker * speller = theSpellChecker();
+ if (speller && !speller->hasDictionary(bv->buffer().language())) {
+ int dsize = speller->numDictionaries();
+ if (0 == dsize) {
+ hide();
+ QMessageBox::information(p,
+ qt_("Spell Checker"),
+ qt_("Spell checker has no dictionaries."));
+ return;
+ }
+ }
+
DocIterator from = bv->cursor();
- DocIterator to = end_;
+ DocIterator to = isCurrentBuffer(from) ? end_ : doc_iterator_end(&bv->buffer());
WordLangTuple word_lang;
docstring_list suggestions;