From 0ae510fdf0b02352d7b4a62e2fcb9e2e6cfe475c Mon Sep 17 00:00:00 2001
From: James Brooks <james@alt-three.com>
Date: Sat, 26 Jan 2019 15:00:42 +0000
Subject: [PATCH] Fix other duplicate order by statements

---
 app/Composers/ComponentsComposer.php          | 2 +-
 resources/views/partials/components.blade.php | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/app/Composers/ComponentsComposer.php b/app/Composers/ComponentsComposer.php
index 7e62715c..1fac9423 100644
--- a/app/Composers/ComponentsComposer.php
+++ b/app/Composers/ComponentsComposer.php
@@ -53,7 +53,7 @@ class ComponentsComposer
     public function compose(View $view)
     {
         $componentGroups = $this->getVisibleGroupedComponents();
-        $ungroupedComponents = Component::ungrouped()->get();
+        $ungroupedComponents = Component::ungrouped()->orderBy('status', 'desc')->get();
 
         $view->withComponentGroups($componentGroups)
             ->withUngroupedComponents($ungroupedComponents);
diff --git a/resources/views/partials/components.blade.php b/resources/views/partials/components.blade.php
index 2fd869af..c35143a9 100644
--- a/resources/views/partials/components.blade.php
+++ b/resources/views/partials/components.blade.php
@@ -12,7 +12,7 @@
     </li>
 
     <div class="group-items {{ $componentGroup->is_collapsed ? "hide" : null }}">
-        @each('partials.component', $componentGroup->enabled_components()->orderBy('order')->get(), 'component')
+        @each('partials.component', $componentGroup->enabled_components, 'component')
     </div>
     @endif
 </ul>
@@ -25,7 +25,7 @@
         <strong>{{ trans('cachet.components.group.other') }}</strong>
 
         <div class="pull-right">
-            <i class="ion ion-ios-circle-filled text-component-{{ $ungroupedComponents->max('status') }} {{ $ungroupedComponents->sortByDesc('status')->first()->status_color }}" data-toggle="tooltip" title="{{ $ungroupedComponents->sortByDesc('status')->first()->human_status }}"></i>
+            <i class="ion ion-ios-circle-filled text-component-{{ $ungroupedComponents->max('status') }} {{ $ungroupedComponents->first()->status_color }}" data-toggle="tooltip" title="{{ $ungroupedComponents->first()->human_status }}"></i>
         </div>
     </li>