sw bibliography: fix missing identifier + local URL in biblio reference

Regression from commit 5fd2f212a1db9e061fb3a886810b427251e8a6bf (sw
bibliography, local copy: handle the bibliography database window,
2021-07-27), the problem was that the newly added LOCAL_URL column
affected BibliographyLoader::getPropertyValue(), which was not adapted,
done now.

Change-Id: Id3bd95bdd5c692048a50ce7ba2e5cdf16e824bd9
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/120400
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>

1 files changed, 50 insertions, 0 deletions

diff --git a/extensions/qa/bibliography/bibliography.cxx b/extensions/qa/bibliography/bibliography.cxx
new file mode 100644
index 000000000000..af68ee95eb70
--- /dev/null
+++ b/extensions/qa/bibliography/bibliography.cxx
@@ -0,0 +1,50 @@
+/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- */
+/*
+ * This file is part of the LibreOffice project.
+ *
+ * This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this
+ * file, You can obtain one at http://mozilla.org/MPL/2.0/.
+ */
+
+#include <test/bootstrapfixture.hxx>
+#include <unotest/macros_test.hxx>
+
+#include <com/sun/star/beans/XPropertySet.hpp>
+#include <com/sun/star/frame/Bibliography.hpp>
+
+using namespace ::com::sun::star;
+
+namespace
+{
+/// Covers extensions/source/bibliography/ fixes.
+class Test : public test::BootstrapFixture, public unotest::MacrosTest
+{
+};
+}
+
+CPPUNIT_TEST_FIXTURE(Test, testBibliographyLoader)
+{
+    // Given a bibliography provider:
+    uno::Reference<container::XNameAccess> xBibAccess
+        = frame::Bibliography::create(mxComponentContext);
+    uno::Reference<beans::XPropertySet> xPropSet(xBibAccess, uno::UNO_QUERY);
+    uno::Sequence<beans::PropertyValue> aSeq;
+
+    // When getting the column names:
+    xPropSet->getPropertyValue("BibliographyDataFieldNames") >>= aSeq;
+
+    // Then make sure we have columns and all have non-empty names:
+    CPPUNIT_ASSERT(aSeq.hasElements());
+
+    // Without the accompanying fix in place, this test would have failed, as the last column
+    // (LOCAL_URL) had an empty field name:
+    for (const auto& rPair : std::as_const(aSeq))
+    {
+        CPPUNIT_ASSERT(!rPair.Name.isEmpty());
+    }
+}
+
+CPPUNIT_PLUGIN_IMPLEMENT();
+
+/* vim:set shiftwidth=4 softtabstop=4 expandtab: */