Login   Register  
PHP Classes
elePHPant
Icontem

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

Recommend this page to a friend!
Stumble It! Stumble It! Bookmark in del.icio.us Bookmark in del.icio.us

      Dev Front  >  All threads  >  Generally a very good idea, has some minor bugs (e.  >  (Un) Subscribe thread alerts  
Subject:Generally a very good idea, has some minor bugs (e.
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 minor bugs (e.   Reply  
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 minor bugs (e.   Reply  
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 minor bugs (e.   Reply  
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.