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

[ticket/12488] Add user warning indication to viewtopic posts #2400

Merged
merged 4 commits into from May 7, 2014

Conversation

PayBas
Copy link
Contributor

@PayBas PayBas commented May 3, 2014

@@ -129,6 +129,7 @@ <h2 class="poll-title">{POLL_QUESTION}</h2>

<!-- IF postrow.POSTER_POSTS != '' --><dd class="profile-posts"><strong>{L_POSTS}{L_COLON}</strong> {postrow.POSTER_POSTS}</dd><!-- ENDIF -->
<!-- IF postrow.POSTER_JOINED --><dd class="profile-joined"><strong>{L_JOINED}{L_COLON}</strong> {postrow.POSTER_JOINED}</dd><!-- ENDIF -->
<!-- IF postrow.POSTER_WARNINGS --><dd class="profile-warnings"><strong>{L_WARNINGS}{L_COLON}</strong> <em>{postrow.POSTER_WARNINGS}</em></dd><!-- ENDIF -->
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the value should not be wrapped by <em>

@PayBas
Copy link
Contributor Author

PayBas commented May 4, 2014

@bantu : it's already in place: https://github.com/phpbb/phpbb/blob/develop-ascraeus/phpBB/viewtopic.php#L1135 so I'll just remove the && $poster_id != ANONYMOUS part. The template knows not to display POSTER_WARNINGS == 0.

@nickvergessen : what would be the use of a non-permission based value? Secondly, which one should be the default POSTER_WARNINGS? The auth based, or the current one? Thirdly, would it be called something like POSTER_WARNINGS_AUTH ?

Don't know what I was thinking with this

PHPBB3-12488
nickvergessen added a commit to nickvergessen/phpbb that referenced this pull request May 7, 2014
[ticket/12488] Add user warning indication to viewtopic posts

* PayBas/ticket/12488:
  [ticket/12488] subSilver2 fix (no longer displays if warnings = 0)
  [ticket/12488] Typo fix
  [ticket/12488] Removed <em> and removed anonymous check
  [ticket/12488] Add user warning indication to viewtopic posts
@nickvergessen nickvergessen merged commit 4d64e09 into phpbb:develop-ascraeus May 7, 2014
@PayBas PayBas deleted the ticket/12488 branch May 7, 2014 19:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants