Skip to content

Idp manager#32

Open
nworr wants to merge 4 commits intojelix:accountsfrom
nworr:idp-manager
Open

Idp manager#32
nworr wants to merge 4 commits intojelix:accountsfrom
nworr:idp-manager

Conversation

@nworr
Copy link
Contributor

@nworr nworr commented Sep 12, 2025

Add an admi module to enable/disable IDP plugins

Enabled IDP are still /app/system/mainconfig.ini.php , but disabled ones still need to be know

So I adddeclareIDPPlugin event

ref #27

@nworr nworr changed the base branch from master to accounts September 12, 2025 10:33



`declareIDPlugin`
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

je propose de plutôt nommer l'évènement AuthAdminGetIDPPlugin

@@ -0,0 +1,109 @@
<?php
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

je propose de nommer le module authadmin au lieu de idpadmin. Il portera d'autres pages de configuration générale pour l'authentification. Le contrôleur pourrait être nommé idpadmin plutôt que default (à terme default sera plutôt une sorte de page d'accueil de l'admin auth)

jAcl2DbManager::createRightGroup($groupName, 'idpadmin~default.rights.group.name');

// Add right subject
jAcl2DbManager::createRight('idpadmin.view', 'idpadmin~default.idp.view', $groupName);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

droits : auth.idpadmin.* plutôt que idpadmin.*

$uim = $event->uiManager;

$adminMenu = new SubMenu('admin', 'Administration', 10);
$adminMenu->addJelixLinkItem(jLocale::get('idpadmin~default.navigation.menu.idp'), 'idpadmin~default:index', array(), 'address-book');
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

intitulé de l'item de menu "authentification"


public function save()
{
$modif = new IniModifier(jApp::appPath('app/system/mainconfig.ini.php'));
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

la sauvegarde doit être faite dans var/config/liveconfig. Par contre pour la lecture, via jApp::config()

@nworr
Copy link
Contributor Author

nworr commented Feb 9, 2026

rebased and updated
Screenshot 2026-02-09 at 15-16-52 Auth test app

@nworr nworr marked this pull request as ready for review February 9, 2026 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants