Stronger typing and function access control in OStatus

diff --git a/plugins/OStatus/actions/salmon.php b/plugins/OStatus/actions/salmon.php
index 7cb4ac2fce3d7684c95763bcfeea31571d97ae7a..2a6fc15edd4e0f17e0a3d05d5db4b1a3050e130d 100644 (file)
--- a/plugins/OStatus/actions/salmon.php
+++ b/plugins/OStatus/actions/salmon.php
@@ -22,9 +22,7 @@
  * @author James Walker <james@status.net>
  */
 
-if (!defined('STATUSNET')) {
-    exit(1);
-}
+if (!defined('GNUSOCIAL')) { exit(1); }
 
 class SalmonAction extends Action
 {
@@ -32,7 +30,7 @@ class SalmonAction extends Action
     var $activity = null;
     var $target   = null;
 
-    function prepare($args)
+    protected function prepare(array $args=array())
     {
         StatusNet::setApi(true); // Send smaller error pages
 
@@ -78,9 +76,9 @@ class SalmonAction extends Action
      * Check the posted activity type and break out to appropriate processing.
      */
 
-    function handle($args)
+    protected function handle()
     {
-        StatusNet::setApi(true); // Send smaller error pages
+        parent::handle();
 
         common_log(LOG_DEBUG, "Got a " . $this->activity->verb);
         if (Event::handle('StartHandleSalmonTarget', array($this->activity, $this->target)) &&

diff --git a/plugins/OStatus/classes/Ostatus_profile.php b/plugins/OStatus/classes/Ostatus_profile.php
index 59eccc52d06b7a6d778edfdd7e54618c3ed00663..7ab5b7c8f030818d3e6fd32ff4caaf230123cd23 100644 (file)
--- a/plugins/OStatus/classes/Ostatus_profile.php
+++ b/plugins/OStatus/classes/Ostatus_profile.php
@@ -342,12 +342,13 @@ class Ostatus_profile extends Managed_DataObject
      * @param Profile $actor
      * @return boolean success
      */
-    public function notifyActivity($entry, $actor)
+    public function notifyActivity($entry, Profile $actor)
     {
         if ($this->salmonuri) {
             $salmon = new Salmon();
             return $salmon->post($this->salmonuri, $this->notifyPrepXml($entry), $actor);
         }
+        common_debug(__CLASS__.' error: No salmonuri for Ostatus_profile uri: '.$this->uri);
 
         return false;
     }

diff --git a/plugins/OStatus/lib/salmon.php b/plugins/OStatus/lib/salmon.php
index c93b314f8ab4161b084eb1e469b84f6180d53fdf..244fe4748b5b4aaa59c2d7d21ea926523c66e0da 100644 (file)
--- a/plugins/OStatus/lib/salmon.php
+++ b/plugins/OStatus/lib/salmon.php
@@ -46,7 +46,7 @@ class Salmon
      * @param Profile $actor local user profile whose keys to sign with
      * @return boolean success
      */
-    public function post($endpoint_uri, $xml, $actor)
+    public function post($endpoint_uri, $xml, Profile $actor)
     {
         if (empty($endpoint_uri)) {
             return false;