[TWiki-Dev] [SVN] HideyoImazu r25779 - twiki/trunk/PreferencesPlugin/lib/TWiki/Plugins

[<de...@de...>]

Author: HideyoImazu
Date: 2013-04-19 02:16:41 -0500 (Fri, 19 Apr 2013)
New Revision: 25779
Trac url: http://develop.twiki.org/trac/changeset/25779

Modified:
   twiki/trunk/PreferencesPlugin/lib/TWiki/Plugins/PreferencesPlugin.pm
Log:
Item7236: PreferencesPlugin not to touch a variable whose type is label

Modified: twiki/trunk/PreferencesPlugin/lib/TWiki/Plugins/PreferencesPlugin.pm
===================================================================
--- twiki/trunk/PreferencesPlugin/lib/TWiki/Plugins/PreferencesPlugin.pm	2013-04-19 03:00:43 UTC (rev 25778)
+++ twiki/trunk/PreferencesPlugin/lib/TWiki/Plugins/PreferencesPlugin.pm	2013-04-19 07:16:41 UTC (rev 25779)
@@ -54,6 +54,46 @@
     return 1;
 }
 
+# Item7008
+my $TO_ADDTOHEAD_WHEN_EDIT = <<'END';
+<style type='text/css'>
+.twikiPrefFieldTable {display: inline-table; vertical-align: top;}
+.twikiPrefFieldDiv {display: inline}
+.twikiPrefFieldLabelDisabled {color: #888;}
+.twikiPrefFieldHash {visibility: hidden;}
+.twikiPrefFieldLabelDisabled .twikiPrefFieldHash {visibility: visible;}
+</style>
+<!--[if IE]>
+<style type='text/css'>
+.twikiPrefFieldTable {display: inline; vertical-align: top;}
+</style>
+<![endif]-->
+<script type='text/javascript'>
+if( window.jQuery ) {
+    $( function () {
+        var disabledClass = 'twikiPrefFieldLabelDisabled';
+        $( '.twikiPrefEnableCheckbox' ).each( function () {
+            var $checkbox = $(this);
+            if( $checkbox.attr( 'name' ).match( /(.*?)!/ ) ) {
+                var name = RegExp.$1;
+                var $label = $( '#twikiPrefFieldLabel-' + name );
+                if( !$checkbox.attr( 'checked' ) ) {
+                    $label.addClass( disabledClass );
+                }
+                $checkbox.change( function () {
+                    if( $checkbox.attr( 'checked' ) ) {
+                        $label.removeClass( disabledClass );
+                    } else {
+                        $label.addClass( disabledClass );
+                    }
+                } );
+            }
+        } );
+    } );
+}
+</script>
+END
+
 sub beforeCommonTagsHandler {
     ### my ( $text, $topic, $web ) = @_;
     my $topic = $_[1];
@@ -99,44 +139,7 @@
         TWiki::Func::setTopicEditLock( $web, $topic, 1 );
         
         # Item7008
-        TWiki::Func::addToHEAD('PreferencesPlugin', <<'END');
-<style type='text/css'>
-.twikiPrefFieldTable {display: inline-table; vertical-align: top;}
-.twikiPrefFieldDiv {display: inline}
-.twikiPrefFieldLabelDisabled {color: #888;}
-.twikiPrefFieldHash {visibility: hidden;}
-.twikiPrefFieldLabelDisabled .twikiPrefFieldHash {visibility: visible;}
-</style>
-<!--[if IE]>
-<style type='text/css'>
-.twikiPrefFieldTable {display: inline; vertical-align: top;}
-</style>
-<![endif]-->
-<script type='text/javascript'>
-if( window.jQuery ) {
-    $( function () {
-        var disabledClass = 'twikiPrefFieldLabelDisabled';
-        $( '.twikiPrefEnableCheckbox' ).each( function () {
-            var $checkbox = $(this);
-            if( $checkbox.attr( 'name' ).match( /(.*?)!/ ) ) {
-                var name = RegExp.$1;
-                var $label = $( '#twikiPrefFieldLabel-' + name );
-                if( !$checkbox.attr( 'checked' ) ) {
-                    $label.addClass( disabledClass );
-                }
-                $checkbox.change( function () {
-                    if( $checkbox.attr( 'checked' ) ) {
-                        $label.removeClass( disabledClass );
-                    } else {
-                        $label.addClass( disabledClass );
-                    }
-                } );
-            }
-        } );
-    } );
-}
-</script>
-END
+        TWiki::Func::addToHEAD('PreferencesPlugin', $TO_ADDTOHEAD_WHEN_EDIT);
 
         # Replace setting values by form fields but not inside comments Item4816
         my $outtext = '';
@@ -239,6 +242,9 @@
             $fieldDef = _getField( $formDef, $name );
         }
         if ( $fieldDef ) {
+            if ( $fieldDef->{type} && $fieldDef->{type} eq 'label' ) {
+                return "$prefix$name = $value";
+            }
             if( defined(&TWiki::Form::renderFieldForEdit)) {
                 # TWiki < 4.2 SMELL: use of unpublished core function
                 ( $extras, $html ) =
@@ -353,6 +359,9 @@
     if( $formDef ) {
         my $fieldDef = _getField( $formDef, $name );
         my $type = $fieldDef->{type} || '';
+        if ( $type eq 'label' ) {
+            return "$prefix$name =$value";
+        }
         if( $type && $type =~ /^(checkbox|select.*\+multi)/ ) {
             my @values = $query->param( $name );
             my %vset = ();