Permalink
Browse files

MDL-30700 Assignment 2.2: Added text_sorting function to tablelib

Allows oracle to sort by text (clob) columns (should be used sparingly).
  • Loading branch information...
damyon committed Dec 11, 2012
1 parent 8bd7536 commit 0887f755877a90d5310155d1c6542e2319c5c21e
Showing with 16 additions and 2 deletions.
  1. +15 −2 lib/tablelib.php
  2. +1 −0 mod/assignment/lib.php
View
@@ -60,6 +60,7 @@ class flexible_table {
var $column_class = array();
var $column_suppress = array();
var $column_nosort = array('userpic');
+ private $column_textsort = array();
var $setup = false;
var $sess = NULL;
var $baseurl = NULL;
@@ -214,6 +215,14 @@ function sortable($bool, $defaultcolumn = NULL, $defaultorder = SORT_ASC) {
$this->sort_default_order = $defaultorder;
}
+ /**
+ * Use text sorting functions for this column (required for text columns with Oracle).
+ * @param string column name
+ */
+ function text_sorting($column) {
+ $this->column_textsort[] = $column;
+ }
+
/**
* Do not sort using this column
* @param string column name
@@ -513,10 +522,14 @@ public static function get_sort_for_table($uniqueid) {
* @param array $cols column name => SORT_ASC or SORT_DESC
* @return SQL fragment that can be used in an ORDER BY clause.
*/
- public static function construct_order_by($cols) {
+ public static function construct_order_by($cols, $textsortcols=array()) {
+ global $DB;
$bits = array();
foreach ($cols as $column => $order) {
+ if (in_array($column, $textsortcols)) {
+ $column = $DB->sql_order_by_text($column);
+ }
if ($order == SORT_ASC) {
$bits[] = $column . ' ASC';
} else {
@@ -531,7 +544,7 @@ public static function construct_order_by($cols) {
* @return SQL fragment that can be used in an ORDER BY clause.
*/
public function get_sql_sort() {
- return self::construct_order_by($this->get_sort_columns());
+ return self::construct_order_by($this->get_sort_columns(), $this->column_textsort);
}
/**
View
@@ -1380,6 +1380,7 @@ function display_submissions($message='') {
$table->no_sorting('finalgrade');
$table->no_sorting('outcome');
+ $table->text_sorting('submissioncomment');
// Start working -- this is necessary as soon as the niceties are over
$table->setup();

0 comments on commit 0887f75

Please sign in to comment.