From b3b7fef6e4a747202aa88d1be741e0f2b1388cfc Mon Sep 17 00:00:00 2001 From: Timmy38 Date: Tue, 12 May 2026 12:01:41 +0200 Subject: [PATCH] =?UTF-8?q?N=C2=B09167=20Use=20ExtensionDetails=20UIBlocks?= =?UTF-8?q?=20instead=20of=20table?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../layout/extension/_extension-details.scss | 13 +- .../DataFeatureRemovalController.php | 175 ++++++++++-------- .../templates/Features.html.twig | 18 +- setup/extensionsmap.class.inc.php | 16 +- .../DryRemovalRuntimeEnvironment.php | 6 +- setup/itopextension.class.inc.php | 8 + .../Layout/Extension/ExtensionDetails.php | 36 ++-- .../ExtensionDetailsUIBlockFactory.php | 30 ++- .../components/input/input-toggler.html.twig | 3 +- .../input/input-toggler.ready.js.twig | 2 + .../Backoffice/RenderAllUiBlocks.php | 4 + 11 files changed, 189 insertions(+), 122 deletions(-) diff --git a/css/backoffice/layout/extension/_extension-details.scss b/css/backoffice/layout/extension/_extension-details.scss index 0c39e8ba6b..3fd5e51c92 100644 --- a/css/backoffice/layout/extension/_extension-details.scss +++ b/css/backoffice/layout/extension/_extension-details.scss @@ -62,4 +62,15 @@ $ibo-extension-details--actions--button--padding-x: $ibo-button--padding-x !defa .ibo-extension-details--actions:has(.toggler-install:not(:disabled)) .ibo-popover-menu--section a[data-resource-id="force_uninstall"] { display: none; -} \ No newline at end of file +} + +.ibo-extension-details .ibo-popover-menu ~ .ibo-button{ + visibility: hidden; +} +.ibo-extension-details .ibo-popover-menu:has(.ibo-popover-menu--item) ~ .ibo-button{ + visibility: visible; +} + +.ibo-extension-details .ibo-toggler--wrapper:has(.ibo-toggler.ibo-is-hidden){ + visibility: hidden; +} diff --git a/datamodels/2.x/combodo-data-feature-removal/src/Controller/DataFeatureRemovalController.php b/datamodels/2.x/combodo-data-feature-removal/src/Controller/DataFeatureRemovalController.php index f7f32be0c8..ed1db5a706 100644 --- a/datamodels/2.x/combodo-data-feature-removal/src/Controller/DataFeatureRemovalController.php +++ b/datamodels/2.x/combodo-data-feature-removal/src/Controller/DataFeatureRemovalController.php @@ -11,6 +11,8 @@ require_once APPROOT.'setup/feature_removal/SetupAudit.php'; require_once APPROOT.'setup/feature_removal/DryRemovalRuntimeEnvironment.php'; use Combodo\iTop\Application\TwigBase\Controller\Controller; +use Combodo\iTop\Application\UI\Base\Layout\Extension\ExtensionDetailsUIBlockFactory; +use Combodo\iTop\DataFeatureRemoval\Helper\DataFeatureRemovalConfig; use Combodo\iTop\DataFeatureRemoval\Helper\DataFeatureRemovalException; use Combodo\iTop\DataFeatureRemoval\Helper\DataFeatureRemovalHelper; use Combodo\iTop\DataFeatureRemoval\Service\DataCleanupService; @@ -27,18 +29,22 @@ use utils; class DataFeatureRemovalController extends Controller { private array $aSelectedExtensionsForCheck = []; + private ?array $aExtensionsToCheck = null; + private bool $bForcedUninstallation = false; private array $aCountClassesToCleanup = []; private array $aAnalysisDataTable = []; private int $iCount = 0; + private int $iColumnCount = 2; public function OperationMain($sErrorMessage = null): void { $aParams = []; - $this->ReadRemovedExtensions(); $this->AddAnalyzeParams(); $aParams['sTransactionId'] = utils::GetNewTransactionId(); - $aParams['aExtensions'] = $this->GetExtensionsTableToSelect(); + $aParams['iColumnCount'] = $this->iColumnCount; + $aParams['aAvailableExtensions'] = $this->SplitArrayIntoColumns($this->GetAvailableExtensions(), $this->iColumnCount); + $aParams['aAnalysisDataTable'] = $this->aAnalysisDataTable; $aParams['aClasses'] = array_keys($this->aCountClassesToCleanup); $aParams['DataFeatureRemovalErrorMessage'] = $sErrorMessage; @@ -70,12 +76,11 @@ class DataFeatureRemovalController extends Controller public function OperationAnalyze(): void { - $this->ReadRemovedExtensions(); + $iCount = $this->ReadExtensionsDiff(); $this->m_sOperation = 'Main'; - try { - if (count($this->aSelectedExtensionsForCheck) > 0) { + if ($iCount > 0) { $this->Analyze(); } $this->OperationMain(); @@ -88,7 +93,7 @@ class DataFeatureRemovalController extends Controller private function Analyze(): void { $sSourceEnv = MetaModel::GetEnvironment(); - $oDryRemovalRuntimeEnvironment = new DryRemovalRuntimeEnvironment($sSourceEnv, $this->aSelectedExtensionsForCheck); + $oDryRemovalRuntimeEnvironment = new DryRemovalRuntimeEnvironment($sSourceEnv, $this->aExtensionsToCheck['to_be_removed']); $oDryRemovalRuntimeEnvironment->CompileFrom($sSourceEnv); $oSetupAudit = new SetupAudit($sSourceEnv); @@ -143,9 +148,9 @@ class DataFeatureRemovalController extends Controller $aParams['aClasses'] = $aGetRemovedClasses; $aParams['iQueryCount'] = $iQueryCount; $aParams['bDeletionPossible'] = !$bHasIssues; - $aParams['aAddedExtensions'] = $aAddedExtensions; - $aParams['aRemovedExtensions'] = $aRemovedExtensions; - $aParams['aExtensions'] = $this->GetExtensionsTableDiff($aAddedExtensions, $aRemovedExtensions); + + $aParams['iColumnCount'] = $this->iColumnCount; + $aParams['aAvailableExtensions'] = $this->SplitArrayIntoColumns($this->GetExtensionsDiff($aAddedExtensions, $aRemovedExtensions), $this->iColumnCount); $this->DisplayPage($aParams); } @@ -209,72 +214,46 @@ class DataFeatureRemovalController extends Controller $this->DisplayPage($aParams); } - private function GetExtensionsTableDiff(array $aAddedExtensions, array $aRemovedExtensions): array + private function GetAvailableExtensions(): array { $aExtensions = []; - $aColumns = ['', 'Name', 'code', 'Badge' ]; - - foreach ($aAddedExtensions as $sAddedExtensionCode => $sAddedExtensionLabel) { - $aExtensions[] = [ - << -HTML, - $sAddedExtensionLabel, - $sAddedExtensionCode, - Dict::S('UI:Layout:ExtensionsDetails:BadgeToBeInstalled'), - ]; - } - foreach ($aRemovedExtensions as $sAddedExtensionCode => $sAddedExtensionLabel) { - $aExtensions[] = [ - << -HTML, - $sAddedExtensionLabel, - $sAddedExtensionCode, - Dict::S('UI:Layout:ExtensionsDetails:BadgeToBeUninstalled'), - ]; - } - - return $this->GetTableData('Extensions', $aColumns, $aExtensions); - } - - /** - * Get installed extensions from disk - * - * @return array structure for twig datatable - */ - private function GetExtensionsTableToSelect(): array - { - $aExtensions = []; - $aColumns = ['', 'Version', 'Name', 'Code']; - foreach (DataFeatureRemoverExtensionService::GetInstance()->ReadItopExtensions() as $sCode => $oExtension) { /** @var \iTopExtension $oExtension */ + $aExtensions[$sCode] = [ + 'version' => $oExtension->sVersion, + 'label' => $oExtension->sLabel, + 'code' => $sCode, + 'description' => $oExtension->sDescription, + 'source' => $oExtension->GetExtensionSourceLabel(), + 'installed' => $oExtension->bInstalled, + 'extra_flags' => [ + //'selected' => in_array($sCode, $this->aSelectedExtensionsForCheck), + 'uninstallable' => $oExtension->CanBeUninstalled(), + 'missing' => $oExtension->bRemovedFromDisk, + ], - $sChecked = ''; - $sDisabledHtml = ''; - if ($oExtension->bRemovedFromDisk) { - $sDisabledHtml = 'disabled=""'; - $sChecked = 'checked'; - } elseif (in_array($sCode, $this->aSelectedExtensionsForCheck)) { - $sChecked = 'checked'; - } - - $sLabel = $oExtension->sLabel; - $sVersion = $oExtension->sVersion; - $sIdEnable = "aExtensions[$sCode][enable]"; - - $aExtensions[] = [ - << -HTML, - $sVersion, - $sLabel, - $sCode, ]; } - return $this->GetTableData('Extensions', $aColumns, $aExtensions); + return $aExtensions; + } + + private function GetExtensionsDiff(array $aAddedExtensions, array $aRemovedExtensions): array + { + $aExtensions = []; + foreach ($this->GetAvailableExtensions() as $sCode => $aExtension) { + if (isset($aAddedExtensions[$sCode])) { + $aExtension['extra_flags']['selected'] = true; + $aExtension['extra_flags']['disabled'] = true; + $aExtensions[$sCode] = $aExtension; + } elseif (isset($aRemovedExtensions[$sCode])) { + $aExtension['extra_flags']['selected'] = false; + $aExtension['extra_flags']['disabled'] = true; + $aExtensions[$sCode] = $aExtension; + } + } + + return $aExtensions; } private function GetTableData(string $sTableName, array $aColumns, array $aData): array @@ -318,27 +297,59 @@ HTML, } /** - * @return void + * Read extensions selected from posted parameters + * @return int Number of extensions to be added or removed */ - public function ReadRemovedExtensions(): void + public function ReadExtensionsDiff(): int { - if (count($this->aSelectedExtensionsForCheck) > 0) { - return; + if (!is_null($this->aExtensionsToCheck)) { + return count($this->aExtensionsToCheck['to_be_installed']) + count($this->aExtensionsToCheck['to_be_removed']); } - $aSelectedExtensionsFromUI = utils::ReadPostedParam('aExtensions', []); - foreach ($aSelectedExtensionsFromUI as $sCode => $aData) { - $sValue = $aData['enable'] ?? 'off'; - if (($sValue) === 'on') { - $this->aSelectedExtensionsForCheck[] = $sCode; + $aAvailableExtensions = $this->GetAvailableExtensions(); + + $aSelectedExtensionsFromUI = utils::ReadPostedParam('aSelectedExtensions', []); + $this->aExtensionsToCheck = [ + 'to_be_installed' => [], + 'to_be_removed' => [], + ]; + foreach ($aAvailableExtensions as $sCode => &$aExtensionData) { + if (!isset($aSelectedExtensionsFromUI[$sCode])) { + continue; } + + if ($aExtensionData['installed'] && $aSelectedExtensionsFromUI[$sCode] !== 'on') { + $aExtensionData['extra_flags']['selected'] = false; + $this->aExtensionsToCheck['to_be_removed'][] = $sCode; + if (!$aExtensionData['extra_flags']['uninstallable']) { + $this->bForcedUninstallation = true; + } + } elseif (!$aExtensionData['installed'] && $aSelectedExtensionsFromUI[$sCode] === 'on') { + $aExtensionData['extra_flags']['selected'] = true; + $this->aExtensionsToCheck['to_be_installed'][] = $sCode; + } + } + return count($this->aExtensionsToCheck['to_be_installed']) + count($this->aExtensionsToCheck['to_be_removed']); + } + + /** + * Divide an array into several sub arrays, distributing elements so that every sub array has an equal number of elements + * @param array $aInput + * @param int $iColNumber + * + * @return array + */ + private function SplitArrayIntoColumns(array $aInput, int $iColNumber) + { + $aOutput = array_fill(0, $iColNumber, []); + + $iIndex = 0; + foreach ($aInput as $mItem) { + //Split extensions in two columns + $aOutput[$iIndex % $this->iColumnCount][] = $mItem; + $iIndex++; } - // Add source removed to check - foreach (DataFeatureRemoverExtensionService::GetInstance()->ReadItopExtensions() as $sCode => $oExtension) { - if ($oExtension->bRemovedFromDisk) { - $this->aSelectedExtensionsForCheck[] = $sCode; - } - } + return $aOutput; } } diff --git a/datamodels/2.x/combodo-data-feature-removal/templates/Features.html.twig b/datamodels/2.x/combodo-data-feature-removal/templates/Features.html.twig index 26c3f15a10..71b06b6b64 100644 --- a/datamodels/2.x/combodo-data-feature-removal/templates/Features.html.twig +++ b/datamodels/2.x/combodo-data-feature-removal/templates/Features.html.twig @@ -6,9 +6,21 @@ {% UIInput ForHidden {sName:'operation', sValue:'Analyze'} %} {% UIInput ForHidden {sName:'transaction_id', sValue:sTransactionId} %} - {% UIFieldSet Standard {sLegend:'DataFeatureRemoval:Features:Title'|dict_s} %} - {% UIDataTable ForForm { sRef:'aExtensions', aColumns:aExtensions.Columns, aData:aExtensions.Data} %}{% EndUIDataTable %} - {% EndUIFieldSet %} + {% UIPanel Neutral { sTitle:'DataFeatureRemoval:Features:Title'|dict_s, sSubTitle: '' } %} + {% UIMultiColumn Standard {} %} + {% for iColumnIndex in 0..iColumnCount-1 %} + {% UIColumn Standard {} %} + {% for aExtension in aAvailableExtensions[iColumnIndex] %} + {% if aExtension['installed'] %} + {% UIExtensionDetails Installed { sCode : aExtension['code'], sLabel : aExtension['label'], sDescription : aExtension['description'], aMetaData : [aExtension['version'], aExtension['source']], aExtraFlags : aExtension['extra_flags']} %}{% EndUIExtensionDetails %} + {% else %} + {% UIExtensionDetails NotInstalled { sCode : aExtension['code'], sLabel : aExtension['label'], sDescription : aExtension['description'], aMetaData : [aExtension['version'], aExtension['source']], aExtraFlags : aExtension['extra_flags']} %}{% EndUIExtensionDetails %} + {% endif %} + {% endfor %} + {% EndUIColumn %} + {% endfor %} + {% EndUIMultiColumn %} + {% EndUIPanel %} {% UIToolbar ForButton {} %} {% UIButton ForPrimaryAction {sLabel:'UI:Button:Analyze'|dict_s, sName:'btn_apply', sId:'btn_apply', bIsSubmit:true} %} diff --git a/setup/extensionsmap.class.inc.php b/setup/extensionsmap.class.inc.php index 8c240f9fc9..b45240c392 100644 --- a/setup/extensionsmap.class.inc.php +++ b/setup/extensionsmap.class.inc.php @@ -437,7 +437,7 @@ class iTopExtensionsMap 'default' => true, // by default offer to install all modules 'modules' => $oExtension->aModules, 'mandatory' => $oExtension->bMandatory, - 'source_label' => $this->GetExtensionSourceLabel($oExtension->sSource), + 'source_label' => $oExtension->GetExtensionSourceLabel(), 'uninstallable' => $oExtension->CanBeUninstalled(), 'missing' => $oExtension->bRemovedFromDisk, 'version' => $oExtension->sVersion, @@ -447,21 +447,7 @@ class iTopExtensionsMap return $aRes; } - protected function GetExtensionSourceLabel($sSource) - { - $sResult = ''; - switch ($sSource) { - case iTopExtension::SOURCE_MANUAL: - $sResult = 'Local extensions folder'; - break; - case iTopExtension::SOURCE_REMOTE: - $sResult = (ITOP_APPLICATION == 'iTop') ? 'iTop Hub' : 'ITSM Designer'; - break; - - } - return $sResult; - } /** * Mark the given extension as chosen diff --git a/setup/feature_removal/DryRemovalRuntimeEnvironment.php b/setup/feature_removal/DryRemovalRuntimeEnvironment.php index ed86e87e2b..4cb8d1d42f 100644 --- a/setup/feature_removal/DryRemovalRuntimeEnvironment.php +++ b/setup/feature_removal/DryRemovalRuntimeEnvironment.php @@ -8,7 +8,7 @@ use SetupUtils; class DryRemovalRuntimeEnvironment extends RunTimeEnvironment { - protected array $aExtensionsByCode; + protected array $aExtensionsToRemoveByCode; /** * Toolset for building a run-time environment @@ -18,7 +18,7 @@ class DryRemovalRuntimeEnvironment extends RunTimeEnvironment public function __construct($sSourceEnv = 'production', array $aExtensionCodesToRemove = []) { parent::__construct($sSourceEnv, false); - $this->aExtensionsByCode = $aExtensionCodesToRemove; + $this->aExtensionsToRemoveByCode = $aExtensionCodesToRemove; $this->Prepare($sSourceEnv, $this->sBuildEnv); } @@ -34,7 +34,7 @@ class DryRemovalRuntimeEnvironment extends RunTimeEnvironment SetupUtils::copydir(APPROOT."/data/$sSourceEnv-modules", APPROOT."/data/$sBuildEnv-modules"); SetupUtils::copydir(APPROOT."/conf/$sSourceEnv", APPROOT."/conf/$sBuildEnv"); - $this->DeclareExtensionAsRemoved($this->aExtensionsByCode); + $this->DeclareExtensionAsRemoved($this->aExtensionsToRemoveByCode); } private function DeclareExtensionAsRemoved(array $aExtensionCodes): void diff --git a/setup/itopextension.class.inc.php b/setup/itopextension.class.inc.php index 81344f9e7c..09a817125a 100644 --- a/setup/itopextension.class.inc.php +++ b/setup/itopextension.class.inc.php @@ -141,4 +141,12 @@ class iTopExtension } return true; } + + public function GetExtensionSourceLabel(): string { + return match ($this->sSource) { + iTopExtension::SOURCE_MANUAL => 'Local extensions folder', + iTopExtension::SOURCE_REMOTE => (ITOP_APPLICATION == 'iTop') ? 'iTop Hub' : 'ITSM Designer', + default => 'Unknown', + }; + } } diff --git a/sources/Application/UI/Base/Layout/Extension/ExtensionDetails.php b/sources/Application/UI/Base/Layout/Extension/ExtensionDetails.php index 0d03ebd6c7..bf82433b74 100644 --- a/sources/Application/UI/Base/Layout/Extension/ExtensionDetails.php +++ b/sources/Application/UI/Base/Layout/Extension/ExtensionDetails.php @@ -179,32 +179,37 @@ class ExtensionDetails extends UIContentBlock protected function InitializeToggler() { + $sName = 'aSelectedExtensions['.$this->GetCode().']'; $this->oToggler = new Toggler(); - $this->oToggler->SetName('ExtensionToggler'); + $this->oToggler->SetName($sName); $this->oToggler->AddCSSClass('toggler-install'); } protected function InitializePopoverMenu() { - $sModalLabel = Dict::Format('UI:Layout:ExtensionsDetails:MenuAboutTitle', $this->sLabel); - $sModalText = $this->sAbout; - $oModifyButton = new JSButtonItem( - 'extension_details', - Dict::S('UI:Layout:ExtensionsDetails:MenuAbout'), - <<oPopoverMenu = new PopoverMenu(); - $this->oPopoverMenu->AddItem('more-actions', PopoverMenuItemFactory::MakeFromApplicationPopupMenuItem($oModifyButton)); $oPopoverOpenButton = ButtonUIBlockFactory::MakeIconAction('fas fa-ellipsis-v', 'Show more actions'); $this->oPopoverMenu->SetTogglerFromBlock($oPopoverOpenButton); $this->oMoreActions = new UIContentBlock(); $this->oMoreActions->AddSubBlock($this->oPopoverMenu); $this->oMoreActions->AddSubBlock($oPopoverOpenButton); + + if (mb_strlen($this->sAbout) > 0) { + $sModalLabel = Dict::Format('UI:Layout:ExtensionsDetails:MenuAboutTitle', $this->sLabel); + $sModalText = $this->sAbout; + $oModifyButton = new JSButtonItem( + 'extension_details', + Dict::S('UI:Layout:ExtensionsDetails:MenuAbout'), + <<oPopoverMenu->AddItem('more-actions', PopoverMenuItemFactory::MakeFromApplicationPopupMenuItem($oModifyButton)); + } + } public function AllowForceUninstall() @@ -213,7 +218,8 @@ JS, 'force_uninstall', Dict::S('UI:Layout:ExtensionsDetails:MenuForce'), <<oPopoverMenu->AddItem('more-actions', PopoverMenuItemFactory::MakeFromApplicationPopupMenuItem($oForceUninstallButton)); diff --git a/sources/Application/UI/Base/Layout/Extension/ExtensionDetailsUIBlockFactory.php b/sources/Application/UI/Base/Layout/Extension/ExtensionDetailsUIBlockFactory.php index 7ec0db7b26..2861a47fe6 100644 --- a/sources/Application/UI/Base/Layout/Extension/ExtensionDetailsUIBlockFactory.php +++ b/sources/Application/UI/Base/Layout/Extension/ExtensionDetailsUIBlockFactory.php @@ -18,6 +18,8 @@ class ExtensionDetailsUIBlockFactory extends AbstractUIBlockFactory $aBadges = []; $bUninstallable = $aExtraFlags['uninstallable'] ?? true; $bMissingFromDisk = $aExtraFlags['missing'] ?? false; + $bSelected = $aExtraFlags['selected'] ?? true; + $bDisabled = $aExtraFlags['disabled'] ?? false; self::AddExtraBadges($aBadges, $bUninstallable, $bMissingFromDisk); $oBadgeInstalled = BadgeUIBlockFactory::MakeGreen(Dict::S('UI:Layout:ExtensionsDetails:BadgeInstalled')); $oBadgeInstalled->AddCSSClass('checked'); @@ -28,10 +30,23 @@ class ExtensionDetailsUIBlockFactory extends AbstractUIBlockFactory $oExtensionDetails = new ExtensionDetails($sCode, $sLabel, $sDescription, $aMetaData, $aBadges, $sAbout); $oExtensionDetails->GetToggler()->SetIsToggled(true); - if (!$bUninstallable) { + if ($bMissingFromDisk) { + $oExtensionDetails->GetToggler()->SetIsToggled(false); + $oExtensionDetails->GetToggler()->SetIsDisabled(true); + } + else if (!$bUninstallable) { $oExtensionDetails->AllowForceUninstall(); $oExtensionDetails->GetToggler()->SetIsDisabled(true); } + + if (!$bSelected) { + $oExtensionDetails->GetToggler()->SetIsToggled(false); + } + if ($bDisabled) { + $oExtensionDetails->GetToggler()->SetIsDisabled(true); + $oExtensionDetails->GetToggler()->AddCSSClass('ibo-is-hidden'); + } + return $oExtensionDetails; } @@ -39,6 +54,8 @@ class ExtensionDetailsUIBlockFactory extends AbstractUIBlockFactory { $aBadges = []; $bUninstallable = $aExtraFlags['uninstallable'] ?? true; + $bSelected = $aExtraFlags['selected'] ?? false; + $bDisabled = $aExtraFlags['disabled'] ?? false; self::AddExtraBadges($aBadges, $bUninstallable, false); $oBadgeInstalled = BadgeUIBlockFactory::MakeGrey(Dict::S('UI:Layout:ExtensionsDetails:BadgeNotInstalled')); $oBadgeInstalled->AddCSSClass('unchecked'); @@ -46,8 +63,17 @@ class ExtensionDetailsUIBlockFactory extends AbstractUIBlockFactory $oBadgeToBeUninstalled = BadgeUIBlockFactory::MakeCyan(Dict::S('UI:Layout:ExtensionsDetails:BadgeToBeInstalled')); $oBadgeToBeUninstalled->AddCSSClass('checked'); $aBadges[] = $oBadgeToBeUninstalled; + $oExtensionDetails = new ExtensionDetails($sCode, $sLabel, $sDescription, $aMetaData, $aBadges, $sAbout); - return new ExtensionDetails($sCode, $sLabel, $sDescription, $aMetaData, $aBadges, $sAbout); + if ($bSelected) { + $oExtensionDetails->GetToggler()->SetIsToggled(true); + } + if ($bDisabled) { + $oExtensionDetails->GetToggler()->SetIsDisabled(true); + $oExtensionDetails->GetToggler()->AddCSSClass('ibo-is-hidden'); + } + + return $oExtensionDetails; } private static function AddExtraBadges(array &$aBadges, bool $bUninstallable, bool $bMissingFromDisk) diff --git a/templates/base/components/input/input-toggler.html.twig b/templates/base/components/input/input-toggler.html.twig index f9bd74c6b5..1674d06cb9 100644 --- a/templates/base/components/input/input-toggler.html.twig +++ b/templates/base/components/input/input-toggler.html.twig @@ -1,7 +1,8 @@ {% extends "base/components/input/layout.html.twig" %} {% block iboInput %} - {{ parent() }} + {{ parent() }} + {% endblock %} diff --git a/templates/base/components/input/input-toggler.ready.js.twig b/templates/base/components/input/input-toggler.ready.js.twig index ec3e102fbc..703c0c5ad9 100644 --- a/templates/base/components/input/input-toggler.ready.js.twig +++ b/templates/base/components/input/input-toggler.ready.js.twig @@ -1,8 +1,10 @@ $('#{{ oUIBlock.GetId() }}').parent().on('click', function() { let oInput = $(this).find('.ibo-toggler'); + let oHiddenInput = $(this).find('.ibo-toggler--hidden'); if (!oInput.prop('disabled')) { oInput.prop('checked', !oInput.prop('checked')); + oHiddenInput.val(oInput.prop('checked') ? 'on' : 'off'); oInput.trigger('change'); } }); \ No newline at end of file diff --git a/tests/manual-visual-tests/Backoffice/RenderAllUiBlocks.php b/tests/manual-visual-tests/Backoffice/RenderAllUiBlocks.php index 2aac9ef2f0..619d831536 100644 --- a/tests/manual-visual-tests/Backoffice/RenderAllUiBlocks.php +++ b/tests/manual-visual-tests/Backoffice/RenderAllUiBlocks.php @@ -639,6 +639,10 @@ $oPage->AddUiBlock($oMultiCol); $oExtensionDetailInstalledFromFactory = ExtensionDetailsUIBlockFactory::MakeInstalled('itop-sample', 'My extension v2', 'This is for test only', ['v1.1.1', 'Designer', '12/12/2012'], ['uninstallable' => false,'missing' => true]); $oColumnLeft->AddSubBlock($oExtensionDetailInstalledFromFactory); +$oExtensionDetailInstalledFromFactory = ExtensionDetailsUIBlockFactory::MakeInstalled('itop-not-uninstallable', 'You cannot uninstall me', 'Click force uninstall to uninstall me', ['v9.9.9', 'Void', '12/12/2012'], ['uninstallable' => false,'missing' => false]); +$oColumnLeft->AddSubBlock($oExtensionDetailInstalledFromFactory); + + $oExtensionDetailInstalledWithLongTitle = ExtensionDetailsUIBlockFactory::MakeNotInstalled('itop-sample', 'My extension with a very long title', 'This is for test only', ['v1.1.1', 'Designer', '12/12/2012'], ['uninstallable' => false]); $oColumnRight->AddSubBlock($oExtensionDetailInstalledWithLongTitle); $oPage->add('
');