Use a different template for the sidebar to display the user's trophies

Author: Cyperghost

com.woltlab.wcf/templates/boxUserTrophyList.tpl

@@ -1,41 +1,12 @@
-{if $boxPosition == 'sidebarLeft' || $boxPosition == 'sidebarRight'}
-	<ol class="sidebarList">
-		{foreach from=$boxUserTrophyList item=boxUserTrophy}
-			<li class="sidebarListItem">
-				<div class="sidebarListItem__avatar">
-					{unsafe:$boxUserTrophy->getTrophy()->renderTrophy(32)}
-				</div>
+<ol class="containerBoxList trophyCategoryList tripleColumned">
+	{foreach from=$boxUserTrophyList item=boxUserTrophy}
+		<li class="box64">
+			<div>{unsafe:$boxUserTrophy->getTrophy()->renderTrophy(64)}</div>
 
-				<div class="sidebarListItem__content">
-					<h3 class="sidebarListItem__title">
-						<a href="{$boxUserTrophy->getTrophy()->getLink()}" class="sidebarListItem__link">
-							{$boxUserTrophy->getTrophy()->getTitle()}
-						</a>
-					</h3>
-				</div>
-
-				<div class="sidebarListItem__meta">
-					<div class="sidebarListItem__meta__author">
-						{user object=$boxUserTrophy->getUserProfile() tabindex='-1'}
-					</div>
-					<div class="sidebarListItem__meta__time">
-						{time time=$boxUserTrophy->time}
-					</div>
-				</div>
-			</li>
-		{/foreach}
-	</ol>
-{else}
-	<ol class="containerBoxList trophyCategoryList tripleColumned">
-		{foreach from=$boxUserTrophyList item=boxUserTrophy}
-			<li class="box64">
-				<div>{@$boxUserTrophy->getTrophy()->renderTrophy(64)}</div>
-
-				<div class="sidebarItemTitle">
-					<h3><a href="{$boxUserTrophy->getTrophy()->getLink()}">{$boxUserTrophy->getTrophy()->getTitle()}</a></h3>
-					<small>{if !$boxUserTrophy->getDescription()|empty}<p>{@$boxUserTrophy->getDescription()}</p>{/if}<p>{user object=$boxUserTrophy->getUserProfile()} <span class="separatorLeft">{@$boxUserTrophy->time|time}</span></p></small>
-				</div>
-			</li>
-		{/foreach}
-	</ol>
-{/if}
+			<div class="sidebarItemTitle">
+				<h3><a href="{$boxUserTrophy->getTrophy()->getLink()}">{$boxUserTrophy->getTrophy()->getTitle()}</a></h3>
+				<small>{if !$boxUserTrophy->getDescription()|empty}<p>{unsafe:$boxUserTrophy->getDescription()}</p>{/if}<p>{user object=$boxUserTrophy->getUserProfile()} <span class="separatorLeft">{time time=$boxUserTrophy->time}</span></p></small>
+			</div>
+		</li>
+	{/foreach}
+</ol>

com.woltlab.wcf/templates/boxUserTrophyListSidebar.tpl

@@ -0,0 +1,26 @@
+<ol class="sidebarList">
+	{foreach from=$boxUserTrophyList item=boxUserTrophy}
+		<li class="sidebarListItem">
+			<div class="sidebarListItem__avatar">
+				{unsafe:$boxUserTrophy->getTrophy()->renderTrophy(32)}
+			</div>
+
+			<div class="sidebarListItem__content">
+				<h3 class="sidebarListItem__title">
+					<a href="{$boxUserTrophy->getTrophy()->getLink()}" class="sidebarListItem__link">
+						{$boxUserTrophy->getTrophy()->getTitle()}
+					</a>
+				</h3>
+			</div>
+
+			<div class="sidebarListItem__meta">
+				<div class="sidebarListItem__meta__author">
+					{user object=$boxUserTrophy->getUserProfile() tabindex='-1'}
+				</div>
+				<div class="sidebarListItem__meta__time">
+					{time time=$boxUserTrophy->time}
+				</div>
+			</div>
+		</li>
+	{/foreach}
+</ol>

wcfsetup/install/files/lib/system/box/UserTrophyListBoxController.class.php

@@ -150,9 +150,13 @@ public function getTemplate()
 
         UserProfileRuntimeCache::getInstance()->cacheObjectIDs(\array_unique($userIDs));
 
-        return WCF::getTPL()->render('wcf', 'boxUserTrophyList', [
+        $templateName = 'boxUserTrophyList';
+        if ($this->getBox()->position == 'sidebarLeft' || $this->getBox()->position == 'sidebarRight') {
+            $templateName = 'boxUserTrophyListSidebar';
+        }
+
+        return WCF::getTPL()->render('wcf', $templateName, [
             'boxUserTrophyList' => $this->objectList,
-            'boxPosition' => $this->box->position,
         ]);
     }
 }