Fix resolveAliasAndColumn to use longest matching relationship path

- Removed early break in path resolution loop
- Now correctly resolves asset.financials.monthly_hire_revenue to the financials table alias
- Previously was breaking after finding 'asset' and returning the asset table alias
- Now continues through all segments and uses the longest matching path
- Fixes 'Unknown column' error when computed columns reference nested relationships in aggregates

Author: Manus AI

src/Support/Reporting/ReportQueryConverter.php

@@ -411,18 +411,15 @@ protected function resolveAliasAndColumn(string $rootTable, string $columnPath):
         $currentTable = $rootTable;
         $pathSegments = [];
 
+        // Loop through all segments and keep updating currentTable with the last successfully resolved alias
         foreach ($parts as $segment) {
             $pathSegments[] = $segment;
             $currentPath = implode('.', $pathSegments);
 
             if (isset($this->joinAliases[$currentPath])) {
                 $currentTable = $this->joinAliases[$currentPath];
-            } else {
-                // Try to find the relationship from the current table
-                // This handles nested relationships like "asset" -> "financials"
-                // where we need to use the asset table alias as the base
-                break;
             }
+            // Don't break - continue to check if there's a longer path that matches
         }
 
         // If we resolved at least part of the path, use that table alias