Feature Wiki
Tabs
Cleaning of Legacy Code for WebDAV
Page Overview
[Hide]1 Reasons to Abandon Feature
WebDAV is an old component, which has a lot of legacy code in the ILIAS core. Some methods seem to be solely made for WebDAV. An indicator for this is a function block with a comment like “BEGIN WebDAV”. A search for the string “BEGIN WebDAV” brings up 57 matches in 28 different files.
For the new release 6.0, we would like to perform a clean-up of this legacy code. The removal of functions are made over pull request and need to be approved by the maintainer of the corresponding component.
We have no intention to change or even abandon any existing features (unless stated in other feature requests).
2 Technical Information
Changes over Pull-Request.
3 Contact
- Author of the Request: Heer, Raphael [rheer]
- Maintainer: Kergomard, Stephan [skergomard], Heer, Raphael [rheer]
- Implementation of the feature is done by: Heer, Raphael [rheer]
4 Funding
Removing the feature from the ILIAS code base might need funding. If you are interest in funding this request, please add your name and institution to this list.
- ...
5 Discussion
Use the following discussion section to express your objections against this request or your consent to get rid of this feature.
JourFixe, ILIAS [jourfixe], 07 MAY 2019: We highly appreciate this suggestion and accept to abandon legacy code of WebDAV for 6.0.
6 Implementation
At the 12. November 2020, some changes for WebDAV on the branches trunk (from which one day later branched release_7) happened:
- The settings for WebDAV were moved from ilObjFileAccess to ilObjWebDAV and therefore got its own menu entry in the administration menu.
- The setting "webDavActionEnabled" was removed, since it is not used anymore
- The setting "webDavEnabled", which previously was stored in the client.ini file, is now stored in the database
- The unused methods "_getDefaultWebfolderInstructions", "setCustomWebfolderInstructionsEnabled", "isCustomWebfolderInstructionsEnabled" and "getDefaultWebfolderInstructions" were removed from ilObjWebDAV respectively ilObjFileAccessSettings (where they were before the transition to ilObjWebDAV)
Approval
Approved at {date} by {user}.
Last edited: 18. Nov 2020, 11:10, Heer, Raphael [rheer]