[Phpgroupware-tracker] [bug #3625] validator's strip_html removes 0 from strings

[nobody]

=================== BUG #3625: LATEST MODIFICATIONS ==================
http://savannah.gnu.org/bugs/?func=detailbug&bug_id=3625&group_id=509

Changes by: Dave Hall <address@hidden>
Date: Fri 06/27/2003 at 14:17 (Australia/Melbourne)

------------------ Additional Follow-up Comments ----------------------------
added the link to http://phpgroupware.org/3rdparty



=================== BUG #3625: FULL BUG SNAPSHOT ===================


Submitted by: olberger                Project: phpGroupWare                 
Submitted on: Thu 05/15/2003 at 23:34
Category:  API - phpGWapi             Bug Group:  None                      
Severity:  5 - Major                  Priority:  Normal                     
Resolution:  None                     Assigned to:  skwashd                 
Status:  Open                         Component Version:  None              
Platform Version:  None               Reproducibility:  Every Time          

Summary:  validator's strip_html removes 0 from strings

Original Submission:  When using strip_html from class validator (from 
phpgwapi) on a string containing the '0' character, it is removed although not 
being part of a HTML tag.

so strip_html('2003') give 23.

Usage of if(!empty($digit)) whould probably be replaced by if(isset($digit) && 
$digit != "") inside strip_html


Follow-up Comments
*******************

-------------------------------------------------------
Date: Fri 06/27/2003 at 14:16       By: skwashd
added the link to http://phpgroupware.org/3rdparty

-------------------------------------------------------
Date: Thu 05/29/2003 at 16:46       By: None
I will follow this one up.  I will also add your project to the list of 3r 
party phpgw apps

-------------------------------------------------------
Date: Thu 05/29/2003 at 16:34       By: olberger
You're right.

I just found this bug when checking various things while testing PicoLibre on 
the latest phpgw ;)

<offtopic>
We're in the process to upgrading dependencies, but that questions alot of 
design issues also along the way... more news later on the phpgw lists I think
</offtopic>

-------------------------------------------------------
Date: Thu 05/29/2003 at 12:05       By: skwashd
picolibre relies on phpgw12.  This is a very old version which we no longer 
support.   I would suggest that you raise the dependency on an unsupported 
version with them.

Cheers

-------------------------------------------------------
Date: Wed 05/28/2003 at 23:22       By: None
Dunno if regular PhpGW apps use it... but it's in the API ;)

I noticed that when using it for PicoLibre (www.picolibre.org), which is built 
on top of PhpGW.

And I checked in the CVS before reporting, and it seems to still have been 
there at that time.

-------------------------------------------------------
Date: Sat 05/17/2003 at 21:45       By: skwashd
hmmm, is this used by an apps?  Or in an app you are building?  Just trying to 
work out how to replicate this one :)

btw what version of phpgw are you using?

Cheers


CC list is empty


No files currently attached


For detailed info, follow this link:
http://savannah.gnu.org/bugs/?func=detailbug&bug_id=3625&group_id=509

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/