Arrow Home arrow Forums
larger font smaller font default font Fixed screen resolution Auto adjust screen size

Joomlapolis Forums  


<< Start < Prev 1 2 Next > End >>
Re:
Date: 2006/08/30 16:54 By: beat Status: Admin  
Karma: 256  
Admin

Posts: 4215
graphgraph
I implemented and thought i tested this in CB 1.0.1 before releasing

Well, please log bug at forge with reference to this thread and server versions information and i will take a look for next version.
Beat - Developer on Community Builder core Team
- If you like CB and this forum, you will love Nick's CB 1.2 RC4 reference manual ! : Click here to Get it now
- Would like to help us move faster ? Get it, and/or help us spend more time coding by helping others in this forum, many thanks
Click here to see the profile of this user The topic has been locked.

Re:Possible solution for the warnings
Date: 2006/08/31 14:09 By: walb Status: User  
Karma: 4  
Fresh Joomlapolitan

Posts: 10
graphgraph
imgToolbox.class.php line 628 holding
Code:

              return in_array'exec'explode',',ini_get'disable_functions' ) ) );

change into
Code:

             return in_array'exec'explode', ',ini_get'disable_functions' ) ) );

Note the extra space behind the comma. This will cause that the exec function is found and no warning is displayed.
Click here to see the profile of this user The topic has been locked.

Re:Possible solution for the warnings
Date: 2006/09/03 03:02 By: beat Status: Admin  
Karma: 256  
Admin

Posts: 4215
graphgraph
walb wrote:
imgToolbox.class.php line 628 holding
Code:

              return in_array'exec'explode',',ini_get'disable_functions' ) ) );

change into
Code:

             return in_array'exec'explode', ',ini_get'disable_functions' ) ) );

Note the extra space behind the comma. This will cause that the exec function is found and no warning is displayed.


Thanks for noticing, fixed in next release with following code:

Code:

     function cbIsExecDisabled(){         if (is_callable("ini_get")) {             $funcs explode',',ini_get'disable_functions' ) );             for ( $i=0$n=count($funcs); $i<$n$i++ ) {                 $funcs[$i] = trim($funcs[$i]);             }             return in_array'exec'$funcs );         } else {             return false;         }     }



Can you please test final fix and report back if ok at your site ?

Thanks
Beat - Developer on Community Builder core Team
- If you like CB and this forum, you will love Nick's CB 1.2 RC4 reference manual ! : Click here to Get it now
- Would like to help us move faster ? Get it, and/or help us spend more time coding by helping others in this forum, many thanks
Click here to see the profile of this user The topic has been locked.

Re:Possible solution for the warnings
Date: 2006/09/04 16:38 By: walb Status: User  
Karma: 4  
Fresh Joomlapolitan

Posts: 10
graphgraph
beat wrote:


Thanks for noticing, fixed in next release with following code:

Code:

     function cbIsExecDisabled(){         if (is_callable("ini_get")) {             $funcs explode',',ini_get'disable_functions' ) );             for ( $i=0$n=count($funcs); $i<$n$i++ ) {                 $funcs[$i] = trim($funcs[$i]);             }             return in_array'exec'$funcs );         } else {             return false;         }     }



Can you please test final fix and report back if ok at your site ?

Thanks

The code is tested and is of course OK.
Thanks,
Walb
Click here to see the profile of this user The topic has been locked.

<< Start < Prev 1 2 Next > End >>

Documentation

Documentation Subscription Service
(updated for CB 1.2 RC4)

What?

Why?

Where?

Just click here for answers!

Click here for a yearly subscription: subscribe now

Download Latest Release

The latest stable Community Builder Release is version 1.1 for Joomla 1.0 and Mambo.
You need to be a registered member of Joomlapolis to download.

The latest release candidate of Community Builder is version 1.2 RC4, native for Joomla 1.0, 1.5 and Mambo.
It is available as "thank you" to all CB documentation subscribers and now also to all registered joomlapolitans.

CB Login