Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Android app doesn't show user icon if user name contains whitespace #1667

Closed
nursoda opened this issue Oct 13, 2017 · 4 comments · Fixed by #1668
Closed

Android app doesn't show user icon if user name contains whitespace #1667

nursoda opened this issue Oct 13, 2017 · 4 comments · Fixed by #1668

Comments

@nursoda
Copy link

nursoda commented Oct 13, 2017

Bug opened as requested in https://help.nextcloud.com/t/solved-android-app-doesnt-show-user-icon-if-user-name-contains-whitespace/22217 by Andy:

Actual behaviour

I have several accounts with profile photos. Only for the account with a space in the username, the profile photo is not shown in the app. It’s shown on the web. The same photo is shown in a test account (without whitespace in its user name) when I upload the identical file.

Expected behaviour

Profile photo should be shown in app, too.

Steps to reproduce

  1. create a user with whitespace, such as "maidenname surname".
  2. assign a profile photo to that user.
  3. add that account to the app. Profile photo is not shown.

Environment data

Android version: 7.1.1
Device model: OP 3T
Stock or customized system: stock OOS
Nextcloud app version: 1.4.x and 2.0 betax
Nextcloud server version: 12.0.3

Logs

Please request which logs you need. But you should be able to reproduce yourself easily ;-)

@AndyScherzinger
Copy link
Member

pinging @tobiasKaminsky since you probably know the exact code part. My guess is it has something to do with escaping (missing)

@dougouk
Copy link

dougouk commented Oct 13, 2017

On the URL, it says this is [solved].. So should this issue be closed?

@nursoda
Copy link
Author

nursoda commented Oct 13, 2017

I did not mark the forum thread solved, don't know who did. Probabl because it moved to this bug. It's NOT solved yet.

@mario
Copy link
Contributor

mario commented Oct 13, 2017 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants