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

Bug when displaying the user name in the poll table #32

Closed
vmassol opened this issue Feb 22, 2019 · 4 comments
Closed

Bug when displaying the user name in the poll table #32

vmassol opened this issue Feb 22, 2019 · 4 comments

Comments

@vmassol
Copy link

@vmassol vmassol commented Feb 22, 2019

team_activity___cooking_workshop__xpoll_team_activity____cooking_workshop_webhome__-_xwiki

@vmassol
Copy link
Author

@vmassol vmassol commented Feb 22, 2019

The problem is that wiki=true is used in {{html clean='false' wiki='true'}} and the URL inside the HREF is parsed as wiki content... (since getUserName() returns HTML).

@vmassol
Copy link
Author

@vmassol vmassol commented Feb 22, 2019

Idea: use a better displayer for the user. We have displayers showing both avatars and name.

@lucaa
Copy link

@lucaa lucaa commented Feb 24, 2019

Quick solution: getUserName() takes a second parameter controlling whether html should be generated or not - so replacing all getUserName(user) with getUserName(user, false) does the job
Long solution: proper sheet that avoids using {{html wiki=true }} (which is generally a bad idea and the proof of that is this issue). This solution could, as Vincent said, also include an improvement of the user display.

@oanalavinia oanalavinia self-assigned this Mar 20, 2019
oanalavinia added a commit to oanalavinia/application-xpoll that referenced this issue Mar 26, 2019
* used html tables for showing the table
* used a macro for displaying the user along with the avatar
* indentation
oanalavinia added a commit to oanalavinia/application-xpoll that referenced this issue Mar 27, 2019
acotiuga added a commit that referenced this issue Mar 27, 2019
* Bug when displaying the user name in the poll table #32
* used html tables for showing the table
* used a macro for displaying the user along with the avatar
* indentation
@acotiuga acotiuga added this to the 11.1 milestone Mar 27, 2019
@acotiuga acotiuga closed this Mar 27, 2019
@acotiuga acotiuga removed this from the 11.1 milestone Mar 27, 2019
@acotiuga acotiuga added this to the 1.11 milestone Mar 27, 2019
@acotiuga acotiuga removed this from the 1.11 milestone Mar 27, 2019
@acotiuga acotiuga added this to the 1.10.3 milestone Mar 27, 2019
@lucaa
Copy link

@lucaa lucaa commented Mar 28, 2019

On a 1.10.4 version of the poll application, on a subwiki of a farm with global users , this issue is not fixed as it's replaced with a new issue: broken avatar macro and broken link to the user page:
image

I am reopening this issue also it's arguable whether the bug is a new one or the same as this one (it's obviously not the same, but the user display is still not ok).

@lucaa lucaa reopened this Mar 28, 2019
oanalavinia added a commit to oanalavinia/application-xpoll that referenced this issue Mar 29, 2019
acotiuga added a commit that referenced this issue Mar 29, 2019
Bug when displaying the user name in the poll table #32
@oanalavinia oanalavinia removed this from the 1.10.3 milestone Mar 29, 2019
@oanalavinia oanalavinia added this to the 1.10.5 milestone Mar 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants