Merge pull request #1491 from matrix-org/luke/groups-fix-optional-profile-properties

Fix bug preventing partial group profile
pull/21833/head
David Baker 2017-10-17 19:30:50 +01:00 committed by GitHub
commit 62a79d823d
1 changed files with 9 additions and 1 deletions

View File

@ -448,8 +448,16 @@ export default React.createClass({
_initGroupStore: function(groupId) {
this._groupStore = GroupStoreCache.getGroupStore(MatrixClientPeg.get(), groupId);
this._groupStore.on('update', () => {
const summary = this._groupStore.getSummary();
if (summary.profile) {
// Default profile fields should be "" for later sending to the server (which
// requires that the fields are strings, not null)
["avatar_url", "long_description", "name", "short_description"].forEach((k) => {
summary.profile[k] = summary.profile[k] || "";
});
}
this.setState({
summary: this._groupStore.getSummary(),
summary,
error: null,
});
});