3,035
edits
Line 525: | Line 525: | ||
===/plugins=== | ===/plugins=== | ||
---- | |||
* In general, it's not in CVS which is bad. | * In general, it's not in CVS which is bad. | ||
** We know what part can't be in CVS (uniqueUrl.php), so it will be committed soon. | ** We know what part can't be in CVS (uniqueUrl.php), so it will be committed soon. | ||
Line 535: | Line 536: | ||
====/plugins/index.php==== | ====/plugins/index.php==== | ||
====/plugins/download.php==== | ====/plugins/download.php==== | ||
* [61-78] PHP is not for serving files. Use a redirect that sends them to a URL instead. | * [61-78] PHP is not for serving files. Use a redirect that sends them to a URL instead. | ||
====/plugins/PluginFinderService.php==== | ====/plugins/PluginFinderService.php==== | ||
Line 547: | Line 546: | ||
* [47] This should be a require_once(). | * [47] This should be a require_once(). | ||
* [394] This could be done more thoroughly with a built-in PHP function instead of using preg_replace. | * [394] This could be done more thoroughly with a built-in PHP function instead of using preg_replace. | ||
====/plugins/uniqueUrl.php==== | ====/plugins/uniqueUrl.php==== | ||
* Can't change this. | * Can't change this. | ||
===/rss=== | ===/rss=== |
edits