diff options
author | Brion Vibber <brion@pobox.com> | 2010-03-29 15:15:51 -0700 |
---|---|---|
committer | Brion Vibber <brion@pobox.com> | 2010-03-29 15:15:51 -0700 |
commit | 01a03e34c8d286011c0f4301e60c103d992a449a (patch) | |
tree | d9c5a44704b83d3ca05ed1f136a3748fb8210f12 /actions/newnotice.php | |
parent | e2d3ebb9f4a3b516067745d250b4723367b928c2 (diff) | |
parent | cfeb1bfa419e886a1b60dfcbd4260d2f5de0515b (diff) |
Merge branch '0.9.x' into 1.0.x
Diffstat (limited to 'actions/newnotice.php')
-rw-r--r-- | actions/newnotice.php | 35 |
1 files changed, 18 insertions, 17 deletions
diff --git a/actions/newnotice.php b/actions/newnotice.php index ed0fa1b2b..2aa354870 100644 --- a/actions/newnotice.php +++ b/actions/newnotice.php @@ -164,22 +164,25 @@ class NewnoticeAction extends Action $replyto = 'false'; } - $upload = null; - $upload = MediaFile::fromUpload('attach'); - - if (isset($upload)) { - - $content_shortened .= ' ' . $upload->shortUrl(); - - if (Notice::contentTooLong($content_shortened)) { + $uploads = array(); + foreach($_FILES as $name => $value) { + if(substr($name, 0, 6) == "attach") { + $upload = MediaFile::fromUpload($name); + if (isset($upload)) { + $content_shortened .= ' ' . $upload->shortUrl(); + } + } + } + if (Notice::contentTooLong($content_shortened)) { + foreach($uploads as $upload) { $upload->delete(); - $this->clientError( - sprintf( - _('Max notice size is %d chars, including attachment URL.'), - Notice::maxContent() - ) - ); } + $this->clientError( + sprintf( + _('Max notice size is %d chars, including attachment URL.'), + Notice::maxContent() + ) + ); } $options = array('reply_to' => ($replyto == 'false') ? null : $replyto); @@ -197,12 +200,10 @@ class NewnoticeAction extends Action $notice = Notice::saveNew($user->id, $content_shortened, 'web', $options); - if (isset($upload)) { + foreach($uploads as $upload) { $upload->attachToNotice($notice); } - - if ($this->boolean('ajax')) { header('Content-Type: text/xml;charset=utf-8'); $this->xw->startDocument('1.0', 'UTF-8'); |