From f387ec077304520abff5565adb4ab6fa919dc3b2 Mon Sep 17 00:00:00 2001 From: xZero707 Date: Wed, 15 Mar 2023 02:54:53 +0100 Subject: [PATCH] Revert "Improve plugin installer reliability" This reverts commit f81ec4c67e48a3da585a6fbae028122c7a013d9b. --- rootfs/etc/cont-init.d/20-install-plugins | 28 +++++++++-------------- 1 file changed, 11 insertions(+), 17 deletions(-) diff --git a/rootfs/etc/cont-init.d/20-install-plugins b/rootfs/etc/cont-init.d/20-install-plugins index 466fd50..bef562a 100755 --- a/rootfs/etc/cont-init.d/20-install-plugins +++ b/rootfs/etc/cont-init.d/20-install-plugins @@ -16,9 +16,7 @@ function scriptExitHandler() { # Check if plugin installed. This is very basic check that doesn't involve database function isPluginInstalled() { - PLUGIN="${1:?PLUGIN is required}" - - if [ -d "${PLUGIN_PATH}" ] && [ -f "${PLUGIN_PATH}/${PLUGIN}.php" ]; then + if [ -d "${PLUGIN_PATH}" ] || [ -f "${PLUGIN_PATH}.php" ]; then return 0 fi return 1 @@ -46,22 +44,20 @@ function main() { for PLUGIN_EXPR in ${PLUGIN_LIST}; do IFS=':' read -ra PLUGIN <<<"${PLUGIN_EXPR}" - PLUGIN_NAME="${PLUGIN[0]}" - PLUGIN_PATH="${WP_CONTENT_PATH}/plugins/${PLUGIN_NAME}" - export PLUGIN_PATH + PLUGIN_PATH="${WP_CONTENT_PATH}/plugins/${PLUGIN[0]}" - if isPluginInstalled "${PLUGIN_NAME}"; then - echo "> Plugin '${PLUGIN_NAME}' already installed and will be skipped." + if isPluginInstalled; then + echo "> Plugin '${PLUGIN[0]}' already installed and will be skipped." continue fi - WP_PLUGIN_INSTALL_ARGS="${PLUGIN_NAME}" + WP_PLUGIN_INSTALL_ARGS="${PLUGIN[0]}" if [ -n "${PLUGIN[1]}" ]; then WP_PLUGIN_INSTALL_ARGS="${WP_PLUGIN_INSTALL_ARGS} --version=${PLUGIN[1]}" fi - echo "> Installing plugin '${PLUGIN_NAME}' version '${PLUGIN[1]}'" + echo "> Installing plugin '${PLUGIN[0]}' version '${PLUGIN[1]}'" installPlugin "${WP_PLUGIN_INSTALL_ARGS}" & done @@ -76,22 +72,20 @@ function main() { for PLUGIN_EXPR in ${PLUGIN_LIST}; do IFS=':' read -ra PLUGIN <<<"${PLUGIN_EXPR}" - PLUGIN_NAME="${PLUGIN[0]}" - PLUGIN_PATH="${WP_CONTENT_PATH}/plugins/${PLUGIN_NAME}" - export PLUGIN_PATH + PLUGIN_PATH="${WP_CONTENT_PATH}/plugins/${PLUGIN[0]}" - if isPluginInstalled "${PLUGIN_NAME}"; then - echo "> Plugin '${PLUGIN_NAME}' installed" + if isPluginInstalled; then + echo "> Plugin '${PLUGIN[0]}' installed" continue fi ((FAILED_COUNT = FAILED_COUNT + 1)) - echo "> Warning: Plugin '${PLUGIN_NAME}' failed to install" + echo "> Warning: Plugin '${PLUGIN[0]}' failed to install" done echo "> Total of ${FAILED_COUNT} plugins failed to install" - if [ "${PLUGIN_STRICT_INSTALL}" = "true" ] && [ "${FAILED_COUNT}" != "0" ]; then + if [ "${PLUGIN_STRICT_INSTALL}" = "true" ] && [ ${FAILED_COUNT} != "0" ]; then echo "> WORDPRESS_PLUGIN_INSTALL_STRICT is set to true. Terminating with non-zero exit code" return 1 fi