diff options
author | Corentin Noël <tintou@noel.tf> | 2022-09-20 18:06:37 +0300 |
---|---|---|
committer | Corentin Noël <corentin.noel@collabora.com> | 2022-09-20 18:41:49 +0300 |
commit | 4a08aa9a3c79964c0b6db535ec0ba071cc0d24e0 (patch) | |
tree | 2b76a1d238102692369eae4b9c5883e25f885359 /src | |
parent | d5cdb7044b8f4fceabac00a0300f00cee85cff6d (diff) |
Properly decode escaped principal urls
Apply the same fix as in https://github.com/nextcloud/calendar/commit/8c294b12e16cf0fb9e4297eb35310d341ec30de3
to allow sharing with groups that contains spaces and escaped characters
Fixes #2411
Signed-off-by: Corentin Noël <tintou@noel.tf>
Diffstat (limited to 'src')
-rw-r--r-- | src/components/AppNavigation/Settings/SettingsAddressbookShare.vue | 12 | ||||
-rw-r--r-- | src/utils/url.js | 32 |
2 files changed, 37 insertions, 7 deletions
diff --git a/src/components/AppNavigation/Settings/SettingsAddressbookShare.vue b/src/components/AppNavigation/Settings/SettingsAddressbookShare.vue index 1ec31a5c..c1583e35 100644 --- a/src/components/AppNavigation/Settings/SettingsAddressbookShare.vue +++ b/src/components/AppNavigation/Settings/SettingsAddressbookShare.vue @@ -34,7 +34,7 @@ :user-select="true" open-direction="bottom" track-by="user" - label="user" + label="displayName" @search-change="findSharee" @input="shareAddressbook" /> <!-- list of user or groups addressbook is shared with --> @@ -53,6 +53,7 @@ import client from '../../../services/cdav' import addressBookSharee from './SettingsAddressbookSharee' import debounce from 'debounce' +import { urldecode } from '../../../utils/url.js' export default { name: 'SettingsAddressbookShare', @@ -100,10 +101,7 @@ export default { * @param {boolean} data.isGroup is this a group ? */ shareAddressbook({ user, displayName, uri, isGroup }) { - const addressbook = this.addressbook - uri = decodeURI(uri) - user = decodeURI(user) - this.$store.dispatch('shareAddressbook', { addressbook, user, displayName, uri, isGroup }) + this.$store.dispatch('shareAddressbook', { addressbook: this.addressbook, user, displayName, uri, isGroup }) }, /** @@ -121,10 +119,10 @@ export default { && !this.addressbook.shares.some((share) => share.uri === result.principalScheme)) { const isGroup = result.calendarUserType === 'GROUP' list.push({ - user: result[isGroup ? 'groupId' : 'userId'], + user: urldecode(result[isGroup ? 'groupId' : 'userId']), displayName: result.displayname, icon: isGroup ? 'icon-group' : 'icon-user', - uri: result.principalScheme, + uri: urldecode(result.principalScheme), isGroup, }) } diff --git a/src/utils/url.js b/src/utils/url.js new file mode 100644 index 00000000..1040d03b --- /dev/null +++ b/src/utils/url.js @@ -0,0 +1,32 @@ +/** + * @copyright Copyright (c) 2021 Richard Steinmetz <richard@steinmetz.cloud> + * + * @author Richard Steinmetz <richard@steinmetz.cloud> + * + * @license GNU AGPL version 3 or any later version + * + * This program is free software: you can redistribute it and/or modify + * it under the terms of the GNU Affero General Public License as + * published by the Free Software Foundation, either version 3 of the + * License, or (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU Affero General Public License for more details. + * + * You should have received a copy of the GNU Affero General Public License + * along with this program. If not, see <http://www.gnu.org/licenses/>. + * + */ + +/** + * Works like urldecode() from php + * + * @see https://www.php.net/manual/en/function.urldecode.php + * @param {string} url The url to be decoded + * @returns {string} The decoded url + */ +export function urldecode(url) { + return decodeURIComponent(url.replace(/\+/g, ' ')) +} |