fixed AssetReport: "Order the results by" issues (#12227)

plainblack · Aug 26, 2011 · 8ae2457 · 8ae2457
1 parent 073093e
commit 8ae2457
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/docs/changelog/7.x.x.txt b/docs/changelog/7.x.x.txt
@@ -2,6 +2,7 @@
  - fixed #12225: Stock asset, multiple instances on a page
  - fixed #12229: Indexed thingy data has gateway url prepended to it
  - fixed #12195: Visitor group by scratch membership shared among all Visitors (Dale Trexel)
+ - fixed #12227: Corrected AssetReport such that OrderBy works correctly.
 
 7.10.22
  - rfe   #12223: Add date type to content profiling (metadata)

diff --git a/lib/WebGUI/Asset/Wobject/AssetReport.pm b/lib/WebGUI/Asset/Wobject/AssetReport.pm
@@ -159,9 +159,8 @@ sub getTemplateVars {
         $rules->{'whereClause'} .= qq{$prop $op $value};
     }
 
-    if($rules->{'whereClause'}) {
-        $rules->{'joinClass'}   = $settings->{className};
-    }
+    # Always join to the class, asset and assetData are excluded by getLineageSql
+    $rules->{'joinClass'}   = $settings->{className};
 
     #Build the order by condition
     my $order                 = $settings->{order};