Sorting by aliases

[kyslik]

Sorting by aliases

[Kyslik]

I will try to make it work. Thank you for suggestion with $sortable array.

Isn't it possible to use https://github.com/Kyslik/column-sortable#columnsortable-overloading-advanced ?

[veganista]

Yes, overloading for the total field works for my example above.

public function totalSortable($query, $direction)
{
    return $query->orderBy('total', $direction);
}

I think that this is probably the best solution rather than allowing the configuration of fields to be set.

public $sortable = [
    'total' => 'total',
    'balance' => 'balance',
    'anotherAliasedField' => 'anotherAliasedField',
    'created_at',
];

Doesn't look great as it's not obvious why you would be repeating the field name and i can't think of another use case for allowing that.

This is related to #24 as it would solve that guy's problem.

[Kyslik]

I think something like introducing new variable would be nice, maybe:

$sortableAliases = []

Or something similar, I will think about it and make it work.

---

WIP, aliases expected in new minor version

[Kyslik]

@veganista Hey, I was thinking about fixing this issue.

Anyway, #24 won't be fixed by this because OP uses accessor (manipulation with data after it is fetched) so there is no (virtual) column that package can sort by; in example below I show query with full_name and package could sort by it...

SELECT neededfield, CONCAT(firstname, ' ', lastname) as full_name ...

For that purpose there is that overloading functionality (which I use a lot in my projects), custom joins, custom selects etc.

---

I will try fix this soon.

[Kyslik]

@veganista try on dev-master, see README.md and please report back if everything OK.

[veganista]

Sorry for the delay in getting back on this, took me a while to get back on the project.

Works as expected and much more concise than overloading.

Thanks