fix problem with new notices

[quix0rs-gnu-social.git] / actions / newnotice.php

diff --git a/actions/newnotice.php b/actions/newnotice.php
index de040ef77a38301d7152c2d9be6043e861dfa6f7..b576e1326641b0c593feae507b644e8742366e72 100644 (file)
--- a/actions/newnotice.php
+++ b/actions/newnotice.php
@@ -20,13 +20,13 @@
 if (!defined('LACONICA')) { exit(1); }
 
 class NewnoticeAction extends Action {
-
+       
        function handle($args) {
                parent::handle($args);
                # XXX: Ajax!
 
                if (!common_logged_in()) {
-                       common_user_error(_t('Not logged in.'));
+                       common_user_error(_('Not logged in.'));
                } else if ($_SERVER['REQUEST_METHOD'] == 'POST') {
                        $this->save_new_notice();
                } else {
@@ -38,48 +38,33 @@ class NewnoticeAction extends Action {
 
                $user = common_current_user();
                assert($user); # XXX: maybe an error instead...
-               $notice = DB_DataObject::factory('notice');
-               assert($notice);
-               $notice->profile_id = $user->id; # user id *is* profile id
-               $notice->created = DB_DataObject_Cast::dateTime();
-               # Default theme uses 'content' for something else
-               $notice->content = $this->trimmed('status_textarea');
-
-               if (!$notice->content) {
-                       $this->show_form(_t('No content!'));
+               $content = $this->trimmed('status_textarea');
+               
+               if (!$content) {
+                       $this->show_form(_('No content!'));
                        return;
-               } else if (strlen($notice->content) > 140) {
-                       $this->show_form(_t('That\'s too long. Max notice size is 140 chars.'));
+               } else if (strlen($content) > 140) {
+                       $this->show_form(_('That\'s too long. Max notice size is 140 chars.'));
                        return;
                }
 
-               $notice->rendered = common_render_content($notice->content, $notice);
+               $notice = Notice::saveNew($user->id, $content, 'web');
                
-               $id = $notice->insert();
-
-               if (!$id) {
-                       common_server_error(_t('Problem saving notice.'));
+               if (is_string($notice)) {
+                       $this->show_form($notice);
                        return;
                }
-
-               $orig = clone($notice);
-               $notice->uri = common_notice_uri($notice);
-
-               if (!$notice->update($orig)) {
-                       common_server_error(_t('Problem saving notice.'));
-                       return;
-               }
-
-        common_save_replies($notice);  
+               
                common_broadcast_notice($notice);
                
                $returnto = $this->trimmed('returnto');
+               
                if ($returnto) {
                        $url = common_local_url($returnto,
                                                                        array('nickname' => $user->nickname));
                } else {
                        $url = common_local_url('shownotice',
-                                                                       array('notice' => $id));
+                                                                       array('notice' => $notice->id));
                }
                common_redirect($url, 303);
        }
@@ -97,7 +82,7 @@ class NewnoticeAction extends Action {
                                $content = '@' . $profile->nickname . ' ';
                        }
                }
-               common_show_header(_t('New notice'), NULL, $content,
+               common_show_header(_('New notice'), NULL, $content,
                                   array($this, 'show_top'));
                if ($msg) {
                        common_element('p', 'error', $msg);