diff options
author | Brion Vibber <brion@pobox.com> | 2010-04-02 15:48:39 -0700 |
---|---|---|
committer | Brion Vibber <brion@pobox.com> | 2010-04-02 15:48:39 -0700 |
commit | e4d934827bd7b2162e870611662549a5df759b57 (patch) | |
tree | b786d0fec05446fd6e1eec9cde463fc62e872beb /actions | |
parent | d844e6bde511595ae05c60406a58b864d2607d8b (diff) | |
parent | b42e044f6e5111fe6cb7d4a7b7431cd9b9b67ccb (diff) |
Merge branch 'testing' into 0.9.x
Diffstat (limited to 'actions')
-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'); |