PHP Classes

Generally a very good idea, has some minor bugs (e.

Recommend this page to a friend!

      Dev Front  >  All threads  >  Generally a very good idea, has some...  >  (Un) Subscribe thread alerts  
Subject:Generally a very good idea, has some...
Summary:Package rating comment
Messages:3
Author:michael
Date:2012-03-06 20:52:55
Update:2012-05-17 19:20:01
 

michael rated this package as follows:

Utility: Good
Consistency: Sufficient
Documentation: Insufficient

  1. Generally a very good idea, has some...   Reply   Report abuse  
Picture of michael michael - 2012-03-06 20:52:55
Generally a very good idea, has some minor bugs (e.g. not checking if a var is an array => php.notice.
Should have the possibility to scan existing projects/folders (after a fresh installation) and assign them automatically as (new) projects.
Missing more/better documentation.

  2. Re: Generally a very good idea, has some...   Reply   Report abuse  
Picture of Gergely Aradszki Gergely Aradszki - 2012-03-07 09:03:09 - In reply to message 1 from michael
Hi!

Thanks for rating.
Would you please spacify where you found the notice error so I could fix it?
Scanning existing projects folder did not really came up for me as I have too many projects and the whole purpose of the script was to show only the few important ones. Anyway, it would be nice to have some options where you could select at installation which folders to include as a project.

  3. Re: Generally a very good idea, has some...   Reply   Report abuse  
Picture of michael michael - 2012-05-17 19:20:01 - In reply to message 2 from Gergely Aradszki
Hello,

and sorry, did no see your reply.

Here is what I did to avoid unwanted error messages:

inside function save_config():

instead:

$config['projects'] = isset($config['projects']) ? array_values($config['projects']) : false;

replaced by this:

protected function save_config($config) {
$config['projects'] =
( isset( $config['projects'] ) && is_array( $config['projects'] ) )
? array_values( $config['projects'] )
: false;

It checks now addiotnally if is is an array.

Maybe this helps.