diff options
author | Evan Prodromou <evan@prodromou.name> | 2008-05-20 15:26:34 -0400 |
---|---|---|
committer | Evan Prodromou <evan@prodromou.name> | 2008-05-20 15:26:34 -0400 |
commit | 5137eb20d876f71de2b55fef3c470edb15e08462 (patch) | |
tree | ed553983f2677c23082fba3dde48ccef3aac0cac /classes | |
parent | 52600ce0b063e68e622b19699841e41b5ddbf2d1 (diff) |
fix validation of url
darcs-hash:20080520192634-84dde-2394dbf8767ff886817a5f20f5c9064ea3a1bc79.gz
Diffstat (limited to 'classes')
-rw-r--r-- | classes/Notice.php | 10 |
1 files changed, 2 insertions, 8 deletions
diff --git a/classes/Notice.php b/classes/Notice.php index f06b21d6c..f742d4326 100644 --- a/classes/Notice.php +++ b/classes/Notice.php @@ -43,14 +43,8 @@ class Notice extends DB_DataObject /* the code above is auto generated do not remove the tag below */ ###END_AUTOCODE - // XXX: if profile_id changes, this goes invalid. To be fair, that's a very edge case - static $profile; - function getProfile() { - if (!$this->profile) { - $this->profile = Profile::staticGet($this->profile_id); - } - return $this->profile; + return Profile::staticGet($this->profile_id); } function validateContent() { @@ -58,7 +52,7 @@ class Notice extends DB_DataObject } function validateUrl() { - return is_null($this->url) || + return is_null($this->url) || (strlen($this->url) == 0) || Validate::uri($this->url, array('allowed_schemes' => array('http', 'https'))); } } |