lp:~zeutschel/goobi-production/refactoring-filesystem-and-shellscript-functions
- Get this branch:
- bzr branch lp:~zeutschel/goobi-production/refactoring-filesystem-and-shellscript-functions
Branch merges
- Ralf Claussnitzer (community): Approve
- Matthias Ronge (community): Needs Resubmitting
- Henning Gerhardt: Disapprove
-
Diff: 825 lines (+379/-194)11 files modifiedsrc/de/sub/goobi/beans/Benutzer.java (+2/-3)
src/de/sub/goobi/beans/Prozess.java (+9/-14)
src/de/sub/goobi/config/ConfigMain.java (+2/-4)
src/de/sub/goobi/export/dms/ExportDms.java (+2/-2)
src/de/sub/goobi/export/download/ExportMets.java (+2/-1)
src/de/sub/goobi/helper/FilesystemHelper.java (+75/-3)
src/de/sub/goobi/helper/Helper.java (+15/-136)
src/de/sub/goobi/helper/HelperSchritte.java (+1/-1)
src/de/sub/goobi/helper/ShellScript.java (+264/-0)
src/de/sub/goobi/helper/WebDav.java (+5/-29)
src/de/sub/goobi/helper/ldap/Ldap.java (+2/-1)
Related bugs
Related blueprints
Branch information
Recent revisions
- 58. By Matthias Ronge
-
This change contains various code restructuring concerning the filesystem operations and shell scripting. It’s also a step towards downsizing the god object “Helper”.
Changes to Filesystem operations:
- File system functionality has been moved from “Helper” and “Prozess” into a new Class “FilesystemHelper”.
- Code has been refactored, checking for existence of a directory has been centralised.
- A locking issue with the file renaming functionality, especially under Windows, has been fixed.Changes to Shell scripting:
- Scripting functionality has been moved from “Helper” into a new Class “ShellScript”.
- White spaces in path names are now correctly passed to the ProcessBuilder.
- The error level return value can now be accessed independend of output to stderr.
- Code has been refactored, the results parsing has been centralised, stub methods isEmptyCommand() and closeProcessStreams() have been re-integrated.
Branch metadata
- Branch format:
- Branch format 7
- Repository format:
- Bazaar repository format 2a (needs bzr 1.16 or later)
- Stacked on:
- lp:goobi-production/1.7