Block Quick Switch users cannot be longer than 128 characters

Quick switch settings are stored in the variables table, which has the structure of this:

CREATE TABLE IF NOT EXISTS `variable` (
  `name` varchar(128) NOT NULL default '',
  `value` longtext NOT NULL,
  PRIMARY KEY  (`name`)
) ENGINE=MyISAM DEFAULT CHARSET=utf8;

The list of quick switch users is stored as a serialized array in the value field, which makes essentially no limit. Then the configuration for setting the form textfield is set as

  $form['masquerade_quick_switches'] = array(
    '#type' => 'textfield',
    '#title' => t('Block <em>Quick Switch</em> users'),
    '#autocomplete_path' => 'masquerade/autocomplete/multiple',
    '#default_value' => variable_get('masquerade_quick_switches', ''),
    '#description' => t('Enter the usernames, separated by commas, of accounts to show as quick switch links in the Masquerade block.')
  );

and there is no #maxlength => 128 there, or 128 anywhere else in the file. You're going to have to clarify how where this 128 limit is.

Try again and paste the error text?

re #5: masquerade_quick_switches isn't 128 chars long, which would be the variable's name. That's not 128 chars. I think this related to #508280: OPML remote location should accept longer URLs, where apparently FAPI defaults to 128 char limit on text fields if a maxlength isn't defined. See line ~172 of system.module.

$type['textfield'] = array('#input' => TRUE, '#size' => 60, '#maxlength' => 128, '#autocomplete_path' => FALSE, '#process' => array('form_expand_ahah'));

Anyway, just set a #maxlength on the fapi element and see what happens.

Here is a patch which sets #maxlength to NULL, removing the FAPI limit for both the "menu" and "block quick switch textfields. As is, they are both limited to 128 characters as mentioned in #6.

Here is a rerolled patch which I will be committing in a moment.