This issue was discussed on the devel-list. (http://lists.drupal.org/archives/drupal-devel/2004-05/msg00132.html) Now I am making a task out of it. The patches will be per-task.

First task: rename Static to sticky.

Comment: I changed only a few lines, only where the word static appears in the UI. I did not dare to change the actual DB scheme. For consistancy's sake we could choose to rename the 'static' column into 'sticky'. But that change will dig deep into drupal. Too much hassle IMO. What should i do?

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Dries’s picture

+1 for updating the database scheme.

Bèr Kessels’s picture

FileSize
16.66 KB

This patch changes all core modules and files that use the term "static". All those references are changed into "sticky". Also the appropriate help-texts and doxygen-texts are changed. All database references to "static" are changed.

I did not yet change any contributions, I tried grepping, but my regexp skills are not that good that i could rule out all "static $variable" calls. Could anybody who knows how to do this post a list here, please?

Chris Johnson’s picture

These look like good possibilties:

modules/authentication/ldap_integration/user.module
modules/event/event.module
modules/howto/howto.module
modules/menus/menus.module
modules/ecommerce/product/product.module
modules/macrotags/macros.inc
modules/macrotags/macrotags.module
modules/summary/summary.module
modules/taxnav/taxnav.module
modules/textile/Textile.php
modules/weblink/weblink.module
modules/xstory/xstory.module
modules/filestore2/filestore2.module
modules/flexinode/flexinode.module
modules/doxygen/doxygen.module
modules/weight/weight.module
modules/wiz/wiz.module
modules/taxonomy_context/taxonomy_context.module
modules/jsdomenu/jsdomenu.module
modules/weather/phpweather-2.1.2/images-test.php
modules/contact/contact.module
modules/product/product.module
modules/navigation/navigation.module
themes/ax_clean/ax_clean.theme
themes/gworks/xtemplate.xtmpl
themes/gworks/xtemplate.theme
themes/polder/polder.inc
themes/polder/polder.theme
themes/smarty/smarty.theme
themes/smarty/includes/PEAR.php
themes/smarty/includes/Smarty.class.php
themes/smooth_blue/smooth_blue.theme
themes/xtemplate/xpushbutton/xtemplate.xtmpl
themes/marvin_2k/marvin_2k.theme
themes/marvin_2k/templates/node.tpl.php
themes/phptemplate/default/node.tpl.php
themes/phptemplate/phptemplate.theme
themes/phptemplate/marvin_2k/node.tpl.php
themes/phptemplate/mobile/node.tpl.php

TDobes’s picture

FileSize
15.69 KB

Here's an updated version of Ber's patch which applies to CVS HEAD as of 20040618 (post-tabs-patch).

Dries’s picture

Committed this patch to HEAD. Thanks Ber and TDobes!

Anonymous’s picture