Kaynağa Gözat

replaced the term "paste" with the more generic "document"

kudos @Ribas160
El RIDO 1 yıl önce
ebeveyn
işleme
bde805d2f1

+ 1 - 9
lib/Model/AbstractModel.php

@@ -88,7 +88,7 @@ abstract class AbstractModel
     public function setId($id)
     {
         if (!self::isValidId($id)) {
-            throw new Exception('Invalid paste ID.', 60);
+            throw new Exception('Invalid document ID.', 60);
         }
         $this->_id = $id;
     }
@@ -129,14 +129,6 @@ abstract class AbstractModel
      */
     abstract public function store();
 
-    /**
-     * Delete the current instance.
-     *
-     * @access public
-     * @throws Exception
-     */
-    abstract public function delete();
-
     /**
      * Test if current instance exists in store.
      *

+ 1 - 12
lib/Model/Comment.php

@@ -71,17 +71,6 @@ class Comment extends AbstractModel
         }
     }
 
-    /**
-     * Delete the comment.
-     *
-     * @access public
-     * @throws Exception
-     */
-    public function delete()
-    {
-        throw new Exception('To delete a comment, delete its parent paste', 64);
-    }
-
     /**
      * Test if comment exists in store.
      *
@@ -131,7 +120,7 @@ class Comment extends AbstractModel
     public function setParentId($id)
     {
         if (!self::isValidId($id)) {
-            throw new Exception('Invalid paste ID.', 65);
+            throw new Exception('Invalid document ID.', 65);
         }
         $this->_data['parentid'] = $id;
     }

+ 1 - 1
lib/Model/Paste.php

@@ -111,7 +111,7 @@ class Paste extends AbstractModel
                 $this->_data
             ) === false
         ) {
-            throw new Exception('Error saving paste. Sorry.', 76);
+            throw new Exception('Error saving document. Sorry.', 76);
         }
     }
 

+ 2 - 2
tst/ControllerTest.php

@@ -671,7 +671,7 @@ class ControllerTest extends TestCase
         ob_end_clean();
         $response = json_decode($content, true);
         $this->assertEquals(1, $response['status'], 'outputs error status');
-        $this->assertEquals('Invalid paste ID.', $response['message'], 'outputs error message');
+        $this->assertEquals('Invalid document ID.', $response['message'], 'outputs error message');
     }
 
     /**
@@ -825,7 +825,7 @@ class ControllerTest extends TestCase
         $content = ob_get_contents();
         ob_end_clean();
         $this->assertMatchesRegularExpression(
-            '#<div[^>]*id="errormessage"[^>]*>.*Invalid paste ID\.#s',
+            '#<div[^>]*id="errormessage"[^>]*>.*Invalid document ID\.#s',
             $content,
             'outputs delete error correctly'
         );

+ 0 - 13
tst/ModelTest.php

@@ -366,19 +366,6 @@ class ModelTest extends TestCase
         $this->assertEquals((float) 300, (float) $paste['meta']['time_to_live'], 'remaining time is set correctly', 1.0);
     }
 
-    public function testCommentDeletion()
-    {
-        $pasteData = Helper::getPastePost();
-        $this->_model->getPaste(Helper::getPasteId())->delete();
-
-        $paste = $this->_model->getPaste();
-        $paste->setData($pasteData);
-        $paste->store();
-        $this->expectException(Exception::class);
-        $this->expectExceptionCode(64);
-        $paste->getComment(Helper::getPasteId())->delete();
-    }
-
     public function testPurge()
     {
         $conf  = new Configuration;