Completed
Push — phpunit ( d909d7...1f30cc )
by Marcos
573:57 queued 562:54
created
lib/Service/ShareService.php 1 patch
Doc Comments   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -49,6 +49,7 @@  discard block
 block discarded – undo
49 49
 	 * @param $target_item_guid string      The shared item GUID
50 50
 	 * @param $request_array    array
51 51
 	 * @param $permissions      integer     Must be created with a bitmask from options on the ShareRequest class
52
+	 * @param string $credential_owner
52 53
 	 * @return array                        Array of sharing requests
53 54
 	 */
54 55
 	public function createBulkRequests($target_item_id, $target_item_guid, $request_array, $permissions, $credential_owner) {
@@ -253,7 +254,6 @@  discard block
 block discarded – undo
253 254
 	/**
254 255
 	 * Delete ACL
255 256
 	 *
256
-	 * @param  ShareRequest $request
257 257
 	 * @return \OCA\Passman\Db\ShareRequest[]
258 258
 	 */
259 259
 	public function deleteShareACL(SharingACL $ACL) {
@@ -277,7 +277,6 @@  discard block
 block discarded – undo
277 277
 	/**
278 278
 	 * Get pending share requests by guid and uid
279 279
 	 *
280
-	 * @param  ShareRequest $request
281 280
 	 * @return \OCA\Passman\Db\ShareRequest[]
282 281
 	 */
283 282
 	public function getPendingShareRequestsForCredential($item_guid, $user_id) {
Please login to merge, or discard this patch.
lib/Service/CredentialService.php 1 patch
Doc Comments   +20 added lines patch added patch discarded remove patch
@@ -43,23 +43,39 @@  discard block
 block discarded – undo
43 43
 	public function updateCredential($credential) {
44 44
 		return $this->credentialMapper->updateCredential($credential);
45 45
 	}
46
+
47
+	/**
48
+	 * @param Credential $credential
49
+	 */
46 50
 	public function upd($credential) {
47 51
 		return $this->credentialMapper->upd($credential);
48 52
 	}
49 53
 
54
+	/**
55
+	 * @param Credential $credential
56
+	 */
50 57
 	public function deleteCredential($credential){
51 58
 		return $this->credentialMapper->deleteCredential($credential);
52 59
 	}
53 60
 
61
+	/**
62
+	 * @param integer $vault_id
63
+	 */
54 64
 	public function getCredentialsByVaultId($vault_id, $user_id) {
55 65
 		return $this->credentialMapper->getCredentialsByVaultId($vault_id, $user_id);
56 66
 	}
57 67
 
68
+	/**
69
+	 * @param integer $vault_id
70
+	 */
58 71
 	public function getRandomCredentialByVaultId($vault_id, $user_id) {
59 72
 		$credentials = $this->credentialMapper->getRandomCredentialByVaultId($vault_id, $user_id);
60 73
 		return array_pop($credentials);
61 74
 	}
62 75
 
76
+	/**
77
+	 * @param integer $timestamp
78
+	 */
63 79
 	public function getExpiredCredentials($timestamp) {
64 80
 		return $this->credentialMapper->getExpiredCredentials($timestamp);
65 81
 	}
@@ -77,6 +93,10 @@  discard block
 block discarded – undo
77 93
 
78 94
         throw new DoesNotExistException("Did expect one result but found none when executing");
79 95
 	}
96
+
97
+	/**
98
+	 * @param integer $credential_id
99
+	 */
80 100
 	public function getCredentialLabelById($credential_id){
81 101
 		return $this->credentialMapper->getCredentialLabelById($credential_id);
82 102
 	}
Please login to merge, or discard this patch.