Problem/Motivation

This is needed in cases of multisite installations where your generated theme is in web/sites/yoursite.com/themes/custom.

CommentFileSizeAuthor
#6 also-scan-origin-site-3226203-6.patch1.78 KBpivica
#5 interdiff-3226203-2-5.txt385 bytespivica
#5 also-scan-origin-site-3226203-5.patch385 bytespivica
#2 also-scan-origin-site-3226203-2.patch1.83 KBpivica

Comments

pivica created an issue. See original summary.

pivica’s picture

pivica commented
Title: _bs_base_drupal_scan should also scan origin of site folder » _bs_base_drupal_scan() should also scan origin of site folder
StatusFileSize
new also-scan-origin-site-3226203-2.patch1.83 KB

Here is a patch, additional code taken from ExtensionDiscovery::scan().

pivica’s picture

pivica commented
Status: Active » Needs review
berdir’s picture

berdir
Primary language German
Location Switzerland
commented 
+++ b/bs_base.drush.inc
@@ -493,8 +496,19 @@ function _bs_base_drupal_scan($type, $reset = FALSE) {
+    ExtensionDiscovery::ORIGIN_CORE => 'core',

I think it's a safe bet that you don't need to search in core, bs base themes aren't going to be in there ;)

pivica’s picture

pivica commented
StatusFileSize
new also-scan-origin-site-3226203-5.patch385 bytes
new interdiff-3226203-2-5.txt385 bytes

You are totally right :)

pivica’s picture

pivica commented
StatusFileSize
new also-scan-origin-site-3226203-6.patch1.78 KB

Uploaded wrong patch, here is a correct one.

  • pivica committed 22a5d3f on 8.x-1.x 
    Issue #3226203 by pivica, Berdir: _bs_base_drupal_scan() should also...
pivica’s picture

pivica commented
Status: Needs review » Fixed

Committed.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.