Skip to content

Commit

Permalink
Add per default the apps-external directory in config.php during inst…
Browse files Browse the repository at this point in the history
…allation
  • Loading branch information
mmattel committed Apr 25, 2018
1 parent 224f968 commit c61ddb6
Show file tree
Hide file tree
Showing 3 changed files with 32 additions and 5 deletions.
10 changes: 5 additions & 5 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ core_vendor=core/vendor

core_doc_files=AUTHORS COPYING README.md
core_src_files=$(wildcard *.php) index.html db_structure.xml .htaccess .user.ini robots.txt
core_src_dirs=apps core l10n lib occ ocs ocs-provider resources settings
core_src_dirs=apps apps-external core l10n lib occ ocs ocs-provider resources settings
core_test_dirs=tests
core_all_src=$(core_src_files) $(core_src_dirs) $(core_doc_files)
dist_dir=build/dist
Expand Down Expand Up @@ -234,7 +234,7 @@ $(dist_dir)/owncloud: $(composer_deps) $(nodejs_deps) $(core_all_src)
rm -Rf $@/settings/tests
rm -Rf $@/core/vendor/*/{.bower.json,bower.json,package.json,testem.json}
find $@/{core/,l10n/} -iname \*.sh -delete
find $@/{apps/,lib/composer/,core/vendor/} \( \
find $@/{apps/,apps-external/,lib/composer/,core/vendor/} \( \
-name bin -o \
-name test -o \
-name tests -o \
Expand All @@ -245,7 +245,7 @@ $(dist_dir)/owncloud: $(composer_deps) $(nodejs_deps) $(core_all_src)
-name travis -o \
-iname \*.sh \
\) -print | xargs rm -Rf
find $@/{apps/,lib/composer/} -iname \*.exe -delete
find $@/{apps/,apps-external/,lib/composer/} -iname \*.exe -delete
# Set build
$(eval _BUILD="$(shell date -u --iso-8601=seconds) $(shell git rev-parse HEAD)")
# Replace channel in version.php
Expand Down Expand Up @@ -284,7 +284,7 @@ $(dist_dir)/qa/owncloud: $(composer_dev_deps) $(nodejs_deps) $(core_all_src) $(c
find $@ -name no-php -delete
rm -Rf $@/core/vendor/*/{.bower.json,bower.json,package.json,testem.json}
find $@/{core/,l10n/} -iname \*.sh -delete
find $@/{apps/,lib/composer/,core/vendor/} \( \
find $@/{apps/,apps-external/,lib/composer/,core/vendor/} \( \
-name test -o \
-name examples -o \
-name demo -o \
Expand All @@ -293,7 +293,7 @@ $(dist_dir)/qa/owncloud: $(composer_dev_deps) $(nodejs_deps) $(core_all_src) $(c
-name travis -o \
-iname \*.sh \
\) -print | xargs rm -Rf
find $@/{apps/,lib/composer/} -iname \*.exe -delete
find $@/{apps/,apps-external/,lib/composer/} -iname \*.exe -delete
# Set build
$(eval _BUILD="$(shell date -u --iso-8601=seconds) $(shell git rev-parse HEAD)")
# Replace channel in version.php
Expand Down
Empty file added apps-external/.gitkeep
Empty file.
27 changes: 27 additions & 0 deletions lib/private/Setup.php
Original file line number Diff line number Diff line change
Expand Up @@ -378,10 +378,37 @@ public function install($options) {
$config->setSystemValue('logtimezone', \date_default_timezone_get());
}

// adding the apps-external directory by default using apps_path
$apps2Key = \OC::$server->getSystemConfig()->getValue('apps_paths', false);

// add the key only if it does not exist (protect against overwriting)
if ($apps2Key === false) {

$defaultAppsPaths = [
'apps_paths' => [
[
"path" => \OC::$SERVERROOT . '/apps',
"url" => "/apps",
"writable" => false
],
[
"path" => \OC::$SERVERROOT . '/apps-external',
"url" => "/apps-external",
"writable" => true
]
]
];

$config->setSystemValues($defaultAppsPaths);
}

self::installBackgroundJobs();

//and we are done
$config->setSystemValue('installed', true);

// finished initial setup

}

return $error;
Expand Down

0 comments on commit c61ddb6

Please sign in to comment.