Update Attribute.php

Without this else, TEXT->DROPDOWN fails.  It just returns a simple success because no database changes are needed other than changing the definition_type which is not done here.  This would also cause all other conversions not listed to automatically succeed, but we don't need to worry because those are restricted in the view to only ones that are actually supported.

application/models/Attribute.php

@@ -303,6 +303,12 @@ class Attribute extends CI_Model
 			$this->db->trans_complete();
 		}
 		
+		//Any other allowed conversion does not get checked here
+		else
+		{
+			$success = TRUE;
+		}
+
 		return $success;
 	}
 	
@@ -542,4 +548,4 @@ class Attribute extends CI_Model
 
 		return $this->db->update('attribute_definitions', array('deleted' => 1));
 	}
-}
+}