Skip to content

N°9167 Use ExtensionDetails UIBlocks instead of table#910

Open
Timmy38 wants to merge 7 commits into
feature/uninstallationfrom
feature/9167_UIBlocks_ext_mgmt
Open

N°9167 Use ExtensionDetails UIBlocks instead of table#910
Timmy38 wants to merge 7 commits into
feature/uninstallationfrom
feature/9167_UIBlocks_ext_mgmt

Conversation

@Timmy38
Copy link
Copy Markdown
Contributor

@Timmy38 Timmy38 commented May 18, 2026

No description provided.

@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented May 18, 2026

Greptile Summary

This PR replaces the feature-removal extension tables with extension detail UI blocks. It changes:

  • Extension selection and analysis-result views now render multi-column extension cards.
  • Extension togglers now submit explicit on/off values through hidden inputs.
  • Extension source labels move onto the iTopExtension model.
  • Extension detail cards hide empty menus and add force-uninstall behavior.

Confidence Score: 3/5

This should be fixed before merging.

  • Forced extension removal can be submitted with the old hidden toggler value.
  • Remote extensions can bypass the force-uninstall guard used by setup.
  • The main UI conversion is otherwise contained to the feature-removal card flow.

Focus on ExtensionDetails.php, the toggler hidden input behavior, and DataFeatureRemovalController.php extension flags.

Important Files Changed

Filename Overview
datamodels/2.x/combodo-data-feature-removal/src/Controller/DataFeatureRemovalController.php Builds the new extension-card data and diff inputs for the feature-removal workflow.
sources/Application/UI/Base/Layout/Extension/ExtensionDetails.php Adds card toggler naming and force-uninstall menu behavior.
templates/base/components/input/input-toggler.html.twig Adds a hidden input so togglers post explicit on/off state.

Reviews (2): Last reviewed commit: "N°9167 Add comments" | Re-trigger Greptile

Comment thread setup/itopextension.class.inc.php Outdated
@Timmy38 Timmy38 closed this May 18, 2026
@Timmy38 Timmy38 reopened this May 18, 2026
Comment thread sources/Application/UI/Base/Layout/Extension/ExtensionDetails.php
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal Work made by Combodo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants