Fixed bug 2553 - truncate_userinfo() was out-of-sync with prefix_user table definition. Added a reminder to the SQL files.

lib/db/mysql.sql

@@ -409,7 +409,9 @@ CREATE TABLE `prefix_sessions` (
 #
 # Table structure for table `user`
 #
-
+# When changing prefix_user, you may need to update
+# truncate_userinfo() in moodlelib.php
+#
 CREATE TABLE `prefix_user` (
   `id` int(10) unsigned NOT NULL auto_increment,
   `auth` varchar(20) NOT NULL default 'manual',

lib/db/postgres7.sql

@@ -276,7 +276,12 @@ CREATE TABLE prefix_cache_text (
 
 CREATE INDEX prefix_cache_text_md5key_idx ON prefix_cache_text (md5key);
 
-
+#
+# Table structure for table `user`
+#
+# When updating field length, modify
+# truncate_userinfo() in moodlelib.php
+#
 CREATE TABLE prefix_user (
    id SERIAL PRIMARY KEY,
    auth varchar(20) NOT NULL default 'manual',

lib/moodlelib.php

@@ -1786,7 +1786,7 @@ function truncate_userinfo($info) {
     // define the limits
     $limit = array(
                     'username'    => 100,
-                    'idnumber'    =>  12,
+                    'idnumber'    =>  64,
                     'firstname'   =>  20,
                     'lastname'    =>  20,
                     'email'       => 100,