Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
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?
Comment | File | Size | Author |
---|---|---|---|
#4 | static_to_sticky_1.patch | 15.69 KB | TDobes |
#2 | static_to_sticky_0.patch | 16.66 KB | Bèr Kessels |
static_to_sticky.patch | 4.07 KB | Bèr Kessels | |
Comments
Comment #1
Dries CreditAttribution: Dries commented+1 for updating the database scheme.
Comment #2
Bèr Kessels CreditAttribution: Bèr Kessels commentedThis 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?
Comment #3
Chris Johnson CreditAttribution: Chris Johnson commentedThese 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
Comment #4
TDobes CreditAttribution: TDobes commentedHere's an updated version of Ber's patch which applies to CVS HEAD as of 20040618 (post-tabs-patch).
Comment #5
Dries CreditAttribution: Dries commentedCommitted this patch to HEAD. Thanks Ber and TDobes!
Comment #6
(not verified) CreditAttribution: commented