| Subject: | great job could you add : defines,... |  
| Summary: | Package rating comment |  
| Messages: | 5 |  
| Author: | zetoun |  
| Date: | 2011-01-07 09:19:40 |  
| Update: | 2011-02-14 19:36:50 |  
|   |  
 
 | 
 | 
zetoun rated this package as follows:
| Utility:  | Good | 
| Consistency:  | Good | 
| Documentation:  | Sufficient | 
| Examples:  | Sufficient | 
| 
 | 
  zetoun - 2011-01-07 09:19:40  
great job  
could you add : defines, globals, requires and includes ? 
  
  Colin McKinnon - 2011-01-25 22:58:00 -  In reply to message 1 from zetoun 
I was rather planning not to changwe the functionality of the code, and do as little maintenance as possible! 
 
But if you want to send me a version with patches applied I'll try to review it. 
  
  Colin McKinnon - 2011-02-04 23:06:08 -  In reply to message 2 from Colin McKinnon 
I've just uploaded a newer versions which adds clickthrough support - I've included an example which generates URLs for PHPXREF generated code documentation.  
  
  Colin McKinnon - 2011-02-07 15:11:41 -  In reply to message 3 from Colin McKinnon 
> could you add : defines, globals, requires and includes ? 
 
I can see that being able to merge several PHP src files into one might be useful, however this is not really possible without runing the code - and how should it deal with a scenario like: 
 
$use_inc=''; 
if (php_sapi_name() == 'cli') { 
  $use_inc="curses_ui.inc.php"; 
} else { 
  if (strstr($_SERVER['HTTP_ACCEPT']), 'text/html') { 
    $use_inc="html_ui.inc.php"; 
  } else { 
    $use_inc="wml_ui.inc.php"; 
  } 
} 
require($use_inc); 
  
  zetoun - 2011-02-14 19:36:50 -  In reply to message 4 from Colin McKinnon 
damn,its exactly what my code looks like ... 
  
   |