Kan geen koppeling maken als een gebruiker lid is van het team dashboard_super_user

[phavekes]

This issue is imported from pivotal - Originaly created at Dec 10, 2015 by Henny Bekker

Als men lid is van het team dashboard_super_user, en men via Admin als een andere instelling is aangelogd, kunnen geen koppelingen gemaakt worden. Dan komt de melding:

---

Something went wrong when opening this page.

Heeft te maken met:

Caused by: com.mysql.jdbc.exceptions.jdbc4.MySQLIntegrityConstraintViolationException: Column \'institutionId\' cannot be null

[phavekes]

@alanvdam Kan je hier naar kijken? (Henny Bekker - Dec 10, 2015)

[phavekes]

Er zijn eigenlijk twee dingen:

• Als je als super user inlogt mag je geen diensten koppelen. (zie selfservice.api.rest.ServicesController#connect) Dit mag alleen als admin. Dasbhoard (war) geeft dan een 403 terug aan de javascript die dit laat zien als een error pagina. Pas bij kijken in de console kan je ziet wat er echt gebeurd. Dit zou gewoon een access denied pagina moeten zijn. Of beter de knop zou disabled moeten zijn?
• De mysql error is veroorzaakt door in te loggen met OneGini. Als er dan een dienst gekoppeld wordt ontbreekt het klantId/insitituteId. OneGini heeft die niet. Ook dit zouden we moeten afvagen voordat er naar de server wordt gegaan..... Dit zou niet helemaal in csa een error moeten geven. (Alan van Dam - Dec 10, 2015)

[phavekes]

Na overleg met @hennybekker 

• Als super user mag je als je de admin role hebt aangenomen voor een instelling wel services koppelen. Ofwel je krijgt geen 403 meer als je een dienst wilt koppelen.
• Als je een admin bent en je huidige idp heeft geen institutionId mag je geen diensten koppelen. De knop is dan niet zichtbaar (Alan van Dam - Dec 15, 2015)

[phavekes]

We hebben het volgende afgesproken: 1: Als dashbooard-super-admin mag je wel \'on behalf of\' diensten aankoppelen aan een IdP van een instelling. 2: Als dashbooard-super-admin neem je de klantId (insitituteId) van de betreffende instelling over als je op "Admin" of op "Viewer" klikt.

Activiteit 1 heeft wel tot gevolg dat de KlantID van de IdP wordt overruled door de KlantID van de instelling waarvoor je het beheer (als viewer of admin) wilt doen. Tevens zal gekeken moeten worden welke IdP\'s de betreffende KlantID heeft en deze zullen dan ook getoond moeten worden in de "IdP Switch" (zie attached image). (Henny Bekker - Dec 15, 2015)

[phavekes]

Om deze bug af te sluiten heb ik het switchen van institutionId nog niet geimplementeerd. Als een super user nu een service will connecten zal hij een 403 pagina krijgen ipv. de error pagina. (Alan van Dam - Dec 16, 2015)

[phavekes]

Ik krijg nog steeds \'access denied\' als ik op acc een dienst wil de-activeren... (Arnout Terpstra - Mar 9, 2016)

[phavekes]

Als super_admin mag je geen koppelingen aanmaken ..... (Henny Bekker - Mar 9, 2016)