diff options
| author | Bharat Mediratta <bharat@menalto.com> | 2009-01-14 04:12:02 +0000 |
|---|---|---|
| committer | Bharat Mediratta <bharat@menalto.com> | 2009-01-14 04:12:02 +0000 |
| commit | f3ba69c1d67c425ffa180d082a373e5cce0c86ce (patch) | |
| tree | 5ecbbe0ba96a94e3f4aadd26042c8ad1a32ef1bc /modules/media_rss/helpers/media_rss.php | |
| parent | 02af2d8b7639fdc18fba3d69a4ca0a3f5c92b948 (diff) | |
Make sure that helper functions are all static. Add new
File_Structure_Test to make sure we don't regress.
According to the PHP docs, the "public" keyword is implied on static
functions, so remove it. Also, require private static functions to
start with an _.
http://php.net/manual/en/language.oop5.visibility.php
Diffstat (limited to 'modules/media_rss/helpers/media_rss.php')
| -rw-r--r-- | modules/media_rss/helpers/media_rss.php | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/modules/media_rss/helpers/media_rss.php b/modules/media_rss/helpers/media_rss.php index c834611a..9522727d 100644 --- a/modules/media_rss/helpers/media_rss.php +++ b/modules/media_rss/helpers/media_rss.php @@ -19,12 +19,12 @@ */ class media_rss_Core { - public static function item_feed($item) { + static function item_feed($item) { $id = $item->type == "album" ? $item->id : $item->parent_id; return url::site("media_rss/albums/$id"); } - public static function tag_feed($tag) { + static function tag_feed($tag) { return url::site("media_rss/tags/$tag->id}"); } }
\ No newline at end of file |