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 |
|
 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.
 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.
 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.
|