diff options
| author | Evan Prodromou <evan@status.net> | 2010-04-01 12:09:33 -0400 |
|---|---|---|
| committer | Evan Prodromou <evan@status.net> | 2010-04-01 12:09:33 -0400 |
| commit | 4b80ce0be89fe50eabec1a19dbf4a0c26a413423 (patch) | |
| tree | 6b7dd62a137bc3daa50643c163bd4353d9e478b0 | |
| parent | 650074c648d98f81674c6e2b0ebf052c473ada6e (diff) | |
if user allows location sharing but turned off browser location use profile location
| -rw-r--r-- | actions/newnotice.php | 24 |
1 files changed, 15 insertions, 9 deletions
diff --git a/actions/newnotice.php b/actions/newnotice.php index ed0fa1b2b..748d104ff 100644 --- a/actions/newnotice.php +++ b/actions/newnotice.php @@ -184,13 +184,21 @@ class NewnoticeAction extends Action $options = array('reply_to' => ($replyto == 'false') ? null : $replyto); - if ($user->shareLocation() && $this->arg('notice_data-geo')) { - - $locOptions = Notice::locationOptions($this->trimmed('lat'), - $this->trimmed('lon'), - $this->trimmed('location_id'), - $this->trimmed('location_ns'), - $user->getProfile()); + if ($user->shareLocation()) { + // use browser data if checked; otherwise profile data + if ($this->arg('notice_data-geo')) { + $locOptions = Notice::locationOptions($this->trimmed('lat'), + $this->trimmed('lon'), + $this->trimmed('location_id'), + $this->trimmed('location_ns'), + $user->getProfile()); + } else { + $locOptions = Notice::locationOptions(null, + null, + null, + null, + $user->getProfile()); + } $options = array_merge($options, $locOptions); } @@ -201,8 +209,6 @@ class NewnoticeAction extends Action $upload->attachToNotice($notice); } - - if ($this->boolean('ajax')) { header('Content-Type: text/xml;charset=utf-8'); $this->xw->startDocument('1.0', 'UTF-8'); |