- 17 Eki, 2015 40 kayıt (commit)
-
-
Julien Nabet yazdı
Auformat list (maData) is defined as "MapType" which itself is defined like this: boost::ptr_map<OUString, ScAutoFormatData> so default sorting is ascii 2 consequences: 1) Default didn't appear first 2) When adding a new autoformat entry when it was new first one of the list wasn't saved because of iterator was incremented first before looping See https://bugs.documentfoundation.org/show_bug.cgi?id=94173#c5 There were some other weird behaviors too according to comments of the bugtracker Regression from http://cgit.freedesktop.org/libreoffice/core/commit/?id=72c1b6141d590fb4479925ed8bc88b79357c2bfc Solution: Add a Compare so Default entry is always first one so the first time incremented iterator is ok and new entry (even if new first one in list) is saved Thank you Markus for the idea! (I was lost in Compare syntax) Reviewed-on: https://gerrit.libreoffice.org/18598Tested-by:
Jenkins <ci@libreoffice.org> Reviewed-by:
Julien Nabet <serval2412@yahoo.fr> (cherry picked from commit 652158c3) use collator for UI visible sorting, tdf#94173 follow-up (cherry picked from commit a71febc9) b9f7ebb087c1ce5008f43b2df3f1fadc41066ed1 Change-Id: I9ba0cdc63c66b747db102bb661cd09fbfe5996ae Reviewed-on: https://gerrit.libreoffice.org/19029Reviewed-by:
Eike Rathke <erack@redhat.com> Tested-by:
Eike Rathke <erack@redhat.com> (cherry picked from commit 457a9f40)
-
Caolán McNamara yazdı
Change-Id: I1b6f8d5190fae16bdeb1def7887d3121dad6bc16 (cherry picked from commit 75304346) Reviewed-on: https://gerrit.libreoffice.org/19147Reviewed-by:
Eike Rathke <erack@redhat.com> Tested-by:
Eike Rathke <erack@redhat.com> (cherry picked from commit ddd5614f)
-
Caolán McNamara yazdı
Change-Id: Ifc1f226a38328e6ce02d3181917dd0530f320914 (cherry picked from commit 1e468ed5) Reviewed-on: https://gerrit.libreoffice.org/19021Reviewed-by:
Miklos Vajna <vmiklos@collabora.co.uk> Tested-by:
Miklos Vajna <vmiklos@collabora.co.uk> (cherry picked from commit 935c1294)
-
Caolán McNamara yazdı
regression from... commit 8a08f686 Author: Mike <mikekaganski@hotmail.com> Date: Mon Apr 27 01:27:05 2015 +1000 tdf#66141: SwTxtFrm::FormatQuick(bool) endless loop If a line happens to be invisible (e.g. in too thin cell of a table) then aLine.FormatLine(nStart) returns nStart, and aLine.Insert( new SwLineLayout() ) is executed until OOM. keep the zero advance loop detection attempt, but allow the first insertion and disallow subsequent inserts Change-Id: I16380588220149cfd0ed0f835f08d2849180fece (cherry picked from commit f06508e2) Reviewed-on: https://gerrit.libreoffice.org/18908Reviewed-by:
Miklos Vajna <vmiklos@collabora.co.uk> Tested-by:
Miklos Vajna <vmiklos@collabora.co.uk> (cherry picked from commit dabd6add)
-
Caolán McNamara yazdı
overwrote crash-1.xls by accident in commit d2281e20 Author: Caolán McNamara <caolanm@redhat.com> Date: Fri Oct 2 10:41:25 2015 +0100 fix crash on loading certain xls Change-Id: I2dd58060c1f3c9f12356c4ab18a0c838e7cdd718 so restore that, add another test case for... commit c4d1a99e Author: Caolán McNamara <caolanm@redhat.com> Date: Sat Oct 3 11:53:48 2015 +0100 fix crash on loading certain xls Change-Id: I4f4563b07109df7e2288458cf8adda37582262cf as crash-4.xls and add d2281e20 test-case in as crash-5.xls Change-Id: I2de3986290bdf5e227e4471e411bb6a9a83b3085 (cherry picked from commit c70c4d20) Reviewed-on: https://gerrit.libreoffice.org/19111Tested-by:
Jenkins <ci@libreoffice.org> Reviewed-by:
David Tardon <dtardon@redhat.com> (cherry picked from commit a47494d5)
-
Caolán McNamara yazdı
Change-Id: I4f4563b07109df7e2288458cf8adda37582262cf (cherry picked from commit c4d1a99e) Reviewed-on: https://gerrit.libreoffice.org/19110Tested-by:
Jenkins <ci@libreoffice.org> Reviewed-by:
David Tardon <dtardon@redhat.com> (cherry picked from commit aaf53996)
-
Caolán McNamara yazdı
Change-Id: I22be90d7c54b1118f81337bd9e1d97ba3a1fd86e (cherry picked from commit 4eb26f9c) Reviewed-on: https://gerrit.libreoffice.org/19109Tested-by:
Jenkins <ci@libreoffice.org> Reviewed-by:
David Tardon <dtardon@redhat.com> (cherry picked from commit ce321f45)
-
Caolán McNamara yazdı
Change-Id: I9b781aaf98c16c83e12406ee38167814fb32cfc9 (cherry picked from commit a28f5f96) Reviewed-on: https://gerrit.libreoffice.org/19119Reviewed-by:
David Tardon <dtardon@redhat.com> Tested-by:
David Tardon <dtardon@redhat.com> (cherry picked from commit a016ffbf)
-
Caolán McNamara yazdı
Change-Id: I2dd58060c1f3c9f12356c4ab18a0c838e7cdd718 (cherry picked from commit d2281e20) Reviewed-on: https://gerrit.libreoffice.org/19097Tested-by:
Jenkins <ci@libreoffice.org> Reviewed-by:
Caolán McNamara <caolanm@redhat.com> Tested-by:
Caolán McNamara <caolanm@redhat.com> (cherry picked from commit 070693d7)
-
Caolán McNamara yazdı
Change-Id: I36f9439e417b380bd3f999c02c55c29337796002 (cherry picked from commit 9b738520) Reviewed-on: https://gerrit.libreoffice.org/19103Tested-by:
Jenkins <ci@libreoffice.org> Reviewed-by:
Markus Mohrhard <markus.mohrhard@googlemail.com> Tested-by:
Markus Mohrhard <markus.mohrhard@googlemail.com> (cherry picked from commit cf54f3a3)
-
Szymon Kłos yazdı
Change-Id: Iabcf17b71b3fe06a25bb7b0e7f3aa65ad0663e69 Reviewed-on: https://gerrit.libreoffice.org/19083Tested-by:
Jenkins <ci@libreoffice.org> Reviewed-by:
Caolán McNamara <caolanm@redhat.com> Tested-by:
Caolán McNamara <caolanm@redhat.com> (cherry picked from commit 7dd84833)
-
Michael Meeks yazdı
Change-Id: I212bb442bdffa38d77e7e7357199c39420fa7951 Reviewed-on: https://gerrit.libreoffice.org/19079Tested-by:
Jenkins <ci@libreoffice.org> Reviewed-by:
Caolán McNamara <caolanm@redhat.com> Tested-by:
Caolán McNamara <caolanm@redhat.com> (cherry picked from commit 1d49c4a5)
-
Markus Mohrhard yazdı
editeng uses special control characters (apparently 0x01) that need to be translated by editeng to the real characters. Let EditEngine handle the conversion from editeng string to OUString. Change-Id: If5014e1ada7f1df933a084b17bb307035ce01cc6 Reviewed-on: https://gerrit.libreoffice.org/19065Tested-by:
Jenkins <ci@libreoffice.org> Reviewed-by:
Caolán McNamara <caolanm@redhat.com> Tested-by:
Caolán McNamara <caolanm@redhat.com> (cherry picked from commit e108b24c)
-
Laurent Godard yazdı
- preserve backward compatibility - nDebugFlag is stored but not used when loaded - Flag nDebugFlag set the top bit to 1 - stores the multiplier of sal_Int16 limit to reach nStart - in load, test this flag bit - rebuild correct nStart - new B_CURVERSION file format for binary storage macro - unit test for big modules Change-Id: Iaa037982d828fef7195615e6eda546b7199a4fe8 Reviewed-on: https://gerrit.libreoffice.org/18926Reviewed-by:
Caolán McNamara <caolanm@redhat.com> Tested-by:
Caolán McNamara <caolanm@redhat.com> (cherry picked from commit db17079f)
-
Julien Nabet yazdı
Inherit from WizardDialog instead of object Define required constants for buttons Remove or tweak calls to super methods Fix call setDialogProperties which was wrong since unification of this method See ee122cfd With the patch, FTP dialog opens, we can also connect successfully to an ftp server Still there are bugs to fix then, first of them directory selection Change-Id: I86c3c9638114216627d92832c26ad780c12860b4 Reviewed-on: https://gerrit.libreoffice.org/18831Reviewed-by:
Caolán McNamara <caolanm@redhat.com> Tested-by:
Caolán McNamara <caolanm@redhat.com> (cherry picked from commit 7e09b365) Reviewed-on: https://gerrit.libreoffice.org/19068 (cherry picked from commit 05f24922)
-
Szymon Kłos yazdı
Change-Id: I94ba2fea74703d69e65c0864744ab81ccf205f9c Reviewed-on: https://gerrit.libreoffice.org/17192Reviewed-by:
Caolán McNamara <caolanm@redhat.com> Tested-by:
Caolán McNamara <caolanm@redhat.com> Reviewed-on: https://gerrit.libreoffice.org/19070 (cherry picked from commit 0f878457)
-
Christian Lohmaier yazdı
Change-Id: I4c21115a9274dbf5fc138705527f2229554ba6f8 (cherry picked from commit 7d596a4b) (cherry picked from commit fd151aa1)
-
Mark Hung yazdı
characters do not take account of hanging punctuation. Note that placing a cursor next to the hanging puncuation is still not possible, but the selection and the non-printing characters works fine now. Change-Id: I22c8c2c5044b46c325236caffec67d9f68c23eed Reviewed-on: https://gerrit.libreoffice.org/17591Tested-by:
Jenkins <ci@libreoffice.org> Reviewed-by:
Norbert Thiebaud <nthiebaud@gmail.com> Reviewed-on: https://gerrit.libreoffice.org/19031Reviewed-by:
Caolán McNamara <caolanm@redhat.com> Tested-by:
Caolán McNamara <caolanm@redhat.com> (cherry picked from commit b63dcee3)
-
Miklos Vajna yazdı
(cherry picked from commit 3e27df10) Conflicts: sw/qa/extras/ooxmlimport/ooxmlimport.cxx writerfilter/source/dmapper/NumberingManager.cxx Change-Id: I0c4366ad0a2f81a198860869f670767343a392be Reviewed-on: https://gerrit.libreoffice.org/19032Reviewed-by:
Caolán McNamara <caolanm@redhat.com> Tested-by:
Caolán McNamara <caolanm@redhat.com> (cherry picked from commit c33af041)
-
Eike Rathke yazdı
Fixes all these test case scenarios: 1. in A1 enter =1E222 * move cell cursor back onto A1 * status bar displays Sum=100000... repeated until filled (or 222 '0' characters) 2. invoke number format dialog on A1 * for General format 100000... is displayed in the preview 3. move cell cursor to A2 * open Function Wizard (Ctrl+F2) * choose (double click) ABS function * enter A1 as parameter * see 100000... displayed as Function result and Result 4. save as .ods * in content.xml see display text of A1 being saved as 100000... Change-Id: I7c22c0461a6783c85c1d51c31e8607fb2edb821c (cherry picked from commit ef0a2683) Reviewed-on: https://gerrit.libreoffice.org/18924Tested-by:
Jenkins <ci@libreoffice.org> Reviewed-by:
Caolán McNamara <caolanm@redhat.com> Tested-by:
Caolán McNamara <caolanm@redhat.com> (cherry picked from commit 746359e4)
-
Michael Stahl yazdı
JunitTest_chart_unoapi fails here with "assert(!hasCurrent())" because the initialization apparently fails due to lack of a wglCreateContextAttribsARB in the VM, so ensure that the hTempRC is not active and also deleted in all cases. (cherry picked from commit 5efad739) vcl: even in the newfangled OpenGL world, initializing members is ... ... still a good idea. (cherry picked from commit 37d24365) Change-Id: I7e03b95d1146af48a24e34692c3c2827298fccee Reviewed-on: https://gerrit.libreoffice.org/19040Reviewed-by:
Markus Mohrhard <markus.mohrhard@googlemail.com> Tested-by:
Markus Mohrhard <markus.mohrhard@googlemail.com> (cherry picked from commit 418d7171)
-
Caolán McNamara yazdı
Change-Id: Id9864f199e270d13d801348b12f1e94dd80558c7 (cherry picked from commit 358ca9ea) Reviewed-on: https://gerrit.libreoffice.org/19042Reviewed-by:
Eike Rathke <erack@redhat.com> Tested-by:
Eike Rathke <erack@redhat.com> (cherry picked from commit 82f88758)
-
Miklos Vajna yazdı
Word has a feature like this: a paragraph style can refer to a numbering style, and both can specify paragraph margins. If that's the case, then the ones from the paragraph style has priority. In Writer, the numbering style has priority, so the only chance for correct import result is to set the margin directly on the paragraph in this case. (cherry picked from commit f4badd9a) Conflicts: sw/qa/extras/ooxmlimport/ooxmlimport.cxx Conflicts: sw/qa/extras/ooxmlimport/ooxmlimport.cxx Change-Id: Iff3b03bcc56e0db3a48452c293acf41c91b8f159 Reviewed-on: https://gerrit.libreoffice.org/18794Reviewed-by:
Andras Timar <andras.timar@collabora.com> Tested-by:
Andras Timar <andras.timar@collabora.com> (cherry picked from commit d14c7b03)
-
Miklos Vajna yazdı
With this, <w:ind w:hanging="..."/> as direct paragraph formatting and <w:ind w:left="..." w:hanging="..."/> as a numbering level formatting is properly merged, i.e. w:left is not lost, defaulting to 0. (cherry picked from commit 56341e5d) Conflicts: sw/qa/extras/ooxmlimport/ooxmlimport.cxx Change-Id: If5534fbd9ee6d41139b0ed3a3df9d0cc5aad3239 Reviewed-on: https://gerrit.libreoffice.org/18906Reviewed-by:
Andras Timar <andras.timar@collabora.com> Tested-by:
Andras Timar <andras.timar@collabora.com> (cherry picked from commit 8e911fe7)
-
Khaled Hosny yazdı
The glyphs for %phi and %varphi symbols were swapped in commit 81001f2c (back in 2009), as part of https://bz.apache.org/ooo/show_bug.cgi?id=105084 (see the document attached there), but no justification was given. Given the shape of the symbols after that change contradicts the ones in the Unicode code charts and other popular math fonts, I simply swapped them back. Change-Id: I0133a3d07df932f144ec5900103e11e27f174a5e Reviewed-on: https://gerrit.libreoffice.org/17822Reviewed-by:
Caolán McNamara <caolanm@redhat.com> Tested-by:
Caolán McNamara <caolanm@redhat.com> Reviewed-on: https://gerrit.libreoffice.org/19000Reviewed-by:
Khaled Hosny <khaledhosny@eglug.org> Tested-by:
Khaled Hosny <khaledhosny@eglug.org> (cherry picked from commit bf6e219e)
-
Michael Stahl yazdı
This was crashing in CppunitTest_sc_macros_test on Windows with --enable-mergelibs (release build), because the first invocation of processWindowResizeEvent() deleted the vcl::Window, and the maControllers.count(pWindow) test creates a VclPtr for it, so ends up with a double-free. TODO: is processWindowResizeEvent() supposed to be idempotent? It would be possible to detect that there is already an event posted by checking m_PostedWindows in postWindowResizeEvent(). Change-Id: I7b72f2baf21bb8223e9fe4bd929d826217b920e5 (cherry picked from commit 520514cf) Reviewed-on: https://gerrit.libreoffice.org/19022Reviewed-by:
Caolán McNamara <caolanm@redhat.com> Tested-by:
Caolán McNamara <caolanm@redhat.com> (cherry picked from commit eb9321d5)
-
Caolán McNamara yazdı
void OutputDevice::acquire() const: Assertion `mnRefCnt>0' failed" Window gets destroyed between postWindowResizeEvent and processWindowResizeEvent Change-Id: I3452a23ad8c3b6d863a56b73166520ef103dad1b (cherry picked from commit 5f36b195) Signed-off-by:
Michael Stahl <mstahl@redhat.com> (cherry picked from commit 2d213a44)
-
Michael Meeks yazdı
Change-Id: I5c331380b872429cf1fa3908c6add8de879c9ed2 Reviewed-on: https://gerrit.libreoffice.org/18927Reviewed-by:
Eike Rathke <erack@redhat.com> Tested-by:
Eike Rathke <erack@redhat.com> (cherry picked from commit 064b759c)
-
Tor Lillqvist yazdı
Change-Id: I60c7540241c41f5063736f2a3d4817371411c8dc (cherry picked from commit a799d7c9) Reviewed-on: https://gerrit.libreoffice.org/18716Tested-by:
Jenkins <ci@libreoffice.org> Reviewed-by:
David Tardon <dtardon@redhat.com> (cherry picked from commit 183129ea)
-
Winfried Donkers yazdı
provide forward compatibility for name change of FINV Reviewed-on: https://gerrit.libreoffice.org/18814Reviewed-by:
Christian Lohmaier <lohmaier+LibreOffice@googlemail.com> Tested-by:
Christian Lohmaier <lohmaier+LibreOffice@googlemail.com> (cherry picked from commit 2cc04087) Conflicts: sc/source/core/tool/compiler.cxx Change-Id: I0afd2555be94c452172e5a357f2f0897be381863 Reviewed-on: https://gerrit.libreoffice.org/18909Reviewed-by:
Eike Rathke <erack@redhat.com> Tested-by:
Eike Rathke <erack@redhat.com> (cherry picked from commit 4a334351)
-
Caolán McNamara yazdı
Change-Id: Ia174feec73ee676567a3632d2f88b11c176b6363 (cherry picked from commit b107353a) Reviewed-on: https://gerrit.libreoffice.org/18902Tested-by:
Jenkins <ci@libreoffice.org> Reviewed-by:
Miklos Vajna <vmiklos@collabora.co.uk> (cherry picked from commit f86ff1a0)
-
Stephan Bergmann yazdı
...it leads to better handling of the failure downstream, if e.g. this is an attempt to copy a file from the presets to an existing file of a previously partly generated UserInstallation, and the presets and the UserInstallation are on different drives and rename(3) fails with EXDEV. (Without this fix, removing the UserInstallation's registrymodifications.xcu caused restarting soffice to fail with "User installation could not be completed" in this case; regression introduced with 6edbcc1b "cid#1242936 handle rename failure.") Change-Id: I409698c4e2e6cc6ccd8b2be80a393c9657b5125d (cherry picked from commit d2a7abeb) Reviewed-on: https://gerrit.libreoffice.org/18898Tested-by:
Jenkins <ci@libreoffice.org> Reviewed-by:
Miklos Vajna <vmiklos@collabora.co.uk> (cherry picked from commit 23be91de)
-
Mark Hung yazdı
vertical layout. Change-Id: I6391e665db205545f0d660e3de826755c954f286 Reviewed-on: https://gerrit.libreoffice.org/17836Reviewed-by:
Caolán McNamara <caolanm@redhat.com> Tested-by:
Caolán McNamara <caolanm@redhat.com> Reviewed-on: https://gerrit.libreoffice.org/18879Reviewed-by:
Miklos Vajna <vmiklos@collabora.co.uk> Tested-by:
Miklos Vajna <vmiklos@collabora.co.uk> (cherry picked from commit ea91af84)
-
Caolán McNamara yazdı
Change-Id: Ia8167833b54bcfc68f1a306e58a1134a38d64ef6 Reviewed-on: https://gerrit.libreoffice.org/18844Tested-by:
Jenkins <ci@libreoffice.org> Reviewed-by:
Cor Nouws <cor.nouws@documentfoundation.org> Reviewed-by:
Adolfo Jayme Barrientos <fitojb@ubuntu.com> (cherry picked from commit ce75966a)
-
Eike Rathke yazdı
Change-Id: Id0fec512b53aab47f9951056e8726af0cce45708 (cherry picked from commit 624ab2e5) Reviewed-on: https://gerrit.libreoffice.org/18664Reviewed-by:
Markus Mohrhard <markus.mohrhard@googlemail.com> Tested-by:
Markus Mohrhard <markus.mohrhard@googlemail.com> (cherry picked from commit 00adc136)
-
Mark Hung yazdı
tdf#40091 Vertical writing inferior to MS Word tdf#40335 Using Right-to-left (vertical) for text, the characters overlap each other. Cursor is placed at the half of the character. It seems that current implementation use HarfBuzz OpenType layout API hb_buffer_get_glyph_positions() to layout the text, however instead of setting text direction to HB_DIRECTION_TTB, it uses HB_DIRECTION_LTR as its direction and rotates each CJK glyph to simulate the HB_DIRECTION_TTB. However the API use font width to position each glyph no matter whether the glyph will be rotated or not. Change-Id: Ie35238dd92218f8c4463c0ae1d356ad8609c5f48 Reviewed-on: https://gerrit.libreoffice.org/17774Tested-by:
Jenkins <ci@libreoffice.org> Reviewed-by:
Norbert Thiebaud <nthiebaud@gmail.com> Reviewed-on: https://gerrit.libreoffice.org/18878Reviewed-by:
Caolán McNamara <caolanm@redhat.com> Tested-by:
Caolán McNamara <caolanm@redhat.com> (cherry picked from commit a4532151)
-
Mark Hung yazdı
Language poolitem will be inserted after user completes editing with IME, making it refer to valid range when undo. Change-Id: Id2876aa74dba6f7d134b8e2df4d9b36a8f429bb1 Reviewed-on: https://gerrit.libreoffice.org/18392Reviewed-by:
Caolán McNamara <caolanm@redhat.com> Tested-by:
Caolán McNamara <caolanm@redhat.com> Reviewed-on: https://gerrit.libreoffice.org/18877Tested-by:
Jenkins <ci@libreoffice.org> (cherry picked from commit 2b136504)
-
László Németh yazdı
AutoCorrect Emoji replacements were incomplete in Calc cells and Impress text boxes, keeping the terminating colon: :omega: -> Ω: Corrected by this patch: :omega: -> Ω Change-Id: I0d1f6f9ec9c31a7b37e0c9afaaad17dcee568dd5 Reviewed-on: https://gerrit.libreoffice.org/18849Reviewed-by:
Adolfo Jayme Barrientos <fitojb@ubuntu.com> Tested-by:
Adolfo Jayme Barrientos <fitojb@ubuntu.com> (cherry picked from commit 1e15cf04)
-
Christian Lohmaier yazdı
Change-Id: I6287394ba04c7ec81ffb39229a0147e829615b07 (cherry picked from commit dfb00ac6) (cherry picked from commit e0910afb)
-
Eike Rathke yazdı
... if more than one argument is passed to PRODUCT() or SUMSQ() and matrix/array arguments are involved the functions returned wrong results. For example, wrong behavior =PRODUCT({2,3},{4,5}) gave 6 =SUMSQ({2,3},{4,5}) gave 34 Correct is =PRODUCT({2,3},{4,5}) result 120 =SUMSQ({2,3},{4,5}) result 54 Change-Id: Iec7b136a5cc678416f28b1abfe19cd0901ef22b6 (cherry picked from commit c1780ad2) Reviewed-on: https://gerrit.libreoffice.org/18864Reviewed-by:
Łukasz Hryniuk <lukasz.hryniuk@wp.pl> Tested-by:
Łukasz Hryniuk <lukasz.hryniuk@wp.pl> (cherry picked from commit 6b8b3ffa)
-