Skip to content

Commit 124777f

Browse files
committed
Extract Magento2 version detection to a function.
This allows us to do more checks regarding if the package exists.
1 parent dbca958 commit 124777f

File tree

2 files changed

+11
-4
lines changed

2 files changed

+11
-4
lines changed

magento2/usr/local/share/env/30-framework

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -172,4 +172,4 @@ export MAGENTO_ENABLE_TARGET_RULE_CACHE
172172
MAGENTO_ENTERPRISE_EDITION=$(convert_to_boolean_string "${MAGENTO_ENTERPRISE_EDITION:-$(run_return_boolean has_composer_package "magento/product-enterprise-edition")}")
173173
export MAGENTO_ENTERPRISE_EDITION
174174

175-
export MAGENTO_VERSION="${MAGENTO_VERSION:-$(composer_package_version magento/product-community-edition | cut -d. -f1,2)}"
175+
export MAGENTO_VERSION="${MAGENTO_VERSION:-$(detect_magento_version)}"

magento2/usr/local/share/magento2/magento_functions.sh

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,13 @@
11
#!/bin/bash
22

3+
function detect_magento_version() {
4+
if has_composer_package magento/product-community-edition; then
5+
composer_package_version magento/product-community-edition | cut -d. -f1,2
6+
else
7+
echo "2.1"
8+
fi
9+
}
10+
311
function do_composer_config() {
412
as_code_owner "composer global config repositories.magento composer https://repo.magento.com/"
513

@@ -408,12 +416,11 @@ function do_magento_download_magerun2() {
408416
chmod +x /app/bin/n98-magerun2.phar
409417
}
410418

411-
function remove_config_template() (
412-
set +e
419+
function remove_config_template() {
413420
if dpkg --compare-versions "$MAGENTO_VERSION" ge 2.2; then
414421
rm -f /etc/confd/conf.d/magento_config.php.toml /etc/confd/templates/magento/config.php.tmpl
415422
fi
416-
)
423+
}
417424

418425
function do_magento2_templating() {
419426
remove_config_template

0 commit comments

Comments
 (0)