Revert "Fixed direct messaging: AjaxWebChannel is now using Action's methods."

This reverts commit 0f2c43bd040437b3e40c706d7c3f4ba163e94a71.

Making Channel a subclass of Action for no other reason than to let
the AjaxWebChannel do some output is the really, really wrong way to
do this. A Channel is not an Action.

I'll change AjaxWebChannel so it takes an Action as a constructor
paramater and uses that Action for its output. We do this for most
Widget subclasses and it makes sense here, too.

diff --git a/classes/Channel.php b/classes/Channel.php
index c03efef481686e2e13bc0dc27d086518d1e771a9..2e3e4e8d4a7cd5c9f44804edf9c10e195bda7189 100644 (file)
--- a/classes/Channel.php
+++ b/classes/Channel.php
@@ -19,7 +19,7 @@
 
 if (!defined('LACONICA')) { exit(1); }
 
-class Channel extends Action
+class Channel
 {
     
     function on($user)
@@ -129,7 +129,6 @@ class WebChannel extends Channel
         # XXX: buffer all output and send it at the end
         # XXX: even better, redirect to appropriate page
         #      depending on what command was run
-
         common_show_header(_('Command results'));
         common_element('p', null, $text);
         common_show_footer();
@@ -147,26 +146,26 @@ class AjaxWebChannel extends WebChannel
 
     function output($user, $text)
     {
-        $this->startHTML('text/xml;charset=utf-8', true);
-        $this->elementStart('head');
-        $this->element('title', null, _('Command results'));
-        $this->elementEnd('head');
-        $this->elementStart('body');
-        $this->element('p', array('id' => 'command_result'), $text);
-        $this->elementEnd('body');
-        $this->elementEnd('html');
+        common_start_html('text/xml;charset=utf-8', true);
+        common_element_start('head');
+        common_element('title', null, _('Command results'));
+        common_element_end('head');
+        common_element_start('body');
+        common_element('p', array('id' => 'command_result'), $text);
+        common_element_end('body');
+        common_element_end('html');
     }
 
     function error($user, $text)
     {
-        $this->startHTML('text/xml;charset=utf-8', true);
-        $this->elementStart('head');
-        $this->element('title', null, _('Ajax error'));
-        $this->elementEnd('head');
-        $this->elementStart('body');
-        $this->element('p', array('id' => 'error'), $text);
-        $this->elementEnd('body');
-        $this->elementEnd('html');
+        common_start_html('text/xml;charset=utf-8', true);
+        common_element_start('head');
+        common_element('title', null, _('Ajax Error'));
+        common_element_end('head');
+        common_element_start('body');
+        common_element('p', array('id' => 'error'), $text);
+        common_element_end('body');
+        common_element_end('html');
     }
 }