summaryrefslogtreecommitdiff
path: root/ucb
AgeCommit message (Collapse)Author
2016-08-16tdf#101094 (25): Make a std::less explicit, add/remove TODOs...Giuseppe Castagno
... and a bit of function member signature change. Change-Id: I0d41277bca3991c7c105400e932df96f7bcf0d79 Reviewed-on: https://gerrit.libreoffice.org/28172 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Giuseppe Castagno <giuseppe.castagno@acca-esse.eu>
2016-08-16tdf#101094 (24): Simplify cache removal: DESTROY, PROPPATCH, MOVEGiuseppe Castagno
Plus another PUT forgotten on a previuos commit. Change-Id: I9e15cae2e858884cfd4c827ae767a2c5fcef1782 Reviewed-on: https://gerrit.libreoffice.org/28158 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Giuseppe Castagno <giuseppe.castagno@acca-esse.eu>
2016-08-15loplugin:unusedenumconstants in unotools..xmlhelpNoel Grandin
Change-Id: Id7ddc0fc1f57c5e8e7fb002e31d54fb8e9f8ffab Reviewed-on: https://gerrit.libreoffice.org/28050 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-08-14tdf#101094 (23): Simplify cache removal: MOVE, COPYGiuseppe Castagno
Change-Id: Ia9ef644359c34ac30b78d48c79e8bc9d5c639f92 Reviewed-on: https://gerrit.libreoffice.org/28111 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Giuseppe Castagno <giuseppe.castagno@acca-esse.eu>
2016-08-12Cppcheck: Variable is reassigned a value before the old one has been usedGiuseppe Castagno
Change-Id: Id49254f3f3b230de4e7d3f116b52a38f6042f6c1 Reviewed-on: https://gerrit.libreoffice.org/28070 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Giuseppe Castagno <giuseppe.castagno@acca-esse.eu>
2016-08-12Related: tdf#99499, add a limit to the number of http redirectionsGiuseppe Castagno
Check for maximum number of redirections according to <https://tools.ietf.org/html/rfc7231#section-6.4>. A practical limit can be 5, due to old RFC: <https://tools.ietf.org/html/rfc2068#section-10.3>, this limit is reported also in more recent RFCs, see final paragraph of RFC7231, 6.4. Change-Id: I2b394ef8d1ef391a527df349aa749819c496657b Reviewed-on: https://gerrit.libreoffice.org/28066 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Giuseppe Castagno <giuseppe.castagno@acca-esse.eu>
2016-08-11tdf#101094 (22): Simplify cache removal: MKCOL, PUTGiuseppe Castagno
Change-Id: Id2c9f87e408b6754e17ef3792691d848deabfcd5 Reviewed-on: https://gerrit.libreoffice.org/28056 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Giuseppe Castagno <giuseppe.castagno@acca-esse.eu>
2016-08-11tdf#101094 (21): Simplify cache removal: LOCK, UNLOCKGiuseppe Castagno
Change-Id: Ib963fc6e0834606dcb6f42b3c0df5a408ebc5351 Reviewed-on: https://gerrit.libreoffice.org/28030 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Giuseppe Castagno <giuseppe.castagno@acca-esse.eu>
2016-08-11loplugin:countusersofdefaultparams in svl..unodevtoolsNoel Grandin
Change-Id: Ie3d2cf29e99b8a51b80246aafa23a92e6c3404b2 Reviewed-on: https://gerrit.libreoffice.org/28017 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-08-10tdf#101385: don't try to get SharePoint folder outside of allowed rangeMike Kaganski
for a given site. This patch uses the fact that getFolderByServerRelativeUrl API returns site's root folder if used with empty path (''). Hopefully this isn't otherwise context-dependent, so that it doesn't return other folders in some unknown circumstances. More correct would be instead to use the site's root path directly, like '/sites/mysite'. But I don't know a reliable way to filter the API part from the base URI. To make use of the corrected SharePoint repository root id, the patch also redirects all cases where "/" object is queried, to session's getRootFolder(). libcmis part of the patch will hopefully soon be merged upstream. Change-Id: I0f3b9930748de3fe4763eb27721739f097b9892d Reviewed-on: https://gerrit.libreoffice.org/27987 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: David Tardon <dtardon@redhat.com>
2016-08-03tdf#101094 (20) OPTIONS: Check for Class1 DAV before unlockGiuseppe Castagno
Change-Id: I890a44bb98577de006355d8394d46baeeaadb13f Reviewed-on: https://gerrit.libreoffice.org/27837 Tested-by: Giuseppe Castagno <giuseppe.castagno@acca-esse.eu> Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Giuseppe Castagno <giuseppe.castagno@acca-esse.eu>
2016-08-01Related: tdf#96410 eXo Platform WebDAV: where lock fails...Giuseppe Castagno
...because apparently eXo Platform WebDAV does not support or supports poorly this section <http://tools.ietf.org/html/rfc4918#section-7.3> of RFC4918. It returns HTTP error 404 instead. Change-Id: I74cc99b29afa62110470e9ca785fce36e3d04bb2 Reviewed-on: https://gerrit.libreoffice.org/27755 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Giuseppe Castagno <giuseppe.castagno@acca-esse.eu>
2016-07-31Revert "tdf#101094 (20) OPTIONS: Add mechanism to manage not found Web URL: ↵Giuseppe Castagno
HEAD" This reverts commit 0acd13ec38fad1a63b86d2428299e469227a9069. Because it introduces a bug in saving a new file on a WebDAV server. Need to find a better solution. Change-Id: Ie353fac7203d599989ca67cf89c81c1a95f5b47e Reviewed-on: https://gerrit.libreoffice.org/27744 Reviewed-by: Giuseppe Castagno <giuseppe.castagno@acca-esse.eu> Tested-by: Giuseppe Castagno <giuseppe.castagno@acca-esse.eu>
2016-07-30tdf#101094 (20) OPTIONS: Add mechanism to manage not found Web URL: HEADGiuseppe Castagno
Change-Id: Ia941cfb68924140c3ab171d4ed2c4e8675947ca3 Reviewed-on: https://gerrit.libreoffice.org/27724 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Giuseppe Castagno <giuseppe.castagno@acca-esse.eu>
2016-07-29tdf#101094 (19) OPTIONS: Add mechanism to manage not found Web URL: GETGiuseppe Castagno
Change-Id: Ie3da55d230631c50968b00cdea176f30051abf37 Reviewed-on: https://gerrit.libreoffice.org/27699 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Giuseppe Castagno <giuseppe.castagno@acca-esse.eu>
2016-07-29tdf#101094 (18) OPTIONS: Update OPTIONS cache lifetime if not foundGiuseppe Castagno
Change-Id: I1b37bba50cd41ab5eeb960927148cf6b005f3fd4 Reviewed-on: https://gerrit.libreoffice.org/27698 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Giuseppe Castagno <giuseppe.castagno@acca-esse.eu>
2016-07-29tdf#101094 (17) OPTIONS: Options cache removal: COPYGiuseppe Castagno
Change-Id: Ifa60eabd0a31b66074d79b30a5be7d497585f602 4a2c233261211b13ed06d05278bf42e446330487 Reviewed-on: https://gerrit.libreoffice.org/27696 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Giuseppe Castagno <giuseppe.castagno@acca-esse.eu>
2016-07-29tdf#101094 (16) OPTIONS: Options cache removal: MOVEGiuseppe Castagno
Change-Id: I7a610003295aa174d655f1e847e6d9a19d84a197 Reviewed-on: https://gerrit.libreoffice.org/27695 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Giuseppe Castagno <giuseppe.castagno@acca-esse.eu>
2016-07-29tdf#101094 (15) OPTIONS: Options cache removal: MKCOLGiuseppe Castagno
Change-Id: I752ab44b88935da75198d27f8b490f97c863408f Reviewed-on: https://gerrit.libreoffice.org/27694 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Giuseppe Castagno <giuseppe.castagno@acca-esse.eu>
2016-07-29tdf#101094 (14) OPTIONS: Options cache removal: PUTGiuseppe Castagno
Change-Id: I251bbb101b4be851deaab66cbe4e10ff1fa3376e Reviewed-on: https://gerrit.libreoffice.org/27691 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Giuseppe Castagno <giuseppe.castagno@acca-esse.eu>
2016-07-29tdf#101094 (13) OPTIONS: Options cache removal: LOCK, UNLOCKGiuseppe Castagno
Change-Id: Ib5e52973252b3af7b0fbe440806eb1e669670299 Reviewed-on: https://gerrit.libreoffice.org/27686 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Giuseppe Castagno <giuseppe.castagno@acca-esse.eu>
2016-07-29tdf#101094 (12) OPTIONS: Add options cache ctrl in getResourceOptionsGiuseppe Castagno
Change-Id: If4f53e8efc51e5ec33747dfcbe8f4294bbd6160b Reviewed-on: https://gerrit.libreoffice.org/27685 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Giuseppe Castagno <giuseppe.castagno@acca-esse.eu>
2016-07-29tdf#101094 (11): Add WebDAV options cache configuration param.Giuseppe Castagno
Added in officecfg five new properties to be able to set cache lifetime if needed. The new properties are available in advanced, expert configuration only, in org.openoffice.Inet.Settings. Default values are as follows (value is in seconds): OptsCacheLifeImplWeb = 300 when the web resource is Web only, implementing OPTIONS. Min. 0 sec (no caching) max. 3600 sec (1h). OptsCacheLifeDAV = 60 when the web resource is WebDAV. Min. 0 sec (no caching) max. 3600 sec (1h). OptsCacheLifeDAVLocked = 600 when the web resource is WebDAV and it's locked by this LO instance (e.g. lock store has a lock to it). Min. 0 sec (no caching) max. 3600 sec (1h). OptsCacheLifeNotImpl = 3600 when the web resource does not implement OPTIONS method. Min. 0 sec (no caching) max. 43200 sec (12h). OptsCacheLifeNotFound = 15 when the requested web resource is not found on server. Min. 0 sec (no caching) max. 30 sec. Change-Id: I719b97645e1d91a29134820b77678fd88fcb9ac2 Reviewed-on: https://gerrit.libreoffice.org/27684 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Giuseppe Castagno <giuseppe.castagno@acca-esse.eu>
2016-07-29tdf#101094 (10) OPTIONS: Add a simple options cache classGiuseppe Castagno
Added behavioral unit tests as well. Change-Id: I30f84c8f814d3460a421428ebe0d2fbc32c5c433 Reviewed-on: https://gerrit.libreoffice.org/27668 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Giuseppe Castagno <giuseppe.castagno@acca-esse.eu>
2016-07-29tdf#101094 (9) OPTIONS: Add options check in Content::getPropertiesGiuseppe Castagno
Change-Id: I1937d51158f67055ed4bbeb60dafbf8ffee090ae Reviewed-on: https://gerrit.libreoffice.org/27667 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Giuseppe Castagno <giuseppe.castagno@acca-esse.eu>
2016-07-29tdf#101094 (8) OPTIONS: Add options check in Content::resourceTypeForLocksGiuseppe Castagno
Change-Id: Ia1bcc1be9e61626da2d60ea679d4b6e7e114ac9f Reviewed-on: https://gerrit.libreoffice.org/27661 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Giuseppe Castagno <giuseppe.castagno@acca-esse.eu>
2016-07-29tdf#101094 (7) OPTIONS: Remove resourceTypeForLocks before unlockGiuseppe Castagno
Not needed, unlock() checks if a lock is present first. If present, it means that LOCK is available, inferring the presence of UNLOCK availability. Change-Id: I007c4b02b27e8a40f15721c0308c894ad6f482fd Reviewed-on: https://gerrit.libreoffice.org/27660 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Giuseppe Castagno <giuseppe.castagno@acca-esse.eu>
2016-07-28tdf#101094 (6) OPTIONS: Do not retry on HTTP error 501Giuseppe Castagno
HTTP error 501 the server tells us the used method is not implemented, non need to retry. Change-Id: I4529d2bc27a87e8f3463ded578c6c55e0701ae17 Reviewed-on: https://gerrit.libreoffice.org/27636 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Giuseppe Castagno <giuseppe.castagno@acca-esse.eu>
2016-07-28tdf#101094 (5) OPTIONS: Add options check in Content::getResourceTypeGiuseppe Castagno
Added some verbose SAL_INFO to detail server responses to PROPFIND. Corrected a wrong comment and beautified a bit code I introduced in commit 3b26a2a403ca5e99b0dd07d042d47501c091af16. Change-Id: Id59749fe60fe0b451d8e0f4093d7902fe94bbed0 Reviewed-on: https://gerrit.libreoffice.org/27635 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Giuseppe Castagno <giuseppe.castagno@acca-esse.eu>
2016-07-28ucb: fix deadlocks in tdoc UCP OfficeDocumentsManagerMichael Stahl
Reduce scope of MutexGuards so that UNO methods are called without the mutex locked. This avoids deadlocks in CppunitTest_sw_mailmerge after using SolarMutex in dbaccess' ModelMethodGuard. Change-Id: I0229ebc2983c85e2003d51053a6bd130240274c7 Reviewed-on: https://gerrit.libreoffice.org/27582 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
2016-07-28tdf#101094 (4) OPTIONS: Add member function getResourceOptionsGiuseppe Castagno
Change-Id: Ia98456842f666ad97a9a8ffc82e787c0c88066f6 Reviewed-on: https://gerrit.libreoffice.org/27608 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Giuseppe Castagno <giuseppe.castagno@acca-esse.eu>
2016-07-28tdf#101094 (3) OPTIONS: Add options implem. in DAV resource access.Giuseppe Castagno
Change-Id: I186a77739b40cd89a8a918fd22379eb4240292ff Reviewed-on: https://gerrit.libreoffice.org/27607 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Giuseppe Castagno <giuseppe.castagno@acca-esse.eu>
2016-07-28tdf#101094 (2) OPTIONS: Add neon session implementationGiuseppe Castagno
Change-Id: I1032f552b8ee2a413fdbfbf414bd7b39d6cbe6e3 Reviewed-on: https://gerrit.libreoffice.org/27606 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Giuseppe Castagno <giuseppe.castagno@acca-esse.eu>
2016-07-28tdf#101094 (1) OPTIONS: New class to store retrieved OPTIONSGiuseppe Castagno
This class describes the DAV options useful to LO. Added behavioural unit tests as well. Change-Id: I7ba0230f1a1469b12f9b3f59f21e5c2730c65b81 Reviewed-on: https://gerrit.libreoffice.org/27595 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Giuseppe Castagno <giuseppe.castagno@acca-esse.eu>
2016-07-27improve passstuffbyref return analysisNoel Grandin
Change-Id: I4258bcc97273d8bb7a8c4879fac02a427f76e18c Reviewed-on: https://gerrit.libreoffice.org/27317 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-07-27loplugin:countusersofdefaultparams in tools..xmlsecurityNoel Grandin
find methods with default params with only zero or one call site Change-Id: Ie5b30f60e9fe00ba1acf0dfc79b005ded46f05a0 Reviewed-on: https://gerrit.libreoffice.org/27512 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-07-25curl 7.50.0 has CURL as typedef struct Curl_easyCaolán McNamara
Change-Id: I22e5e2cdf78c38087579071c1b1570a8adc7d3c4
2016-07-25new loplugin overrideparamNoel Grandin
verify that parameters on override methods have the same set of default values for their params as their parent/super-methods do. Change-Id: Ibdbc1c6e417fbaa680ea025a6bbf5ba9c2e5bcd2 Reviewed-on: https://gerrit.libreoffice.org/27437 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-07-15new loplugin unnecessary overrideNoel Grandin
Change-Id: I88d3e33823d68745b98625050a8a274f9ef04bcb Reviewed-on: https://gerrit.libreoffice.org/27135 Reviewed-by: Stephan Bergmann <sbergman@redhat.com> Tested-by: Stephan Bergmann <sbergman@redhat.com>
2016-07-13Remove __cplusplus-checks from apparently-C++-only filesStephan Bergmann
Change-Id: I87e9ce45cf945e75e8140a9d4608da8abcddada6 Reviewed-on: https://gerrit.libreoffice.org/27187 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2016-07-08loplugin:redundantcast: redundant static_casts in ucbStephan Bergmann
Change-Id: Ifd6a843ce5af9409512c4303a103fdc158e38c40
2016-07-07tdf#100783 Avoid infinite recursion in Content::getObject()Mike Kaganski
With certain CMIS host URLs, it is possible that exception handler in Content::getObject() will recursively call itself with the same data (xParent from the same URL). This patch checks if the new URL is not the same as own, to avoid this. Change-Id: Ifaeb4ff27a9c809c5c96fa35ec190c3263a8fe62 Reviewed-on: https://gerrit.libreoffice.org/26977 Reviewed-by: Andras Timar <andras.timar@collabora.com> Tested-by: Andras Timar <andras.timar@collabora.com>
2016-07-07loplugin:passstuffbyref also for {css::uno,rtl}::ReferenceStephan Bergmann
Change-Id: I9adb9cce899591380c18fa8a336408ba9b21defd
2016-07-05Fix link in commentStephan Bergmann
Change-Id: Iaa7bef1330e7bb1e3e5c3b01685540867af2bdb8
2016-07-05tdf#94378: Respond to OnStorageChangedStephan Bergmann
...as emitted from SfxObjectShell::SaveCompleted when toggling edit mode. Change-Id: I4d99554c3587cf3fd67993ce94ccd77629b5e259
2016-07-05Clarify calculation precedence tdf#39440Muhammet Kara
Use parentheses to clarify the code. Change-Id: I864dc6dacadb5b9ba9dca8e0abd9fa4e6db1eddc Reviewed-on: https://gerrit.libreoffice.org/25677 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: jan iversen <jani@documentfoundation.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2016-07-05merge tdoc_ucp::OfficeDocumentsEventListener withNoel Grandin
tdoc_ucp::ContentProvider Change-Id: I4c743ce3d61d048ae9cf762462d9042337cd9d47 Reviewed-on: https://gerrit.libreoffice.org/26903 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-07-04merge ftp::ResultSetFactory with ResultSetFactoryINoel Grandin
Change-Id: I061145fefef33aa3e7349adf7da03222c3b2a911 Reviewed-on: https://gerrit.libreoffice.org/26898 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-06-28loplugin:singlevalfields in variousNoel Grandin
Change-Id: Ia0d8f463a4dba9ec63aa0159441e3e607dd3bf5e Reviewed-on: https://gerrit.libreoffice.org/26738 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-06-27Clean up uses of Any::getValue() in ucbStephan Bergmann
Change-Id: Ifeae43257ce9c2ba8513652c21b512bcffd87841