Stefan Schimanski wrote:

Am 12.08.2008 um 17:24 schrieb rgheck:

Stefan Schimanski wrote:
Not sure this very commit is the reason. But I was bitten today several times from the following ugly bug:

Whenever I want to insert a citation, nothing is inserted, instead the citation in front of the cursor is changed.

This bug existed before, but was fixed some time ago I think. But now it's even worse than before. In addition to the inability to insert a new citation, copying an old citation inset fails with an assertion which finally closes the document.

No, this commit isn't it. This has to do with bibitems---the bibliography environment---not with citations. But yes, I've seen that bug, too, and it is VERY annoying. It has to be fixed for 1.6.

I reverted the commit locally and now it works as before. So there is some connection. Did not have to time to really investigate it.

I don't think it can have had to do with that. But the attached patch fixes the bug for me.

Here's a recipe to trigger it, first. MODIFY a citation. Then try to insert a new one. See if that gets you the bug. Then try the patch.

The problem is that GuiView::disconnectDialog() is never being called in this case.

rh

Index: GuiCitation.cpp
===================================================================
--- GuiCitation.cpp	(revision 26122)
+++ GuiCitation.cpp	(working copy)
@@ -167,14 +167,14 @@
 {
 	applyView();
 	clearSelection();
-	hide();
+	hideView();
 }
 
 
 void GuiCitation::on_cancelPB_clicked()
 {
 	clearSelection();
-	hide();
+	hideView();
 }
 
 

Reply via email to