Skip to content

Fix the escapeHTML issue with files_external app - #23293

Closed
sharidas wants to merge 1 commit into
nextcloud:masterfrom
sharidas:fix-escapehtml-filesexternam
Closed

Fix the escapeHTML issue with files_external app#23293
sharidas wants to merge 1 commit into
nextcloud:masterfrom
sharidas:fix-escapehtml-filesexternam

Conversation

@sharidas

Copy link
Copy Markdown
Contributor

The files_external app uses escapeHTML. This change
helps fix the UI of the external storage.

Signed-off-by: Sujith Haridasan sujith.h@gmail.com

The files_external app uses escapeHTML. This change
helps fix the UI of the external storage.
Signed-off-by: Sujith Haridasan <sujith.h@gmail.com>
@sharidassharidas self-assigned this Oct 8, 2020
@sharidassharidas linked an issue Oct 8, 2020 that may be closed by this pull request
@sharidas

Copy link
Copy Markdown
ContributorAuthor

Summary

The user when tries to add external storage in the admin->settings page, the Available for field does not show up the users/groups.

Description

The Available for field in the external storage does not show up the users/groups field. The reason for this is escapeHTML usage in the settings.js file (of files_external app). And hence I have added a function (inspired from apps/settings) escapeHTML.

Note: Let me know if this looks ok?

How was it tested?

  1. Created an external storage with SFTP
  2. Added user admin and another user user1
  3. Verified no errors were logged.

@sharidassharidas added the 2. developing Work in progress label Oct 8, 2020
@sharidassharidas added 3. to review Waiting for reviews bug and removed 2. developing Work in progress labels Oct 8, 2020
@kesselb

kesselb commented Oct 8, 2020

Copy link
Copy Markdown
Contributor

#23264

We can also go for this approach. I don't have a strong opinion ;)

@sharidas

Copy link
Copy Markdown
ContributorAuthor

#23264

We can also go for this approach. I don't have a strong opinion ;)

I didn't knew about your PR :( Sorry for my duplicate PR ...

@sharidas

Copy link
Copy Markdown
ContributorAuthor

I am closing this PR as it becomes a duplicate to #23264.

@sharidassharidas closed this Oct 9, 2020
@sharidas
sharidas deleted the fix-escapehtml-filesexternam branch October 9, 2020 04:40
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsbug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Select for users/groups in external storage ui does not work

3 participants

@sharidas@kesselb@sharidasan