cvs: smarty / NEWS /libs/plugins function.popup.php

messju		Tue Jan 27 17:55:41 2004 EDT

  Modified files:              
    /smarty	NEWS 
    /smarty/libs/plugins	function.popup.php 
  Log:
  fixed quoting of values in smarty_function_popup()
  
  
  
http://cvs.php.net/diff.php/smarty/NEWS?r1=1.431&r2=1.432&ty=u
Index: smarty/NEWS
diff -u smarty/NEWS:1.431 smarty/NEWS:1.432
--- smarty/NEWS:1.431	Thu Jan 22 18:47:45 2004
+++ smarty/NEWS	Tue Jan 27 17:55:41 2004
@@ -1,3 +1,4 @@
+  - fix quoting of values in smarty_function_popup() (messju)
   - fix handling of hidden sections in Config_File (messju)
   - add handling of resources for {config_load} (messju)
   - fix bug when using arrays with tr_attr and td_attr in {html_table} (messju)
http://cvs.php.net/diff.php/smarty/libs/plugins/function.popup.php?r1=1.10&r2=1.11&ty=u
Index: smarty/libs/plugins/function.popup.php
diff -u smarty/libs/plugins/function.popup.php:1.10 smarty/libs/plugins/function.popup.php:1.11
--- smarty/libs/plugins/function.popup.php:1.10	Fri Dec 19 12:20:19 2003
+++ smarty/libs/plugins/function.popup.php	Tue Jan 27 17:55:41 2004
@@ -31,7 +31,7 @@
             case 'caption':
             case 'closetext':
             case 'status':
-                $append .= ',' . strtoupper($_key) . "','" . str_replace("'","\'",$_value) . "'";
+                $append .= ',' . strtoupper($_key) . ",'" . str_replace("'","\'",$_value) . "'";
                 break;
 
             case 'fgcolor':
@@ -65,7 +65,8 @@
             case 'frame':
             case 'timeout':
             case 'delay':
-                $append .= ',' . strtoupper($_key) . "','$_value'";
+            case 'function':
+                $append .= ',' . strtoupper($_key) . ",'$_value'";
                 break;
 
             case 'sticky':
@@ -83,10 +84,6 @@
                 if ($_value) $append .= ',' . strtoupper($_key);
                 break;
 
-            case 'function':
-                $append .= ',' . strtoupper($_key) . "',$_value";
-                break;
-
             default:
                 $smarty->trigger_error("[popup] unknown parameter $_key", E_USER_WARNING);
         }

-- 
Smarty CVS Mailing List (http://cvs.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php