Age | Commit message (Collapse) | Author | Files | Lines |
|
This avoid Python warnings about this during checks
Change-Id: I6647d9c93f3c2fbc1af475f8bc45e03300ee8b79
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100518
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
|
|
Change-Id: I0fef17ccc8f110d1d8bf629737807f1bcc618b76
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/93685
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
The 'is' operator should not be used for comparison on some types of
literals. In CPython this works by accident and CPython 3.8 introduced a
SyntaxWarning (see https://bugs.python.org/issue34850).
Change-Id: Ifabe22e4ae3b9e1d3bfd5a663f39500e381cfa6e
Reviewed-on: https://gerrit.libreoffice.org/83034
Tested-by: Jenkins
Reviewed-by: Xisco Faulí <xiscofauli@libreoffice.org>
|
|
Change-Id: I309ce0b4aade85a510836be0d04bf66c916a464a
Reviewed-on: https://gerrit.libreoffice.org/75507
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
Tested-by: Jenkins
Reviewed-by: Andrea Gelmini <andrea.gelmini@gelma.net>
|
|
Change-Id: I11c2ac4855a5fec2d26255ec3ab14ffe50e0fbbe
Reviewed-on: https://gerrit.libreoffice.org/71066
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
|
|
Change-Id: I5905dd94843595b229046c0b4a0f16181c16fd9e
Reviewed-on: https://gerrit.libreoffice.org/69869
Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
Tested-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
|
|
Create a brand new odb file (embedded HSQLDB)
Create a simple table in it
Launch Query wizard, you'll see these on console:
...
at com.sun.proxy.$Proxy69.addNew(Unknown Source)
at com.sun.star.wizards.common.NumberFormatter.defineNumberFormat(NumberFormatter.java:96)
at com.sun.star.wizards.ui.FilterComponent.addNumberFormats(FilterComponent.java:504)
at com.sun.star.wizards.query.QueryWizard.buildSteps(QueryWizard.java:233)
...
...
at com.sun.proxy.$Proxy69.addNew(Unknown Source)
at com.sun.star.wizards.common.NumberFormatter.defineNumberFormat(NumberFormatter.java:96)
at com.sun.star.wizards.ui.FilterComponent.addNumberFormats(FilterComponent.java:506)
at com.sun.star.wizards.query.QueryWizard.buildSteps(QueryWizard.java:233)
...
Root cause is FilterComponent.java uses "YYYY-MM-DD" and "YYYY-MM-DD HH:MM:SS" (lines 504 and 506)
If it's ok for English locales, xNumberFormats.queryKey calls fail at least for French locales.
Use addNewConverted instead of addNew method to take into account the difference of locales
and avoid these stacktraces
Change-Id: I5ef6325b8a213ae6758e378c4d3b8560c117042f
Reviewed-on: https://gerrit.libreoffice.org/69131
Tested-by: Jenkins
Reviewed-by: Lionel Elie Mamane <lionel@mamane.lu>
|
|
This code was automatically converted from Java to Python.
It might be used at some point if the remaining java wizards
are converted to python, however that's not the case as
per today, thus this code is unused.
Let's get rid of it as it might create confusion
Change-Id: Ic58ade24b10f7a0654b3e8ae39f6afa712589c3d
Reviewed-on: https://gerrit.libreoffice.org/67868
Tested-by: Jenkins
Reviewed-by: Xisco Faulí <xiscofauli@libreoffice.org>
|
|
it now looks old-fashioned hyphenated
Change-Id: I5b2b905277356c1b986f97de29f82ac1c21b1709
Reviewed-on: https://gerrit.libreoffice.org/66796
Tested-by: Jenkins
Reviewed-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
|
|
Change-Id: I4ac17e520a93678a1a0e8dfd5fd849eaf276219d
Reviewed-on: https://gerrit.libreoffice.org/63776
Reviewed-by: Heiko Tietze <tietze.heiko@gmail.com>
Tested-by: Heiko Tietze <tietze.heiko@gmail.com>
|
|
Unless the underlying column is not nullable and has no default value
this is only a guess, form designer can change it
Change-Id: Ifa403e00b21fdaf86aef383503d54879b25ac62b
Reviewed-on: https://gerrit.libreoffice.org/62969
Reviewed-by: Lionel Elie Mamane <lionel@mamane.lu>
Tested-by: Lionel Elie Mamane <lionel@mamane.lu>
|
|
Change-Id: If89ec640901253121529c09aef4537c6b13cee1d
Reviewed-on: https://gerrit.libreoffice.org/52280
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
Tested-by: Jenkins <ci@libreoffice.org>
|
|
This code got unused after 5b3022860abfe7eefd2e25b60b5b2a563b42b3c0
Change-Id: I6c198ab359f8468a9f4d9041b9bc4feb88eedd74
Reviewed-on: https://gerrit.libreoffice.org/49109
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Xisco Faulí <xiscofauli@libreoffice.org>
|
|
Some HIDs were a bit off in these wizards
- Report Wizard 'Grouping' page
- Table Wizard 'Set primary key' page
- Query Wizard 'Grouping' page
All these contain a subclass of a FieldSelection.
Due to the implementation of HID handling there, we need 4 consecutive
HIDs in HelpIds.java for the buttons cmdMoveSelected, cmdMoveAll,
cmdRemoveSelected and cmdRemoveAll.
Above pages contain subclasses without the MoveAll and RemoveAll buttons
so we need empty placeholders in their places to cater for the assumptions
of the parent class.
This also means that HID numbers must be adjusted a bit, and in ReportWizard.java
we need to shift the base HID number of the Sorting page that follows
the Grouping page.
Change-Id: I944dd4f781aff2c0b0cfd99c3386001506c01440
Reviewed-on: https://gerrit.libreoffice.org/46270
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Andras Timar <andras.timar@collabora.com>
|
|
...which had accidentally been removed completely from this part of
cb2550750386f51ef7a46a8a664a4ab9de614f8d "Drop lazywrite property, which is
silently ignored by configmgr anyway"
Change-Id: Ibacacc7b04bbd0a0cc819530594dd042329b366f
Reviewed-on: https://gerrit.libreoffice.org/47600
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
Tested-by: Jenkins <ci@libreoffice.org>
|
|
Change-Id: I82676e4a0b3dee33e248b3ba81e99f949ee9227a
|
|
Since we use contextual messages, we need to concatenate:
- context
- EOT character
- the message itself
see https://www.gnu.org/software/gettext/manual/html_node/MO-Files.html
A priori, regression from 00657aef09d854c74fb426a935a3e8b1fc390bb0
Change-Id: Ie3e781e30ce09f0121afff2f0eb0769b1fbd2114
Reviewed-on: https://gerrit.libreoffice.org/47521
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Jean-Baptiste Faure <jbfaure@libreoffice.org>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
A still actively used HID was accidentally removed in commit:
https://cgit.freedesktop.org/libreoffice/core/commit/wizards/com/sun/star/wizards/common/HelpIds.java?id=82d863b4a410fa78120d2efc41bd2b3b61f21cee
This caused some HIDs in the Form wizard to slide off by one
Change-Id: I0142a150f1c65456ba9affe5d997ddabe37d1b3e
Reviewed-on: https://gerrit.libreoffice.org/45057
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Katarina Behrens <Katarina.Behrens@cib.de>
|
|
In commit b3f96d2ccba362a2898bc3a4d365ce7c4d0698e2
I made a mistake that caused a lot of Wizard HIDS go off by one
i.e. they were attached to the next UI item.
I noticed this in the Fax wizard.
Adding one more empty array item seems to fix this.
Problem was caused by the replacement of this line with an empty item:
"", "HID:WIZARDS_HID_IS_BTN_NONE", # HID:41002
Change-Id: Id7df502ac843ab0e572ca59a046f5bacdda7e9bc
Reviewed-on: https://gerrit.libreoffice.org/44075
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Andras Timar <andras.timar@collabora.com>
|
|
...at least ever since the "new" configmgr reimplementation,
6b849a6aeeb9ea8b1e25e28d5a8be390e425f84e "#i101955# initial work in progress of
a configmgr reimplementation (for now in an extra module 'configmgr2')" et al
Change-Id: I43430d991647fb2e26762463d51175247db0604b
|
|
* all .ui files go from <interface> to <interface domain="MODULE"> e.g. vcl
* all .src files go away and the english source strings folded into the .hrc as NC_("context", "source string")
* ResMgr is dropped in favour of std::locale imbued by boost::locale::generator pointed at matching
MODULE .mo files
* UIConfig translations are folded into the module .mo, so e.g. UIConfig_cui
goes from l10n target to normal one, so the res/lang.zips of UI files go away
* translation via Translation::get(hrc-define-key, imbued-std::locale)
* python can now be translated with its inbuilt gettext support (we keep the name strings.hrc there
to keep finding the .hrc file uniform) so magic numbers can go away there
* java and starbasic components can be translated via the pre-existing css.resource.StringResourceWithLocation
mechanism
* en-US res files go away, their strings are now the .hrc keys in the source code
* remaining .res files are replaced by .mo files
* in .res/.ui-lang-zip files, the old scheme missing translations of strings
results in inserting the english original so something can be found, now the
standard fallback of using the english original from the source key is used, so
partial translations shrink dramatically in size
* extract .hrc strings with hrcex which backs onto
xgettext -C --add-comments --keyword=NC_:1c,2 --from-code=UTF-8 --no-wrap
* extract .ui strings with uiex which backs onto
xgettext --add-comments --no-wrap
* qtz for gettext translations is generated at runtime as ascii-ified crc32 of
content + "|" + msgid
* [API CHANGE] remove deprecated binary .res resouce loader related uno apis
com::sun::star::resource::OfficeResourceLoader
com::sun::star::resource::XResourceBundleLoader
com::sun::star::resource::XResourceBundle
when translating strings via uno apis
com.sun.star.resource.StringResourceWithLocation
can continue to be used
Change-Id: Ia2594a2672b7301d9c3421fdf31b6cfe7f3f8d0a
|
|
Change-Id: Ic1393da64328c0dc7e2860334b204139bf537d81
Reviewed-on: https://gerrit.libreoffice.org/38401
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
like LibreOffice java extensions, and nlpsolver and librelogo, do, removes the
magic number problem
Change-Id: I561249ccebe41c787cd8cdaa80e1d035f27052cc
Reviewed-on: https://gerrit.libreoffice.org/37986
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
Change-Id: Ie81495a15376b01b0e87ee84628b8c6945897a3d
Reviewed-on: https://gerrit.libreoffice.org/37877
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
These were used on the recently dropped help pages
Change-Id: I391c881b43c49000c37cbe53b111f686eaa64843
Reviewed-on: https://gerrit.libreoffice.org/36093
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
|
|
Change-Id: Ibc81246e615d7c68c93e01b84584e78a79bece16
Reviewed-on: https://gerrit.libreoffice.org/34779
Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
Tested-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
|
|
There are still webwizard*.xhp in help/source/text/shared/autopi/
Change-Id: If54d7bfc093c8c0765fd881eeb14f62bc3517194
Reviewed-on: https://gerrit.libreoffice.org/34402
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
|
|
as shown by
commit a96c308e52983b7bc0275e31e50a9b04e5805852
Author: Noel Grandin <noel@peralex.com>
Date: Wed Dec 17 10:33:26 2014 +0200
java: these fields can be converted to local variables
Change-Id: Ifefb5de196a3e5cbaa8945759da42886c69daacf
Change-Id: I4da6c42e0114a891a6b6af8889c94b4332b0b40b
|
|
Change-Id: I860cc4aa04b4e5cd1437b2f24176ee2ccdec3266
Reviewed-on: https://gerrit.libreoffice.org/25420
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>
|
|
found by UCDetector
Change-Id: Ic0295a24b26e206eb53eda2da540755477df86df
|
|
Change-Id: I7d19233bb0f63f55c87d83e50b7f865f5c7a8ea4
|
|
Change-Id: Iea0dc27bf32499e1f8b38d8a0e90cebe6ab8b31f
|
|
(cherry picked from commit 95efcaf575dced8cc18f2ca9fb874d772ad23595)
Change-Id: I8bb247c92f0326da916940f4fc7999d81998297a
|
|
Change-Id: I291db1dff5ab3ce40fda6f822a0e3ff1e6ef36c1
Reviewed-on: https://gerrit.libreoffice.org/19855
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>
|
|
Change-Id: I0457b81668e9427a3c8d6a4af93438b7fb2bb7ba
|
|
Change-Id: Iafb022cf3bbdbaa8a6b230cb60377d90e64fe142
|
|
Change-Id: I1661514bf3002b79544615620843d1e535aabfce
|
|
also
coverity#1326574 Useless call
Change-Id: Ida860ca8ed1475c88e90fc8aaf608c440453a9a0
|
|
Change-Id: I832b2fc243404a6ae9796ad994e901880d3b9260
|
|
1) Fix value of constants RES_OK, RES_CANCEL and RES_HELP
http://opengrok.libreoffice.org/xref/core/offapi/com/sun/star/awt/PushButtonType.idl#29
2) self.updatePublishUI was wrong because FTP appears in second but there are 2 elements
for each module (local dir, zip, ftp) in pubAware vector
comment was wrong:
- local takes elements 0 and 1
- zip takes elements 2 and 3
- ftp takes elements 4 (that's why "4" is the right value) and 5
see http://opengrok.libreoffice.org/xref/core/wizards/com/sun/star/wizards/web/WWD_Startup.py#509
So fix value passed to self.updatePublishUI + comment
Just a bit of history:
before fix of tdf#92934: Web Wizard: Cannot Export Zip archive (http://cgit.freedesktop.org/libreoffice/core/commit/?id=e72a4a3c2e1608e301daef0b6e87cc70d814e736),
the order seemed to be this one for pubAware: local, ftp, zip (whereas UI order was already local, zip and ftp)
Perhaps the fix was wrong since there was a similar pb for tdf#94421: Web Wizard, option shift when loading saved session
Anyway now the pubAware order is the same as UI order, it may help for readability.
minor
a) increase space for lblStatus (there weren't enough room for some locales, eg French)
b) replace use of copy3 (deprecated) by copy2
Change-Id: I27d3c2dd699fbe2d51a63e462c7b39e78895d1be
Reviewed-on: https://gerrit.libreoffice.org/19115
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
|
|
Call getSelectedFiles instead of getFiles+adapt behavior so it deals
with an array of files with their full path
Change-Id: I21b6c32dea85460461925a9b44dac195d79b7eac
Reviewed-on: https://gerrit.libreoffice.org/18714
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
Change-Id: I3afde8e7f8b0b88e8f1a5d1ad71429c2aa7d7a94
|
|
The hard-coded "/" separator causes getFileURLFromSystemPath to fail,
and it's not obvious why the URL is being converted to a system path
in the first place.
Change-Id: I06ebe6d92954c4f3c884ae6f0b327ea8a36c10a4
|
|
found with
$ git grep -lP 'return\s*\(\s*\w+\s*\)\s*;'
Change-Id: Ic51606877a9edcadeb647c5bf17bc928b69ab60e
|
|
found by UCDetector
Change-Id: I6b0f49529379072da566e927b86815f173e7a90b
|
|
found by UCDetector
Change-Id: I0a0a00d1fae1fed2e6aef198b7a1482d7e7e29f0
|
|
Change-Id: Ifefb5de196a3e5cbaa8945759da42886c69daacf
|
|
Change-Id: Ie2da7cbbd1733881ff2da6fa095c37c21cf19fe8
Reviewed-on: https://gerrit.libreoffice.org/13435
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>
|
|
found by PMD
Change-Id: Id6737916b68ccbdbdeec5d314747a38410923ac6
Reviewed-on: https://gerrit.libreoffice.org/13409
Tested-by: LibreOffice gerrit bot <gerrit@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
|
|
Change-Id: I727e7f2a7527c60bb45f0ba5d0e88a66c5ccdd6f
|