[Erp5-dev] [Erp5-report] r21739 - in /erp5/trunk/products/ERP5/bootstrap/erp5_core: SkinTemplateItem/p...

[Yoshinori Okuji]

This fix is not right. Note that Base.log is deprecated. Use ERPType.Log.log 
instead.

YO

On Monday 23 June 2008 14:18:03 nobody at svn.erp5.org wrote:
> Author: vincent
> Date: Mon Jun 23 14:18:01 2008
> New Revision: 21739
>
> URL: http://svn.erp5.org?rev=21739&view=rev
> Log:
> Fix an undeclared variable error in a rarely taken execution path.
> Discovered & fix by Łukasz Nowak. Thanks !
>
> Modified:
>    
> erp5/trunk/products/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/
>erp5_core/Base_callDialogMethod.xml
> erp5/trunk/products/ERP5/bootstrap/erp5_core/bt/revision
>
> Modified:
> erp5/trunk/products/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/
>erp5_core/Base_callDialogMethod.xml URL:
> http://svn.erp5.org/erp5/trunk/products/ERP5/bootstrap/erp5_core/SkinTempla
>teItem/portal_skins/erp5_core/Base_callDialogMethod.xml?rev=21739&r1=21738&r
>2=21739&view=diff
> ===========================================================================
>=== ---
> erp5/trunk/products/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/
>erp5_core/Base_callDialogMethod.xml (original) +++
> erp5/trunk/products/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/
>erp5_core/Base_callDialogMethod.xml Mon Jun 23 14:18:01 2008 @@ -203,7
> +203,7 @@
>    can_redirect = 0\n
>    # Warn if there are more than one listbox in form ...\n
>    if len(listbox_id_list) > 1:\n
> -    log(\'Base_callDialogMethod\', \'There are %s listboxes in form %s.\'
> % (len(listbox_id_list), form.id))\n +   
> context.log(\'Base_callDialogMethod\', \'There are %s listboxes in form
> %s.\' % (len(listbox_id_list), form.id))\n # ... but handle them anyway.\n
>    for listbox_id in listbox_id_list:\n
>      listbox_line_list = []\n
>
> Modified: erp5/trunk/products/ERP5/bootstrap/erp5_core/bt/revision
> URL:
> http://svn.erp5.org/erp5/trunk/products/ERP5/bootstrap/erp5_core/bt/revisio
>n?rev=21739&r1=21738&r2=21739&view=diff
> ===========================================================================
>=== --- erp5/trunk/products/ERP5/bootstrap/erp5_core/bt/revision (original)
> +++ erp5/trunk/products/ERP5/bootstrap/erp5_core/bt/revision Mon Jun 23
> 14:18:01 2008 @@ -1,1 +1,1 @@
> -855
> +856
>
> _______________________________________________
> Erp5-report mailing list
> Erp5-report at erp5.org
> http://mail.nexedi.com/mailman/listinfo/erp5-report



-- 
Yoshinori Okuji, Nexedi KK President / Nexedi SA CTO
Nexedi: Consulting and Development of Free / Open Source Software
http://www.nexedi.co.jp/
ERP5: Full Featured High End Open Source ERP
http://www.erp5.com/
ERP5 Wiki: Developer Zone for ERP5 Community
http://www.erp5.org/