summaryrefslogtreecommitdiff
path: root/fpicker
AgeCommit message (Collapse)AuthorFilesLines
2017-03-27Resolves: tdf#106729 ensure VistaFilePicker init before accessCaolán McNamara2-2/+19
(cherry picked from commit a15ec82edebf14baa41a888ead7da8e0556ab0bb) Change-Id: I238599cf5ed464c4bb99a894ac47126fbc32796d Reviewed-on: https://gerrit.libreoffice.org/35746 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
2016-05-30valgrind: Invalid read of size 8Caolán McNamara1-0/+1
saw this in passing checking something else ==12972== at 0x6DEF40B0: rtl::Reference<PlacesListBox>::get() const (ref.hxx:169) ==12972== by 0x6DEEF221: VclPtr<PlacesListBox>::operator bool() const (vclptr.hxx:189) ==12972== by 0x6DEED3DA: CustomContainer::GetFocus() (iodlg.cxx:429) ==12972== by 0xD9EC83B: vcl::Window::CompatGetFocus() (window.cxx:3734) ==12972== by 0xD93A604: vcl::Window::ImplGrabFocus(GetFocusFlags) (mouse.cxx:383) ... ==12972== Address 0x2e4b4690 is 288 bytes inside a block of size 424 free'd ==12972== at 0x4C2D22A: operator delete(void*) (vg_replace_malloc.c:576) ==12972== by 0x6DED9F14: SvtFileDialog::dispose() (iodlg.cxx:518) ==12972== by 0xDB04B1F: OutputDevice::disposeOnce() (outdev.cxx:161) ==12972== by 0x6DED1954: VclPtr<SvtFileDialog_Base>::disposeAndClear() (vclptr.hxx:208) Change-Id: I702eab8dbcf07885e4a3415ace6706c25854174e (cherry picked from commit 5b342e10cbec2487085216346c055875a8eb9657)
2016-05-26Resolves: tdf#98436 crash trying To create remote folder before connectedCaolán McNamara1-0/+3
Change-Id: I953167c9b78f9c9eba560390ecef7ccd8e5f1f68 (cherry picked from commit 7d397201060ba71a188b5ef9853984a635e70531) Reviewed-on: https://gerrit.libreoffice.org/25501 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2016-05-23in unlikely case >>= failsCaolán McNamara1-1/+1
Change-Id: I368456acb085905736f1e221eeb3c79df375c7f0
2016-05-13convert FilePicker flags to scoped enumNoel Grandin11-110/+76
and merged into the other flags variable. Change-Id: I23ed21936b9de1537a06168de04530b9fb6bcff1 Reviewed-on: https://gerrit.libreoffice.org/24889 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2016-05-12mpDefDialogParent is not used since 2002Caolán McNamara1-15/+0
i.e. commit c0ae87cb5f2989bc4b8dff4907994d513ee87e39 Author: Stephan Schäfer <ssa@openoffice.org> Date: Tue Oct 22 08:39:05 2002 +0000 #103442# choose DefDialogParent on-the-fly and so calling SetDefDialogParent doesn't achieve anything Change-Id: I2ec72da47b2dac03fdddbdb9eba5ae4bf205eb22
2016-05-12Convert SFX_EXTRA to scoped enumNoel Grandin5-37/+43
Change-Id: I57fa684d91d10a132c718152ca646edd1c7bd189 Reviewed-on: https://gerrit.libreoffice.org/24882 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-05-11set parent window of windows system file picker on save(as)Oliver Specht3-1/+28
If no parent is provided the Windows system file picker uses either the current foreground window or the desktop. The decision depends on the thread id of the foreground window. To make sure the document window is used as parent this is now also sent to the system file picker as it was already done for the internal file picker. Change-Id: Id589cbc1f91db30e065175aaea42ef1512ffb1b9 Reviewed-on: https://gerrit.libreoffice.org/24635 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
2016-05-11remove unused FILEDLG_STATE constants and fieldsNoel Grandin4-14/+0
Change-Id: I2d7460e575b0935e5001b2c78d0ffb0dd33f3c39 Reviewed-on: https://gerrit.libreoffice.org/24880 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-05-04use Any constructor instead of temporariesNoel Grandin1-16/+5
Change-Id: Iffb82a2cee1a28d89eeea2b905aaa14086ee475a
2016-04-28RemoteFilesDialog: icons for view mode buttonsSzymon Kłos1-2/+18
Start Center > Remote files Deleted text and icons inserted into view mode buttons. Change-Id: I0f3caad792d2b87993c08014f38d4f68599132ae Reviewed-on: https://gerrit.libreoffice.org/23249 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
2016-04-27clang-tidy modernize-loop-convert in f*Noel Grandin1-4/+4
Change-Id: Id866aa244378758e3bdb9e99d02cdd2ae6104e16
2016-04-22loplugin:passstuffbyrefStephan Bergmann3-4/+4
Change-Id: I45cc7c949dd8288bcc39f23c224370764c5cf0ec
2016-04-20loplugin:salbool: Automatic rewrite of sal_False/TrueStephan Bergmann1-2/+2
Change-Id: I58d7c95c77e67f315b33d1f83ac80148f93af450
2016-04-18loplugin:passstuffbyrefStephan Bergmann1-2/+2
Change-Id: Ib4c63e4942acec1b02bac1a5bc387e1931eaae12
2016-04-18Avoid reserved identifierStephan Bergmann1-3/+3
Change-Id: I8b85f65e5d3e66aa93e161ce1895547d7d863ca1
2016-04-18clean-up: unused using declarations and includesJochen Nitschke5-9/+0
Searched source for using declarations. Checked if those symbols reappear in the source file, even in comments or dead code but not in #include statements. If they don't reappear, remove the declaration. Remove includes whose symbol got removed. Change-Id: Ibb77163f63c1120070e9518e3dc0a78c6c59fab0 Reviewed-on: https://gerrit.libreoffice.org/24148 Reviewed-by: Jochen Nitschke <j.nitschke+logerrit@ok.de> Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2016-04-14clang-tidy performance-unnecessary-value-param in variousNoel Grandin3-5/+5
Change-Id: I7168d44dab8e6a8e37bb7920d744ff32f5e52907 Reviewed-on: https://gerrit.libreoffice.org/24019 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-04-14loplugin:passstuffbyref in variousNoel Grandin2-5/+5
Change-Id: I80070c83204e531c2f599f8a56193d6ffe0e5022
2016-04-13tdf#98395 : Remote File dialog showing wrong details in columnsSzymon Kłos1-0/+2
Before patch: - File type in the Size column (only first launch) After patch: - File type is not visible Change-Id: Ia2e3775f49dd9bfcc77a985812e040f70201f45c Reviewed-on: https://gerrit.libreoffice.org/22921 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: David Tardon <dtardon@redhat.com>
2016-04-12Avoid reserved identifiersStephan Bergmann2-4/+4
Change-Id: I65681e938c809467f9497568149d25f5f68ed5bf
2016-04-12WNT: check result of PostMessage()Michael Stahl2-4/+8
... so the next time something overflows the main thread's message queue it's easier to debug. Change-Id: I90a35e46f3b2cab190c7da5e53024ec549159ac6
2016-04-12cleanup: remove unused com/sun/star/uno includesJochen Nitschke6-4/+2
Sequence.h(xx), Any.h(xx) and Type.h(xx) and remove unused using-declarations from these files. Add a few missing includes provided by them. Change-Id: I6b91b6d1fdf9d0496dd546c0aab9bdcc6831a5d4 Reviewed-on: https://gerrit.libreoffice.org/23805 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-04-02cleanup: remove unused Reference.h(xx) includesJochen Nitschke1-1/+0
and unused using-declarations from Reference.h Change-Id: I297a7ae6044fa329d245ecf08fd5c4cb930f5b19 Reviewed-on: https://gerrit.libreoffice.org/23735 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
2016-04-01tdf#97966 Drop 'static' keywordsWastack1-1/+1
Including no keywords from extern "C" blocks Change-Id: I8bcd7e8a492ee4dbfa5141416c7c038a1391cf20 Reviewed-on: https://gerrit.libreoffice.org/23673 Reviewed-by: Stephan Bergmann <sbergman@redhat.com> Tested-by: Stephan Bergmann <sbergman@redhat.com>
2016-03-31use SAL_N_ELEMENTS more widelyNoel Grandin2-5/+4
found using git grep -n 'sizeof.*/.*sizeof.*[0]' Change-Id: Icd4a6cc1ca8ec8ebd68e1701a02789c74cf0eb2a
2016-03-23-Werror=unused-but-set-variableStephan Bergmann1-14/+0
Change-Id: I77a73caf56878060ba3688c87e9b6d5dd3ae28b1
2016-03-23loplugin:constantparam in fpickerNoel Grandin2-21/+8
Change-Id: I7225c996030eed494e3ca8a9dabc501fc505e101
2016-03-23remove unused AUTOSELECT_USERFILTER codeNoel Grandin1-33/+0
added in commit 90fcf5718668f0f9f41ac141a7fc4306633e693a Author: Oliver Bolte <obo@openoffice.org> Date: Wed Apr 13 07:54:11 2005 +0000 and untouched since then Change-Id: I995c094edbd3d85a6f637f7cb3e7aa4a0326c8bb
2016-03-15tdf#91794 remove OSL_DEBUG_LEVEL > 1 conditionalsRohan Kumar1-4/+4
I replaced OSL_DEBUG_LEVEL > 1 with OSL_DEBUG_LEVEL > 0 and added SAL_INFO statements intead of OSL_TRACE Change-Id: Id3fda00e64eacc952e9317d854b179ac8f26d6ed Reviewed-on: https://gerrit.libreoffice.org/23229 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>
2016-03-14inline some use-once macrosNoel Grandin1-15/+12
Change-Id: I7ebd8fe70b083a772118a1aab8cdfbf795d6f1e5 Reviewed-on: https://gerrit.libreoffice.org/23235 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-03-09icon view for RemoteFilesDialogSzymon Kłos3-4/+51
New view type can be accessed in: Start Center > Remote Files > Icon View Cloud service for testing purposes: 1. Start Center > Remote Files > Add service 2. Set type to: Alfresco 4 3. Set host to: http://cmis.alfresco.com/cmisatom 4. Click refresh icon and enter login:admin pass:admin 5. Click OK and open this service Change-Id: I3d4674bb84d1df1b678111adffe2f835d2bb2356 Reviewed-on: https://gerrit.libreoffice.org/22437 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2016-03-09loplugin:constantparam in svtoolsNoel Grandin2-6/+6
Change-Id: Ica82c2758030729ee0d1480c16720cf6c8fed487
2016-03-07Bin overly verbose OSL_TRACEs of questionable usefulnessTor Lillqvist4-51/+2
Change-Id: Ic535aef31c2a8d73f9c496ffdc535494c15d3c12
2016-03-07Bin overly verbose function entry/exit debug print macrosTor Lillqvist8-516/+36
Change-Id: Icdbb9344a669e5f78d5db85dc555e91a56ded34c
2016-03-04-Werror,-Wnon-virtual-dtorStephan Bergmann1-0/+1
This looks like a real bug, causing leaks in the derived VistaFilePickerImpl (but harmless for the derived AsyncPickerEvents). Change-Id: Ic0de3f56574b89fb45eccb09fb27b78427f712d4
2016-03-04-Werror,-Wnon-virtual-dtorStephan Bergmann3-11/+7
Why was CCustomControlFactory a class with virtual functions in the first place? Change-Id: I4449a6f69f45a5b294b1906d0ce8a1a8a52c1d7f
2016-03-04Silence -Werror,-Wnon-virtual-dtorStephan Bergmann2-0/+6
Change-Id: I643c7e8ad12f255822c51a8f295e95f8799ef36a
2016-03-04-Werror,-Wunused-macrosStephan Bergmann1-6/+0
Change-Id: Ia8ea71174361b1f685a1a9c038c7ee809e54e0ef
2016-02-17Typo: namesapce(s)->namespace(s)Julien Nabet2-2/+2
Change-Id: Ic7c14c2e39a5ade1f5622a8350f9197d84cf9cc8
2016-02-17use consistent #define checks for the Windows platformNoel Grandin1-1/+1
stage 2 of replacing usage of various checks for the windows platform with the compiler-defined '_WIN32' macro In this stage we focus on replacing usage of the WIN macro Change-Id: Ie8a4a63198a6de96bd158ecd707dadafb9c8ea84 Reviewed-on: https://gerrit.libreoffice.org/22393 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-02-13vcl: include vcl headers via triangular brackets instead of quotesChris Sherlock1-1/+1
Change-Id: I311f7db622ce341527fe12a92b516f800b602f92
2016-02-09Remove excess newlinesChris Sherlock52-1187/+0
A ridiculously fast way of doing this is: for i in $(pcregrep -l -M -r --include='.*[hc]xx$' \ --exclude-dir=workdir --exclude-dir=instdir '^ {3,}' .) do perl -0777 -i -pe 's/^ {3,}/ /gm' $i done Change-Id: Iebb93eccbee9e4fc5c4380474ba595858a27ac2c Reviewed-on: https://gerrit.libreoffice.org/22224 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
2016-02-09Fix tdf#97500 Reinstate missing file error dialog on WebDAVGiuseppe Castagno1-3/+23
Bug introduced with my fix to tdf#96669 in commit 5fc2910fc872bbd1184aaab7c842dff593d2449b. Reverted previous fix and write a new version in fpicker instead. On WebDAV better check if the stream opens first, then check IsDocument property, because it may be a folder name. Change-Id: I72ce728329e4194080db6fa4cc4d98fecf7672e9 Reviewed-on: https://gerrit.libreoffice.org/22214 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: jan iversen <jani@documentfoundation.org>
2016-02-05Fix typosAndrea Gelmini2-2/+2
Change-Id: Ice72f8d9971e15dd6ef365e64cd567b8581a92d3 Reviewed-on: https://gerrit.libreoffice.org/21797 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
2016-02-04tdf#43157 Cleanup DBG_ASSERT for fpicker and frameworkBerk Gureken6-22/+22
Change-Id: I1b79a39a9d709184cc72a905647f73dbf85eef27 Signed-off-by: Berk Gureken <berkgureken@gmail.com> Reviewed-on: https://gerrit.libreoffice.org/21991 Reviewed-by: Michael Stahl <mstahl@redhat.com> Tested-by: Michael Stahl <mstahl@redhat.com>
2016-01-31unnecessary use of OUString constructorNoel Grandin2-6/+4
Change-Id: Idd31b0a53c8318af69bbcd32f6798721ec8eb8e1 Reviewed-on: https://gerrit.libreoffice.org/21945 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-01-26-Werror,-Wsign-compare (clang-cl)Stephan Bergmann1-1/+1
According to MSDN: "If [StretchDIBits] fails, or no scan lines are copied, the return value is 0." Change-Id: I882b97b55f92618f0163f8829bf779a35b5a1452
2016-01-25more Windows build fixesNoel Grandin3-3/+3
after my OInterfaceContainerHelper2 change Change-Id: Id63fd81643b030e0570f00d6e0506cb8ea5d3742
2016-01-25InterfaceContainer2 with vector instead of SequenceNoel Grandin3-3/+3
create an InterfaceContainer2 class to replace InterfaceContainer. It uses a std::vector instead of a Sequence for the mutable listener list, which provides far better performance. Switch all our internal use-sites to the new class. Change-Id: I6b56cfa511ded2395faa22e68fab3b2f16c3cb88
generated by cgit v1.2.3 (git 2.39.1) at 2024-04-27 20:56:00 +0000