Why does this module require the Fields UI module? I like to disable that when its not in use to reduce the amount of PHP needed to compile each page (think "disable developer modules"). Can you remove that dependency?

CommentFileSizeAuthor
#6 field-ui-1114136-6.patch235 bytesjoelstein
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

KarenS’s picture

Status: Active » Fixed

The only module that requires Fields UI is the CCK module that alters the fields UI. Without Fields UI that module does nothing.

joelstein’s picture

That's not entirely true. For instance, suppose I use CCK to add some allowed values via PHP. I only need Fields UI when I'm configuring these additions, but not when the allowed values are actually being evaluated in a node form.

Fields UI adds a large amount of unnecessary code to compile, and it should only be installed when actually configuring fields, a task which takes up around 1% of the lifetime of the site. Many people will install CCK on D7 (even if they don't really need to), so you'd reduce a fair amount of Drupal 7 slowness on the web by lifting this dependency.

joelstein’s picture

PS, other modules are also doing this:

Field Permissions: http://drupal.org/node/1114134
Meta tags quick: http://drupal.org/node/1114120

And, Admin Menu has added Fields UI to its list of developer modules: http://drupal.org/node/1114132#comment-4432890.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

joelstein’s picture

Status: Closed (fixed) » Active

Sorry to pester this issue, but Field UI adds lots of code that doesn't need to be compiled for most requests. CCK will certainly get installed by LOTS of installations... will you please give this more consideration?

joelstein’s picture

Status: Active » Needs review
FileSize
235 bytes

Here's a patch. I still think this idea has a lot of merit, and would like to see it committed.

geerlingguy’s picture

I'd like to support this too, as I have been disabling Fields UI on all the sites I can lately - it does add some overhead (and uses a bit more memory - every byte counts!).

DamienMcKenna’s picture

Version: 7.x-2.x-dev » 7.x-3.x-dev
Issue summary: View changes
Parent issue: » #1277262: Plan for CCK 7.x-3.0 release