API notice access was not access restricted

diff --git a/actions/apistatusesshow.php b/actions/apistatusesshow.php
index 3a60b87637e571dd27a6c5653441b59afa380f57..2a7702ee3c7a8735a25892416fb48e639b22590b 100644 (file)
--- a/actions/apistatusesshow.php
+++ b/actions/apistatusesshow.php
@@ -34,9 +34,7 @@
  * @link      http://status.net/
  */
 
-if (!defined('STATUSNET')) {
-    exit(1);
-}
+if (!defined('GNUSOCIAL')) { exit(1); }
 
 /**
  * Returns the notice specified by id as a Twitter-style status and inline user
@@ -65,7 +63,7 @@ class ApiStatusesShowAction extends ApiPrivateAuthAction
      *
      * @return boolean success flag
      */
-    function prepare($args)
+    protected function prepare(array $args=array())
     {
         parent::prepare($args);
 
@@ -76,12 +74,12 @@ class ApiStatusesShowAction extends ApiPrivateAuthAction
 
         $this->notice_id = (int)$this->trimmed('id');
 
-        if (empty($notice_id)) {
-            $this->notice_id = (int)$this->arg('id');
+        $this->notice = Notice::getKV($this->notice_id);
+        if (!$this->notice->inScope($this->scoped)) {
+            // TRANS: Client exception thrown when trying a view a notice the user has no access to.
+            throw new ClientException(_('Access restricted.'), 403);
         }
 
-        $this->notice = Notice::getKV((int)$this->notice_id);
-
         return true;
     }
 
@@ -90,13 +88,11 @@ class ApiStatusesShowAction extends ApiPrivateAuthAction
      *
      * Check the format and show the notice
      *
-     * @param array $args $_REQUEST data (unused)
-     *
      * @return void
      */
-    function handle($args)
+    protected function handle()
     {
-        parent::handle($args);
+        parent::handle();
 
         if (!in_array($this->format, array('xml', 'json', 'atom'))) {
             // TRANS: Client error displayed when coming across a non-supported API method.

diff --git a/actions/shownotice.php b/actions/shownotice.php
index 1d01244cc145b845efb01a9ebe3e1f7194291596..2576bcc111fc65b394f48f2acd74b8f763803087 100644 (file)
--- a/actions/shownotice.php
+++ b/actions/shownotice.php
@@ -77,7 +77,7 @@ class ShownoticeAction extends ManagedAction
 
         if (!$this->notice->inScope($this->scoped)) {
             // TRANS: Client exception thrown when trying a view a notice the user has no access to.
-            throw new ClientException(_('Not available.'), 403);
+            throw new ClientException(_('Access restricted.'), 403);
         }
 
         $this->profile = $this->notice->getProfile();