[Libreoffice-bugs] [Bug 123975] New: "Support primary keys" checkbox in Advanced Settings dialog must be tri-state

[bugzilla-daemon at bugs.documentfoundation.org]

https://bugs.documentfoundation.org/show_bug.cgi?id=123975

            Bug ID: 123975
           Summary: "Support primary keys" checkbox in Advanced Settings
                    dialog must be tri-state
           Product: LibreOffice
           Version: 6.2.1.2 release
          Hardware: All
                OS: All
            Status: UNCONFIRMED
          Keywords: regression
          Severity: normal
          Priority: medium
         Component: Base
          Assignee: libreoffice-bugs at lists.freedesktop.org
          Reporter: mikekaganski at hotmail.com
                CC: caolanm at redhat.com

Created attachment 149851
  --> https://bugs.documentfoundation.org/attachment.cgi?id=149851&action=edit
A dummy ODBC connection to test Advanced Settings dialog

In Edit->Database->Advanced Settings... dialog, Support primary keys checkbox
must be a tri-state [1]. In 6.2, it's a simple toggle, which makes it
impossible to enable the heuristics described in [1].

The underlying pool item is indeed OptionalBoolItem - see
ODbAdminDialog::createItemSet. But the weld::CheckButton never gets initialized
as a tri-state control; and even if it would, all its methods disable the
tri-state mode, even initially set by set_inconsistent().

Use attached dummy database to test the dialog.

This is regression from welding work; Caolan - could you please look at this?

Version: 6.2.2.1 (x64)
Build ID: fcd633fb1bf21b0a99c9acb3ad6e526437947b01
CPU threads: 12; OS: Windows 10.0; UI render: GL; VCL: win; 
Locale: ru-RU (ru_RU); UI-Language: en-US
Calc: CL

[1] https://help.libreoffice.org/Common/Special_Settings

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/libreoffice-bugs/attachments/20190309/86ddedbe/attachment.html>