can i suggest a character limit validator for the machine name in the features create/recreate form?
following the logic found here:
http://drupal.org/node/1062422
http://drupal.org/node/1316154

what about for the other form fields? would a character limit help?
i haven't found an article specfying limits for machine names, descriptions, project, etc in .info files. the official docs about .info files are here: http://drupal.org/node/542202

Comments

mpotter’s picture

Status: Active » Closed (works as designed)

With the new tar file format, there no longer needs to be a limit on machine name or other fields, so closing this issue.

lpalgarvio’s picture

yep, see #1529066: Features implementation of tar truncates file-paths at 100 characters

but mpotter, could we apply the correct (255) character limit to the form fields anyway?

name, module_name, description
size="60" maxlength="128"
->
size="60" maxlength="255"

7.x

--
CREATE TABLE IF NOT EXISTS `system` (
`filename` varchar(255) NOT NULL default '' COMMENT 'The path of the primary file for this item, relative to the Drupal root; e.g. modules/node/node.module.',
`name` varchar(255) NOT NULL default '' COMMENT 'The name of the item; e.g. node.',
`type` varchar(12) NOT NULL default '' COMMENT 'The type of the item, either module, theme, or theme_engine.',
`owner` varchar(255) NOT NULL default '' COMMENT 'A theme’s ’parent’ . Can be either a theme or an engine.',
`status` int(11) NOT NULL default '0' COMMENT 'Boolean indicating whether or not this item is enabled.',
`bootstrap` int(11) NOT NULL default '0' COMMENT 'Boolean indicating whether this module is loaded during Drupal’s early bootstrapping phase (e.g. even before the page cache is consulted).',
`schema_version` smallint(6) NOT NULL default '-1' COMMENT 'The module’s database schema version number. -1 if the module is not installed (its tables do not exist); 0 or the largest N of the module’s hook_update_N() function that has either been run or existed when the module was first installed.',
`weight` int(11) NOT NULL default '0' COMMENT 'The order in which this module’s hooks should be invoked relative to other modules. Equal-weighted modules are ordered by name.',
`info` blob COMMENT 'A serialized array containing information from the module’s .info file; keys can include name, description, package, version, core, dependencies, and php.',
PRIMARY KEY (`filename`),
KEY `system_list` (`status`,`bootstrap`,`type`,`weight`,`name`),
KEY `type_name` (`type`,`name`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8 COMMENT='A list of all modules, themes, and theme engines that are...';

6.x

--
CREATE TABLE IF NOT EXISTS `system` (
`filename` varchar(255) NOT NULL default '',
`name` varchar(255) NOT NULL default '',
`type` varchar(255) NOT NULL default '',
`owner` varchar(255) NOT NULL default '',
`status` int(11) NOT NULL default '0',
`throttle` tinyint(4) NOT NULL default '0',
`bootstrap` int(11) NOT NULL default '0',
`schema_version` smallint(6) NOT NULL default '-1',
`weight` int(11) NOT NULL default '0',
`info` text,
PRIMARY KEY (`filename`),
KEY `modules` (`type`(12),`status`,`weight`,`filename`),
KEY `bootstrap` (`type`(12),`status`,`bootstrap`,`weight`,`filename`),
KEY `type_name` (`type`(12),`name`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8;