Bug 353899 - several user configurable settings are ignored
Summary: several user configurable settings are ignored
Status: REPORTED
Alias: None
Product: KBibTeX
Classification: Applications
Component: User interface (show other bugs)
Version: git (master)
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Thomas Fischer
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-10-14 15:08 UTC by Gandalf Lechner
Modified: 2019-05-31 12:39 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
screenshot (142.35 KB, image/png)
2016-08-15 09:57 UTC, Gandalf Lechner
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Gandalf Lechner 2015-10-14 15:08:53 UTC
When making adjustments to the settings of kbibtex, several of these are ignored. 

For example, after setting double click to open the associated file still the editor is launched upon double click.

Or, when selecting some id pattern as default, closing settings and opening it again, it is not selected as default anymore.

Reproducible: Always
Comment 1 Thomas Fischer 2015-10-24 21:19:33 UTC
I can confirm the bug, it is due to some "FIXME"s in class KBibTeXPreferencesDialog, file kbibtexpreferencesdialog.cpp around lines 135 to 145.
Comment 2 Thomas Fischer 2015-10-25 19:49:32 UTC
Please test the updated Git master (commit 8a6fc36e6c81f6dfaa0da5 or later) if the problem has been fixed.
Comment 3 Thomas Fischer 2016-02-15 19:29:03 UTC
Has this problem been fixed for your in commit 8a6fc36e6c8 or later?

(In reply to Gandalf Lechner from comment #0)
> When making adjustments to the settings of kbibtex, several of these are
> ignored. 
> 
> For example, after setting double click to open the associated file still
> the editor is launched upon double click.
> 
> Or, when selecting some id pattern as default, closing settings and opening
> it again, it is not selected as default anymore.
> 
> Reproducible: Always
Comment 4 Thomas Fischer 2016-08-11 19:55:24 UTC
Hello Gandalf,

any news on this bug? Can you please confirm whether it is fixed or not?

(In reply to Thomas Fischer from comment #3)
> Has this problem been fixed for your in commit 8a6fc36e6c8 or later?
> 
> (In reply to Gandalf Lechner from comment #0)
> > When making adjustments to the settings of kbibtex, several of these are
> > ignored.
Comment 5 Gandalf Lechner 2016-08-15 09:57:28 UTC
Created attachment 100603 [details]
screenshot
Comment 6 Gandalf Lechner 2016-08-15 09:58:40 UTC
Hi Thomas,

I recently switched to kde neon (git stable), and now have further problems with kbibtex. Using a recent checkout (git rev 2eeb6ec, master), I have only limited functionality - the element editor is not working, etc. So at the moment, I cannot comment on these older bug reports.
Comment 7 Thomas Fischer 2016-08-16 20:33:24 UTC
> I recently switched to kde neon (git stable), and now have further problems
> with kbibtex. Using a recent checkout (git rev 2eeb6ec, master), I have only
> limited functionality - the element editor is not working, etc. So at the
> moment, I cannot comment on these older bug reports.
I think I know which issue you are referring to. This was due to a typo in a recent commit and the problem was fixed in 01e4d3c8b610a483c2 four days ago. Please fetch the most recent master and test for yourself.
Comment 8 Gandalf Lechner 2016-08-16 20:50:57 UTC
After a fresh checkout just now, the program crashes immediately after starting it, with the message 

"ASSERT failure in MenuLineEdit::slotTextChanged: "d->m_multiLineEditText is NULL", file /home/gandalf/down/git/kbibtex/src/gui/widgets/menulineedit.cpp, line 314"
Comment 9 Thomas Fischer 2016-08-18 07:42:25 UTC
(In reply to Gandalf Lechner from comment #8)
> After a fresh checkout just now, the program crashes immediately after
> starting it, with the message 
> 
> "ASSERT failure in MenuLineEdit::slotTextChanged: "d->m_multiLineEditText is
> NULL", file /home/gandalf/down/git/kbibtex/src/gui/widgets/menulineedit.cpp,
> line 314"

Sorry, this was due to a regression introduced in a recent refactoring. Commit c6855b146640e7efd7ebbf8fc6abd should fix this issue.
Please test again.
Comment 10 Gandalf Lechner 2016-08-18 09:46:00 UTC
After the last checkout, the crash has disappeared. And also the bug in this report has been fixed, thanks :-)
Comment 11 Gandalf Lechner 2016-08-18 09:56:07 UTC
Sorry, I spoke to soon - I just saw that the selection which columns are shown in the seach result panel is still forgotten after closing and restarting the program.