Update:Archive/1.0/Developers: Difference between revisions

Jump to navigation Jump to search
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===
3,035

edits

Navigation menu