-
-
Notifications
You must be signed in to change notification settings - Fork 408
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
Working on #2972 Template Conversion to Twig Format (portal.php) #3019
Conversation
This is finished |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, just one minor change needed.
inc/views/base/portal/portal.twig
Outdated
{% block head %} | ||
<title>{{ mybb.settings.bbname }}</title> | ||
<link rel="alternate" type="application/rss+xml" title="{{ lang.latest_announcements }} (RSS 2.0)" href="{{ mybb.settings.bburl }}/syndication.php?portal=1" /> | ||
<link rel="alternate" type="application/atom+xml" title="{{ lang.latest_announcements }} (Atom 1.0)" href="{{ mybb.settings.bburl }}/syndication.php?portal=1&type=atom1.0" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
&
should be &
inc/views/base/portal/portal.twig
Outdated
</tr> | ||
<tr> | ||
<td class="trow1"> | ||
<span class="smalltext">{{ lang.pms_received_new|raw }}<br /><br /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rather than using |raw
here, it looks like this is a fairly simple language variable that could be built in the template using the trans()
function. One of my PRs from today marks the trans()
function as being HTML safe, so as long as $mybb->user['username']
is HTML safe, it should be fine here.
Done |
Would you mind updating the list of templates in #2972 with the ones you've finished with the PRs I've merged too please? |
* Working on #2972 Template Conversion to Twig Format (portal.php) * Updated attachments * Coding standards
* Working on #2972 Template Conversion to Twig Format (portal.php) * Updated attachments * Coding standards
* Working on #2972 Template Conversion to Twig Format (portal.php) * Updated attachments * Coding standards
#2972 for portal.php
This is still incomplete, I need #2988 merged before I can finish this.