SURFnet / sp-dashboard

SURFconext Service Provider Dashboard
Apache License 2.0
4 stars 2 forks source link

SP dashboard gaat over z\'n nek als er geen contact type is ingevuld in manage #955

Closed phavekes closed 2 hours ago

phavekes commented 2 hours ago

This issue is imported from pivotal - Originaly created at May 24, 2019 by Teun Fransen

Als er in manage voor een entity een contact ingevuld wordt zonder contacttype, dan gaat het SP Dashboard over z\'n nek als de entity bekeken wordt. Dit is de error: {"datetime":"2019-05-23T16:28:27+02:00","channel":"php","level":"WARNING","message":"Notice: Undefined index: contactType","context":{"exception":{"class":"ErrorException","message":"Notice: Undefined index: contactType","code":0,"file":"/opt/openconext/spdashboard/releases/sp-dashboard-2.0.9/src/Surfnet/ServiceProviderDashboard/Infrastructure/Manage/Dto/Contact.php:33"}},"extra":[]}

Als een dergelijk contact aanwezig is mag deze worden genegeerd.

phavekes commented 2 hours ago
@teunf  als wij hem negeren en iemand bewerkt een entity in SPD en slaat hem op dan wordt hij gepubliseerd naar Manage met de gegevens zoals deze dan als laatste in SPD zijn gezet. De contacten met onbekend contacttypen zullen hierdoor niet meer in manage staan.

Dat is gedrag is dan zoals verwacht? (bstrooband - Sep 4, 2019)

phavekes commented 2 hours ago

Het zou mooi zijn als dit contact-type loze entry meteen rechtgezet kan worden in het SP Dashboard. (Teun Fransen - Sep 5, 2019)

phavekes commented 2 hours ago

De vraag is dan als welk contacttype dit zou moeten worden omgezet en wat er zou moeten gebeuren als dit contacttype al bestaat? (bstrooband - Sep 5, 2019)

phavekes commented 2 hours ago

Herstel. Ik heb nog even met Bart overlegd. Dit contact mag worden genegeerd en worden weggegooid (Teun Fransen - Sep 5, 2019)

phavekes commented 2 hours ago

9 van de 10 gevallen is het een handmatige fout in Manage waarbij de entry enkel bestaat uit een contact-type, zonder verdere informatie. (Teun Fransen - Sep 5, 2019)

phavekes commented 2 hours ago

Bedankt Teun, dan negeren we de betreffende contact gegevens en zullen we deze bij de mogelijk daaropvolgende push niet meesturen.. (bstrooband - Sep 6, 2019)

phavekes commented 2 hours ago

👍 (Teun Fransen - Sep 6, 2019)