| 
| 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:40great job could you add : defines, globals, requires and includes ?
  Colin McKinnon - 2011-01-25 22:58:00 - In reply to message 1 from zetounI 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 McKinnonI'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 McKinnondamn,its exactly what my code looks like ... |