BannersAll sort order fix

[twitchtoo]

Minor change may help correct sort order issues in BannersAll

Condition: Sort order is not being followed - Stock Responsive Classic Template.

Found in:
includes/templates/YOUR_TEMPLATE/sideboxes/tpl_banner_box_all.php
line 22
$sql = "select banners_id from " . TABLE_BANNERS . " where status = 1 " . $new_banner_search . $my_banner_filter . " order by banners_sort_order";

The Fix:
$sql = "select banners_id from " . TABLE_BANNERS . " where status = 1 " . $new_banner_search . $my_banner_filter . " order by banners_sort_order asc"; // Twitch fix banners display sort order


Now my banners follow the correct sort order.

[lat9]

What version of MySQL? The default for the ORDER BY clause is ASC (https://www.w3schools.com/sql/sql_orderby.asp_/

[mc12345678]

What version of MySQL? The default for the ORDER BY clause is ASC (https://www.w3schools.com/sql/sql_orderby.asp_/

It may be an indexing issue based on this mysql document: https://dev.mysql.com/doc/refman/8.0...imization.html

Effectively indicating that optimization may use the index instead of the value(s) in the order by field if optimization identifies that the index is faster...

[twitchtoo]

There is an SQL version mismatch, this is the fix for when you have no choice and it needs to work. This also prevents 'guessing' or any random/inconsistent looking sort order.

Solid state coding is more reliable.