PrivateCoffee/element-web
4
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 3

Fix ability to feature self in a group summary

Browse source 
By default the AddressPicker would omit the currently logged-in user. This adds a property to override that to allow "self" to be picked.
This commit is contained in:
Luke Barnard 2017-09-27 11:04:41 +01:00
parent 1ca9102442
commit 6bc9b32ab7
2 changed files with 9 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
src/components/structures/GroupView.js
View file

@ -239,6 +239,7 @@ const RoleUserList = React.createClass({
button: _t("Add to summary"),
validAddressTypes: ['mx'],
groupId: this.props.groupId,
shouldOmitSelf: false,
onFinished: (success, addrs) => {
if (!success) return;
const errorList = [];

9
src/components/views/dialogs/AddressPickerDialog.js
View file

@ -41,7 +41,11 @@ module.exports = React.createClass({
validAddressTypes: PropTypes.arrayOf(PropTypes.oneOf(addressTypes)),
onFinished: PropTypes.func.isRequired,
groupId: PropTypes.string,
// The type of entity to search for. Default: 'user'.
pickerType: PropTypes.oneOf(['user', 'room']),
// Whether the current user should be omitted from the address returned. Only
// applicable when pickerType is `user`. Default: true.
shouldOmitSelf: PropTypes.bool,
},
getDefaultProps: function() {
@ -50,6 +54,7 @@ module.exports = React.createClass({
focus: true,
validAddressTypes: addressTypes,
pickerType: 'user',
shouldOmitSelf: true,
};
},
@ -366,7 +371,9 @@ module.exports = React.createClass({
});
return;
}
if (result.user_id === MatrixClientPeg.get().credentials.userId) {
if (this.props.shouldOmitSelf &&
result.user_id === MatrixClientPeg.get().credentials.userId
) {
return;
}