Refactor user ClubLog fields migration

Update migration to use array definitions for new user ClubLog fields instead of raw SQL strings. This improves compatibility and clarity when adding columns to the 'users' table.

Author: magicbug

application/migrations/018_clubloguserfields.php

@@ -8,17 +8,29 @@ public function up()
         {
                 // Check and add each field only if it does not exist
                 $fields_to_add = array(
-                        'user_clublog_name' => 'VARCHAR(255) DEFAULT NULL',
-                        'user_clublog_password' => 'VARCHAR(255) DEFAULT NULL',
-                        'user_clublog_callsign' => 'VARCHAR(255) DEFAULT NULL'
+                        'user_clublog_name' => array(
+                                'type' => 'VARCHAR',
+                                'constraint' => 255,
+                                'null' => TRUE,
+                        ),
+                        'user_clublog_password' => array(
+                                'type' => 'VARCHAR',
+                                'constraint' => 255,
+                                'null' => TRUE,
+                        ),
+                        'user_clublog_callsign' => array(
+                                'type' => 'VARCHAR',
+                                'constraint' => 255,
+                                'null' => TRUE,
+                        ),
                 );
 
                 // Get current columns in 'users' table
                 $fields = $this->db->list_fields('users');
 
                 foreach ($fields_to_add as $field => $definition) {
                         if (!in_array($field, $fields)) {
-                                $this->dbforge->add_column('users', array($field . ' ' . $definition));
+                                $this->dbforge->add_column('users', array($field => $definition));
                         }
                 }
         }