certat / do-portal

This project is in maintenance mode and will only receive bug fixes, but no new features. A new version of this software is being developed.
5 stars 5 forks source link

JS error in user_list: organization undefined #45

Closed ghost closed 5 years ago

ghost commented 5 years ago

We get a JS error because of an undefined property:

TypeError: a.organizations[d.organization_id] is undefined
Stack trace:
@https://cp-aec.cert.at/scripts/scripts.96600f9a.js:1:16654
@https://cp-aec.cert.at/scripts/scripts.96600f9a.js:1:16592
@https://cp-aec.cert.at/scripts/scripts.96600f9a.js:1:16570
i@https://cp-aec.cert.at/scripts/vendor.d5ec3447.js:5:31837
k/<@https://cp-aec.cert.at/scripts/vendor.d5ec3447.js:6:296
$digest@https://cp-aec.cert.at/scripts/vendor.d5ec3447.js:6:6128
$apply@https://cp-aec.cert.at/scripts/vendor.d5ec3447.js:6:8120
g@https://cp-aec.cert.at/scripts/vendor.d5ec3447.js:5:14309
r@https://cp-aec.cert.at/scripts/vendor.d5ec3447.js:5:16872
Yb/</w.onload@https://cp-aec.cert.at/scripts/vendor.d5ec3447.js:5:17296
 Possibly unhandled rejection: {}
vendor.d5ec3447.js:5:25613
e/<
https://cp-aec.cert.at/scripts/vendor.d5ec3447.js:5:25613
Lb/this.$get</<
https://cp-aec.cert.at/scripts/vendor.d5ec3447.js:5:10222
j
https://cp-aec.cert.at/scripts/vendor.d5ec3447.js:6:116
$digest
https://cp-aec.cert.at/scripts/vendor.d5ec3447.js:6:6128
$apply
https://cp-aec.cert.at/scripts/vendor.d5ec3447.js:6:8120
g
https://cp-aec.cert.at/scripts/vendor.d5ec3447.js:5:14309
r
https://cp-aec.cert.at/scripts/vendor.d5ec3447.js:5:16872
Yb/</w.onload
https://cp-aec.cert.at/scripts/vendor.d5ec3447.js:5:17296

Not sure what the reason is. The user_list has successfully been shown at least once after the upgrade.

If the data is bad it shouldn't have been accepted.

ghost commented 5 years ago

One user has been added inbetween. In the database table users I see that role_id is null for this user.

EDIT: That's not the reason. But why can the role_id be NULL? #46

ghost commented 5 years ago

missing migration