Refactor: we had three similar functions, refactored into just one.

Also, these functions returned false, or a string containing the type,
but the string was only used to indirectly mean a true value; now
we explicitely return true in this case.

Signed-off-by: Marc Delisle <marc@infomarc.info>

1 files changed, 4 insertions, 1 deletions

diff --git a/tbl_change.php b/tbl_change.php
index c0f468d355..e3f4a433e2 100644
--- a/tbl_change.php
+++ b/tbl_change.php
@@ -135,7 +135,10 @@ $url_params = PMA_urlParamsInEditMode(
 
 $has_blob_field = false;
 foreach ($table_columns as $column) {
-    if (PMA_isColumnBlob($column)) {
+    if (PMA_isColumn(
+        $column,
+        array('blob', 'tinyblob', 'mediumblob', 'longblob')
+    )) {
         $has_blob_field = true;
         break;
     }