Reduce redudant code around displaying SQL results rows

[Christopher Kramer]

Originally reported on Google Code with ID 236

We have three ways of displaying SQL result sets as tables:
- view=sql
- action=table_sql
- action=row_view

All three share a big bunch of code. We really should reduce redundant code here. This
would avoid issues like issue #223.

Reported by crazy4chrissi on 2013-12-24 12:54:12

[Christopher Kramer]

Reported by crazy4chrissi on 2013-12-25 20:37:12 - Labels added: Maintainability

[dreadnaut]

Just a reminder: we could work on Issue #78 at the same time, when we fix this.

Reported by dreadnaut on 2014-01-03 14:51:39

[phpLiteAdmin]

Since commit 4a3f7b68ef59ba4bd88e94f90530c287e551f842, the table_search-action uses the same code as row_view to display rows, thus removing lots of redundant code. table_sql and sql have not been worked on yet.