Merge pull request #17274 from bdsumon4u/fix-17210-summarize-on-prefixed-table

fix: Remove redundant column prefixing in table summaries
This commit is contained in:
Dan Harrin
2025-08-12 13:23:44 +01:00
committed by GitHub
3 changed files with 2 additions and 7 deletions
@@ -277,7 +277,7 @@
if (! grouping) {
group = null
direction = null
return
}
@@ -4,7 +4,6 @@ namespace Filament\Tables\Concerns;
use Closure;
use Filament\Support\Services\RelationshipJoiner;
use Illuminate\Database\Connection;
use Illuminate\Database\Eloquent\Builder;
use Illuminate\Database\Query\Expression;
use Illuminate\Support\Str;
@@ -43,10 +42,7 @@ trait CanSummarizeRecords
continue;
}
/** @var Connection $queryConnection */
$queryConnection = $query->getConnection();
$qualifiedAttribute = $queryConnection->getTablePrefix() . $query->getModel()->qualifyColumn($column->getName());
$qualifiedAttribute = $query->getModel()->qualifyColumn($column->getName());
foreach ($summarizers as $summarizer) {
if ($summarizer->hasQueryModification()) {
@@ -262,7 +262,6 @@ class UpgradeDirectoryStructureToV4Command extends Command
}
/**
* @param string $directory
* @return array<int, string>
*/
protected function findPhpFiles(string $directory): array